楼主:
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
完全切中我找人的心路历程，可怕...