1

我最近在 Eclipse 中设置了查找错误以查看它生成的报告。我已将所有设置设置为尽可能敏感。如果我创建一个写入文件的小型应用程序并且不关闭流它会拾取它,这一切都很好。

但是,使用一个已经编写的项目,我们没有几个错误,特别是在输出中,我们根本没有错误(就查找错误而言)

我想知道是否有人可以通过他们的版本运行它并报告我是否发现错误设置不正确,或者实际上它是否找不到错误?

import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;


public class SpellUtil {

    private final static String teenSpelling[] = {"Zero", "One", "Two", "Three",
        "Four", "Five", "Six", "Seven", "Eight", "Nine", "Ten", "Eleven",
        "Twelve", "Thirteen", "Fourteen", "Fifteen", "Sixteen",
        "Seventeen", "Eighteen", "Nineteen"};

    private final static String centSpelling[] = {"Twenty", "Thirty", "Forty",
        "Fifty", "Sixty", "Seventy", "Eighty", "Ninety"};

    private final static String suffixSpelling[] = {
        "", // Dummy! no level 0 (added for nicer indexing in code)
        "", // Nothing for level 1
        " Thousand, ", " Million, ", " Billion, ", " Trillion, ", " Quadrillion, ",
        " Quintillion, "};



    public static String spell(int number) {

        int rem, placeIndicator = 1;
        boolean isNegative = false;
        List<String> spelling = new ArrayList<String>();

        if (number < 0) {
            isNegative = true;
            number = Math.abs(number);
        }

        while (number > 0) {
            rem = number % 1000;
            number = number / 1000;

            spelling.add(suffixSpelling[placeIndicator]);

            try {
                spelling.add(spellBelow1000(rem));
            } catch (SpellingException e) {
                System.out.println(e.getMessage());
            }

            placeIndicator++;
        }

        StringBuilder sb = new StringBuilder();
        if (isNegative) sb.append("Minus ");
        for (int i = spelling.size() - 1; i >= 0; i--) {
            sb.append(spelling.get(i));
        }

        return sb.toString();
    }

    private static String spellBelow1000(int number) throws SpellingException {

        if (number < 0 || number >= 1000)
            throw new SpellingException("Expecting a number between 0 and 999: " + number);

        if (number < 20) {
            // if number is a teen,
            // find it in teen table and return its equivalent text (word).
            return teenSpelling[number];
        } else if (number < 100) {
            // otherwise, if it is a cent,
            // find the most (div) and least (rem) significant digits (MSD/LSD)
            int div = (int) number / 10;
            int rem = (int) number % 10;

            if (rem == 0) {
                // if LSD is zero, return the cent key word directly (like
                // fifty).
                return centSpelling[div-2];
            } else {
                // otherwise, return the text as cent-teen (like fifty-one)
                return centSpelling[div-2] + "-" + teenSpelling[rem];
            }
        } else {
            // otherwise, it is a mil;
            // find it's MSD and remaining cent.
            int div = number / 100;
            int rem = (int) number % 100;  // TODO will findbugs detect unnecessary (int)?

            // Prepare the mil prefix:
            String milText = teenSpelling[div] + " Hundred";

            // decide whether to append the cent tail or not.
            if (rem == 0) {
                // if it does have a non-zero cent, that's it.
                // return the mil prefix, for example three hundred:
                return milText;
            } else {
                // otherwise, spell the cent and append it to mil prefix.
                // (now, rem is a cent).
                // For example, three Hundred and Sixty-Four:
                return milText + " and " + spellBelow1000(rem);
            }
        }
    }
}
4

4 回答 4

2

您期望在此行中找到错误:

int rem = (int) number % 100;  // TODO will findbugs detect unnecessary (int)?

是错误的,因为%运算的结果通常不是整数。

CandC++中,余数运算符只接受整数操作数,但在 Java 中,它也接受浮点操作数。这意味着诸如此类的语句double x = 8.2 % 4;在 Java 中非常有效,并且结果可能是非整数值。(0.1999999999999993在这种情况下)

请在此处查看 Java 语言规范。

于 2012-10-13T12:52:42.223 回答
1

您似乎对 findbugs 配置有问题。我建议通过声纳使用 Findbugs。它更容易配置,并且您可以获得 checkstyle、pmd 以及用于管理和解决违规的系统。

声纳 findbugs 页面

于 2012-10-13T13:19:39.193 回答
1

我认为问题在于您误解了 FindBugs 的功能及其功能。

基本上,FindBugs 解析每个类以生成解析树 - 程序结构的内存表示。然后尝试在树中找到与已知模式匹配的位置,这些模式表示不正确或有问题的编程。例如:

    if (someString == "42") {
        ....
    }

FindBugs 很可能会告诉您使用 '==' 运算符比较字符串是错误的。它所做的是在类中查找运算符为“==”且其中一个或两个操作数为字符串的任何表达式节点。它将针对已编程检测和报告的大量模式重复此过程。有些会比这个更复杂,但基本上,FindBug 只是做结构模式匹配的一种形式。

FindBugs 没有也不能做的是了解您的程序实际上应该做什么。例如:

    public boolean isOdd(int arg) {
        return (arg % 2) == 0;
    }

这对于任何了解简单数学的人来说显然是不正确的……但 FindBugs 不会注意到它。那是因为 FindBugs 不知道该方法实际上应该做什么。此外,它无法进行基本语义分析,以找出代码没有实现数学。


我这样做的原因是因为我需要演示查找错误,并且我需要一个应用程序来生成一些错误来展示它是如何工作的。

也许你需要作弊:

  • 阅读 Findbugs 文档以了解它能够找到的内容。
  • 编写一些带有您知道 Findbugs 可以为您找到的错误的“玩具”应用程序。

包括你知道它不会找到的例子也是值得的......这样你就可以解释 Findbugs 的局限性。

于 2012-10-13T13:50:59.650 回答
0

findbugs 所做的是寻找一些可能(并且很可能会)导致意外/不需要的行为的常见错误。大多数这些错误是技术性的或对 java 及其 API 的错误使用。可以在此处找到所有 findbugs 检查的列表换句话说,如果您以正确的方式做了您不想要的事情,fin​​dbugs 不会检测到它。在您的代码中,我看不到 findbugs 会检测到的任何内容。您在评论中提到的不必要的演员表不是 findbugs 规则,因为它不会改变您的代码的行为。它更多是一种风格或效率错误,会被checkstylePMD等工具检测到。

于 2012-10-13T12:53:39.570 回答