PHPでのリトライ処理について

Question

質問お願いします。

現在、業務でシステムを作っていてリトライ処理を実装することになりました。

内容としては、カードデータを取得し、['resultCode']が200であれば$successCardsに格納、200以外であれば$getDataの取得をやり直し、3回試みてもダメだったらfalseを返すという内容です（$getDataはモックデータで、これからリクエスト処理を書きます）。

処理の途中までは記載できているのですが、肝心のリトライ処理の実装がイマイチ分からず困っているので分かる方いましたらご教授お願いしたいです🙇‍♂️

            $successCards = [];

            do {
                $retryCount    = 0;
                $maxRetryCount = 3;

                $getData = [
                    "Result" => [
                        "resultCode" => 200,
                    ]
                ];

                // ステータスコードが200であれば配列に格納
                if ($getData['Result']['resultCode'] === '200') {
                    array_push($successCards, $getData);
                };

                // 200以外であればリトライ処理
                if ($getData['Result']['resultCode'] !== '200') {
                    ++$retryCount;
                };

            } while ($maxRetryCount >= $retryCount);

Comments

[AbeTakashi]

なんか、あまり意味のあるリトライ処理に見えませんね。通常はウェイトを入れるなりがありますが、なぜリトライ処理を必要とするのかの事情も第三者からは分かりませんし、なんとも言えないところです。まずはarray_pushしたあとにループを抜ける処理（break）を追加だけして、テストやチーム内でレビューしてみたらどうでしょうか。リトライの仕様が曖昧なら、チーム内で議論すべきでしょう。第三者には何が正しいか分からないので答えにくいし、teratail向けの質問じゃないかぁという気がします。

[nakatievent]

確かに、第三者からすると分からない部分がありますよね。。
一旦breakを追加してプルリク投げて反応を見てみます。
ありがとうございました！

Answer 1

PHP: do-while - Manual
doループ内に、カウンター類の初期化を入れてはいけません。
ループの戦闘に戻ったときにカウンターがリセットされてしまうので、
doの外（先）に置くべきです。

doループの中で、データ取得処理をやってください。
そして、成功と判定した場合にはbreak;でdoループを抜けてください。

カウンター初期化
do {
データ取得
成功判定→成功ならループを抜ける
} while (カウンター超えてないか判定)

もしくは

カウンター初期化
do {
データ取得
} while (成功判定 アンド カウンター超えてないか判定)

という形になるかと。