重構筆記 | 🌹 喇賽的人 Blog 🌹

最近接了個超級複雜程式碼翻寫任務 , 算是最害怕的任務吧 , 就順便重構一些過去的技術債 , 所以筆記一下

好的程式碼原則

如果遇到這種 if 結構可以用 Dictionary 重構 , 消除 switch-case 可以參考這篇(https://ronsun.github.io/content/20180412-anti-switch-case.html)

if(x == "Dev"){
	return 1;
}

if(x == "Prod"){
	return 2;
}

var dict = new Dictionary<string,int>(){
	{"Dev",1},
	{"Prod",2}
};

if (dict.ContainsKey(x)){
	return dict[x];
}

工作常我 99% 自己不會用 out , 偏偏老舊 code 一堆 out

1
2
3

public string ToolMan(int money , Food food , out string byebye){
	//todo
}

改建立類別 , 把掰掰變成屬性即可成為沒出局的工具人 ~

public class Man {
	public string ByeBye { get; set; }
	public string ToolMan(int money , Food food){
		//todo
	}
}

我曾經接過一包很爛的 code 一堆朝狀 if else , 變數名稱是用 fuck 來命名 , code 爛不說連變數都這樣搞
就不能乖乖取名 _30cm 這種有意義又好看的命名嗎?

在鄉民的世界裡 30cm 幾乎是標準配備 , 在 coding 的世界裡越是行數多的程式碼越難一眼理解
依據大師原則 , 少於 10 行程式碼的方法 (最好不超過 4 行)
所以大師規定 10cm 才是好的 code 最好不要超過 4cm ~ 這下館長有救了!

依據大師原則 => 不超過兩個參數的方法 , 超過的話重構為物件 DTO

我不太會形容 , 感覺有點像是這樣 , 明明可以直接在 Ruler 搞定的事硬要搬到 Man 裡面 , 像是 30cm 情節

class Man{
	public static void HowLong(Ruler r){
		return r.Meaure();
	}
}

class Ruler{
	public int Measure(){ return 30; }
}

大師意思就是寫 for or foreach 的 code 都是垃圾 ~