2013-10-09 123 views
0

这里编程noob。我试图编写一个简单的程序,随机生成一个井字棋板,显示X和Os,然后确定游戏的结果。我的问题是,当我将字符串变量放在if/else语句的大括号中时,它告诉我它们没有被初始化。即,它表示变量s11,s13,s21,s22,s31,s32和s33尚未初始化。然而,s12和s23是初始化的,我能看到的唯一区别是它们不在大括号内。我在哪里犯错了?谢谢。一些变量可能没有被初始化?为什么?

import java.util.Random; 

public class Lab7 { 

public static void main(String[] args) { 

    int b11, b12, b13, b21, b22, b23, b31, b32, b33; 
    int x, o; 
    String s11, s12, s13, s21, s22, s23, s31, s32, s33; 

    x = 0; 
    o = 0; 

    Random r = new Random(); 
    b11 = r.nextInt(3); 
    b12 = r.nextInt(3); 
    b13 = r.nextInt(3); 
    b21 = r.nextInt(3); 
    b22 = r.nextInt(3); 
    b23 = r.nextInt(3); 
    b31 = r.nextInt(3); 
    b32 = r.nextInt(3); 
    b33 = r.nextInt(3); 

    if ((b11 == b12) && (b12 == b13)) 
     if (b11 == 0) 
     { 
      o = o+1; 
      s11 = "O"; 
     } 
     else if (b11 == 1) 
     { 
      x = x+1; 
      s11 = "X"; 
     } 
     else 
      s11 = ""; 
    if ((b21 == b22) && (b22 == b23)) 
     if (b21 == 0) 
     { 
      o = o+1; 
      s21 = "O"; 
     } 
     else if (b21 == 1) 
     { 
      x = x+1; 
      s21 = "X"; 
     } 
     else 
      s21 = ""; 
    if ((b31 == b32) && (b32 == b33)) 
     if (b31 == 0) 
     { 
      o = o+1; 
      s31 = "O"; 
     } 
     else if (b31 == 1) 
     { 
      x = x+1; 
      s31 = "X"; 
     } 
     else 
      s31 = ""; 

    if ((b11 == b21) && (b21 == b31)) 
     if (b11 == 0) 
      o = o+1; 
     else if (b11 == 1) 
      x = x+1; 
    if ((b32 == b22) && (b22 == b12)) 
     if (b32 == 0) 
     { 
     o = o+1; 
     s32 = "O"; 
     } 
     else if (b32 == 1) 
     { 
     x = x+1; 
     s32 = "X"; 
     } 
     else 
      s32 = ""; 
    if ((b33 == b23) && (b23 == b13)) 
     if (b33 == 0) 
     { 
     o = o+1; 
     s33 = "O"; 
     } 
     else if (b33 == 1) 
     { 
      x = x+1; 
      s33 = "X"; 
     } 
     else 
      s33 = ""; 

    if ((b22 == b11) && (b22 == b33)) 
     if (b22 == 0) 
     { 
     o = o+1; 
     s22 = "O"; 
     } 
     else if (b22 == 1) 
     { 
     x = x+1; 
     s22 = "X"; 
     } 
     else 
      s22 = ""; 
    if ((b13 == b22) && (b22 == b31)) 
     if (b13 == 0) 
     { 
      o = o+1; 
      s13 = "O"; 
     } 
     else if (b13 == 1) 
     { 
      x = x+1; 
      s13 = "X"; 
     } 
     else 
      s13 = ""; 

    if (b12 == 0) 
     s12 = "O"; 
    else if (b12 == 1) 
     s12 = "X"; 
    else 
     s12 = ""; 

    if (b23 == 0) 
     s23 = "O"; 
    else if (b23 == 1) 
     s23 = "X"; 
    else 
     s23 = ""; 

    System.out.println(s11+" "+s12+" "+s13); 
    System.out.println(s21+" "+s22+" "+s23); 
    System.out.println(s31+" "+s32+" "+s33); 
} 
} 
+5

考虑执行所有可能的路径。 –

+0

'if/else'的's12'和's23'链总是导致分配。 – SJuan76

+1

即使你可以证明对于所有路径,所有使用的变量都已经在该路径的前面进行了分配(我没有检查,代码太多),但java中的初始化检查器非常简单并且相当悲观。如果您确定,只需初始化为null或其他。 – harold

回答

0

如果if表达式的条件是假的,sxx变量将不会被初始化。他们必须始终被初始化。您可以简单地将它们初始化为null或空字符串。

0

它们被声明,但不是必然初始化。为了消除这种编译错误,他们必须在中每初始化通过代码的可能路径。做到这一点最简单的方法是简单地设置您的Stringnull""当你声明它们,即

String s11 = null; 

String s11 = ""; 
1

使用

String s11="", s12="", s13="", s21="", s22="", s23="", s31="", s32="", s33=""; 

,而不是

String s11, s12, s13, s21, s22, s23, s31, s32, s33; 
0

编译器认识到在任何情况下s12s23都被赋予一个值。对于其他人,他们不需要,他们需要一个价值才能使用。只需写入即可初始化每个变量。

String s11=""; 
String s12=""; 
... 
1

这是因为您的变量可能不会被初始化,具体取决于您的代码所需的条件。

只需在它们中加上一个空值,你的代码应该很好。

但请记住,使用==运算符来比较两个String对象可能会失败。

0

错误意味着根据输入,这些变量可能不会被初始化。

例如if ((b13 == b22) && (b22 == b31))它可能导致错误,s13不会被初始化。

String s11 = "", s12 ="" . . . .;将解决问题,但可能会导致逻辑错误,您可以稍后检查。现在就去做吧。

0

使用嵌套if语句时,悬挂其他属于内部if,而不是两者都是。例如,如果b11不等于b12,则不会将任何内容分配给s11。最简单的办法是这样的:

if ((b11 == b12) && (b12 == b13)) { 
    if (b11 == 0) 
    { 
     o = o+1; 
     s11 = "O"; 
    } 
    else if (b11 == 1) 
    { 
     x = x+1; 
     s11 = "X"; 
    } else { 
     s11 = ""; 
    } 
} 
else { 
     s11 = ""; 
} 

而且,使用大括号磨片if语句的部分横跨多行。

+0

非常感谢。我需要学习如何记住这样的事情。 – CSpadawan

相关问题