Re: [請益] 如何沉住氣讀別人的 code消失

看板Soft_Job作者時間6年前 (2018/01/05 06:54), 編輯推噓7(26推 19噓 105→)
留言150則, 40人參與, 最新討論串3/7 (看更多)
※ 引述《godspeedlee (妳，我可以)》之銘言： : ※ 引述《p52189 (鼠霸)》之銘言： : : 如題……原始碼不長 : : 但撰碼習慣跟我完全不同 : : 到現在我也還沒找到規則 : : 以至於想改一小部分都因為不知道規則而必須幾乎全部爬一遍才改得動 : : 每次讀都很難沉住氣一下就很煩躁不想做了 : : 一直這樣很像也不是辦法 : : 請問大家有什麼方法可以坐得住嗎= = : : 謝謝…… : : ----- : : Sent from JPTT on my HTC_M9u. : 1. 排版看不順眼可以用 astyle 排版 : 2. 可以用 SourceInsight 追蹤程式 : 3. 抓出 Data Flow，畫出 Data Flow Diagram : 4. 了解系統內有哪些 process(es)/thread(s) 互動 : 5. 如果是 C/C++，可以用 gdb 追蹤了解程式行為 : 6. 某些情形下可以用 gprof + graphivz 了解程式 : run-time call graph: https://goo.gl/uZGr2q : 一些開發環境如 Keil C 可以輸出 static call graph : 7. 使用雙螢幕，一個螢幕追蹤程式，一個螢幕用來作筆記 : 8. 買本 Code Reading https://goo.gl/zYsvfj : (有簡體版) 老實說 我很討厭因為自己的喜好而調整排版的人 尤其那種，為了修幾個bug，加點功能就順手排版的敗類 等到production出事，回頭緊急追查上版改了什麼code，在版控下diff一看，發現因為重 現排版，整個源碼檔案從上到下都是變動，真的會大罵一聲：「幹你娘！」 奉勸各位，要重新排版讓code閱讀方便不是不行，請把這種重排版的動作，單純地做一次 單一的commit，comment請寫「純排版異動，未做任何邏輯變動」。 然後不要把修bug加功能和這件事混合在一起。 這樣你的母親日子會過得比較安穩 -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 114.32.139.28 ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1515106493.A.37A.html
推
01/05 08:18, , 1F
但我更討厭連好好縮排都不會的人……orz.
01/05 08:18, 1F
→
01/05 08:20, , 2F
不過同意 reformat 要獨立一個 commit, 不然看 diff log
01/05 08:20, 2F
→
01/05 08:20, , 3F
會想罵幹，兩種都很幹。
01/05 08:20, 3F
推
01/05 08:21, , 4F
推！
01/05 08:21, 4F
推
01/05 08:27, , 5F
只能推了 為了媽媽好XD!!
01/05 08:27, 5F
推
01/05 08:34, , 6F
我指的排版是自己看，請不要隨便問候別人媽媽
01/05 08:34, 6F
推
01/05 08:51, , 7F
貴司的管理問題干我何事，我只是提出方案
01/05 08:51, 7F
推
01/05 08:55, , 8F
整個檔案都變動是IDE搞得吧，誰會沒事在那邊排版
01/05 08:55, 8F
→
01/05 08:56, , 9F
所以應該要"問候"的是上一個作者為什麼不排版(ry
01/05 08:56, 9F
推
01/05 09:09, , 10F
有沒有排版是主管負責監督，別自己找事
01/05 09:09, 10F
→
01/05 09:10, , 11F
動了別人的code後面就要負責
01/05 09:10, 11F
推
01/05 09:15, , 12F
其實這些眉眉角角還蠻重要的
01/05 09:15, 12F
→
01/05 09:25, , 13F
排版是IDE送的 為甚麼不要用lint規定格式?
01/05 09:25, 13F
→
01/05 09:26, , 14F
馬上驗證上一篇 嗆別人的自己也通常沒照軟工開發
01/05 09:26, 14F
→
01/05 09:29, , 15F
commit的summary要寫好 不要包裹一堆進去同一個 我認為是基本
01/05 09:29, 15F
→
01/05 09:30, , 16F
的基本耶 連這種事情都不辦的 我會認為他都不知道自己在做什
01/05 09:30, 16F
→
01/05 09:37, , 17F
麼 或是連用打字表達自己在做什麼都有障礙..
01/05 09:37, 17F
→
01/05 09:43, , 18F
空白跟斷行都是有選項可以去忽略的
01/05 09:43, 18F
噓
01/05 09:50, , 19F
不會忽略空白也很扯
01/05 09:50, 19F
→
01/05 09:53, , 20F
你是不是沒遇過IDE會幫你排版的
01/05 09:53, 20F
→
01/05 10:08, , 21F
排版獨立commit的確比較好，尤其是整頁都變動的
01/05 10:08, 21F
推
01/05 10:51, , 22F
嗚嗚 我要怒改我習慣了
01/05 10:51, 22F
推
01/05 11:09, , 23F
有沒有見過把astyle塞在makefile裡的?
01/05 11:09, 23F
→
01/05 11:09, , 24F
結果弄到不小心很多地方需要重新編譯
01/05 11:09, 24F
→
01/05 11:10, , 25F
整個編譯速度還變得超慢
01/05 11:10, 25F
→
01/05 11:12, , 26F
好7pupu 沒有統合coding style嗎？
01/05 11:12, 26F
→
01/05 11:25, , 27F
哈哈 這個是真的
01/05 11:25, 27F
→
01/05 11:27, , 28F
我有次上版順便用第三方排版工具修過 後來看變動紀錄
01/05 11:27, 28F
→
01/05 11:27, , 29F
自己都覺得好笑
01/05 11:27, 29F
→
01/05 11:43, , 30F
這種東西都馬 linter 要規定好 沒format都不準 check i
01/05 11:43, 30F
→
01/05 11:43, , 31F
n就好了
01/05 11:43, 31F
推
01/05 11:54, , 32F
推 有時候想排版都會猶豫
01/05 11:54, 32F
推
01/05 11:56, , 33F
看到整塊紅的以為改了什麼 結果只是多了空格
01/05 11:56, 33F
推
01/05 12:12, , 34F
----現在宣布 空白跟tab 大戰開始-----
01/05 12:12, 34F
→
01/05 12:36, , 35F
tab應該是3個空格 4個的tab是怎樣
01/05 12:36, 35F
推
01/05 16:13, , 36F
在路上亂拉屎 罵清的人刮壞草皮
01/05 16:13, 36F
推
01/05 17:47, , 37F
推薦全形空白 我幫你加一個:)
01/05 17:47, 37F
推
01/05 17:48, , 38F
這篇讓人莫名心有戚戚焉
01/05 17:48, 38F
→
01/05 20:36, , 39F
git diff -w就好啦 這都不會用喔？
01/05 20:36, 39F
還有 71 則推文
→
01/07 20:26, , 111F
看來樓上都是沒有出過社會的學生，沒有聽過什麼叫
01/07 20:26, 111F
→
01/07 20:26, , 112F
做production support, 不然就是窮途末路的低薪碼農
01/07 20:26, 112F
→
01/07 20:26, , 113F
，公司連個在線營運的產品都沒有，才會這麼沒見識。
01/07 20:26, 113F
→
01/07 20:26, , 114F
我談production support, 你們卻都在以悠悠哉哉的開
01/07 20:26, 114F
→
01/07 20:26, , 115F
發時期。
01/07 20:26, 115F
→
01/07 20:27, , 116F
樓上還好你沒來我公司，我怕你要做production suppo
01/07 20:27, 116F
→
01/07 20:27, , 117F
rt就嚇到尿出來喔
01/07 20:27, 117F
噓
01/07 20:41, , 118F
哪間公司產品在線的時候source不能讓看code的人fork出
01/07 20:41, 118F
→
01/07 20:41, , 119F
來檢查? 麻煩報個名號讓大家知道
01/07 20:41, 119F
→
01/07 20:43, , 120F
我第一次知道不能fork的公司耶 是不是也不能merge回
01/07 20:43, 120F
→
01/07 20:44, , 121F
去 有bug通通不能改唷^.<
01/07 20:44, 121F
→
01/07 20:44, , 122F
我還真是沒見識 謝謝你告訴我有這種公司
01/07 20:44, 122F
→
01/07 20:48, , 123F
樓上已經在胡言亂語了, 看 Diff 跟 fork 甚麼關聯?
01/07 20:48, 123F
噓
01/07 20:49, , 124F
據我所知FLAGM都可以fork出來 看來你的公司比這些公司
01/07 20:49, 124F
→
01/07 20:49, , 125F
fork一隻出來 然後在那個commit之前開一個branch
01/07 20:49, 125F
→
01/07 20:50, , 126F
然後改之前改之後用格式整理再做diff
01/07 20:50, 126F
→
01/07 20:50, , 127F
很困難?
01/07 20:50, 127F
→
01/07 20:50, , 128F
真是不好意思 會這個可能要年薪四百萬
01/07 20:50, 128F
→
01/07 20:51, , 129F
對年薪三百萬的人而言這個操作太困難了
01/07 20:51, 129F
→
01/07 20:52, , 130F
附帶一提 把這個功能寫成自動化可能要年薪五百萬
01/07 20:52, 130F
噓
01/07 20:56, , 131F
我記得你的年薪想要升到四百萬 會了這個然後不會寫
01/07 20:56, 131F
→
01/07 20:57, , 132F
自動化可以剛剛好
01/07 20:57, 132F
→
01/07 21:04, , 133F
樓上真是太閒了, production support 為了看 diff
01/07 21:04, 133F
→
01/07 21:05, , 134F
浪費時間這樣大費周章, 這閒情逸致公司不知道多損
01/07 21:05, 134F
→
01/07 21:05, , 135F
多損失幾百萬只因為你慢慢搞這些
01/07 21:05, 135F
→
01/07 21:07, , 136F
這些損失扣你的薪資不知道白做幾年工
01/07 21:07, 136F
噓
01/07 21:10, , 137F
這個應該是天天需要的功能吧 花一天寫一個自動化手稿
01/07 21:10, 137F
→
01/07 21:11, , 138F
損失幾百萬? 那不知道會省下幾千萬
01/07 21:11, 138F
→
01/07 21:11, , 139F
啊我忘了你的年薪只有三百萬不會寫自動化手稿
01/07 21:11, 139F
→
01/07 21:11, , 140F
抱歉抱歉
01/07 21:11, 140F
→
01/07 22:09, , 141F
只會維護技術容易停滯，跟不上時代
01/07 22:09, 141F
推
01/08 23:08, , 142F
我真的覺得druid0214給的建議很好，不過她年薪很高不會
01/08 23:08, 142F
→
01/08 23:09, , 143F
用GIT很正常好嗎，不關你ide的是沒錯，但你又愛在那邊
01/08 23:09, 143F
→
01/08 23:09, , 144F
抱怨，改個ide機處設定就好有多難，笑死人
01/08 23:09, 144F
→
01/08 23:10, , 145F
還有我進公司第一天真的是自己開發一個專案沒錯
01/08 23:10, 145F
→
01/08 23:11, , 146F
損失幾百萬我笑了，你以為你是政府單位接10e以上案子?
01/08 23:11, 146F
→
01/08 23:11, , 147F
就算被改調自己ide存一下都會有基本規格是會差到哪?
01/08 23:11, 147F
→
01/08 23:12, , 148F
不過聽accessdenied講說公司不能fork又年領400萬台幣
01/08 23:12, 148F
→
01/08 23:12, , 149F
我猜各位可能是宇宙等級的公司上萬兆台幣資本額
01/08 23:12, 149F
→
01/08 23:12, , 150F
才會讓年薪400萬階級的連開fork的權利都沒有
01/08 23:12, 150F
‣ 更多 accessdenied 的文章
文章代碼(AID): #1QJh2zDw (Soft_Job)
討論串 (同標題文章)
本文引述了以下文章的的內容：
完整討論串 (本文為第 3 之 7 篇)：
排序： 最舊先 | 最新先 | 留言數
在新視窗開啟完整討論串 (共7篇)
‣ 更多 accessdenied 的文章
文章代碼(AID): #1QJh2zDw (Soft_Job)