Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handling timeout #19

Merged
merged 30 commits into from Nov 3, 2017
Merged

Handling timeout #19

merged 30 commits into from Nov 3, 2017

Conversation

arukuka
Copy link
Member

@arukuka arukuka commented Oct 28, 2017

fix #18

該当commit : f46ae06

以下への対処を実装しました。

  • 時間切れ
  • 不正なレスポンス
  • 実行時エラー

上記の事象が発生した場合、ゲームログには
そのプレイヤーのlogの末尾のresultに-1をセットしています。

細かくエラーの種類をresultにセットすることや
ゲームログに残り時間(timeLeft)を追加することを考えましたが
あまりオープンにしすぎると嫌がる方もいらっしゃるかなと思い
見送りました。

実装を急いだだめ、for_windows branchから派生させていただいています。

ktanaka and others added 27 commits October 10, 2017 15:30
…11 specification, extra parentheses indicate the value-initialization instead of the list-initialization, if the GNU extension [-Wgnu-array-member-paren-init] is not specified
debugしていたらobstacledの配列アクセスで
Segmentation faultになっていました。
その修整と、y = length はマップに含まれる
(障害点が存在しうる)ことと
実装が食い違っていたためcommit内容のように修整しました。
Windows環境でもエラーになったため
このbranchをmergeします。
Windowsはファイル・ディレクトリの
大文字・小文字の区別がないため
makeが走りません。そのため
.PHONYを用いるように修整しました。
I beg ya pardon...
相手がもといた位置を通過しているかどうかの判定を先に行い、
両者が動くときに限り動線の交差判定を行う。
@arukuka
Copy link
Member Author

arukuka commented Oct 28, 2017

サンプルコースファイルのthinkTimeが恐らく2秒に設定されていたため、
20秒に設定した方がよろしいかと思います。

@arukuka
Copy link
Member Author

arukuka commented Oct 28, 2017

viewerの表示もエラー時に対応させます。

@arukuka arukuka merged commit 4a368fc into master Nov 3, 2017
@arukuka arukuka deleted the handle_timeout branch November 3, 2017 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In manager, processes for player's timeout, runtime error are unimplemented
1 participant