自拍偷在线精品自拍偷,亚洲欧美中文日韩v在线观看不卡

記錄第一次給開源項目提 PR

開源
本篇文章算是該系列的一個彩蛋篇,記錄一下第一次給開源項目提 PR 的過程(之前好像也有過,不過那個非常小的一個改動),希望能夠幫助更多的人參與到開源項目中來。

注:本系列對 ahooks 的源碼解析是基于 v3.3.13。自己 folk 了一份源碼,主要是對源碼做了一些解讀,可見 詳情[1]。

本篇文章算是該系列的一個彩蛋篇,記錄一下第一次給開源項目提 PR 的過程(之前好像也有過,不過那個非常小的一個改動),希望能夠幫助更多的人參與到開源項目中來。

起因

在寫了幾篇關于 ahooks 的文章之后,收到了官方同學的私信。

圖片

這讓我受寵若驚的同時也有點小興奮和惶恐。

興奮是,之前感覺參與開源是一件遙不可及的事情,現在似乎我也能夠去做了。當然也有私心,假如我的簡歷上有給開源項目做貢獻的經歷,那豈不是一個不錯的加分項?

惶恐的是,我之前沒有參與過開源項目,擔心自己不能做好這件事。

根據大佬的建議,我決定先從一些 issue 入手,也就是幫忙解決一下 issue。

明確問題OR需求

于是我抱著試試看的態(tài)度,看了一下官方的 issue,看到這么一條。issue 詳情[2]。

圖片

剛好我之前對 useRequest 源碼做過一些分析——如何使用插件化機制優(yōu)雅的封裝你的請求[3]。于是我決定 fix 一下這個 issue。

這個 issue 的需求很簡單,就是希望輪詢失敗后,能夠支持最大的輪詢次數,假如失敗的次數大于這個值,則停止輪詢。

編碼前準備

首先,從 ahooks 官方 GitHub 中 folk 一份。這個操作我之前已經做了。

圖片

第二步,基于 master 切換一個功能分支。如下:

git checkout -b fix/pollingSupportRetryCount

最后就是環(huán)境的一些初始化操作,不同的倉庫不同,ahooks 如下:

yarn run init
yarn start

功能實現

