team_battle.php のスクリプト内でエラーが起きた時にcatchしてエラー出力しスクリプト実行は継続する対応 #2530

概要

team_battleスクリプト内の一騎討ちで戦闘でエラーが発生した場合、全部の一騎討ちが実行されなくなるため、
一つの戦闘でエラーが発生しても一騎討ちが行われるように修正する

また、エラーが起きた戦闘のuser_noとwar_idとエラー内容をログに出力させる。

テスト方法、確認した内容

2箇所以上のチームで戦争を開始し、jrye_teamWar.teamWar_lockのsingle_combat_time_1 ~ 3を1に。
一騎討ちをさせるユーザーのデッキに含まれて発動するスキルSkillFunc.incにエラーを仕込んで
team_battle.phpを実行。
↓
一騎討ちのエラーが出力されることと、
問題がない次の一騎討ちが行われていればOK

こちらではdivision by zero のエラーを仕込んでみて確認しました。

TODO

エラーが起きた時にSlackで通知をするなど検知できるようにしたい

@wada
レビューお願いいたします

@kouho
92行目以降の各if文の中身はやっていることが同じなので統一できないでしょうか？

@wada
確認ありがとうございます。
以下対応しました。
> continueの前でrollbackをお願いします。
> 92行目以降の各if文の中身はやっていることが同じなので統一できないでしょうか？

@kouho
対応ありがとうございます。

変更後にfor文で3回メインループと一騎討ちを実行していますが、メインループは1回で良いと思います。
変更前の一騎討ちは、if文で1つだけ実行するようになっていましたが、3回連続実行による問題はないでしょうか？

@wada
確認ありがとうございます。
3回のfor文は一騎討ちが実行済みかどうかの判定にのみ使い、

1回のスクリプト実行で一つの戦争につきメインループと一騎討ちの処理は1回ずつ実行して、
136行目のcontinue 2; で次の戦争の処理へ移り、

エラーが起きた場合は、 single_combat_time_{combat_rank}に0を入れて
continue 2;で次の戦争へとなるため、1戦争につき、3回連続実行にはならない想定でした。

@wada
> 一騎討ち終了後の条件も追加お願いします。

確認ありがとうございます。追加しました。

@kouho
ロールバックの順番以外はOKです。

@wada
確認ありがとうございます。ロールバックの順番も修正しました。

@kouho
対応ありがとうございます。

@kouho
continueの前でrollbackをお願いします。

@kouho
一騎討ち終了後の条件も追加お願いします。

@kouho
37の前に39のロールバックをお願いします。