2

我在让这个项目正常工作时遇到了很多问题,但我目前一直坚持让这个类正常工作。它的假设是从广播类中获取当前电台并将其传递给该类。问题是我试图在 AM 和 FM 之间进行选择,但每次运行它时,它只显示 AM 电台。我不明白为什么它会自动设置到那个电台。

public class AutoRadioSystem
{
  private Radio selectedRadio;
  private AMRadio radioAM;
  private FMRadio radioFM; 
  private XMRadio radioXM;

  //is this the correct place to initialize these? 
  Radio amRadio = new AMRadio();
  Radio fmRadio = new FMRadio();
  public AutoRadioSystem()
  {
   //even making the selected radio FM still produces values for AM
   selectedRadio = radioFM;
  }

  // this is where my problem currently lies and probably much more. Shouldn't it return 0.0 without any station being selected.  
  public double getCurrentStation()
  {
    if (selectedRadio == radioAM)
    {
      return amRadio.getCurrentStaion();
    }
    else if (selectedRadio == radioFM)
    {
      return fmRadio.getCurrentStaion();
    }
    return 0.0;
  }

  //I'm not sure if i'm setting this up correctly to switch the radio from am to fm 
  public void selectRadio()
  {
    if (selectedRadio == radioAM)
      selectedRadio = radioFM;
  }
  public static void main (String [] args) { 
    AutoRadioSystem c = new AutoRadioSystem();
    c.selectRadio();
    double b = c.getCurrentStation();
    System.out.println(b);
  }
}

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()
  {
    currentStation = Min_Station;
  }
  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 class FMRadio extends Radio
{
  private static final double Max_Station = 108.0;
  private static final double Min_Station = 88.0;
  private static final double Increment = .01;
  public FMRadio()
  {
    currentStation = Min_Station;
  }
  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 = ("FM " + this.currentStation);
    return message;
  } 
}


public abstract class Radio
{
 double currentStation;

 RadioSelectionBar radioSelectionBar;
 public Radio()
 {

 }
 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)
 {
   this.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;
  } 
 }
4

3 回答 3

1

问题是,在 .getCurrentStation() 中,selectedRadio 和 radioAM 都不是初始化且为空。

错误开始于:

public void selectRadio()
{
    if (selectedRadio == radioAM)
    {
        selectedRadio = radioFM;
    }
}

在这里,selectedRadio = null,所以它永远不会被赋值。

编辑:我相信你才刚刚开始,所以更多细节会有所帮助。

  1. 当声明两个字段 amRadio 和 radioAM 然后初始化其中一个并使用另一个时,您会犯错误。
  2. 您没有为 selectedRadio 设置值并进行比较,这总是返回 false
  3. 为实例初始化值的最佳位置是构造方法,这里是 AutoRadioSystem()

您可能希望将代码更改为:

private Radio selectedRadio;
public AutoRadioSystem()
{
    selectedRadio = new FMRadio();
}
// To compare, using instanceOf, but better design will use enum value instead, up to you
于 2013-09-24T18:31:58.263 回答
1

我想我找到了问题

每个Radio重载都有 2 个字段

private AMRadio radioAM;
...
Radio amRadio = new AMRadio();

但是您要与之比较的那个:radioAM永远不会被实例化,因此始终为空。


你打电话时

if (selectedRadio == radioAM)

两者selectedRadioradioAM都是null,所以它们当然是相等的


除非您打算radioAMamRadio成为完全不同的实例,否则您不应该有 2 个这样的字段。

由于您使用的是多态性,因此您可能希望使用后者

Radio amRadio = new AMRadio();
于 2013-09-24T18:32:09.170 回答
0

selectedRadio、radioAM 和 RadioFM 的所有属性都为空。构造函数中的代码没有效果,因为 selectedRadio = RadioFM。这意味着 selectedRadio 的值不会改变并且保持为零。

因此 getCurrentStation 中的 selectedRadio == radioAM (null == null) 始终为真。这将始终应用您的方法 getCurrentStation 中的第一个 if 块,并且始终返回“amradio”。

蔡欧

于 2013-09-24T19:09:47.530 回答