PTT評價

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

看板Soft_Job標題Re: [心得] 非本科系生真的這麼劣勢嗎?作者
tbpfs
( http://pse.is/tbpfs )
時間推噓 5 推:10 噓:5 →:33

※ 引述《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 留言評論
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 1.174.50.22 (臺灣)
PTT 網址

MonkeyCL10/07 00:09罩杯吧

chen0988510/07 00:28我也覺得網站滿漂亮的,應該是可以找切板工作程度了

nttu9450710/07 00:59又你...

bill40377710/07 01:14show e cpu

bill40377710/07 01:14*cup

superpandal10/07 04:56漂亮? 感覺是demo 不過以demo來說不用太要求...

jyuan199310/07 07:26理學院不覺得罩門,大學只上過algo+ds+一些簡單的小課

jyuan199310/07 07:27大學只學過C++/Perl/Java,現在工作用Scala+Js

newhandfun10/07 07:46某樓以人廢言?什麼時候code review也要噓了

Y7810/07 10:03會是 A 是因為那個 html 元素就叫做 a

Y7810/07 10:10然後會是 e 是因為他是 event listener, 命名習慣通常會叫做

Y7810/07 10:10 event evt 或是 e,有在寫 js 的應該都知道是什麼,就跟迴

Y7810/07 10:10圈會寫 i 差不多

ericrobin10/07 10:10e不是慣例嗎= =

energyy110410/07 10:11不是 你自己說你不懂js還幫人code review幹嘛

day83123110/07 10:12程式碼覺得還可以,做靜態網頁做個dummy資料也可以接受

day83123110/07 10:13但是也分個檔案吧,然後註解過的程式碼就刪掉不要留

day83123110/07 10:15最後是字串串接,感覺就有機會被XSS

bcjohn10/07 10:20e 就是 event 的意思,這應該不算問題

nekosgr9310/07 10:39我寫js第一眼看也不知道e是三小

nekosgr9310/07 10:39e跟event才叉四個字是有必要縮成這樣?

kyoe10/07 10:41這篇有點硬挑毛病..

nekosgr9310/07 10:43不過我也很久沒寫純JS

leolarrel10/07 11:13就像我前老闆不懂韌體卻老愛用純軟角度review MCU code

content7110/07 11:31你說的都是前端JS的慣例阿,這也好批評

spfy10/07 11:38很棒 你說的很對 大家都應該學習

johnny914410/07 13:00推,命名很重要,誰知道e是error,還是event

johnny914410/07 13:00Nodejs寫很久,沒這種慣例,多人合作的時候,不要妄想

johnny914410/07 13:00大家想的慣例長一樣

aa098316317810/07 13:04

kingofsdtw10/07 13:18github本來就沒DB自己模擬一個

pride82910/07 16:07請問注釋都用英文寫真的很重要嗎?

superpandal10/07 19:44我也會直接寫e 代碼補全完善的環境才會考慮寫完整

superpandal10/07 19:44基本上原po如果遇上樓主肯定被打槍 這世界就是這樣

superpandal10/07 19:52隨時都會有人以自身作法套在別人身上

energyy110410/07 20:18這種就雞蛋裡挑骨頭吧 每間公司命名慣例都不一樣 進

energyy110410/07 20:18去再改就好 這又不是什麼大問題

saladim10/07 23:03單個字母的名字是不好的 就算是 i j k l, 不過這邊都說慣

saladim10/07 23:04例屌打XD 每家公司就算有各自的style 裡面還是有人說跟

saladim10/07 23:05這邊一樣的話: 慣例你不懂喔? 整個很有趣~~~

newhandfun10/07 23:24小的還以為好的程式碼是好讀懂,看來我的程式之路還

newhandfun10/07 23:24遠著呢

ekids123410/08 06:11才不會因為 e 而影響錄取

chatnoir10/08 09:39半桶水的人拜託不要review...

chatnoir10/08 09:40e真的是超慣例, 有在寫js都懂, evt跟event也常有人用

laputaflutin10/08 21:29連webstorm trycatch自動補全都用e啊

pkro1234510/12 19:15寫e怎麼了嗎?不要不懂裝懂害人

b8504031210/13 22:29e 我猜是 event