美文网首页TDD(测试驱动开发)
Switch语句这个坏味道,到底坏在哪里?

Switch语句这个坏味道,到底坏在哪里?

作者: 袁慎建 | 来源:发表于2020-05-29 19:56 被阅读0次

    在老马的《重构》第1版中,将Switch Statement定义成一个代码坏味道。而且里面提到一句:

    大多数时候,一看到Switch语句,你就应该考虑以多态替换它。

    我最初看到这个描述的时候,我持有一点疑虑的。你仔细想,如果Switch Statement是一个坏味道的话,那程序中的条件分支(if-else)也是一种坏味道,因为if-else和switch从本质上都是条件分支。然而,if-else没有被公认为一种代码坏味道,而且在程序中不可避免会用到这个。

    在《重构》第2版中,老马直接将这个坏味道改成 Repeated Switches,并做了澄清:

    因为在20世纪90年代末,程序员太过于忽视多态的价值,我们希望矫枉过正。如今的程序员已经更多地使用多态,switch语句也不像15年前那样有害无益,很多语言支持更复杂的switch语句,而不只是根据基本类型值来做判断条件。因此。我们现在更关注重复的Switch

    实际上**重复才是罪魁祸首,重复会让代码修改起来更加困难。在第一版中,他其实也提到这点:

    从本质上讲,switch语句的问题在于重复,你会经常发现同样的switch语句散布在不同的地方。如果为它添加一个新的case字句,就必须找到所有switch语句并修改它们。

    作为重构的初学者,当你看到wwitch坏味道的时候,要抓住重复这个关键点,而且重复的switch有可能以if-else体现出来。只要有相同的条件判断值在多处出现,就散发出Repeated Switch的味道。没必要的重复的代码,是你应该要提高警惕的地方,接下来可能要想点办法消除它,比如使用多态来取代。

    这里附上一个重复Switch的案例:

    案例一:

    public class Item {
    
        private String name;
    
        private int sellIn;
    
        private int quality;
    
        public Item(String name, int sellIn, int quality) {
            this.name = name;
            this.sellIn = sellIn;
            this.quality = quality;
        }
    
        @Override
        public String toString() {
            return this.name + ", " + this.sellIn + ", " + this.quality;
        }
    
        void update() {
            updateQuality();
    
            updateSellIn();
    
            if (isExpired()) {
                updateQualityAfterExpired();
            }
        }
    
        private boolean isExpired() {
            return sellIn < 0;
        }
    
        private void updateQuality() {
            if (isAgedBrie()) {
                if (quality < 50) {
                    quality = quality + 1;
                }
                return;
            }
            if (isBackstagePass()) {
                increaseQuality();
                if (sellIn < 11) {
                    increaseQuality();
                }
                if (sellIn < 6) {
                    increaseQuality();
                }
                return;
            }
            if (isSulfuras()) {
                return;
            }
            if (quality > 0) {
                quality = quality - 1;
            }
        }
    
        private void updateSellIn() {
            if (isSulfuras()) {
                return;
            }
            sellIn = sellIn - 1;
        }
    
        private void updateQualityAfterExpired() {
            if (isAgedBrie()) {
                increaseQuality();
                return;
            }
            if (isBackstagePass()) {
                quality = 0;
                return;
            }
            if (isSulfuras()) {
                return;
            }
            if (quality > 0) {
                quality = quality - 1;
            }
        }
    
        private void increaseQuality() {
            if (quality < 50) {
                quality = quality + 1;
            }
        }
    
        private boolean isSulfuras() {
            return name.equals("Sulfuras, Hand of Ragnaros");
        }
    
        private boolean isBackstagePass() {
            return name.equals("Backstage passes to a TAFKAL80ETC concert");
        }
    
        private boolean isAgedBrie() {
            return name.equals("Aged Brie");
        }
    }
    

    案例二:

    public class CheckInSystem {
        private Map<String, String> checkInRecords = new HashMap<>();
    
        public  boolean checkIn(String fingerprint){
            Employee employee = EmployeeRepository.query(fingerprint);
            int type = employee.getType();
            String record;
            switch (type) {
                case Employee.ENGINEER:
                    record = "I am an Engineer, My Name is" + employee.getName();
                    break;
                case Employee.SALESMAN:
                    record = "I am a Salesman, My Name is" + employee.getName();
                    break;
                case Employee.MANAGER:
                    record = "I am a Manager, My Name is" + employee.getName();
                    break;
                default:
                    record = "";
            }
            if (checkInRecords.isEmpty()) {
                return false;
            }
            checkInRecords.put(fingerprint, record);
    
            return true;
        }
    }
    
    public class Employee {
        static final int ENGINEER = 0;
        static final int SALESMAN = 1;
        static final int MANAGER = 2;
    
        private int type;
        private String name;
        private int monthlySalary;
        private int commission;
        private int bonus;
    
        public Employee(int type) {
            this.type = type;
        }
    
        public int getType() {
            return type;
        }
    
        public String getName() {
            return name;
        }
    
        public int payAmount() {
            switch (type) {
                case ENGINEER:
                    return monthlySalary;
                case SALESMAN:
                    return monthlySalary + commission;
                case MANAGER:
                    return monthlySalary + bonus;
                default:
                    throw new RuntimeException("Invalid employee");
            }
        }
    }
    

    以上代码,你找到了重复的Switch了吗?

    相关文章

      网友评论

        本文标题:Switch语句这个坏味道,到底坏在哪里?

        本文链接:https://www.haomeiwen.com/subject/iomfzhtx.html