编译问题：无法访问的语句
Posted
技术标签:
【中文标题】编译问题：无法访问的语句【英文标题】：Compilation issue: Unreachable statement 【发布时间】：2013-10-29 07:59:48 【问题描述】：大家晚上好，
我遇到了一些用于介绍性 Java 类的代码的编译问题。手头的应用程序创建了一个计算器。在尝试编译时，我收到一条错误消息，指出我有一个“无法访问的语句”，这让我相信我在某个地方陷入了循环（同样，我正在上入门课程，所以我的假设可能是错误的）。我已经无数次地查看了代码，但找不到问题所在。编译器指向第 99 行，keypad.add(keys[12]); //减去，作为错误的来源。我还在这一行的末尾添加了“/ERROR/”以帮助指出这一点。除了帮助定位此编译错误的来源之外，您能否告诉我，除了对代码进行细致的梳理之外，我是否应该采用任何技术来查找此类错误？
import java.awt.*;
import java.awt.event.*;
import java.awt.datatransfer.*;
import java.text.DecimalFormat;
import javax.swing.JOptionPane;
public class Calculator extends Frame implements ActionListener
private Button keys[];
private Panel keypad;
private TextField lcd;
private double opl;
private boolean first;
private boolean foundKey;
private boolean clearText;
private int lastOp;
private DecimalFormat calcPattern;
public Calculator()
// create an instance of the menu
MenuBar mnuBar = new MenuBar();
setMenuBar(mnuBar);
// construct and populate the File menu
Menu mnuFile = new Menu("File", true);
mnuBar.add(mnuFile);
MenuItem mnuFileExit = new MenuItem("Exit");
mnuFile.add(mnuFileExit);
// construct and populate the Edit menu
Menu mnuEdit = new Menu("Edit", true);
mnuBar.add(mnuEdit);
MenuItem mnuEditClear = new MenuItem("Clear");
mnuEdit.add(mnuEditClear);
mnuEdit.insertSeparator(1);
MenuItem mnuEditCopy = new MenuItem("Copy");
mnuEdit.add(mnuEditCopy);
MenuItem mnuEditPaste = new MenuItem("Paste");
mnuEdit.add(mnuEditPaste);
// construct and populate the About menu
Menu mnuAbout = new Menu("About", true);
mnuBar.add(mnuAbout);
MenuItem mnuAboutCalculator = new MenuItem("About Calculator");
mnuAbout.add(mnuAboutCalculator);
// add the ActionListener to each menu item
mnuFileExit.addActionListener(this);
mnuEditClear.addActionListener(this);
mnuEditCopy.addActionListener(this);
mnuEditPaste.addActionListener(this);
mnuAboutCalculator.addActionListener(this);
// assign an ActionCommand to each menu item
mnuFileExit.setActionCommand("Exit");
mnuEditClear.setActionCommand("Clear");
mnuEditCopy.setActionCommand("Copy");
mnuEditPaste.setActionCommand("Paste");
mnuAboutCalculator.setActionCommand("About");
// constuct components and initialize beginning values
lcd = new TextField(20);
lcd.setEditable(false);
keypad = new Panel();
keys = new Button[16];
first = true;
opl = 0.0;
clearText = true;
lastOp = 0;
calcPattern = new DecimalFormat("########.########");
// consturct and assign captions to the Buttons
for (int i=0; i<=9; i++)
keys[i] = new Button(String.valueOf(i));
keys[10] = new Button("/");
keys[11] = new Button("*");
keys[12] = new Button("-");
keys[13] = new Button("+");
keys[14] = new Button("=");
keys[15] = new Button(".");
// set Frame and keypad layout to grid layout
setLayout(new BorderLayout());
keypad.setLayout(new GridLayout(4,4,10,10));
for (int i=7; i<=10; i++) // 7, 8, 9, divide
keypad.add(keys[i]);
for (int i=4; i<=6; i++) // 4, 5, 6
keypad.add(keys[i]);
keypad.add(keys[11]); // multiply
for (int i=1; 1<=3; i++) // 1, 2, 3
keypad.add(keys[i]);
keypad.add(keys[12]); // subtract /*ERROR*/
keypad.add(keys[0]); // 0 key
for (int i=15; i>=13; i--) // decimal point, =, +
keypad.add(keys[i]);
for (int i=0; i<keys.length; i++)
keys[i].addActionListener(this);
add(lcd, BorderLayout.NORTH);
add(keypad, BorderLayout.CENTER);
addWindowListener(
new WindowAdapter()
public void windowClosing(WindowEvent e)
System.exit(0);
);
// end of constructor method
public void actionPerformed(ActionEvent e)
// test for menu item clicks
String arg = e.getActionCommand();
if (arg == "Exit")
System.exit(0);
if (arg == "Clear")
clearText = true;
first = true;
opl = 0.0;
lcd.setText("");
lcd.requestFocus();
if (arg == "Copy")
Clipboard cb = Toolkit.getDefaultToolkit().getSystemClipboard();
StringSelection contents = new StringSelection(lcd.getText());
cb.setContents(contents, null);
if (arg == "Paste")
Clipboard cb = Toolkit.getDefaultToolkit().getSystemClipboard();
Transferable content = cb.getContents(this);
try
String s = (String)content.getTransferData(DataFlavor.stringFlavor);
lcd.setText(calcPattern.format(Double.parseDouble(s)));
catch (Throwable exc)
lcd.setText("");
if (arg == "About")
String message = "Calculator ver.1.0\nOpenExhibit Softwar\nCopyright 2007\nAll rights Reserved";
JOptionPane.showMessageDialog(null,message,"About Calculator", JOptionPane.INFORMATION_MESSAGE);
// test for button clicks
foundKey = false;
// search for the clicked key
for (int i=0; i<keys.length && !foundKey; i++)
if(e.getSource() == keys[i])
foundKey = true;
switch(i)
// number and decimal point buttons
case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 8: case 9: case 15:
if(clearText)
lcd.setText("");
clearText = false;
lcd.setText(lcd.getText() + keys[i].getLabel());
break;
// operator buttons
case 10: case 11: case 12: case 13: case 14:
clearText = true;
if (first) // first operand
if(lcd.getText().length()==0) opl = 0.0;
else opl = Double.parseDouble(lcd.getText());
first = false;
clearText = true;
lastOp = i; // save the last operator
else // second operand
switch(lastOp)
case 10: // divide button
opl /= Double.parseDouble(lcd.getText());
break;
case 11: // multiply button
opl *= Double.parseDouble(lcd.getText());
break;
case 12: // minus button
opl -= Double.parseDouble(lcd.getText());
break;
case 13: // plus button
opl += Double.parseDouble(lcd.getText());
break;
// end of switch(lastOp)
lcd.setText(calcPattern.format(opl));
clearText = true;
if(i==14) first = true; // equal button
else lastOp = i; // save last operator
// end else
break;
// end of switch(i)
// end of if
// end of for
// end of actionPerformed
public static void main(String args[])
// set frame properties
Calculator f = new Calculator ();
f.setTitle("Calculator Application");
f.setBounds(200,200,300,300);
f.setVisible(true);
// set image properties and add to frame
Image icon = Toolkit.getDefaultToolkit().getImage("calcImage.gif");
f.setIconImage(icon);
// end of main
// end of class
一如既往，感谢您的帮助。
杰瑞
【问题讨论】：
我建议您创建一个 Swing GUI，而不是 AWT。您也不能通过== 比较字符串。请改用equals(...) 或equalsIgnoreCase(...) 方法。即，而不是if (arg == "Clear") 做if (arg.equalsIgnoreCase("Clear"))。
@HovercraftFullOfEels 感谢您的意见。我会尽量记住这一点，以备将来使用。话虽如此，我需要以一种非常具体的方式为此类创建应用程序，以最好地使我能够查看和使用各种不同的包、技术等。
??请明确要求。这是给学校的吗？您是否绝对需要使用 AWT 而不是 Swing？如果不是，您是否觉得有些事情是 AWT 允许您做而不是 Swing？至于我对字符串的其他建议，您最好更改当前代码，否则可能无法正常工作。
@HovercraftFullOfEels 代码工作正常。关于要求，这是针对学校的。是的，对于这个特定的实验室，我需要在 Swing 上使用 AWT。此外，我们使用的文本具有我们用于开发应用程序的特定代码。当然，这并不总是最好的方法，但它是向我介绍我可以使用的各种技术的好方法。一旦我的实验室完成，我的导师将通过比较工具运行我的代码和“正确”代码（根据教科书），以确保它们匹配。这让我在效率方面基本上没有灵活性。
感谢您的信息。那么你当然需要遵守要求，认为这确实让我质疑你的导师的判断。祝你好运！
【参考方案1】：
在该语句之前的循环中有一个小错字。看看你能不能发现它：
for (int i=1; 1<=3; i++) // 1, 2, 3
keypad.add(keys[i]);
你的测试是1<=3，这总是正确的，因为1总是小于或等于3。我想你的意思是：
for (int i=1, i<3; i++)
keypad.add(keys[i]);
【讨论】：
正确。我在发布后立即注意到它，并试图在任何人花费太多时间之前发布答案。只要允许我，我就会将您的答案标记为正确。感谢您的宝贵时间！【参考方案2】：我发布问题的那一刻，我就看到了我的问题。在我的问题之前的行中，我输入了：
for (int i=1; 1<=3; i++) // 1, 2, 3
keypad.add(keys[i]);
当我打算打字时：
for (int i=1; i<=3; i++) // 1, 2, 3
keypad.add(keys[i]);
1
再次感谢。
【讨论】：
以上是关于编译问题：无法访问的语句的主要内容，如果未能解决你的问题，请参考以下文章