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

[tbpfs ( http://pse.is/tbpfs )]

※ 引述《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

應該可以幫你的程式加些分

--
紫楓碎碎念
YouTube頻道：https://www.youtube.com/user/tbpfs
FB粉專：https://www.facebook.com/tbpfs2/
blog: http://tbpfs1.blogspot.com/
從零開始學會寫程式:http://l.ovoy.click/itiron
軟體工程師薪資大解密: http://l.ovoy.click/R3M42

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 1.174.50.22 (臺灣)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1601998659.A.93D.html
推 MonkeyCL: 罩杯吧
推 chen09885: 我也覺得網站滿漂亮的，應該是可以找切板工作程度了
噓 nttu94507: 又你...
→ bill403777: show e cpu
→ bill403777: *cup
→ superpandal: 漂亮？ 感覺是demo 不過以demo來說不用太要求...
→ jyuan1993: 理學院不覺得罩門，大學只上過algo+ds+一些簡單的小課
→ jyuan1993: 大學只學過C++/Perl/Java，現在工作用Scala+Js
推 newhandfun: 某樓以人廢言？什麼時候code review也要噓了
→ Y78: 會是 A 是因為那個 html 元素就叫做 a
→ Y78: 然後會是 e 是因為他是 event listener, 命名習慣通常會叫做
→ Y78:  event evt 或是 e，有在寫 js 的應該都知道是什麼，就跟迴
→ Y78: 圈會寫 i 差不多
→ ericrobin: e不是慣例嗎= =
噓 energyy1104: 不是 你自己說你不懂js還幫人code review幹嘛
→ day831231: 程式碼覺得還可以，做靜態網頁做個dummy資料也可以接受
→ day831231: 但是也分個檔案吧，然後註解過的程式碼就刪掉不要留
→ day831231: 最後是字串串接，感覺就有機會被XSS
推 bcjohn: e 就是 event 的意思，這應該不算問題
→ nekosgr93: 我寫js第一眼看也不知道e是三小
→ nekosgr93: e跟event才叉四個字是有必要縮成這樣？
推 kyoe: 這篇有點硬挑毛病..
→ nekosgr93: 不過我也很久沒寫純JS
→ leolarrel: 就像我前老闆不懂韌體卻老愛用純軟角度review MCU code
噓 content71: 你說的都是前端JS的慣例阿，這也好批評
→ spfy: 很棒 你說的很對 大家都應該學習
推 johnny9144: 推，命名很重要，誰知道e是error,還是event
→ johnny9144: Nodejs寫很久，沒這種慣例，多人合作的時候，不要妄想
→ johnny9144: 大家想的慣例長一樣
推 aa0983163178: 推
→ kingofsdtw: github本來就沒DB自己模擬一個
推 pride829: 請問注釋都用英文寫真的很重要嗎？
→ superpandal: 我也會直接寫e 代碼補全完善的環境才會考慮寫完整
→ superpandal: 基本上原po如果遇上樓主肯定被打槍 這世界就是這樣
→ superpandal: 隨時都會有人以自身作法套在別人身上
→ energyy1104: 這種就雞蛋裡挑骨頭吧  每間公司命名慣例都不一樣 進
→ energyy1104: 去再改就好 這又不是什麼大問題
→ saladim: 單個字母的名字是不好的 就算是 i j k l, 不過這邊都說慣
→ saladim: 例屌打XD  每家公司就算有各自的style 裡面還是有人說跟
→ saladim: 這邊一樣的話: 慣例你不懂喔? 整個很有趣~~~
→ newhandfun: 小的還以為好的程式碼是好讀懂，看來我的程式之路還
→ newhandfun: 遠著呢
噓 ekids1234: 才不會因為 e 而影響錄取
推 chatnoir: 半桶水的人拜託不要review...
→ chatnoir: e真的是超慣例, 有在寫js都懂, evt跟event也常有人用
→ laputaflutin: 連webstorm trycatch自動補全都用e啊
噓 pkro12345: 寫e怎麼了嗎？不要不懂裝懂害人
推 b85040312: e 我猜是 event