链表实现中的 NumberFormatException

发布于 2024-12-20 02:57:40 字数 664 浏览 1 评论 0原文

我用Java实现了一个循环列表。该代码要求输入 int 值，我希望用 "END" 终止输入列表。该代码可以工作，但会引发运行时异常：NumberFormatException。

try{
    while(true){
        newnode=new Node();
        oldnode.next=newnode;
        newnode.prev=oldnode;               
        System.out.print("Enter value:");
        try{
            ctrlstr=bfr.readLine();
        }
        catch(Exception ex){
            ex.printStackTrace();
        }
        if (ctrlstr=="END") break;
        newnode.val=Integer.parseInt(ctrlstr);
        oldnode=newnode;                        
        i++;    
    }
}
catch(Exception ex){
    ex.printStackTrace();
}

原文

I implemented a circular list in Java. The code asks for the int values to be entered, and I wish to terminate the input list with an "END". The code works but throws a runtime exception: NumberFormatException.

try{
    while(true){
        newnode=new Node();
        oldnode.next=newnode;
        newnode.prev=oldnode;               
        System.out.print("Enter value:");
        try{
            ctrlstr=bfr.readLine();
        }
        catch(Exception ex){
            ex.printStackTrace();
        }
        if (ctrlstr=="END") break;
        newnode.val=Integer.parseInt(ctrlstr);
        oldnode=newnode;                        
        i++;    
    }
}
catch(Exception ex){
    ex.printStackTrace();
}

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

薄暮涼年 2024-12-27 02:57:40

在这里：

if (ctrlstr=="END")

您使用 == 比较字符串，它始终检查引用标识。相反，您应该使用 equals:

if (ctrlstr.equals("END"))

ctrlstr 为 null 时您只想得到 false 结果）

if ("END".equals(ctrlstr))

一些额外的注意事项：

或者也许（如果当code>Exception 通常是一个坏主意 - 您应该捕获更具体的异常
捕获异常，然后在打印后继续无论如何通常是一个坏主意
如果您适当缩进，您的代码将更具可读性
您的代码将更具可读性如果您总是使用if 语句等的大括号

Here:

if (ctrlstr=="END")

you're comparing strings using ==, which always checks for reference identity. Instead, you should use equals:

if (ctrlstr.equals("END"))

or perhaps (if you want just a false result when ctrlstr is null)

if ("END".equals(ctrlstr))

A few extra notes:

Catching Exception is usually a bad idea - you should catch more specific exceptions
Catching an exception and then continuing anyway after printing it is usually a bad idea
Your code will be more readable if you indent appropriately
Your code will be more readable if you always use braces for if statements etc

回复收藏 0 原文