0

我必须实现这个方法来给出数组“planeta”中的行星数量。

public int size() {

    int size=planeta.length-1;

    for (int k=planeta.length-1;k>0;k--){
        if (planeta[k].name().equals(noPlanet))
        size--;
    }
    return size;

}

它返回“planeta.length-1”的值，而不是它应该返回的值（例如，我成功添加了 2 个行星，它总是返回我在初始化时赋予 size 的值）。我究竟做错了什么？

注意：我用 NP 初始化数组“planeta”，这意味着没有行星。当我添加新行星时，它们会转到数组“planeta”中的第一个空闲位置，从 0 开始。请帮忙！！！！！谢谢你=)

addPlanet 方法： public void addPlanet(String planetName, BigInt x, BigInt y, BigInt z) {

    if(!planetExists(planetName) && !complete()){

        SpacePoint newSP= new SpacePoint(x,y,z);
        Planet newPlanet= new Planet(planetName,newSP);

        s=this.size();
        planeta[s]=newPlanet;

    }

}

我如何初始化数组“planeta”： private Planet noPlanet=new Planet("NP");

/** 
 * Creates a new Galaxy with no associated Planets and a given name
 * @param name the name of the Galaxy
 */
public Galaxy(String name) {


    this.name=name;


    //if the galaxy has no planets in it,it's array if filled with planet NP-No Planet
    for (int i=0;i<planeta.length;i++)
        planeta[i]=noPlanet;

}
4

3 回答 3

0

最好使用nulls 而不是特殊值对象，即new Planet("NP"). 我想这是你正在寻找的东西：

public class Galaxy {
  // Empty array of size = 10, i.e. { null, null, .. , null }
  private Planet[] planeta = new Planet[10];

  public Galaxy(String name) {
    this.name = name;
  }

  public int size() {
    int size = 0;

    for (int k = 0; k < planeta.length; k++) {
      // Is there a planet or is it empty?
      if (planeta[k] != null) {
        // There's a planet.
        size++;
      }
    }
    return size;
  }

  public boolean planetExists(String planetName) {
    // Go through the array looking for that planet...
    for (int k = 0; k < this.size(); k++) {
  // NOTICE that we're going only through the non-empty array elements,
      // but we'll check for null either way.
      if (planeta[k] != null && planeta[k].name().equals(planetName)) {
        // Found it.
        return true;
      }
    }

    // Didn't find the planet.
    return false;
  }

  public void addPlanet(String planetName, BigInt x, BigInt y, BigInt z) {
    if (!planetExists(planetName) && !complete()) {
      SpacePoint newSP = new SpacePoint(x, y, z);
      Planet newPlanet = new Planet(planetName, newSP);

      int position = this.size();
      planeta[position] = newPlanet;
    }
  }
}

至于调试，大多数开发环境（IDE）都有Run > Debug命令Run > Run。首先，您需要设置一个断点，通常是Run > Toggle Breakpoint在选定的行上。使用Run > Step into和Run > Step over命令逐行进行。

于 2013-04-12T23:16:17.103 回答
0

这行得通！（至少在我的情况下）我将向您展示更改：

public int size() {

    int size = planeta.length;

    for (int k=planeta.length-1; k>=0; k--) {
          // Is there a planet or is it empty?
          if (planeta[k].equals(noPlanet)) {
            // There's no Planet
            size--;

          }
        }
        return size;
}

方法planetExists：

public boolean planetExists(String planetName) {

    for (int i=0;i<planeta.length;i++)
        if (planeta[i].name().equals(planetName))
            return true;

    return false;



}
于 2013-04-13T15:03:12.757 回答
0

如果您声明数组Planet[] planeta = new Planet[12];，则数组的长度将始终为12. 该数组已经填充了未初始化的值。在这种情况下null。

改为使用ArrayList<Planet> planeta = new ArrayList<Planet>();。您可以使用 添加新行星planeta.add( new Planet() );并使用 返回大小planeta.size()。

如果你必须坚持使用静态数组，你应该测试一个索引是否null在循环中有一个值，for并在找到一个值时减小它的大小。

于 2013-04-12T22:29:44.453 回答