返回错误值的方法

时间:2015-10-18 06:27:52

标签: java class methods

我在课堂上遇到这个奇怪的错误。我有这个IRNode类,我有3个字符串:op1,op2和op3。当我打印出来时,在将它们分配给不同的字符串然后将它们打印出来时给出不同的值。它将它们存储为null。

print语句的输出是这样的:

OP1: 20, OP2: a, OP3: null

;STOREI 

虽然它应该是:

OP1: 20, OP2: a, OP3: null

;STOREI 20 a

我的代码:

public IRNode (String op, String o1, String o2, String o3) {

    opcode = op;
    op1 = o1;
    op2 = o2;
    op3 = o3;
    //IRNodeNext = null;

}

public void printIR () {
    System.out.format("OP1: %s, OP2: %s, OP3: %s\n",op1, op2,op3);
    String o1 = this.op1;
    String o2 = this.op2;
    String o3 = this.op3;
    System.out.print(";"+opcode);
    if (o1 != null) System.out.format(" ",o1);
    if (o2 != null) System.out.format(" ",o2);
    if (o3 != null) System.out.format(" ",o3);
    System.out.format("\n");
}

PS。这是我的编译器项目。这是在我创建AST和IRNode列表之后。

***** EDIT ****** 实际上发现这是一个愚蠢的错误。但我现在面临另一个奇怪的错误。

op1 = thisNode.returnOP1();
op2 = thisNode.returnOP2();
op3 = thisNode.returnOP3();

在这里,我像以前一样得到op1,op2和op3。返回时,我检查了,我正在返回正确的值。 例如, 对于STOREI 20 a 我正在返回op1 = 20和op2 = a

但是当我在这3行代码之后收到它时,我得到了错误的值。我得到op1 = 20,op2 = $ s,op3 = a

为什么会这样?

1 个答案:

答案 0 :(得分:3)

我在这个例子中看到的唯一问题是:

if (o1 != null) System.out.format(" ",o1);

这些行中的每一行都应在空字符串中有%s

if (o1 != null) System.out.format(" %s",o1);
if (o2 != null) System.out.format(" %s",o2);
if (o3 != null) System.out.format(" %s",o3);

没有它,你就不会打印任何东西。

输出:

OP1: 20, OP2: a, OP3: null
;STOREI 20 a

您可能还想考虑这个问题:

System.out.format("%n");

不要使用\n,而是使用%n表示格式化字符串中的换行符。这是平台无关的。

  

*****编辑******实际上发现这是一个愚蠢的错误。但我现在面临另一个奇怪的错误。

     

op1 = thisNode.returnOP1(); op2 = thisNode.returnOP2(); op3 =   thisNode.returnOP3();在这里,我得到像之前的op1,op2和op3。什么时候   返回,我检查,我正在返回正确的值。例如,   对于STOREI 20 a,我返回op1 = 20和op2 = a

     

但是当我在这三行代码之后收到它时,我就是   错误的价值观。我得到op1 = 20,op2 = $ s,op3 = a

     

为什么会这样?

如果没有returnOP方法的代码,很难说,但根据之前的问题,我愿意打赌问题类似。

鉴于此:

public String returnOP1() { return op1; }
public String returnOP2() { return op2; }
public String returnOP3() { return op3; }

而且:

    System.out.println(irNode.returnOP1());
    System.out.println(irNode.returnOP2());
    System.out.println(irNode.returnOP3());

我明白了:

20
a
null

这告诉我,对您而言,returnOP2()会被写入以返回$s而不是op2op2被设置为$s,{写入{1}}以返回returnOP3()

但没有代码,就无法确定。