【Day13】重構程式碼,減少歷史業障
這篇文章寫於 2020 年,文中的工具與版本可能已經更新,實作時請以官方文件為準。
本文目錄
數據顯示,工程師完成最後 20% 的工作時間跟之前的 80% 一樣多
完成功能後,重構是另一個開始
昨天我們完成了一個跑得動的程式,但很明顯這不是一個好的程式
大家應該有感受到隨著程式越來越長,我們對他的掌握力逐步下降
根據 Clean Code 的原則而言就是說這程式寫得很爛XD
很多人為了盡快完成功能而埋頭狂寫,但這樣容易導致你日後維護以及交接的困難性,你所負責的專案越大這個感受會更明顯
所以在完成功能後重構是必要的任務,他能提升你日後解決 bug、需求變更的工作效率
接下來就以昨天程式為範例來重構吧!
今日目標
了解 & 修正目前專案所犯下的錯誤
- 主程式做了太多事情
- 一個函式做了太多事情
- 物件宣告的方式太複雜且多次被使用
1. 主程式做了太多事情
主程式就像是專案的總經理,他只需要知道大方向的架構;而架構裡面的細節就交給各部門處理,他只是溝通資料的橋樑
根據上述的邏輯,我們應該把在主程式中的函式轉移到另外的資料夾(ex:tools),依據功能去命名檔名,範例如下:
- 主程式 - index.js
- 初始化瀏覽器 - tools/initDrive.js
- Facebook爬蟲 - tools/crawlerFB.js
- Instagram爬蟲 - tools/crawlerIG.js
在這樣的分工後我們的主程式是不是變得很乾淨呢?
index.js
require('dotenv').config(); //載入.env環境檔
const { initDrive } = require("./tools/initDrive.js");
const { crawlerFB } = require("./tools/crawlerFB.js");
const { crawlerIG } = require("./tools/crawlerIG.js");
async function crawler () {
const driver = await initDrive();
if (!driver) {//driver不存在就結束程式
return
}
//因為有些人是用FB帳號登入IG,為了避免增加FB登出的動作,所以採取先對IG進行爬蟲
await crawlerIG(driver)
await crawlerFB(driver)
driver.quit();
}
crawler()
2. 一個函式做了太多事情
記住一個大原則:一個函式只需要做一件事情!
現在我們把 FB 登入並取得追蹤人數寫在同一隻函式,這樣會增加你日後維護的困難度,因為函式越長,你越難除錯(debug);以 loginFacebookGetTrace 這隻函式舉例,它實際上可以解構成好幾個功能:
- 取得 Facebook(經典版/新版) 的爬蟲路徑 - getCrawlerPath
- Facebook爬蟲 - crawlerFB
- 登入Facebook - loginFacebook
- 前往粉絲頁 - goFansPage
- 取得追蹤人數 - getTrace
把每個功能獨立成函式,你就能輕鬆除錯(debug)。下面程式以 FB 爬蟲作為解構範例,你可以自己練習改寫 IG 的部分喔
crawlerFB.js
const fb_username = process.env.FB_USERNAME
const fb_userpass = process.env.FB_PASSWORD
const { By, until } = require('selenium-webdriver') // 從套件中取出需要用到的功能
exports.crawlerFB = crawlerFB;//讓其他程式在引入時可以使用這個函式
// FB有經典版以及新版的區分,兩者的爬蟲路徑不同,我們藉由函式取得各自的路徑
const { fb_head_path, fb_trace_path } = getCrawlerPath();
function getCrawlerPath () {
if (process.env.FB_VERSION === 'new') {//如果是新版FB
return {
"fb_head_path": `//*[contains(@class,"fzdkajry")]`,
"fb_trace_path": `//*[contains(@class,"knvmm38d")]`
}
} else {//如果為設定皆默認為舊版
return {
"fb_head_path": `//*[contains(@class,"_1vp5")]`,
"fb_trace_path": `//*[@id="PagesProfileHomeSecondaryColumnPagelet"]//*[contains(@class,"_4bl9")]`
}
}
}
async function crawlerFB(driver) {
await loginFacebook(driver)
const fanpage = "https://www.facebook.com/baobaonevertell/"
await goFansPage(driver, fanpage)
await driver.sleep(3000)
await getTrace(driver)
}
async function loginFacebook(driver) {
const web = 'https://www.facebook.com/login';//我們要前往FB
await driver.get(web)//在這裡要用await確保打開完網頁後才能繼續動作
//填入fb登入資訊
const fb_email_ele = await driver.wait(until.elementLocated(By.xpath(`//*[@id="email"]`)));
fb_email_ele.sendKeys(fb_username)
const fb_pass_ele = await driver.wait(until.elementLocated(By.xpath(`//*[@id="pass"]`)));
fb_pass_ele.sendKeys(fb_userpass)
//抓到登入按鈕然後點擊
const login_elem = await driver.wait(until.elementLocated(By.xpath(`//*[@id="loginbutton"]`)))
login_elem.click()
//因為登入這件事情要等server回應,你直接跳轉粉絲專頁會導致登入失敗
//用登入後才有的元件,來判斷是否登入
await driver.wait(until.elementLocated(By.xpath(fb_head_path)))
}
async function goFansPage(driver, web_url) {
//登入成功後要前往粉專頁面
await driver.get(web_url)
}
async function getTrace(driver) {
let fb_trace = 0;//這是紀錄FB追蹤人數
//因為考慮到登入之後每個粉專顯示追蹤人數的位置都不一樣,所以就採用全抓在分析
const fb_trace_eles = await driver.
wait(until.elementsLocated(By.xpath(fb_trace_path)))
for (const fb_trace_ele of fb_trace_eles) {
const fb_text = await fb_trace_ele.getText()
if (fb_text.includes('人在追蹤')) {
fb_trace = fb_text.replace(/\D/g, '')//只取數字
break
}
}
console.log(`FB追蹤人數:${fb_trace}`)
}
3. 物件宣告的方式太複雜且多次被使用
我們要將建立瀏覽器的部分獨立成一隻js,原因如下
- 建立瀏覽器前需要做事前確認及複雜的設定
- 這個瀏覽器作為物件會被多次使用
- 如果瀏覽器的參數需要變更可以在一個檔案集中管理
基於上述原因,我們應該把他獨立出來,這樣你只需要在一個檔案管理你宣告的物件(瀏覽器)設定,而主程式很單純的使用回傳的物件(瀏覽器)即可
initDrive.js
exports.initDrive = initDrive;//讓其他程式在引入時可以使用這個函式
const webdriver = require('selenium-webdriver') // 加入虛擬網頁套件
const chrome = require('selenium-webdriver/chrome');
const options = new chrome.Options();
options.setUserPreferences({ 'profile.default_content_setting_values.notifications': 1 });//因為FB會有notifications干擾到爬蟲,所以要先把它關閉
const path = require('path');//用於處理文件路徑的小工具
const fs = require("fs");//讀取檔案用
async function initDrive () {
if (!checkDriver()) {// 檢查Driver是否是設定,如果無法設定就結束程式
return
}
// 建立這個broswer的類型
let driver = await new webdriver.Builder().forBrowser("chrome").withCapabilities(options).build();
//考慮到ig在不同螢幕寬度時的Xpath不一樣,所以我們要在這裡設定統一的視窗大小
await driver.manage().window().setRect({ width: 1280, height: 800, x: 0, y: 0 });
return driver
}
function checkDriver() {
try {
chrome.getDefaultService()//確認是否有預設
} catch {
console.warn('找不到預設driver!');
//'../chromedriver.exe'記得調整成自己的路徑
const file_path = '../chromedriver.exe'
//請確認印出來日誌中的位置是否與你路徑相同
console.log(path.join(__dirname, file_path));
//確認路徑下chromedriver.exe是否存在
if (fs.existsSync(path.join(__dirname, file_path))) {
const service = new chrome.ServiceBuilder(path.join(__dirname, file_path)).build();//設定driver路徑
chrome.setDefaultService(service);
console.log('設定driver路徑');
return true
} else {
console.error('無法設定driver路徑');
return false
}
}
return true
}
這篇文章是以自己的程式作為範例講解,如果你想更深入了解重構請閱讀重構—改善既有的程式設計,他深入淺出說明重構的原則讓我受益良多
程式碼往往沒有所謂最好的寫法,只有當下最合理的選擇。而需求永遠在變動,就算當下重構到很滿意,可能下個月再回來看又覺得不夠了。 - YH Yu
執行程式
-
在專案資料夾的終端機(Terminal)執行指令
yarn start -
觀察執行順序:Instagram 自動登入 → 跳轉到指定帳號 → Facebook自動登入 → 跳轉到粉絲頁 → 關閉
-
確定與程式重構後能順利取得FB & IG的追蹤人數

