假设我是一位主管
有人传了一份神秘履历给我
里面附了这个 github repo
先随便打开一个专案
[ https://github.com/jim55167/BMI-Calculator/commits/master ]
整体看起来唯一的特色就是 `vanilla js, 没 jQuery`
但冒出了几个疑问
1. 为什么 .vscode 也在 repo 里,
这有需要放在 git 做版本控管吗?
2. 专案都只有 3~5 个 commit,
看不出做这专案的过程与思路
3. 很长一包的 if else 很难阅读, 快有一点波动拳的味道 (nested if hell)
[ https://github.com/jim55167/BMI-Calculator/blob/master/js/main.js#L29-L92 ]
if (tall !== '' && weight !== '') {
... (60 line)
else {
alert('please take a note');
}
其他专案
4. 在 js 里面塞一包 html 字串的动作, 阅读性差
[ https://github.com/jim55167/Mask-Map/blob/master/js/main.js#L86-L88 ]
5. 既然有提到学了 vue, es6 应该也会一点吧
[ https://github.com/jim55167/Mask-Map/blob/master/js/main.js ]
但在这还大量的使用 var 的原因是什么 ?
不会连 var let const 都搞不清楚吧 ?
6. `tab` `space` 混杂
[ https://github.com/jim55167/Taipei-Travel/blob/master/index.html#L22-L33 ]
7. 随便点点看看 很多细节都非常的粗糙