作者:
yigeo (yigeo)
2020-10-07 19:33:50假設我是一位主管
有人傳了一份神秘履歷給我
裡面附了這個 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. 隨便點點看看 很多細節都非常的粗糙
作者:
final01 (牛頓運動定律)
2020-10-07 19:57:00非本科常常有個問題就是看不出自己的問題，你人真好XD
作者: newhandfun (新手方) 2020-10-07 20:13:00
認真推，我還都沒人幫我看QQ
作者:
bcjohn (bc)
2020-10-07 20:15:00好奇問一下，有時候 vscode 的設定檔每個人設的參數都不一樣，所以如果把 .vscode 加入 git ，統一整個專案的設定應該是合理的吧？
為什麼要大家設定都一樣？傳上去的 code 風格一致就夠了，可以用工具 auto format
作者:
yigeo (yigeo)
2020-10-07 20:49:00我個人的看法是 IDE 的 config 不該出現在 codebase, 除非團隊有共識, 不然每個 IDE/editor 都來一個 config ...
作者:
yigeo (yigeo)
2020-10-07 21:05:00新手很難 因為盲點太多了, 入門後可以靠挖別人的 code誰的 code 優雅就用誰, 但新手無法知道 自己有沒有走錯路方向歪了 自學越久越歪, 除非他是天才 不然很容易走火入魔
新手通常進去不好公司, 別人的 CODE 自然不會太好不要期待遇到好 CODE 轉職到現在, 我一直覺得在見不賢而內自省 連新手都看的出來是 爛CODE 還資深工程師...
.vscode? 這有人問過了 請google github vscodestackoverflow 還是哪 我記得有人討論過了 結論是官方都放了 你愛放不放 隨你
作者:
yigeo (yigeo)
2020-10-07 22:26:00我前面有說 團隊有共識放也沒差 只要說得出合理的原因 都可但這裡的例子 "git.ignoreLimitWarning" 不太認為需要入git
.vscode裡面有debugger設定我就會放進repo裡了
但問題是 根據你前面的分析 你把.vscode有沒有放git考慮在要不要面這個人的選項裡面這樣這個人有機會被問為什麼要放嗎(?)
作者:
yigeo (yigeo)
2020-10-07 22:57:00這是我看完心中的疑惑 如果要面試這個人我會從1問到6
作者:
uioty (uioty)
2020-10-07 22:57:00推這篇 有些非本科系真的沒人幫review品質就不太好 常常說“完成了”結果去用一下他們寫的東西明明很多都沒寫好...後來開始寫自己的project之後才了解，寫給自己用的東西就是會跑就好，一些非本科系這樣過來的人，可能也習慣了「會跑就好」的模式，哪像以前大學的時候還會遇過丟大量測資把你程式玩爆的助教
作者:
yigeo (yigeo)
2020-10-07 22:58:00如果 1~6 都能有合理的答覆 我覺得也是能接受的但殘酷的事實是 看完了有1~6的疑惑通常就不會想找他來面試另外如果只有兩三個疑惑 且有亮點 應該就會想找來聊聊(但我不是面試官啊 我只是假設)
作者: yerym 2020-10-07 23:46:00
推這篇，有用心看&給較具體的建議
作者:
gn60311 (Peterman)
2020-10-08 00:19:00好心推
作者:
chuegou (chuegou)
2020-10-08 02:15:00ide設定我會傳(我的case是IAR專案設定)
作者:
IllMOR (九六三七年五八月二一日)
2020-10-08 03:40:00推好心
一定要上傳的，IDE設定這麼難的事情要整個部門通力合作
作者:
azzc1031 (azzc1031)
2020-10-08 09:56:00推這篇！
作者: tkueeby 2020-10-08 10:03:00
要不要放IDE config要看情況, 大型團隊不會只有一種IDE
作者:
godddddd (howudoing)
2020-10-08 10:25:00不放confit+1 除非是公司指定放
我覺得config要不要傳看專案性質,像我們公司專案的compile要設定比較多東西,所以是會跟著專案一起上,這樣後面的新人進來只要從gitlab clone下來就可以直接compile 然後執行專案
作者:
Hack (RYeh)
2020-10-08 10:43:00推！
作者:
for5566 (Yo)
2020-10-08 10:54:00es6用var怎麼了嗎？我用來區分block跟global scope不行？
作者: Onetwoone121 2020-10-08 11:37:00
推好心人
我是準備要投履歷的新人，我想說用中文做comment比較方便閱讀，但看完你的回覆是比較適合直接用英文嗎～？
寫中文也不是不可 不過通常我都寫英文除了業務上的專有名詞以外
作者:
oooo551 (oooo551)
2020-10-08 15:38:00推
作者:
xlf (Cote rocks!)
2020-10-08 16:11:00放.vscode有什麼問題嗎
作者: vani1995 (澎澎兒) 2020-10-08 16:17:00
面試官人設有點真實
用var定義global怎麼不直接用window?
作者:
TAKADO (朕沒給的你不能搶)
2020-10-09 17:58:00看到面試官人設內牛滿面給推
作者:
ShenJing (ShenJing)
2020-10-10 21:34:00推用心詳細回覆
作者: liangnash (涼奶昔) 2020-10-12 03:58:00
或者用個.config.example
作者: streakray (條紋衣boy) 2020-10-16 03:22:00
推
作者:
ai86109 (derekyen)
2020-10-16 14:37:00推推
作者: npkalala 2020-10-21 08:39:00
完全切中我找人的心路歷程，可怕...