A few comments...

1) Why do you make the array public?

2) Why do you put "public" in so many times? Once will do...

3) Why do you use a for-loop in the constructor? Is this

a[0] = a[1] = a[2] = w;

not possible, and perhaps clearer?

4) Instead of "equals", "add", and "sub", wouldn't it make more sense to
overload ==, +, and -?

--
Anthony Bennett

Post a reply to this message