一个工作了 6 年的同事写的代码，不看 git 记录我还以为是实习生写的

[garlics]

第一次见到那么离谱的代码，完全没有封装的意思。

        if ($config['theme'] == 1) {
            if ($config['plugin']['PG_SHOPPING_CART']) {
                $tabIndex = [
                    'index' => 0,
                    'collection' => 1,
                    'bestforyou' => 2,
                    'cart' => 3,
                    'mine' => 4
                ];
                $tablist = [
                    [
                        'pagePath' => '/pages/index/index',
                        'text' => '首页',
                        'iconPath' => '/assets/images/home_unactived.png',
                        'selectedIconPath' => '/assets/images/theme1_home_actived.png',
                    ],
                    [
                        "pagePath" => "/pages/collection/collection",
                        "iconPath" => "/assets/images/collection.png",
                        "selectedIconPath" => "/assets/images/collection_actived.png",
                        "text" => "收藏"
                    ],
                    [
                        "pagePath" => "/pages/bestforyou/bestforyou",
                        "iconPath" => "/assets/images/bestforyou_unactived.png",
                        "selectedIconPath" => "/assets/images/bestforyou_actived.png",
                        "text" => "为你优选"
                    ],
                    [
                        'pagePath' => '/pages/cart/cart',
                        'text' => '购物车',
                        'iconPath' => '/assets/images/cart_unactived.png',
                        'selectedIconPath' => '/assets/images/cart_actived.png',
                    ],
                    [
                        'pagePath' => '/pages/mine/mine',
                        'text' => '我的',
                        'iconPath' => '/assets/images/mine_unactived.png',
                        'selectedIconPath' => '/assets/images/mine_actived.png',
                    ]
                ];
            } else {
                $tabIndex = [
                    'index' => 0,
                    'collection' => 1,
                    'bestforyou' => 2,
                    'mine' => 3
                ];
                $tablist = [
                    [
                        'pagePath' => '/pages/index/index',
                        'text' => '首页',
                        'iconPath' => '/assets/images/home_unactived.png',
                        'selectedIconPath' => '/assets/images/theme1_home_actived.png',
                    ],
                    [
                        "pagePath" => "/pages/collection/collection",
                        "iconPath" => "/assets/images/collection.png",
                        "selectedIconPath" => "/assets/images/collection_actived.png",
                        "text" => "收藏"
                    ],
                    [
                        "pagePath" => "/pages/bestforyou/bestforyou",
                        "iconPath" => "/assets/images/bestforyou_unactived.png",
                        "selectedIconPath" => "/assets/images/bestforyou_actived.png",
                        "text" => "为你优选"
                    ],
                    [
                        'pagePath' => '/pages/mine/mine',
                        'text' => '我的',
                        'iconPath' => '/assets/images/mine_unactived.png',
                        'selectedIconPath' => '/assets/images/mine_actived.png',
                    ]
                ];
            }
        } else {
            if ($config['plugin']['PG_SHOPPING_CART']) {
                $tabIndex = [
                    'index' => 0,
                    'cart' => 1,
                    'mine' => 2
                ];
                $tablist = [
                    [
                        'pagePath' => '/pages/index/index',
                        'text' => '首页',
                        'iconPath' => '/assets/images/home_unactived.png',
                        'selectedIconPath' => '/assets/images/home_actived.png',
                    ],
                    [
                        'pagePath' => '/pages/cart/cart',
                        'text' => '购物车',
                        'iconPath' => '/assets/images/cart_unactived.png',
                        'selectedIconPath' => '/assets/images/cart_actived.png',
                    ],
                    [
                        'pagePath' => '/pages/mine/mine',
                        'text' => '我的',
                        'iconPath' => '/assets/images/mine_unactived.png',
                        'selectedIconPath' => '/assets/images/mine_actived.png',
                    ]
                ];
            } else {
                $tabIndex = [
                    'index' => 0,
                    'mine' => 1
                ];
                $tablist = [
                    [
                        'pagePath' => '/pages/index/index',
                        'text' => '首页',
                        'iconPath' => '/assets/images/home_unactived.png',
                        'selectedIconPath' => '/assets/images/home_actived.png',
                    ],
                    [
                        'pagePath' => '/pages/mine/mine',
                        'text' => '我的',
                        'iconPath' => '/assets/images/mine_unactived.png',
                        'selectedIconPath' => '/assets/images/mine_actived.png',
                    ]
                ];
            }
        }

