3 ms·
FTA: void boundingBox(Circle c) { if (c instanceof Circle(Point(int x, int y), double radius)) { That looks weird. An if whose condition always is rig
by Someone 7mo ago
FTA:
void boundingBox(Circle c) {
if (c instanceof Circle(Point(int x, int y), double radius)) {
That looks weird. An if whose condition always is right looks superfluous. It also makes it look as if one could write things such as
if (c instanceof Circle(Point(int x, int y), double 1.0)) {
…
} else {
…
}
Finally, that code is wordy. I know that is the Java way, but couldn’t it be slightly shorter by not requiring specifying basic types as in
if (c instanceof Circle(Point(x, y), radius)) {
? (I wouldn’t go as far as
if (c instanceof Circle((x, y), radius)) {
. That doesnt feel Java to me)