несколько конструкторов и если выписка

Question

Я хочу, в пределах моего текста class, определить два конструктора.

При создании нового экземпляра class, я хочу, чтобы ToString вернуть что-то другое в зависимости от того, что называется:

пожарный автомобиль Class

public class FireEngine extends EmergencyVehicle { 

    private String colour; 

    public FireEngine(String colour) { 
     super (colour); 

    } 
    public FireEngine() { 
     this("red"); 

    } 

    public String toString() { 
     if (colour == "red") { 
      return "red"; 
    } else 
     return "no"; 
    } 
} 

EmergencyVehicle класс:

public abstract class EmergencyVehicle extends RoadVehicle { 

    public boolean codeBlue = false; 

    public EmergencyVehicle(String colour){ 
     super(colour); 
    } 

    public boolean isEmergency() { 
     if (codeBlue == true) { 
      return true; 
     } else { 
      return false; 
     } 
    } 

    public void setEmergency(boolean newEmergency) { 
     codeBlue = newEmergency; 
    } 

} 

Это домашнее задание, поэтому я не хочу отвечать сам по себе, но имеет ли смысл этот код?

Например, если я добавляю новый аварийный автомобиль, я хочу оператор if в зависимости от того, какой цвет я добавляю.

Answer 1

первое замечание

Не называйте

this("red"); 

в конструктор по умолчанию, сделайте

colour = "red"; 

если конструктор EmergencyVehicle(String colour) RoadVehicle(String colour) не делает что-то еще.

второе замечание

Не сравнить с использованием

if (colour == "red") 

Используйте

if ("red".equals(colour)) 

3rd замечанием

Метод

public String toString() 

Предполагается вернуть строковое представление экземпляра. Вы осуществляете только возврат red или no, который не очень информативен. Используйте что-то вроде

return("FireEngine(colour=" + colour + ")");