我們先來看下現在 useRequest 的輪詢的實現,其原理主要是在一個請求結束的時候(不管成功與失?。?,通過 setTimeout 進行重新請求,達到輪詢的效果。

onFinally: () => {
// 省略部分代碼...
// 通過 setTimeout 進行輪詢
timerRef.current = setTimeout(() => {
fetchInstance.refresh();
}, pollingInterval);
},

我的想法是,定義一個 options 參數,pollingErrorRetryCount,默認為 -1,代表沒有限制。

另外定義一個變量,記錄當前重試的次數:

const countRef = useRef<number>(0);

當開發(fā)者設置了 pollingErrorRetryCount,并且重試的數量大于該值,我們就直接返回,不執(zhí)行輪詢的邏輯。

當成功或者失敗的時候,更新當前重試的次數:

onError: () => {
countRef.current += 1;
},
onSuccess: () => {
countRef.current = 0;
},

然后在請求結束的時候,判斷重試的次數有沒有達到了開發(fā)設置的次數,假如沒有則執(zhí)行重試操作。有則重置重試的次數,停止輪詢。

onFinally: () => {
if (
pollingErrorRetryCount === -1 ||
// When an error occurs, the request is not repeated after pollingErrorRetryCount retries
(pollingErrorRetryCount !== -1 && countRef.current <= pollingErrorRetryCount)
) {
// 忽略部分代碼
timerRef.current = setTimeout(() => {
fetchInstance.refresh();
}, pollingInterval);
} else {
countRef.current = 0;
}
},

測試用例

上述整體的改造并不困難,但是我在寫測試用例的時候,就開始踩坑了,因為我很少書寫前端的測試用例,還是針對于 hooks 的測試用例。這里是我耗時最多的地方。

最終用例如下:

// 省略部分代碼...
// if request error and set pollingErrorRetryCount
// and the number of consecutive failures exceeds pollingErrorRetryCount, polling stops
let hook2;
let errorCallback;
act(() => {
errorCallback = jest.fn();
hook2 = setUp(() => request(0), {
pollingErrorRetryCount: 3,
pollingInterval: 100,
pollingWhenHidden: true,
onError: errorCallback,
});
});
expect(hook2.result.current.loading).toEqual(true);
expect(errorCallback).toHaveBeenCalledTimes(0);
act(() => {
jest.runAllTimers();
});
await hook2.waitForNextUpdate();
expect(hook2.result.current.loading).toEqual(false);
expect(errorCallback).toHaveBeenCalledTimes(1);
act(() => {
jest.runAllTimers();
});
await hook2.waitForNextUpdate();
expect(errorCallback).toHaveBeenCalledTimes(2);
act(() => {
jest.runAllTimers();
});
await hook2.waitForNextUpdate();
expect(errorCallback).toHaveBeenCalledTimes(3);
act(() => {
jest.runAllTimers();
});
await hook2.waitForNextUpdate();
expect(errorCallback).toHaveBeenCalledTimes(4);
act(() => {
jest.runAllTimers();
});
expect(errorCallback).toHaveBeenCalledTimes(4);
act(() => {
hook2.result.current.run();
});
act(() => {
jest.runAllTimers();
});
await hook2.waitForNextUpdate();
expect(errorCallback).toHaveBeenCalledTimes(5);
hook2.unmount();
// 省略部分代碼...

大致解釋下該測試用例的邏輯,我設置了重試三次,錯誤之后,運行了三次,errorCallback 就會被調用了 4 次(包括錯誤那次)。在第五次執(zhí)行的時候,就不會執(zhí)行 errorCallback,也就還是 4 次。然后我們手動 run 一次請求,期待 errorCallback 應該執(zhí)行 5 次。

這里踩了一個坑,就是第五次請求的時候,我之前是會寫一個等待定時器執(zhí)行的操作,但實際上這里它是不會執(zhí)行定時器的,導致一直報錯,在這里折騰了很久。后來刪除了下面的代碼才執(zhí)行成功。

act(() => {
jest.runAllTimers();
});
- await hook2.waitForNextUpdate();
expect(errorCallback).toHaveBeenCalledTimes(4);

文檔以及 Demo 補充

畢竟加了一個新的 API 參數,需要在文檔中注明,而且中英文文檔都需要補充,還加上了一個 Demo 示例。

圖片圖片

提 PR

上述都完成之后,就可以提交你的代碼了,提交完,去到在你 folk 過來的項目中,可以看到這個。

圖片

我們需要點擊圖中框起來的「Compare & pull request 」,之后就會出現如下圖

圖片

默認會幫我們選好分支的,我們只需要完善其中的信息,還有我們之前提交的 message 也可以修改。最好可以用英文來解釋,本次提交的內容。

最后點擊提交之后就好了。

還有一個提 PR 的入口,如下所示:

圖片圖片

最后等待官方 CR 就可以了(上面的實現其實部分是 CR 后改的)。目前該 PR 已經被合入到 master。

總結思考

給開源項目提 PR 操作過程不是一件很復雜的事情,重點在于需求的修改。往往需要考慮到多種邊界場景,這個時候,我們就需要前端的單元測試來幫助我們覆蓋全面的場景。

另外,對于一些還沒有參與開源項目經驗的同學來講,我覺得類似 ahooks 這種工具庫是一個不錯的選擇:

  • 它的模塊劃分更加清晰,你改了一個模塊的功能,影響面可以更好的預估。對新人比較友好。
  • 邏輯相對簡單,其實你會發(fā)現很多代碼說不定在你們的業(yè)務項目中的 utils/hooks 文件夾中就有。
  • 社區(qū)比較活躍,維護者能夠較快的響應。

希望對大家有所幫助。

參考資料

[1]詳情: https://github.com/GpingFeng/hooks。

[2]issue 詳情: https://github.com/alibaba/hooks/issues/1645。

[3]如何使用插件化機制優(yōu)雅的封裝你的請求: https://juejin.cn/post/7105733829972721677。

責任編輯:姜華 來源: 前端雜貨鋪
相關推薦

2011-07-21 21:01:37

諾基亞塞班蘋果

2017-03-22 15:38:28

代碼架構Java

2023-09-11 00:14:46

后端團隊項目

2015-08-17 10:10:10

創(chuàng)業(yè)失敗經驗

2014-05-19 10:16:56

WinJS開源TypeScript

2012-04-13 10:11:58

Windows 8泄露

2022-03-16 14:59:28

打包debian模板文件

2022-08-15 08:16:56

shiroWeb認證

2021-08-05 08:18:02

開源項目 PR

2015-10-26 16:38:17

2021-02-05 08:35:21

私活程序員

2016-07-14 14:07:04

云計算OpenStack

2013-06-03 09:28:49

游戲設計

2013-05-13 11:35:53

獨立開發(fā)開發(fā)經驗開發(fā)感悟

2018-09-11 17:05:12

戴爾

2024-07-09 10:20:05

VueJSX函數

2024-01-03 21:50:32

緩存機制請求

2022-05-06 11:27:23

虛擬人白皮書行業(yè)

2010-05-27 10:00:09

2012-01-18 11:18:12

Web App
點贊
收藏

51CTO技術棧公眾號