RustによるGUIアプリケーションを追加 #5
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Walkthroughこの変更は、大規模なマージャンゲームアプリケーションのセットアップを強化します。設定ファイルの微調整、Rustアプリケーションの新しいモジュール、インターフェースの改善、モーダルウィジェットの導入が含まれています。これにより、ユーザーインターフェースのインタラクションが向上し、プロジェクト全体の機能性が拡張されました。 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant App
participant GameCore
participant Modal
User->>App: ゲームを開始
App->>GameCore: ゲーム状態の初期化
GameCore-->>App: ゲーム状態を返す
App->>User: ゲーム画面を表示
User->>App: タイルを捨てる
App->>GameCore: タイルを処理
GameCore-->>App: 更新されたゲーム状態
App->>User: 更新されたゲーム画面を表示
User->>App: モーダルを表示
App->>Modal: モーダルウィジェットを表示
Modal-->>User: ユーザーへ情報提供
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (2)
app/Cargo.toml (2)
7-8: 依存関係のバージョンを固定することを検討してください。
icedとiced_awのバージョンが固定されていますが、将来の互換性の問題を避けるために、特定のパッチバージョンを指定することを検討してください。- iced = { version = "0.12.1", default-features = false, features = ["image", "debug"] } - iced_aw = { version = "0.8.0", default-features = false, features = ["tabs"] } + iced = { version = "0.12.1", default-features = false, features = ["image", "debug"], patch = "0" } + iced_aw = { version = "0.8.0", default-features = false, features = ["tabs"], patch = "0" }
15-16: パス依存関係のバージョン管理に注意してください。
ai_bridgeとmahjong_coreはパス依存関係として指定されています。これらのモジュールが頻繁に変更される場合、バージョン管理や変更履歴の管理が重要です。
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files ignored due to path filters (274)
Cargo.lockis excluded by!**/*.lockapp/fonts/Mamelon-5-Hi-Regular.otfis excluded by!**/*.otfapp/images/haiga/haku.gifis excluded by!**/*.gifapp/images/haiga/hakured.gifis excluded by!**/*.gifapp/images/haiga/hatu.gifis excluded by!**/*.gifapp/images/haiga/hatured.gifis excluded by!**/*.gifapp/images/haiga/man1.gifis excluded by!**/*.gifapp/images/haiga/man2.gifis excluded by!**/*.gifapp/images/haiga/man3.gifis excluded by!**/*.gifapp/images/haiga/man4.gifis excluded by!**/*.gifapp/images/haiga/man5.gifis excluded by!**/*.gifapp/images/haiga/man6.gifis excluded by!**/*.gifapp/images/haiga/man7.gifis excluded by!**/*.gifapp/images/haiga/man8.gifis excluded by!**/*.gifapp/images/haiga/man9.gifis excluded by!**/*.gifapp/images/haiga/manred1.gifis excluded by!**/*.gifapp/images/haiga/manred2.gifis excluded by!**/*.gifapp/images/haiga/manred3.gifis excluded by!**/*.gifapp/images/haiga/manred4.gifis excluded by!**/*.gifapp/images/haiga/manred5.gifis excluded by!**/*.gifapp/images/haiga/manred6.gifis excluded by!**/*.gifapp/images/haiga/manred7.gifis excluded by!**/*.gifapp/images/haiga/manred8.gifis excluded by!**/*.gifapp/images/haiga/manred9.gifis excluded by!**/*.gifapp/images/haiga/nan.gifis excluded by!**/*.gifapp/images/haiga/nanred.gifis excluded by!**/*.gifapp/images/haiga/pei.gifis excluded by!**/*.gifapp/images/haiga/peired.gifis excluded by!**/*.gifapp/images/haiga/pin1.gifis excluded by!**/*.gifapp/images/haiga/pin2.gifis excluded by!**/*.gifapp/images/haiga/pin3.gifis excluded by!**/*.gifapp/images/haiga/pin4.gifis excluded by!**/*.gifapp/images/haiga/pin5.gifis excluded by!**/*.gifapp/images/haiga/pin6.gifis excluded by!**/*.gifapp/images/haiga/pin7.gifis excluded by!**/*.gifapp/images/haiga/pin8.gifis excluded by!**/*.gifapp/images/haiga/pin9.gifis excluded by!**/*.gifapp/images/haiga/pinred1.gifis excluded by!**/*.gifapp/images/haiga/pinred2.gifis excluded by!**/*.gifapp/images/haiga/pinred3.gifis excluded by!**/*.gifapp/images/haiga/pinred4.gifis excluded by!**/*.gifapp/images/haiga/pinred5.gifis excluded by!**/*.gifapp/images/haiga/pinred6.gifis excluded by!**/*.gifapp/images/haiga/pinred7.gifis excluded by!**/*.gifapp/images/haiga/pinred8.gifis excluded by!**/*.gifapp/images/haiga/pinred9.gifis excluded by!**/*.gifapp/images/haiga/sha.gifis excluded by!**/*.gifapp/images/haiga/shared.gifis excluded by!**/*.gifapp/images/haiga/sou1.gifis excluded by!**/*.gifapp/images/haiga/sou2.gifis excluded by!**/*.gifapp/images/haiga/sou3.gifis excluded by!**/*.gifapp/images/haiga/sou4.gifis excluded by!**/*.gifapp/images/haiga/sou5.gifis excluded by!**/*.gifapp/images/haiga/sou6.gifis excluded by!**/*.gifapp/images/haiga/sou7.gifis excluded by!**/*.gifapp/images/haiga/sou8.gifis excluded by!**/*.gifapp/images/haiga/sou9.gifis excluded by!**/*.gifapp/images/haiga/soured1.gifis excluded by!**/*.gifapp/images/haiga/soured2.gifis excluded by!**/*.gifapp/images/haiga/soured3.gifis excluded by!**/*.gifapp/images/haiga/soured4.gifis excluded by!**/*.gifapp/images/haiga/soured5.gifis excluded by!**/*.gifapp/images/haiga/soured6.gifis excluded by!**/*.gifapp/images/haiga/soured7.gifis excluded by!**/*.gifapp/images/haiga/soured8.gifis excluded by!**/*.gifapp/images/haiga/soured9.gifis excluded by!**/*.gifapp/images/haiga/thaku.gifis excluded by!**/*.gifapp/images/haiga/thakured.gifis excluded by!**/*.gifapp/images/haiga/thatu.gifis excluded by!**/*.gifapp/images/haiga/thatured.gifis excluded by!**/*.gifapp/images/haiga/tman1.gifis excluded by!**/*.gifapp/images/haiga/tman2.gifis excluded by!**/*.gifapp/images/haiga/tman3.gifis excluded by!**/*.gifapp/images/haiga/tman4.gifis excluded by!**/*.gifapp/images/haiga/tman5.gifis excluded by!**/*.gifapp/images/haiga/tman6.gifis excluded by!**/*.gifapp/images/haiga/tman7.gifis excluded by!**/*.gifapp/images/haiga/tman8.gifis excluded by!**/*.gifapp/images/haiga/tman9.gifis excluded by!**/*.gifapp/images/haiga/tmanred1.gifis excluded by!**/*.gifapp/images/haiga/tmanred2.gifis excluded by!**/*.gifapp/images/haiga/tmanred3.gifis excluded by!**/*.gifapp/images/haiga/tmanred4.gifis excluded by!**/*.gifapp/images/haiga/tmanred5.gifis excluded by!**/*.gifapp/images/haiga/tmanred6.gifis excluded by!**/*.gifapp/images/haiga/tmanred7.gifis excluded by!**/*.gifapp/images/haiga/tmanred8.gifis excluded by!**/*.gifapp/images/haiga/tmanred9.gifis excluded by!**/*.gifapp/images/haiga/tnan.gifis excluded by!**/*.gifapp/images/haiga/tnanred.gifis excluded by!**/*.gifapp/images/haiga/ton.gifis excluded by!**/*.gifapp/images/haiga/tonred.gifis excluded by!**/*.gifapp/images/haiga/tpei.gifis excluded by!**/*.gifapp/images/haiga/tpeired.gifis excluded by!**/*.gifapp/images/haiga/tpin1.gifis excluded by!**/*.gifapp/images/haiga/tpin2.gifis excluded by!**/*.gifapp/images/haiga/tpin3.gifis excluded by!**/*.gifapp/images/haiga/tpin4.gifis excluded by!**/*.gifapp/images/haiga/tpin5.gifis excluded by!**/*.gifapp/images/haiga/tpin6.gifis excluded by!**/*.gifapp/images/haiga/tpin7.gifis excluded by!**/*.gifapp/images/haiga/tpin8.gifis excluded by!**/*.gifapp/images/haiga/tpin9.gifis excluded by!**/*.gifapp/images/haiga/tpinred1.gifis excluded by!**/*.gifapp/images/haiga/tpinred2.gifis excluded by!**/*.gifapp/images/haiga/tpinred3.gifis excluded by!**/*.gifapp/images/haiga/tpinred4.gifis excluded by!**/*.gifapp/images/haiga/tpinred5.gifis excluded by!**/*.gifapp/images/haiga/tpinred6.gifis excluded by!**/*.gifapp/images/haiga/tpinred7.gifis excluded by!**/*.gifapp/images/haiga/tpinred8.gifis excluded by!**/*.gifapp/images/haiga/tpinred9.gifis excluded by!**/*.gifapp/images/haiga/tsha.gifis excluded by!**/*.gifapp/images/haiga/tshared.gifis excluded by!**/*.gifapp/images/haiga/tsou1.gifis excluded by!**/*.gifapp/images/haiga/tsou2.gifis excluded by!**/*.gifapp/images/haiga/tsou3.gifis excluded by!**/*.gifapp/images/haiga/tsou4.gifis excluded by!**/*.gifapp/images/haiga/tsou5.gifis excluded by!**/*.gifapp/images/haiga/tsou6.gifis excluded by!**/*.gifapp/images/haiga/tsou7.gifis excluded by!**/*.gifapp/images/haiga/tsou8.gifis excluded by!**/*.gifapp/images/haiga/tsou9.gifis excluded by!**/*.gifapp/images/haiga/tsoured1.gifis excluded by!**/*.gifapp/images/haiga/tsoured2.gifis excluded by!**/*.gifapp/images/haiga/tsoured3.gifis excluded by!**/*.gifapp/images/haiga/tsoured4.gifis excluded by!**/*.gifapp/images/haiga/tsoured5.gifis excluded by!**/*.gifapp/images/haiga/tsoured6.gifis excluded by!**/*.gifapp/images/haiga/tsoured7.gifis excluded by!**/*.gifapp/images/haiga/tsoured8.gifis excluded by!**/*.gifapp/images/haiga/tsoured9.gifis excluded by!**/*.gifapp/images/haiga/tton.gifis excluded by!**/*.gifapp/images/haiga/ttonred.gifis excluded by!**/*.gifapp/images/haiga/ttyun.gifis excluded by!**/*.gifapp/images/haiga/tura.gifis excluded by!**/*.gifapp/images/haiga/tyhaku.gifis excluded by!**/*.gifapp/images/haiga/tyhakured.gifis excluded by!**/*.gifapp/images/haiga/tyhatu.gifis excluded by!**/*.gifapp/images/haiga/tyhatured.gifis excluded by!**/*.gifapp/images/haiga/tyman1.gifis excluded by!**/*.gifapp/images/haiga/tyman2.gifis excluded by!**/*.gifapp/images/haiga/tyman3.gifis excluded by!**/*.gifapp/images/haiga/tyman4.gifis excluded by!**/*.gifapp/images/haiga/tyman5.gifis excluded by!**/*.gifapp/images/haiga/tyman6.gifis excluded by!**/*.gifapp/images/haiga/tyman7.gifis excluded by!**/*.gifapp/images/haiga/tyman8.gifis excluded by!**/*.gifapp/images/haiga/tyman9.gifis excluded by!**/*.gifapp/images/haiga/tymanred1.gifis excluded by!**/*.gifapp/images/haiga/tymanred2.gifis excluded by!**/*.gifapp/images/haiga/tymanred3.gifis excluded by!**/*.gifapp/images/haiga/tymanred4.gifis excluded by!**/*.gifapp/images/haiga/tymanred5.gifis excluded by!**/*.gifapp/images/haiga/tymanred6.gifis excluded by!**/*.gifapp/images/haiga/tymanred7.gifis excluded by!**/*.gifapp/images/haiga/tymanred8.gifis excluded by!**/*.gifapp/images/haiga/tymanred9.gifis excluded by!**/*.gifapp/images/haiga/tynan.gifis excluded by!**/*.gifapp/images/haiga/tynanred.gifis excluded by!**/*.gifapp/images/haiga/typei.gifis excluded by!**/*.gifapp/images/haiga/typeired.gifis excluded by!**/*.gifapp/images/haiga/typin1.gifis excluded by!**/*.gifapp/images/haiga/typin2.gifis excluded by!**/*.gifapp/images/haiga/typin3.gifis excluded by!**/*.gifapp/images/haiga/typin4.gifis excluded by!**/*.gifapp/images/haiga/typin5.gifis excluded by!**/*.gifapp/images/haiga/typin6.gifis excluded by!**/*.gifapp/images/haiga/typin7.gifis excluded by!**/*.gifapp/images/haiga/typin8.gifis excluded by!**/*.gifapp/images/haiga/typin9.gifis excluded by!**/*.gifapp/images/haiga/typinred1.gifis excluded by!**/*.gifapp/images/haiga/typinred2.gifis excluded by!**/*.gifapp/images/haiga/typinred3.gifis excluded by!**/*.gifapp/images/haiga/typinred4.gifis excluded by!**/*.gifapp/images/haiga/typinred5.gifis excluded by!**/*.gifapp/images/haiga/typinred6.gifis excluded by!**/*.gifapp/images/haiga/typinred7.gifis excluded by!**/*.gifapp/images/haiga/typinred8.gifis excluded by!**/*.gifapp/images/haiga/typinred9.gifis excluded by!**/*.gifapp/images/haiga/tysha.gifis excluded by!**/*.gifapp/images/haiga/tyshared.gifis excluded by!**/*.gifapp/images/haiga/tysou1.gifis excluded by!**/*.gifapp/images/haiga/tysou2.gifis excluded by!**/*.gifapp/images/haiga/tysou3.gifis excluded by!**/*.gifapp/images/haiga/tysou4.gifis excluded by!**/*.gifapp/images/haiga/tysou5.gifis excluded by!**/*.gifapp/images/haiga/tysou6.gifis excluded by!**/*.gifapp/images/haiga/tysou7.gifis excluded by!**/*.gifapp/images/haiga/tysou8.gifis excluded by!**/*.gifapp/images/haiga/tysou9.gifis excluded by!**/*.gifapp/images/haiga/tysoured1.gifis excluded by!**/*.gifapp/images/haiga/tysoured2.gifis excluded by!**/*.gifapp/images/haiga/tysoured3.gifis excluded by!**/*.gifapp/images/haiga/tysoured4.gifis excluded by!**/*.gifapp/images/haiga/tysoured5.gifis excluded by!**/*.gifapp/images/haiga/tysoured6.gifis excluded by!**/*.gifapp/images/haiga/tysoured7.gifis excluded by!**/*.gifapp/images/haiga/tysoured8.gifis excluded by!**/*.gifapp/images/haiga/tysoured9.gifis excluded by!**/*.gifapp/images/haiga/tyton.gifis excluded by!**/*.gifapp/images/haiga/tytonred.gifis excluded by!**/*.gifapp/images/haiga/tytyun.gifis excluded by!**/*.gifapp/images/haiga/tyun.gifis excluded by!**/*.gifapp/images/haiga/tyura.gifis excluded by!**/*.gifapp/images/haiga/ura.gifis excluded by!**/*.gifapp/images/haiga/yhaku.gifis excluded by!**/*.gifapp/images/haiga/yhakured.gifis excluded by!**/*.gifapp/images/haiga/yhatu.gifis excluded by!**/*.gifapp/images/haiga/yhatured.gifis excluded by!**/*.gifapp/images/haiga/yman1.gifis excluded by!**/*.gifapp/images/haiga/yman2.gifis excluded by!**/*.gifapp/images/haiga/yman3.gifis excluded by!**/*.gifapp/images/haiga/yman4.gifis excluded by!**/*.gifapp/images/haiga/yman5.gifis excluded by!**/*.gifapp/images/haiga/yman6.gifis excluded by!**/*.gifapp/images/haiga/yman7.gifis excluded by!**/*.gifapp/images/haiga/yman8.gifis excluded by!**/*.gifapp/images/haiga/yman9.gifis excluded by!**/*.gifapp/images/haiga/ymanred1.gifis excluded by!**/*.gifapp/images/haiga/ymanred2.gifis excluded by!**/*.gifapp/images/haiga/ymanred3.gifis excluded by!**/*.gifapp/images/haiga/ymanred4.gifis excluded by!**/*.gifapp/images/haiga/ymanred5.gifis excluded by!**/*.gifapp/images/haiga/ymanred6.gifis excluded by!**/*.gifapp/images/haiga/ymanred7.gifis excluded by!**/*.gifapp/images/haiga/ymanred8.gifis excluded by!**/*.gifapp/images/haiga/ymanred9.gifis excluded by!**/*.gifapp/images/haiga/ynan.gifis excluded by!**/*.gifapp/images/haiga/ynanred.gifis excluded by!**/*.gifapp/images/haiga/ypei.gifis excluded by!**/*.gifapp/images/haiga/ypeired.gifis excluded by!**/*.gifapp/images/haiga/ypin1.gifis excluded by!**/*.gifapp/images/haiga/ypin2.gifis excluded by!**/*.gifapp/images/haiga/ypin3.gifis excluded by!**/*.gifapp/images/haiga/ypin4.gifis excluded by!**/*.gifapp/images/haiga/ypin5.gifis excluded by!**/*.gifapp/images/haiga/ypin6.gifis excluded by!**/*.gifapp/images/haiga/ypin7.gifis excluded by!**/*.gifapp/images/haiga/ypin8.gifis excluded by!**/*.gifapp/images/haiga/ypin9.gifis excluded by!**/*.gifapp/images/haiga/ypinred1.gifis excluded by!**/*.gifapp/images/haiga/ypinred2.gifis excluded by!**/*.gifapp/images/haiga/ypinred3.gifis excluded by!**/*.gifapp/images/haiga/ypinred4.gifis excluded by!**/*.gifapp/images/haiga/ypinred5.gifis excluded by!**/*.gifapp/images/haiga/ypinred6.gifis excluded by!**/*.gifapp/images/haiga/ypinred7.gifis excluded by!**/*.gifapp/images/haiga/ypinred8.gifis excluded by!**/*.gifapp/images/haiga/ypinred9.gifis excluded by!**/*.gifapp/images/haiga/ysha.gifis excluded by!**/*.gifapp/images/haiga/yshared.gifis excluded by!**/*.gifapp/images/haiga/ysou1.gifis excluded by!**/*.gifapp/images/haiga/ysou2.gifis excluded by!**/*.gifapp/images/haiga/ysou3.gifis excluded by!**/*.gifapp/images/haiga/ysou4.gifis excluded by!**/*.gifapp/images/haiga/ysou5.gifis excluded by!**/*.gifapp/images/haiga/ysou6.gifis excluded by!**/*.gifapp/images/haiga/ysou7.gifis excluded by!**/*.gifapp/images/haiga/ysou8.gifis excluded by!**/*.gifapp/images/haiga/ysou9.gifis excluded by!**/*.gifapp/images/haiga/ysoured1.gifis excluded by!**/*.gifapp/images/haiga/ysoured2.gifis excluded by!**/*.gifapp/images/haiga/ysoured3.gifis excluded by!**/*.gifapp/images/haiga/ysoured4.gifis excluded by!**/*.gifapp/images/haiga/ysoured5.gifis excluded by!**/*.gifapp/images/haiga/ysoured6.gifis excluded by!**/*.gifapp/images/haiga/ysoured7.gifis excluded by!**/*.gifapp/images/haiga/ysoured8.gifis excluded by!**/*.gifapp/images/haiga/ysoured9.gifis excluded by!**/*.gifapp/images/haiga/yton.gifis excluded by!**/*.gifapp/images/haiga/ytonred.gifis excluded by!**/*.gifapp/images/haiga/ytyun.gifis excluded by!**/*.gifapp/images/haiga/yura.gifis excluded by!**/*.gif
Files selected for processing (7)
- .coderabbit.yaml (1 hunks)
- Cargo.toml (1 hunks)
- app/Cargo.toml (1 hunks)
- app/images/haiga/readme.txt (1 hunks)
- app/src/main.rs (1 hunks)
- app/src/modal.rs (1 hunks)
- mahjong-app-engine (1 hunks)
Files skipped from review due to trivial changes (4)
- .coderabbit.yaml
- Cargo.toml
- app/images/haiga/readme.txt
- mahjong-app-engine
Additional context used
Path-based instructions (2)
app/src/modal.rs (1)
Pattern
**/*.rs: あなたは @coderabbitai（別名 github-actions[bot]）で、OpenAIによって訓練された言語モデルです。
あなたの目的は、非常に経験豊富なソフトウェアエンジニアとして機能し、コードの一部を徹底的にレビューし、
以下のようなキーエリアを改善するためのコードスニペットを提案することです：
- ロジック
- セキュリティ
- パフォーマンス
- データ競合
- 一貫性
- エラー処理
- 保守性
- モジュール性
- 複雑性
- 最適化
- ベストプラクティス: DRY, SOLID, KISS
些細なコードスタイルの問題や、コメント・ドキュメントの欠落についてはコメントしないでください。
重要な問題を特定し、解決して全体的なコード品質を向上させることを目指してくださいが、細かい問題は意図的に無視してください。app/src/main.rs (1)
Pattern
**/*.rs: あなたは @coderabbitai（別名 github-actions[bot]）で、OpenAIによって訓練された言語モデルです。
あなたの目的は、非常に経験豊富なソフトウェアエンジニアとして機能し、コードの一部を徹底的にレビューし、
以下のようなキーエリアを改善するためのコードスニペットを提案することです：
- ロジック
- セキュリティ
- パフォーマンス
- データ競合
- 一貫性
- エラー処理
- 保守性
- モジュール性
- 複雑性
- 最適化
- ベストプラクティス: DRY, SOLID, KISS
些細なコードスタイルの問題や、コメント・ドキュメントの欠落についてはコメントしないでください。
重要な問題を特定し、解決して全体的なコード品質を向上させることを目指してくださいが、細かい問題は意図的に無視してください。
Additional comments not posted (10)
app/src/modal.rs (3)
11-16:Modal構造体の設計が適切です。
Modal構造体は、ベース要素とモーダル要素を持ち、背景が押されたときのメッセージをオプションで設定できるように設計されています。設計に問題はありません。
41-161:Widgetトレイトの実装が適切です。
ModalのWidgetトレイト実装は、必要なメソッドを適切にオーバーライドしており、icedフレームワークの標準に従っています。
163-295:Overlay構造体とその実装が適切です。
Overlay構造体は、モーダルの位置や内容を管理し、イベント処理や描画を行うために適切に設計されています。特に問題は見当たりません。app/src/main.rs (7)
84-88:unsafeブロックの使用に注意してください。
player_is_riichi関数でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
97-132:doraメソッドのunsafe使用に注意してください。
doraメソッド内でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
133-161:kawahaiメソッドのunsafe使用に注意してください。
kawahaiメソッド内でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
163-215:tehaiメソッドのunsafe使用に注意してください。
tehaiメソッド内でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
230-314:updateメソッドのunsafe使用に注意してください。
updateメソッド内でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
317-357:viewメソッドのunsafe使用に注意してください。
viewメソッド内でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
75-82:unsafeブロックの使用に注意してください。
player_shanten関数でunsafeブロックを使用しています。G_STATEのアクセスが安全であることを確認してください。
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (2)
- app/src/main.rs (1 hunks)
- mahjong-app-engine (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- app/src/main.rs
- mahjong-app-engine
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
新機能
ドキュメンテーション
readme.txtを追加しました。