劣质代码的产生原因（2）编程语言知识不足 （续2）

 

3.正则表达式

    a. 白名单

    下列代码是用来进行白名单校验的，即如果输入的文字符合条件的话就返回true,否则返回false。

private static final String[] ENABLED_TEXTS = new String[]{"a", "b", "c", "d", "e"};

public boolean isEnabled(String input) {
    for (String text : ENABLED_TEXTS) {
        if (text.equals(input)) {
            return true;
        }
    }
    return false;
}

    当改成使用正则表达式进行校验的时候，代码变成下面这样：

public boolean isEnabled(String input) {
    return input.matches("a|b|c|d|e");
}

 

4.用异常代替错误号

   有一段报名课程的代码：

public String register(int userId, int lessonId) {
    int ret = checkUser(userId);
    if (ret != 0) {
         return "User does not exist.";
    }

    ret = checkLesson(lessonId);
    if (ret != 0) {
         return "Lesson does not exist.";
    }

    ret = checkLessonExpire(lessonId);
    if (ret != 0) {
         return "Lesson is expired.";
    }

    ret = register(userId, lessonId);
    if (ret != 0) {
         return "Register failed.";
    }

    return "";
}

    其中有若干处用到了ret这个变量，ret的值检查代码使得代码变长，而且业务流程代码距离拉开了，变得不容易理解。

public void register(int userId, int lessonId) throws BusinessException {
    checkUser(userId);
    checkLesson(lessonId);
    checkExpire(lessonId);
    registerLesson(userId, lessonId);
}

    当把上述方法原来的int类型返回值去掉，改成throws BusinessException。修改之后的代码变得紧凑多了。

5.位运算

    计算机表示数据是采用二进制的，几乎所有的计算机培训课程都从二进制开始，但是当写程序到一定程度之后，却忘记了利用二进制的基本运算来简化代码，加快效率。

    a.奇偶运算

     思路1：常见做法是数字除以2求余数

public class OddEven {

    public boolean isEven(int num) {
        return num % 2 == 0;
    }
    
    public static void main(String[] args) {
        OddEven instance = new OddEven();
        System.out.println("3 is " + (instance.isEven(3)?"Even":"Odd"));
    }
}

   思路2：采用位运算之后，代码变成下面这样：

public class OddEven {

    public boolean isEven(int num) {
        return (num & 1) == 0;
    }
    
    public static void main(String[] args) {
        OddEven instance = new OddEven();
        System.out.println("3 is " + (instance.isEven(3)?"Even":"Odd"));
        System.out.println("36 is " + (instance.isEven(3)?"Even":"Odd"));
    }
}

进一步，我们来看一个斑马线的例子。
    思路1的做法：

 

import java.awt.Color;

public class Zembra {

    public Color getZembraColor(int index) {
        if (index % 2 ==0) {
            return Color.GRAY;
        } else {
            return Color.WHITE;
        }
    }
    
    public static void main(String[] args) {
        Zembra zembra = new Zembra();
        System.out.println(zembra.getZembraColor(23).toString());
    }
}

思路2的做法：

import java.awt.Color;

public class Zembra {

    public Color getZembraColor(int index) {
        Color[] colors = new Color[]{Color.GRAY, Color.WHITE};
        return colors[index & 1];
    }
    
    public static void main(String[] args) {
        Zembra zembra = new Zembra();
        System.out.println(zembra.getZembraColor(23).toString());
    }
}

代码行数比思路1短了，并且，代码的执行效率提高了。但是易懂性有所降低。

 b. Bit Mask

  有些时候我们需要有多种不同层级的属性来表征事物的属性，或者对于同一事物的多种可以并存的属性。

  比如：字体的下划线、粗体、斜体。

  如果采用若干个变量来定义各种属性则每种属性都要设立一个变量。随着可以扩展的属性越来越多，所需要的变量也越来越多，将降低代码的可读性，可理解性。

  为了防止这种属性扩大引起的变量增多的麻烦，以及为未来可能的扩展留有余地，可以采用Bit Mask。

  比如：字体来说每一位(bit)表示一种属性。例如从最后一位开始向左，依次为：下划线、粗体、斜体、删除线、双删除线、上标、下标...

   那么想要知道某一位是否被设置可以通过位运算来获取。

   例如：Font.style来定义那么，计算是否为粗体，那么下列算式用来计算。

   Font.style & 2 == 2

   同样，该技术被应用于窗口属性、文本框语种限制等处。

 c. 四则运算

  当需要计算一个中间值的时候，采用的是(a + b) / 2的形式，但是这个算法有一个Bug，当a+b > Integer.Max的时候，会将计算结果变成负数。

  正确的形式应该是 (a + b) >>> 1;

  具体可以参照Java的二分法Bug报告：

     http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=5045582