[BeFun]

通俗易懂，我觉得挺好，至少自己三个月后再来看，还能一眼看懂

[stach]

不管是实习生, 还是资深工程师代, 码逻辑清晰是业务系统最关键的地方. 当然楼主可以优化代码组织方式, 封装一下, 我相信改起来没有任何压力 (这代码蛮清晰的嘛, 没有暗坑).

不过要提醒的是, 楼主也不要过度封装, 没有哪种模式是普适的, 过度封装的副作用更大.

[chenyu0532]

记得前几年有一个巨火的手游，用 cocos2d 写的，被别人破解了。发现里面极其多的 if elseif，一层层的包着，看着都头大。。但是人家就是很火。。
个人认为，如果为了自己的代码精进，那相信在座的各位的代码都会优化；如果侧重点在业务上，就可以怎么快怎么来，自己和别人看得懂。
以前我也觉得代码优化最重要。。。但是慢慢觉得业务 /策略 /想法才是最重要的。。

[tydl]

工整、清晰，一目了然。不建议封装~~

[dustinth]

封装意味着多一层抽象, 会多一层心智负担, 所以需要有代码清晰度的提高或者减少重复来作为补偿(比较明显的结果就是封装后代码行数明显变短). 楼主同事的代码没什么大问题, 更谈不上"离谱", 经过优化后代码并没有明显的代码量的降低.

[coderljx]

@kera0a 哈哈，很现实了

[Varobjs]

这不算什么，毕竟 if else 嵌套的只是简单赋值；

你是没见过 很多个 if else 结构，每个包含数百行业务逻辑

然后各个业务逻辑 80-90%是相似的，

[a194259440]

面向过程得通俗易懂，就能说明那代码还挺好得？面向对象学的啥，我更赞同这代码不是一次写成得，导致多层嵌套了 ifelse 语句

[weichengwu]

写成这样了叫逻辑清晰？如果 if 代码块里面的行数超过一屏能显示的长度，逻辑是否还清晰？

[huihuilang]

建议楼主把自己的代码贴出来。。另外同事代码写得好写的不好有啥关系？自己拿自己的工资，干好自己的事情就行

[garlics]

@huihuilang 因为现在我在维护这个代码

[FallenTy]

把 tableList 里面的东西封装下，根据条件直接引用观感好很多，这也能能有人吐槽事逼？还是说你们成天就是写这种代码。不过这不算太离谱的代码，起码逻辑清晰，你同事估计就是犯懒不想优化，顺便还显得代码多

[huihuilang]

@garlics 那是够惨的。。。

[beingbin]

6 年经验×
1 年经验重复 6 年√

[tulongtou]

@weichengwu 这说明该换高分辨率的显示屏了 （手动狗头

[Shing]

莫笑前辈，总有一天你会经历一样的事情的。

[Felldeadbird]

楼主，如果你不是管理的话，那么这代码是正常的。
如果你是管理的话，push 内容时，你应该审查代码。

所以，楼主不要按工龄看代码。这个完全看个人态度的。 我有时候也会写这种代码，因为心情烦躁 + 赶工期。一般我会注明 @todo 形式，为何要这么写。

[cmdOptionKana]

@weichengwu 这个代码最开始可能很短的，后来加着加着变成这么长，如果继续加需求发现代码已经变得很不合理（比如楼主现在就刚好是这个阶段），此时可以重构。

反正感觉问题不是很大的时候就按简单快速的方法先写，可以等后面眼看越来越不合理了再改。（有时等不到那个时候项目就黄了）

[dustinth]

再看了一下楼主的补充, 自己也说了有"新"的需求, 新需求带来代码或者结构的变化是正常现象. 以我多年编程经验来看, 过早的优化比不优化其实危害更大, 不优化大不了重写一部分逻辑, 过早优化改的时候还得把所有优化过的逻辑拆开重装.

[dethan]

各位天天各种研究算法 结果遇到这种代码 还说好????