yo lo haria de la segunda forma q has colocado


puesto q segun lei, la clase debe dibujar el rectangulo de dos colores diferentes, y si te cambian el main deberia ser capaz de pintar el rectangulo de otros colores y no solamente rojo y verde

La aplicacion los define o el main, pero no por ello son constantes.

Bueno en conclusion yo personalmente utilizaria la segunda opcion, resuminedo lo haria por portabilidad, si el dia de mañana te piden un rectangulo de otro color y has utilizado constantes tendrias que modificar todas las veces la clase, mientras que si lo haces de la segunda forma, no tendras que modificar la clase.

Salu2

Bueno los getters y los setters es bueno ponerlos... claro para programas pequeños seguramente no le veras mucha utilidad.

Pero por ejemplo si te piden en medio del programa que cambies el color del contorno... ya se me diras que llamas al construcctor parametrizado nuevamente, pero ¿llamaras al constructor de toda una clase para modificar solamente uno de los datos del objeto? estaras accediendo a todos sus miembros para cambiar solo un dato, eso no tiene sentido.

Claro para tu programa seguramente no habra problema, no es un programa complejo, pero si el problema fuera de una complegidad alta o un software muy robusto, ufff andar llamando al constructor solo para modificar una pequeña variable seria una muy mala practica. (incluso en programas pequeños me parece mala practica deprogramación)