作者tbpfs ( http://pse.is/tbpfs )
看板Soft_Job
標題Re: [心得] 非本科系生真的這麼劣勢嗎？
時間Tue Oct 6 23:37:34 2020
※ 引述《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://webptt.com/m.aspx?n=bbs/Soft_Job/M.1601998659.A.93D.html
1F：推 MonkeyCL: 罩杯吧 10/07 00:09
2F：推 chen09885: 我也覺得網站滿漂亮的，應該是可以找切板工作程度了 10/07 00:28
3F：噓 nttu94507: 又你... 10/07 00:59
4F：→ bill403777: show e cpu 10/07 01:14
5F：→ bill403777: *cup 10/07 01:14
6F：→ superpandal: 漂亮？ 感覺是demo 不過以demo來說不用太要求... 10/07 04:56
7F：→ jyuan1993: 理學院不覺得罩門，大學只上過algo+ds+一些簡單的小課 10/07 07:26
8F：→ jyuan1993: 大學只學過C++/Perl/Java，現在工作用Scala+Js 10/07 07:27
9F：推 newhandfun: 某樓以人廢言？什麼時候code review也要噓了 10/07 07:46
10F：→ Y78: 會是 A 是因為那個 html 元素就叫做 a 10/07 10:03
11F：→ Y78: 然後會是 e 是因為他是 event listener, 命名習慣通常會叫做 10/07 10:10
12F：→ Y78: event evt 或是 e，有在寫 js 的應該都知道是什麼，就跟迴 10/07 10:10
13F：→ Y78: 圈會寫 i 差不多 10/07 10:10
14F：→ ericrobin: e不是慣例嗎= = 10/07 10:10
15F：噓 energyy1104: 不是 你自己說你不懂js還幫人code review幹嘛 10/07 10:11
16F：→ day831231: 程式碼覺得還可以，做靜態網頁做個dummy資料也可以接受 10/07 10:12
17F：→ day831231: 但是也分個檔案吧，然後註解過的程式碼就刪掉不要留 10/07 10:13
18F：→ day831231: 最後是字串串接，感覺就有機會被XSS 10/07 10:15
19F：推 bcjohn: e 就是 event 的意思，這應該不算問題 10/07 10:20
20F：→ nekosgr93: 我寫js第一眼看也不知道e是三小 10/07 10:39
21F：→ nekosgr93: e跟event才叉四個字是有必要縮成這樣？ 10/07 10:39
22F：推 kyoe: 這篇有點硬挑毛病.. 10/07 10:41
23F：→ nekosgr93: 不過我也很久沒寫純JS 10/07 10:43
24F：→ leolarrel: 就像我前老闆不懂韌體卻老愛用純軟角度review MCU code 10/07 11:13
25F：噓 content71: 你說的都是前端JS的慣例阿，這也好批評 10/07 11:31
26F：→ spfy: 很棒 你說的很對 大家都應該學習 10/07 11:38
27F：推 johnny9144: 推，命名很重要，誰知道e是error,還是event 10/07 13:00
28F：→ johnny9144: Nodejs寫很久，沒這種慣例，多人合作的時候，不要妄想 10/07 13:00
29F：→ johnny9144: 大家想的慣例長一樣 10/07 13:00
30F：推 aa0983163178: 推 10/07 13:04
31F：→ kingofsdtw: github本來就沒DB自己模擬一個 10/07 13:18
32F：推 pride829: 請問注釋都用英文寫真的很重要嗎？ 10/07 16:07
33F：→ superpandal: 我也會直接寫e 代碼補全完善的環境才會考慮寫完整 10/07 19:44
34F：→ superpandal: 基本上原po如果遇上樓主肯定被打槍 這世界就是這樣 10/07 19:44
35F：→ superpandal: 隨時都會有人以自身作法套在別人身上 10/07 19:52
36F：→ energyy1104: 這種就雞蛋裡挑骨頭吧 每間公司命名慣例都不一樣 進 10/07 20:18
37F：→ energyy1104: 去再改就好 這又不是什麼大問題 10/07 20:18
38F：→ saladim: 單個字母的名字是不好的 就算是 i j k l, 不過這邊都說慣 10/07 23:03
39F：→ saladim: 例屌打XD 每家公司就算有各自的style 裡面還是有人說跟 10/07 23:04
40F：→ saladim: 這邊一樣的話: 慣例你不懂喔? 整個很有趣~~~ 10/07 23:05
41F：→ newhandfun: 小的還以為好的程式碼是好讀懂，看來我的程式之路還 10/07 23:24
42F：→ newhandfun: 遠著呢 10/07 23:24
43F：噓 ekids1234: 才不會因為 e 而影響錄取 10/08 06:11
44F：推 chatnoir: 半桶水的人拜託不要review... 10/08 09:39
45F：→ chatnoir: e真的是超慣例, 有在寫js都懂, evt跟event也常有人用 10/08 09:40
46F：→ laputaflutin: 連webstorm trycatch自動補全都用e啊 10/08 21:29
47F：噓 pkro12345: 寫e怎麼了嗎？不要不懂裝懂害人 10/12 19:15
48F：推 b85040312: e 我猜是 event 10/13 22:29