PTT職涯區 / Soft_Job (軟體人)

[請益] 短期間用的網站的code review

看板Soft_Job (軟體人)作者 (逃亡的旅者)時間9年前 (2017/01/25 20:43), 編輯推噓11(11推 0噓 30→)
留言41則, 14人參與, 最新討論串1/1
各位大大 由於公司希望能對公司品牌行銷案進行code review , 此類型案件通常使用約1個月到3個月。 但製作期以及素材入手的期間會比較短 （素材可能製作期間或是製作前一週才來） 製作期約3~7天以內 會建議我提先以完成為優先， 事後一週內code review 提改善建議嗎？ ----- Sent from JPTT on my Samsung SM-T715C. -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 101.9.99.12 ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1485348221.A.557.html
推
01/25 21:04, , 1F
通常一週後又有別的案子進來 code review 就別想了
01/25 21:04, 1F
→
01/25 21:32, , 2F
我自己主要是研發系統+管理職，原本是覺得行銷或是業務
01/25 21:32, 2F
→
01/25 21:32, , 3F
案這種短期間的沒有很大code review 的必要，想建議訂
01/25 21:32, 3F
→
01/25 21:32, , 4F
前端html tag規範以及事後檢討寫法就好
01/25 21:32, 4F
推
01/25 21:51, , 5F
目的是?
01/25 21:51, 5F
→
01/25 22:01, , 6F
上頭覺得負責的junior 前端有些html tag的用法不合理，
01/25 22:01, 6F
→
01/25 22:01, , 7F
切版跟設計稿誤差有點大，速度不夠快，要盯一下效率跟
01/25 22:01, 7F
→
01/25 22:01, , 8F
品質，提出改善建議
01/25 22:01, 8F
→
01/25 22:02, , 9F
外包的CODE? 外包就呵呵了
01/25 22:02, 9F
→
01/25 22:11, , 10F
前端是內聘的，專門處理行銷之類短期間的工作
01/25 22:11, 10F
推
01/25 23:17, , 11F
短期案子還想要code review
01/25 23:17, 11F
→
01/25 23:19, , 12F
看你們 team 開發有幾人吧 沒幾個人兼顧這些工作
01/25 23:19, 12F
推
01/25 23:23, , 13F
一個案子先切得夠細，再丟給 junior, 資深的就可以先 run
01/25 23:23, 13F
→
01/25 23:23, , 14F
但如果前端只有一個 junior 就別想了，這是管理上的問題
01/25 23:23, 14F
→
01/25 23:24, , 15F
如果 junior 的能力跟需求不同，就是管理層面試錯誤
01/25 23:24, 15F
推
01/25 23:46, , 16F
趕工誰還會在意code品質...code review更不用想了...
01/25 23:46, 16F
推
01/26 00:31, , 17F
你應該是需要對那個junior做教育訓練吧..
01/26 00:31, 17F
→
01/26 00:58, , 18F
三到七天交件，你需要有研發專用的開發框架或元件模組才有
01/26 00:58, 18F
→
01/26 00:58, , 19F
辦法加速跟標準化
01/26 00:58, 19F
→
01/26 00:59, , 20F
不然就別期望別人會真的照你設計方式走 訓練成本太高
01/26 00:59, 20F
→
01/26 01:01, , 21F
誤差大 要檢討原因 但是只能做到正確 但不會快
01/26 01:01, 21F
推
01/26 02:56, , 22F
再開始考慮code review 之前，已經有ci機制了嗎？
01/26 02:56, 22F
→
01/26 09:30, , 23F
Junior 做的類型是像這種，但動態多一些
01/26 09:30, 23F
→
01/26 09:30, , 25F
bile/index.html
01/26 09:30, 25F
→
01/26 09:30, , 26F
（隨機google搜尋找來的網頁）
01/26 09:30, 26F
→
01/26 09:30, , 27F
這種有需要CI嗎XD
01/26 09:30, 27F
推
01/26 10:22, , 28F
github私有倉庫一個月7美金阿
01/26 10:22, 28F
→
01/26 10:24, , 29F
用github上的pull requests功能，提報修改，在合併前
01/26 10:24, 29F
→
01/26 10:24, , 30F
都能留言提意見並修改，最後在合併到專案裡，而且修
01/26 10:24, 30F
→
01/26 10:24, , 31F
改都有紀錄
01/26 10:24, 31F
→
01/26 11:09, , 32F
沒問題 當然可以 反正沒下資源不講實效 大家演戲爽一下罷
01/26 11:09, 32F
→
01/26 11:11, , 33F
大家技術背景都認真了 滿足要求方法很多 腳踏實地成本高啊
01/26 11:11, 33F
推
01/26 16:33, , 34F
樓上講的真中肯,企業都是要馬兒跑,又要馬兒不吃草
01/26 16:33, 34F
→
01/26 16:34, , 35F
儘量榨乾,等累死了再換下一隻
01/26 16:34, 35F
推
01/26 19:04, , 36F
原po 要求的是品質不是嗎？與其用人去code review不如在
01/26 19:04, 36F
→
01/26 19:04, , 37F
ci server加入驗證品質工具，把bad smells抓出來。另外c
01/26 19:04, 37F
→
01/26 19:04, , 38F
ode review也通常是在ci後的版本拿出來看的。
01/26 19:04, 38F
→
01/26 19:05, , 39F
另外ci成本一定比code review 低，畢竟都自動...
01/26 19:05, 39F
→
01/28 12:46, , 40F
感覺光做就快吐了 還code review....
01/28 12:46, 40F
推
01/28 14:48, , 41F
最可憐的是那隻馬，被當免洗用，上頭要求又高
01/28 14:48, 41F
‣ 更多 travelerX 的文章
文章代碼(AID): #1OY9rzLN (Soft_Job)
Soft_Job 近期熱門文章
更多 近期熱門文章 >>
PTT職涯區 即時熱門文章
更多 即時熱門文章 >>
‣ 更多 travelerX 的文章
文章代碼(AID): #1OY9rzLN (Soft_Job)