OCLint的部分规则(Basic 部分)

OCLint的部分规则(Basic 部分)

对OCLint的部分规则进行简单翻译解释,有部分进行了验证以及进一步分析、测试。OCLint其他相关内容如下:

- -
OCLint-iOS-OC项目几种简单使用 OCLint的部分规则(Basic 部分)
OCLint的部分规则(Unuseed 部分) OCLint的部分规则(Size 部分)
OCLint的部分规则(Redundant 部分) OCLint的部分规则(Naming 部分)
OCLint的部分规则(Migration 部分) OCLint的部分规则(Empty 部分)
OCLint的部分规则(Design 部分) OCLint的部分规则(Convention 部分)
OCLint的部分规则(CoCoa 部分)



1、bitwise operator in conditional

      since:0.6 定义类传送门~点击

Checks for bitwise operations in conditionals. Although being written on purpose in some rare cases, bitwise opera- tions are considered to be too “smart”. Smart code is not easy to understand.

简单解释:OCLint认为位运算符可读性不好

    void example(int a, int b) {
        if (a | b) {
        }
        if (a & b) {
        }
     }

2、broken null check

      since:0.7 定义类传送门~点击

The broken null check itself will crash the program.

简单解释:错误的null检查会导致程序crash

    void m(A *a, B *b) {
        if (a != NULL || a->bar(b))
        {
        }
        if (a == NULL && a->bar(b))
        {
        }
    }   

PS:OC(objective-c)向空对象可以发消息,不会发生crash,但是上面这样的代码的逻辑是错误的。

3、broken nil check

      since:0.7 定义类传送门~点击

The broken nil check in Objective-C in some cases returns just the opposite result.

简单解释:Objective-C中判空,有些情况下会返回预期相反的结果

    + (void)compare:(A *)obj1 withOther:(A *)obj2 {
        if (obj1 || [obj1 isEqualTo:obj2])
        {
        }
        if (!obj1 && ![obj1 isEqualTo:obj2])
        {
        }
    }

PS:这样写应该是逻辑错误

4、broken oddness check

      since:0.6 定义类传送门~点击

Checking oddness by x % 2 == 1won’t work for negative numbers. Use x & 1 == 1,or x % 2 != 0 instead.

简单解释:对数字奇数性进行检查(x % 2 == 1),如果是负数,会有异常,应使用x & 1 == 1或者x % 2 != 0

    void example() {
        if (x % 2 == 1)         // violation
        {
        }
        if (foo() % 2 == 1)     // violation
        {
        }
    }

PS:在Objective-C 上的测试结果上看,并没有什么大的异常。应该主要的问题是负数进行取模区的结果不会出现正值。
比如(x % 2)的结果是-1或者0

5、collapsible if statements

      Since: 0.6 定义类传送门~点击

This rule detects instances where the conditions of two consecutive if statements can be combined into one in order to increase code cleanness and readability.

简单解释:两个连续if条件可以合并的应该合并,可以提高可读性和代码整洁。

    void example(bool x, bool y){
        if (x)              // these two if statements can be
        {
            if (y)          // combined to if (x && y)
            {
                foo();
            }
        }
    }

6、constant conditional operator

      Since: 0.6 定义类传送门~点击

conditional operator whose conditionals are always true or always false are confusing.

简单解释:检查始终为true或者false的操作,会使人疑惑。

    void example() {
        int a = 1 == 1 ? 1 : 0;     // 1 == 1 is actually always true
    }

7、constant if expression

      Since: 0.2 定义类传送门~点击

if statements whose conditionals are always true or always false are confusing.

简单解释:if始终为true或者false的操作,会使人疑惑。

    void example(){
        if (true) {      // always true
            foo();
        }
        if (1 == 0) {    // always false
            bar();
        }
    }

8、 dead code

      Since: 0.4 定义类传送门~点击

Code after return, break, continue, and throw statements is unreachable and will never be executed.

简单解释:在 returnbreakcontinue and throw 之后的代码都是无效的

    void example(id collection) {
        for (id it in collection) {
            continue;
            int i1;                 // dead code
        }
        return;
        int i2;                     // dead code
    }

9、double negative

      Since:0.6 定义类传送门~点击

There is no point in using a double negative, it is always positive.

简单解释:代码中双重否定没有意义

    void example() {
        int b1 = !!1;
        int b2 = ~~1;
    }

10、for loop should be while loop

      Since:0.6 定义类传送门~点击

Under certain circumstances, some for loops can be simplified to while loops to make code more concise.

简单解释:在有些情况下使用while比使用for更简洁

    void example(int a) {
        for (; a < 100;) {
            foo(a);
        }
    }

11、 goto statement

      Since:0.6 定义类传送门~点击

Go To Statement Considered Harmful

简单解释:go to被认为有害的

    void example() {
        A:
            a();
        goto A;     // Considered Harmful
    }

12、jumbled incrementer

      Since:0.7 定义类传送门~点击

Jumbled incrementers are usually typos. If it’s done on purpose, it’s very confusing for code readers.

简单解释:混乱的迭代器通常会出现错误,可读性不好。

    void aMethod(int a) {
        for (int i = 0; i < a; i++) {
            for (int j = 0; j < a; i++) {     // references both 'i' and 'j'
            }
        }
    }

13、misplaced null check

      Since:0.7 定义类传送门~点击

The null check is misplaced. In C and C++, sending a message to a null pointer could crash the program. When null is misplaced, either the check is useless or it’s incorrect.

简单解释:CC++在条件判断的时候注意null判断放在前面,逻辑短路

    void m(A *a, B *b) {
        if (a->bar(b) && a != NULL) { // violation 
        }
        if (a->bar(b) || !a) {        // violation
        }
    }

14、misplaced nil check

      Since:0.7 定义类传送门~点击

The nil check is misplaced. In Objective-C, sending a message to a nil pointer simply does nothing. But code readers may be confused about the misplaced nil check.

简单解释:Objective-C在条件判断的时候注意nil判断放在前面,逻辑短路

    + (void)compare:(A *)obj1 withOther:(A *)obj2 {
        if ([obj1 isEqualTo:obj2] && obj1) {
        }
        if (![obj1 isEqualTo:obj2] || obj1 == nil)   {
        }
    }

15、 multiple unary operator

      Since:0.6 定义类传送门~点击

Multiple unary operator can always be confusing and should be simplified.

简单解释:多重的运算符不易理解,应该简化。

    void example() {
        int b = -(+(!(~1)));
    }

16、return from finally block

      Since:0.6 定义类传送门~点击

Returning from a finally block is not recommended.

简单解释:不建议在finally中返回

    void example() {
        @try {
            foo();
        }
        @catch(id ex) {
            bar();
        }
        @finally {
            return;         // this can discard exceptions.
        }
    }

17、throw exception from finally block

      Since:0.6 定义类传送门~点击

Throwing exceptions within a finally block may mask other exceptions or code defects.

简单解释:从finally块抛出异常,可能忽略其他的错误

    void example() {
        @try {;}
        @catch(id ex) {;}
        @finally {
            id ex1;
            @throw ex1;                              // this throws an exception
            NSException *ex2 = [NSException new];
            [ex2 raise];                             // this throws an exception, too
        }
    }
發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章