Невозможно вычислить мою логическую ошибку Java

Question

Когда я иду, чтобы запустить это значение, оно возвращается в два раза больше текущейStation плюс инкрементное значение. Вместо просто currentStation плюс добавочное значение. Я не могу понять, есть ли проблема в классе AutoRadioSystem или Radio.

public class AutoRadioSystem 
{ 

    private Radio selectedRadio; 
    private AMRadio radioAM = new AMRadio(); 
    private FMRadio radioFM = new FMRadio(); 
    private XMRadio radioXM = new XMRadio(); 

    public AutoRadioSystem() 
    { 
    selectedRadio = new AMRadio(); 
    } 
    public double getCurrentStation() 
    { 
    if (selectedRadio.equals(radioAM)) 
    { 
     return radioAM.getCurrentStaion(); 
    } 
    else if (selectedRadio.equals(radioFM)) 
    { 
     return radioFM.getCurrentStaion(); 
    } 
    else if (selectedRadio.equals(radioXM)) 
    { 
     return radioXM.getCurrentStaion(); 
    } 
    return 0.0; 
    } 
    public void selectRadio() 
    { 
    if (selectedRadio.equals(radioAM)) 
     selectedRadio = radioFM; 
    else if (selectedRadio.equals(radioFM)) 
     selectedRadio = radioXM; 
    else if (selectedRadio.equals(radioXM)) 
     selectedRadio = radioAM; 
    } 

    public void upCategory() 
    { 
    double catUp = radioXM.getCurrentStaion(); 
    catUp += 10; 
    if (catUp > 199.0) 
    { 
     catUp = 1; 
     radioXM.setCurrentStation(catUp); 
    } 
    radioXM.setCurrentStation(catUp); 
    } 

    public void up() 
    { 
    if (selectedRadio.equals(radioAM)) 
    { 
     double stationUp = radioAM.getCurrentStaion(); 
     stationUp += radioAM.getIncrement(); 
     radioAM.setCurrentStation(stationUp); 
     // System.out.println(stationUp); 
    } 
    } 
    public boolean equals (Object o) 
    { 
    if (o == null) 
     return false; 
    if (! (o instanceof AutoRadioSystem)) 
     return false; 
    AutoRadioSystem other = (AutoRadioSystem) o; 
    return this.selectedRadio == other.selectedRadio; 
    } 

    public static void main (String [] args) { 
    AutoRadioSystem c = new AutoRadioSystem(); 
    c.selectRadio(); 
    double b = c.getCurrentStation(); 
    System.out.println(b); 
    c.selectRadio(); 
    double d = c.getCurrentStation(); 
    System.out.println(d); 
    c.upCategory(); 
    double f = c.getCurrentStation(); 
    System.out.println(f); 
    c.selectRadio(); 
    double e = c.getCurrentStation(); 
    System.out.println(e); 
    c.up(); 
    double g = c.getCurrentStation(); 
    System.out.println(g); 
    c.up(); 
    double t = c.getCurrentStation(); 
    System.out.println(t); 
    } 
} 

дополнительный код

public class AMRadio extends Radio 
{ 
    private static final double Max_Station = 1605; 
    private static final double Min_Station = 535; 
    private static final double Increment = 10; 
    public AMRadio() 
    { 
    } 
    public double getMax_Station() 
    { 
    return this.Max_Station; 
    } 
    public double getMin_Station() 
    { 
    return this.Min_Station; 
    } 
    public double getIncrement() 
    { 
    return this.Increment; 
    } 
    public String toString() 
    { 
    String message = ("AM " + this.currentStation); 
    return message; 
    } 

} 


public abstract class Radio 
{ 
double currentStation; 

RadioSelectionBar radioSelectionBar; 
public Radio() 
{ 
    this.currentStation = getMin_Station(); 
} 
public abstract double getMax_Station(); 
public abstract double getMin_Station(); 
public abstract double getIncrement(); 
public void up() 
{ 

} 

public void down() 
{ 

} 

public double getCurrentStaion() 
{ 
    return this.currentStation; 
} 
public void setCurrentStation(double freq) 
{ 
    currentStation += freq; 
} 
public void setStation(int buttonNumber, double station) 
{ 
} 
public double getStation(int buttonNumber) 
{ 
    return 0.0; 
} 
public String toString() 
{ 
    String message = ("" + currentStation); 
    return message; 
} 
    public boolean equals (Object o) 
    { 
    if (o == null) 
     return false; 
    if (! (o instanceof Radio)) 
     return false; 
    Radio other = (Radio) o; 
    return this.currentStation == other.currentStation; 
    } 

Answer 1

Проблема ниже метода:

public void setCurrentStation(double freq) 
{ 
    // This is equal to currentStation = currentStation + freq 
    currentStation += freq; 
} 

должно быть

public void setCurrentStation(double freq) 
{ 
    currentStation = freq; 
} 

Поскольку вы уже добавления currentStation в up.

всего предложения, на самом деле не связанные с вопросом:

• Думай о методе 'selectRadio'. Каково намерение иметь его.
• У вас есть Radio как собственность и другие три реализации этого класса. Я думаю, вам стоит подумать об использовании оператора «instance of».