---

※ 引述《justsing (只是唱首歌)》之銘言：
: 最近進入一家蠻大的公司
: 才進去沒多久已經把目前正在開發中的專案幾乎重構過了
: 原因是因為某前輩寫的code問題非常多
: 所有變數都用全域變數，導致資料常常出錯
: 命名也很愛取那種
: abc1
: abc2
: 這種完全看不出來在幹嘛的東西
: 開發完也完全不測試就推到正式branch上

講到這個我就心有戚戚

最近在公司接手一些案子，這個案子有多厲害

  * 用純 PHP 義大利麵方法寫

  * 無OOP，無框架

  * 資料庫無正規化、該索引的沒索引，不該索引亂索引

  * 因為太多專有名詞，DB欄位直接用 optA、optB、optC..... optH (option?)

  * 重覆coding的東西太多
    例如查詢資料庫的部份只有欄位不同，就乾脆直接copy整份程式碼，改欄位->存檔
    (說好的 class、function 呢？ 每支程式都在 create connection or die )

    例
    get_user
    edit_user
    del_user
    get_article
    edit_article
    del_article
    ...         你他媽每一個 CRUD 就給我寫一個檔案 (裡面99%一樣，只欄位不同)


  * N+1 query，比如先查詢產品列表，再在迴圈查詢每個產品的producer_id
    (明明 JOIN 一次或 eager loading 就可以辦到的東西)

  * 開發途中才進版控，commit 記錄全部 "no message"

  * 公司把系統某部份再外包給下游接案公司，給PM接洽發包
    接案公司心態：交差
    程式碼收回來之後，PM不懂coding，對方亂寫看不懂，現在要維護、加新功能
    花更多時間在還技術債，而且對案主還沒結案

  * 因為外包的關係，所以資料夾裡面有好幾包，都是獨立的一支系統卻串在一起
    例如後台用某個寫法，有自己的library資料夾等；
    前台某個功能又自己一包，每個系統各自成一個生態，然後串在一起



: 後來主管們看不下去就請我幫忙改
: 改完後目前系統改善很多
: 但好像也惹得他很不高興
: 雖然他現在幾乎是半冷凍狀態
: 但他偶爾只要寫個新功能就會產生一堆Bug
: 之前跟主管反應過也是不了了之
: 畢竟主管不會寫程式，也很難看懂他的問題在哪
: 通常大家遇到這種情形，會怎麼做呢
: 是不是只能快逃了啊



我在進來現在公司以前，公司主管是有直接告訴我
「我們公司現在很多事情沒有成型，如果有新的想法的話，可以直接提出」

確實，在進來的時候也討論了滿多的想法，建議導入開發框架跟SA的概念
明明感覺是業界已經非常簡單行之有年的東西，卻像登天一樣難
我真的是活在平行世界嗎

可能公司的心態也有差
公司認為簡單的東西可以外包減少工程師 loading，結果程式碼品質很難去控管
寧願花一筆錢外包，也不願意再請人，不就是飲鴆止渴嗎？
(第二手接案公司賺到錢，結果債是我們在還)


不過現在看起來，是有慢慢在變好啦... 有吧。





回到你的問題

我接手這個專案之後，什麼修改都是假的
到處 include 比亂繼承類別還要難 code review，完全不知道變數從哪裡開始出現

更別說程式碼混著 SQL 跟 js 了

找 PM 要原始的需求，現任 PM 也不太瞭解 (因為換好幾個PM接手)

後來我 refactor 的作法是：

舊程式碼直接放一邊，在前端把網站點一點，摸一摸 猜他要幹麻
把需求反推回去，看著輸入的表單，把 schema 推出來

等於是在大便裡面挑黃金一樣，最後剩下那一點點稀少的 就是最核心的東西
其他都是 a piece of shit    bullshit

一邊重寫，一邊SA
想想什麼功能有可能擴充，什麼功能是重覆的
把邏輯抽出來，變得抽象一點，再各自實作

↑然後你會發現 這些東西不都重覆的可以共用 class or method 嗎
  那些 copy & paste 重覆的檔案是...


大概就是這樣吧

重構真的很累，邊寫你就會邊感嘆


以前覺得世界奇觀要出國才有

結果發現一堆公司的程式碼裡面也有


(攤手)

--




--
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 123.241.170.131
※ 文章代碼(AID): #1QpsC8RQ (Soft_Job)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1523540744.A.6DA.html