Java中的Atoi用于负值

Question

我正在用 Java 编写一个 Atoi 函数。对于 +ve 整数，它运行良好。但是我想要的是当我输入一个负整数时它应该给我一个错误。所以我尝试在我的 Atoi 类中包含 continue 语句。实现的类是：

class Atoi {

    int atoi(String tmp) {

    int result = 0;

        for (int i = 0; i < tmp.length(); i++) {

            char digit = (char)(tmp.charAt(i) - '0');

        if(digit == '-')

        continue;
        }

        else {

            result += (digit * Math.pow(10, (tmp.length() - i - 1)));
        }

    return result;

    }
}   

但不幸的是，它给了我这个字符的负数，即 -12 它给了我 655312！帮助。

编辑：假设我需要检查浮点数我应该怎么做？如果我输入 12.1 或 123.2 它应该分别返回 12.1 和 123.2 ！！

Answer 1

您应该给出错误而不是 continue（抛出异常，return -1 或您所说的“给出错误”）。

如果您想忽略-，您可以将 else 子句更改为：

result = digit + result * 10;

Answer 2

快速修复明显问题：逻辑顺序错误...

代替

    char digit = (char)(tmp.charAt(i) - '0');
    if(digit=='-')
    continue;

试试

    char origChar=tmp.charAt(i);
    if(origChar=='-')
        continue;
    char digit = (char)(origChar - '0');

但是还有两个问题：

• 如果存在“-”字符，它不会否定该值！
• 如果这是输入字符串：-1-2-3-4-5，该怎么办？结果会很有趣！ 编辑：也试试这个输入：'répa'...更有趣的结果！

不要忘记使用不正确的输入进行测试，正如@Klaus 建议的那样，如果向函数提供了不正确的输入，请不要犹豫抛出异常（最好是IllegalArgumentException）并提供正确的错误消息...

Answer 3

如果这不是作为编程练习来完成的，还有一个更简单的解决方案：

  static int atoi(String tmp)
  {
      int result = Integer.parseInt(tmp);
      if(result >= 0) {
         return result;
      } else {
         throw new IllegalArgumentException("Negative string "+"\"" + tmp + "\"");
      }
  }

在否定结果情况下替换适当的异常或其他操作。如果您只想忽略 '-'，就像在发布的代码中一样，请将 if-then-else 替换为：

      return Math.abs(result);

此代码还会对“abc”等字符串引发异常。

更一般地说，如果一个库方法不能完全按照您的要求做，通常很容易在修改其行为的方法中使用它，而不是重新编写它。

Answer 4

当然，您可以编写这样的代码，但您需要检查 tmp 是否为有效数字。

int atoi(String tmp) {

    int result = 0;

    int factor = tmp.charAt(0) == "-" ? -1 : 1;

        for (int i = 0; i < tmp.length(); i++) {

            if (tmp.chatAt(i) < '0' ||  tmp.chatAt(i) > '9') 

                continue;

                char digit = (char)(tmp.charAt(i) - '0');

                result += (digit * Math.pow(10, (tmp.length() - i - 1)));
        }

        return result * factor;
}

Answer 5

if(digit=='-')

有

(char)(tmp.charAt(i)

你的代码假设没有 -'s

(char)(tmp.charAt(i) - '0');

是一种盲目地将“数字”变量限制为数字的优化。

您需要逐步了解您的代码实际执行的操作，搜索 ASCII 图表并完成减法 '0' 的作用（'0' == 48），因此 '1' (49) - '0 ' (48) = 1 等等...

Answer 6

如果您不想转换负数，那么只要遇到- 符号就返回0，而不是进一步循环。将此代码放在 if-else 块之前。

     if(tmp.charAt(i)=='-')  
         return 0;