如果你還有什麼問題或是覺得有可以改善的地方歡迎在下方留言討論
ℹ️ 專案原始碼
- 今天的完整程式碼可以在這裡找到喔
- 我也貼心地把昨天的把昨天的程式碼打包成壓縮檔,你可以用裡面乾淨的環境來實作今天的重構喔
- 請記得在終端機下指令 yarn 才會把之前的套件安裝
- windows需下載與你chrome版本相同的chrome driver放在專案根目錄
- 調整.env檔
- 填上FB登入資訊
- 填上FB版本(classic/new)
- 填上IG登入資訊
參考資源
免責聲明:文章技術僅抓取公開數據作爲研究,任何組織和個人不得以此技術盜取他人智慧財產、造成網站損害,否則一切后果由該組織或個人承擔。作者不承擔任何法律及連帶責任!
在許多人的幫助下,本系列文章已出版成書,並添加了新的篇章與細節補充:
- 加入更多實務經驗,用完整的開發流程讓讀者了解專案每個階段要注意的事項
- 將爬蟲的步驟與技巧做更詳細的說明,讓讀者可以輕鬆入門
- 調整專案架構
- 優化爬蟲程式,以更廣的視角來擷取網頁資訊
- 增加資料驗證、錯誤通知等功能,讓爬蟲執行遇到問題時可以第一時間通知使用者
- 排程部分改用 node-schedule & pm2 的組合,讓讀者可以輕鬆管理專案程序並獲得更精確的 log 資訊
有興趣的朋友可以到天瓏書局選購,感謝大家的支持。

留言載入中…