• checkstyle简单使用说明


    checkstyle对检查代码规范问题的总结,虽然还不够只能,但已经比较强大。
    1.Cyclomatic Complexity is X (max allowed is X).
    问题说明:圈复杂度过高。所反映的是“判定条件”的数量。圈复杂度大说明程序代码可能质量低且难于测试和维护。
    修改建议:降低判定条件的数量。对方法进行拆分。


    2.Nested if-else depth is X (max allowed is X).
    问题说明:if-else嵌套太多
    修改建议:把嵌套的逻辑用一个分支或多个分支来实现


    3.Line matches the illegal pattern 'X'.
    问题说明:含有非法字符,一般是system类的东西,程序提交时进行删除。


    4.Must have at least one statement
    问题说明:空语句。必须添加业务逻辑。有时每行后不小心添加一个【;】,或者异常处理没有进行。


    5.Empty statement
    问题说明:空语句。必须添加业务逻辑。


    6.Literal Strings should be compared using equals(), not 'X'.
    问题说明:字符串比较不能用==,用equals


    7.switch without "default" clause.
    问题说明:switch语句判断没有default的情况处理。


    8.Found duplicate of X lines in X, starting from line X
    问题说明:包含有重复的语句,应该提取公共方法。


    9.Assignment of parameter 'X' is not allowed.
    问题说明:不允许对函数的参数进行修改。


    10.'X' is a magic number.
    问题说明:对于直接用代码中的数字,进行常量的定义。


    11.Missing a Javadoc comment.
    问题说明:缺少javadoc的注释。


    12.Expected X tag for 'X'.
    问题说明:注释中缺少对变量,异常,返回值的说明


    13.Name 'X' must match pattern 'X'.
    问题说明:变量不符合命名规则


    14.其他的都比较好理解。对于类,方法长度的控制,注释中各参数的说明等。
    问题说明:比较难于修改的为圈复杂度过高,以及嵌套if-else的问题,需要根据实际的业务逻辑进行拆分方法,并进行方法参数的传递。
    而对于圈复杂度过高,判断较复杂(if -elseif -elseif ...,switch case较多的判断)修改时还没有合适的方式。

  • 相关阅读:
    彻底解决IE6下Asp.net Ajax客户端框架未能加载
    mootools【二】 Array篇
    SQL Server存储过程编写和优化措施
    Calendar ——漂亮的MooTools 日历插件
    免费使用和 Ajax & Javascript 操纵和显示图片的代码
    26步获得网站流量
    SEO中的十个“没必要”
    如何选择关键字
    建设一个成功的网站
    jQuery 编辑器(转载)
  • 原文地址:https://www.cnblogs.com/pureEve/p/6667447.html
Copyright © 2020-2023  润新知