为什么我不能返回在toString方法中有价值的字符串类型?

时间:2019-04-28 04:48:38

标签: java string compiler-errors tostring

我正在尝试为名为ThreeDVector的对象编写toString方法,该对象可以根据i,j和k打印出3-d矢量,例如“ -2i + 3.8k-j”或“ 7i-5j”。但是,在第96行上,总是存在一个错误,提示s1,s2,并且s3可能尚未初始化。由于我已经初始化了它,所以我猜这些变量的变量类型有问题,但是我不知道如何解决。

class ThreeDVector
{
  double x;    // x-component of vector
  double y;    // y-component of vector
  private double z;  // z-component of vector
// For the purposes of this lab the z component must be between -1000 
// and 1000 (non-inclusive). 

  public ThreeDVector(){
    x=0; 
    y=0;
    z=0;
  }
  public ThreeDVector(double x, double y, double z)
  {
    this.x = x;
    this.y = y;
    if (z>(-1000)&&z<(1000))
      this.z = z;
    else{
      throw new RuntimeException(); 
    }
  }

  public void setZvalue(double z) throws Exception
  {
    if( z>(-1000)&&z<1000 )
      this.z= z;
    else{
      throw new Exception("z value has to be in the range of -1000 to 1000, non-inclusve");
    }
  }


  public boolean isWholenum (double n){
    if(Math.round(n) == n)
      return true;
    else 
      return false;
  }

  public String toString(){

    String s1, s2, s3;

    if(this.z>=1000||z<=(-1000)){
      return "undefied";
    }else{
      if (x!=0){
        if(isWholenum(x)==true){
          s1=String.valueOf(Math.round(x))+"i";
        }else{
          s1=String.valueOf(String.format("%.3f", x))+"i";
        }
      }else if (x==0)
        s1=null;//if the coefficient is 0, do not print out that term 

      if (y>0){
        if(isWholenum(y)==true){
          s2="+"+String.valueOf(Math.round(y))+"j";
        }else{
          s2="+"+String.valueOf(String.format("%.3f", y))+"j";
        }
      }
      else if (y==0)
        s2=null; 
      else if (y<0){
        if(isWholenum(y)==true){
          s2="-"+String.valueOf(Math.round(y))+"j";
        }else{
          s2="-"+String.valueOf(String.format("%.3f", y))+"j";
        }
      } 
      if (z>0){
        if(isWholenum(z)==true){
          s3="+"+String.valueOf(Math.round(z))+"k";
        }else{
          s3="+"+String.valueOf(String.format("%.3f", y))+"k";
        }
      }
      else if (z==0)
        s3=null; 
      else if (z<0){
        if(isWholenum(z)==true){
          s3="-"+String.valueOf(Math.round(z))+"k";
        }else{
          s3="-"+String.valueOf(String.format("%.3f", z))+"k";
        }
      } 


      return "("+ s1+s2+ s3+")"; 

    }
  }


}

2 个答案:

答案 0 :(得分:2)

变量类型没有问题,但它们不应为null。在toString方法的开头将s1,s2和s3初始化为空字符串。另外,不要将它们设置为null,而应将它们设置为空字符串,在该字符串中您将无法获得x,y或z的适当值。

答案 1 :(得分:0)

代替:

String s1, s2, s3;

使用:

String s1 = "", s2 = "", s3 = "";

此外,我不会使用将条件之一设置为null的条件块,因为将它们初始化为空字符串会使此操作变得多余。将!放在每个条件的前面,并将它们添加到将字符串设置为不同值的块中。