楼主:
tbpfs (http://0rz.tw/Uk989)
2020-10-06 23:37:34※ 引述《ZZ321 (爱情就像是便便)》之铭言：
: 大家好
: 小弟商学院毕业，非本科科系！
: 使用框架：Vue.js
: Git：https://github.com/jim55167
我不熟Js，或是可以说不懂JS了XD
不过可以来帮你做个简单的code review
一开始我点开这个网页:https://jim55167.github.io/Taipei-Travel/
做的还挺有模有样的，至少我做不出来~~哈
然后点进去看code
https://github.com/jim55167/Taipei-Travel/blob/master/js/data.js
这个资料全部放在这里也太狂了吧，应该有些简易的数据库可以放吧?
https://github.com/jim55167/Taipei-Travel/blob/master/js/main.js
function show(e) e是什么? 还蛮常看到的
请写好名字
注释用英文录取率应该有两倍以上
if (clickTarget == "A") {
刚刚来个e 现在来个A
然后加一些error handling
应该可以帮你的程式加些分
作者:
bill403777 (squatting corner)
2020-10-07 01:14:00show e cpu*cup
作者: superpandal 2020-10-07 04:56:00
漂亮？ 感觉是demo 不过以demo来说不用太要求...
作者: jyuan1993 (Yuan) 2020-10-07 07:26:00
理学院不觉得罩门，大学只上过algo+ds+一些简单的小课大学只学过C++/Perl/Java，现在工作用Scala+Js
作者: newhandfun (新手方) 2020-10-07 07:46:00
某楼以人废言？什么时候code review也要嘘了
作者:
Y78 (Y78)
2020-10-07 10:03:00会是 A 是因为那个 html 元素就叫做 a然后会是 e 是因为他是 event listener, 命名习惯通常会叫做event evt 或是 e，有在写 js 的应该都知道是什么，就跟循环会写 i 差不多
不是 你自己说你不懂js还帮人code review干嘛
程式码觉得还可以，做静态网页做个dummy资料也可以接受但是也分个档案吧，然后注解过的程式码就删掉不要留最后是字串串接，感觉就有机会被XSS
作者:
bcjohn (bc)
2020-10-07 10:20:00e 就是 event 的意思，这应该不算问题
我写js第一眼看也不知道e是三小e跟event才叉四个字是有必要缩成这样？
作者:
kyoe (缘份‧不再)
2020-10-07 10:41:00这篇有点硬挑毛病..
就像我前老板不懂韧体却老爱用纯软角度review MCU code
作者:
spfy (spfy)
2020-10-07 11:38:00很棒 你说的很对 大家都应该学习
推，命名很重要，谁知道e是error,还是eventNodejs写很久，没这种惯例，多人合作的时候，不要妄想大家想的惯例长一样
作者:
kingofsdtw (ä¸èƒ½é–’下來!!)
2020-10-07 13:18:00github本来就没DB自己模拟一个
作者: superpandal 2020-10-07 19:44:00
我也会直接写e 代码补全完善的环境才会考虑写完整基本上原po如果遇上楼主肯定被打枪 这世界就是这样随时都会有人以自身作法套在别人身上
这种就鸡蛋里挑骨头吧 每间公司命名惯例都不一样 进去再改就好 这又不是什么大问题
作者:
saladim (杀拉顶)
2020-10-07 23:03:00单个字母的名字是不好的 就算是 i j k l, 不过这边都说惯例屌打XD 每家公司就算有各自的style 里面还是有人说跟这边一样的话: 惯例你不懂喔? 整个很有趣~~~
作者: newhandfun (新手方) 2020-10-07 23:24:00
小的还以为好的程式码是好读懂，看来我的程式之路还远着呢
半桶水的人拜托不要review...e真的是超惯例, 有在写js都懂, evt跟event也常有人用
连webstorm trycatch自动补全都用e啊