作者:
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啊