ebolaman
級別: 副版主
版區: 程式設計
x38
x458
|
分享:
▲
▼
小建議: 1. 核心功能和使用者部分完全分隔,像是 iWeb.vb 內 If 工作完成 Then MessageBox.Show("下載結束!") 可以將 Jump_Page 改成 Function 傳回一個值,在上一層函式提醒使用者
2. 將資料和核心功能完全分隔,像是 Web_Login.Navigate(New Uri("http://www22.eyny.com/home.php?mod=spacecp&ac=usergroup")) 可以改成 Web_Login.Navigate(New Uri(Settings.Url1)) 避免 eyny 改版後網址改變
3. 變數/函式名稱一致性 Sub Jump_Page 是核心功能 Sub Web_Login_Navigated 是 .NET 內建觸發程序 雖然有用 Region 隔開,但是一眼看上去兩個屬於同個性質,但實際上不是 建議核心功能函式不要加底線
有的地方變數開頭大寫,容易和函式混淆,建議以小寫開頭
4. 部分宣告如果只有該模組自己用,建議不要用 Public, 改用 Private
以上都是為了可讀性/機動性,能讓未來寫程式更順手,修改更容易!
|