Re: [心得] 非本科系生真的這麼劣勢嗎?

作者: 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
應該可以幫你的程式加些分
作者: MonkeyCL (猴總召)   2020-10-07 00:09:00
罩杯吧
作者: chen09885 (阿喜)   2020-10-07 00:28:00
我也覺得網站滿漂亮的,應該是可以找切板工作程度了
作者: nttu94507 (連城)   2020-10-07 00:59:00
又你...
作者: bill403777 (squatting corner)   2020-10-07 01:14:00
show 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 差不多
作者: ericrobin   2020-10-07 10:10:00
e不是慣例嗎= =
作者: energyy1104 (Bill Wang)   2020-10-07 10:11:00
不是 你自己說你不懂js還幫人code review幹嘛
作者: day831231 (下個地點)   2020-10-07 10:12:00
程式碼覺得還可以,做靜態網頁做個dummy資料也可以接受但是也分個檔案吧,然後註解過的程式碼就刪掉不要留最後是字串串接,感覺就有機會被XSS
作者: bcjohn (bc)   2020-10-07 10:20:00
e 就是 event 的意思,這應該不算問題
作者: kyoe (緣份‧不再)   2020-10-07 10:41:00
這篇有點硬挑毛病..
作者: leolarrel (真.粽子無雙)   2020-10-07 11:13:00
就像我前老闆不懂韌體卻老愛用純軟角度review MCU code
作者: content71 (羅莉飼養中...)   2020-10-07 11:31:00
你說的都是前端JS的慣例阿,這也好批評
作者: spfy (spfy)   2020-10-07 11:38:00
很棒 你說的很對 大家都應該學習
作者: johnny9144 (Johnny)   2020-10-07 13:00:00
推,命名很重要,誰知道e是error,還是eventNodejs寫很久,沒這種慣例,多人合作的時候,不要妄想大家想的慣例長一樣
作者: aa0983163178 (Yuki)   2020-10-07 13:04:00
作者: kingofsdtw (不能閒下來!!)   2020-10-07 13:18:00
github本來就沒DB自己模擬一個
作者: pride829 (竹鼠)   2020-10-07 16:07:00
請問注釋都用英文寫真的很重要嗎?
作者: superpandal   2020-10-07 19:44:00
我也會直接寫e 代碼補全完善的環境才會考慮寫完整基本上原po如果遇上樓主肯定被打槍 這世界就是這樣隨時都會有人以自身作法套在別人身上
作者: energyy1104 (Bill Wang)   2020-10-07 20:18:00
這種就雞蛋裡挑骨頭吧 每間公司命名慣例都不一樣 進去再改就好 這又不是什麼大問題
作者: saladim (殺拉頂)   2020-10-07 23:03:00
單個字母的名字是不好的 就算是 i j k l, 不過這邊都說慣例屌打XD 每家公司就算有各自的style 裡面還是有人說跟這邊一樣的話: 慣例你不懂喔? 整個很有趣~~~
作者: newhandfun (新手方)   2020-10-07 23:24:00
小的還以為好的程式碼是好讀懂,看來我的程式之路還遠著呢
作者: ekids1234 (∵:☆星痕╭☆)   2020-10-08 06:11:00
才不會因為 e 而影響錄取
作者: chatnoir (對不起)   2020-10-08 09:39:00
半桶水的人拜託不要review...e真的是超慣例, 有在寫js都懂, evt跟event也常有人用
作者: laputaflutin (很恐怖,不要問)   2020-10-08 21:29:00
連webstorm trycatch自動補全都用e啊
作者: pkro12345 (席龍)   2020-10-12 19:15:00
寫e怎麼了嗎?不要不懂裝懂害人
作者: b85040312 (萬年newman)   2020-10-13 22:29:00
e 我猜是 event

Links booklink

Contact Us: admin [ a t ] ucptt.com