StringBuilder没有打印它的值?

时间:2015-07-16 00:16:02

标签: java string for-loop int stringbuilder

我是java的菜鸟,所以如果这是一个简单的问题,我很抱歉。

我总觉得有趣的是,你可以通过将数字加在一起将数字压缩成一个数字。因此,我决定尝试为我做一个程序!这是一个例子。

  

输入: 557

     

5 + 5 + 7 = 17

     

1 + 7 = 8

     

答案: 8

请参阅!这显然适用于任何数字。但是,我的程序终止没有输出。谁能帮我吗?我不习惯tringBuilder,所以我认为这可能是问题。

import java.util.Scanner;
import java.lang.StringBuilder;

public class MagicNumberApp
{
    public static void main (String [] args)
    {
        int number;
        String numberstring;
        boolean keepGoing = false;

        Scanner input = new Scanner(System.in);
        StringBuilder builder = new StringBuilder();

        sopl("Welcome to Magic Number! \nThe idea is to add the idividual digits of a number "
                + "\nuntil it is condensed into a one digit number.\n\nInput a number...");
        sop(">");

        number = input.nextInt();

        numberstring = Integer.toString(number);

        if (numberstring.length() < 1)
            keepGoing = true;

        sopl("");

        number = 0;

        while (keepGoing)
        {
            for (int i = 0; i < numberstring.length(); i++)
            {
                number += Character.getNumericValue(numberstring.charAt(i));
                builder.append("+" + numberstring.charAt(i) + " ");
            }
            builder.append("=" + number);
            sopl(builder);

            if (numberstring.length() > 1)
            {
                numberstring = Integer.toString(number);
                number = 0;
                sopl("");
            }
            else
            {
                keepGoing = false;
            }
        }

    }

    public static void sop (Object o)
    {
        System.out.print(o);
    }

    public static void sopl (Object o)
    {
        System.out.println(o);
    }

}

3 个答案:

答案 0 :(得分:2)

您的keepGoing逻辑是倒退的。您将keepGoing设置为true,输入的数字少于1位,您将其初始化为false

if (numberstring.length() < 1)
    keepGoing = false;

所有数字至少有一位数,甚至0,所以在while循环之前不需要进行上述测试。去掉它。但您必须将keepGoing初始化为true

答案 1 :(得分:1)

我认为问题出在这一行:

if (numberstring.length() < 1)
            keepGoing = true;

现在它说keepGoing只有在numberstring的长度为0时才会为真。您可以将其更改为:

if (numberstring.length() > 1)
                keepGoing = true;

编辑:我还有其他建议。如果用户输入有一个数字,则可以添加else语句来打印消息:

else
    sopl(number + " only has one digit. Try again!");

答案 2 :(得分:0)

除了所有的答案。我建议您将输入读作String并保存自己初始转换为字符串。

    numberstring = input.nextLine();