tmcit-2024-adv-prog / quantum-gomoku

量子五目並べ
MIT License
0 stars 0 forks source link

TCP通信用の機能の追加 #26

Closed 5hrz closed 1 day ago

5hrz commented 2 weeks ago

本PRでは以下の機能を追加します。

なお、本PRでは他クラスとの連携を行なっていないため、TODOが残っています。

本機能を追加する際には以下のように記載します。


public void someSenderSideMethod() {
    // 接続を最初に受け待つ側の処理
    TCPSocket tcpSocket = new TCPSocket(new TCPSocketProps(null, 5050));
    // ソケットの初期化
    tcpSocket.initSender();
    // 接続相手からの通信を待ち受ける
    tcpSocket.startReceive();
}

public void someReceiverSideMethod() {
    // 接続を最初に受け待つ側の処理
    TCPSocket tcpSocket = new TCPSocket(new TCPSocketProps("192.168.100.1", 5050));
    // ソケットの初期化
    tcpSocket.initReceiver();
    // 接続相手からの通信を待ち受ける
    tcpSocket.startReceive();
}
github-actions[bot] commented 2 weeks ago
Overall Project 41.67% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 week ago
Overall Project 41.67% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 week ago
Overall Project 41.67% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 week ago
Overall Project 41.67% :x:

There is no coverage information present for the Files changed

5hrz commented 2 days ago
  • rebaseしてほしい

  • Receiver Senderのインターフェイスはinterfacesディレクトリに移動してほしい

後ほど対応します

  • GameStatusはGameStateに改名されました

    • GameStateは試合の状態(盤上の石の配置とかプレイヤーの情報とか手番の情報とか)を持つオブジェクトにしてほしい(具体的なクラスの実装は試合担当の実装を待つ)

Serializableを保ってもらえればどのような構造であっても送受信できるようにしてあるので一旦TODO統合時に調整したいです

cffnpwr commented 2 days ago
  • rebaseしてほしい
  • Receiver Senderのインターフェイスはinterfacesディレクトリに移動してほしい

後ほど対応します

  • GameStatusはGameStateに改名されました

    • GameStateは試合の状態(盤上の石の配置とかプレイヤーの情報とか手番の情報とか)を持つオブジェクトにしてほしい(具体的なクラスの実装は試合担当の実装を待つ)

Serializableを保ってもらえればどのような構造であっても送受信できるようにしてあるので一旦TODO統合時に調整したいです

把握しました 対応後にapproveしてマージします

あと、完全に忘れてましたが、GameState(GameStatusだったもの)はmodelsディレクトリに移動してほしいです

github-actions[bot] commented 1 day ago
Overall Project 10.64% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 day ago
Overall Project 10.64% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 day ago
Overall Project 10.64% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 day ago
Overall Project 10.64% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 day ago
Overall Project 10.64% :x:

There is no coverage information present for the Files changed

github-actions[bot] commented 1 day ago
Overall Project 10.64% -6.38% :x:
Files changed 0% :x:


File Coverage
GameState.java 0% :x:
5hrz commented 1 day ago

ごめんなさいめちゃくちゃコミットログ荒らしました

cffnpwr commented 1 day ago

Overall Project 10.64% -6.38% ❌ Files changed 0% ❌

File Coverage
GameState.java 0% ❌

カバレッジちゃんと出てないな