假設我是一位主管
有人傳了一份神秘履歷給我
裡面附了這個 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. 隨便點點看看 很多細節都非常的粗糙