JOINコマンドを実装

[paphio-kitsuki]

JOINコマンドを実装するにあたり、様々な変更を加えました。 JOIN系関数(CkJoin~, ExJoin~)以外の部分での変更は以下の通りです。 ・Eventにis_do_nothing変数を追加 (コマンドを実行する(Executeに回す)意味がないことを示す変数。JOINでそれを判定するとき、変数を使用しないと実装できなかったため。) -> 認証前で実行しても意味がない場合を判定する文も、現在はEventHandlerに記述していますが、この変数を利用したいです。 ・Channelを、オペレーターが複数いても大丈夫なように変更 ・Channelのモードの管理方法をmapに変更 ・Channelのnicknameによる検索において、一致条件を変更 (こちらの問題に対応しました) ・EventHandler内に、必要に応じてChannelを生成する関数を作成 (こちらに対応) 　↑これDatabase内でも良いかも？ ・UserにChannelを持たせるよう変更 (こちらに対応) ・DatabaseのCheck内で、EventConfiguratorをすべて回した後に実行する関数「AfterCheck」の実装 ・utilsにStrToLowerや、vectorを継承して新たに「Contains」と「Remove」関数を追加したクラスを実装

makeによるコンパイルは確認済みです。

[rinonaito]

・Channelのnicknameによる検索において、一致条件を変更 (こちらの問題に対応しました)

調査ありがとう！これ、知らなかった。。。 では、ニックネームはレターケースを無視した場合にも一意であることが保証されている、という想定ですよね？ であれば、NICKコマンドの実装を一部修正します。 現状、ニックネーム設定時の一意性確認はレターケースのみでも差異があれば一意であるとして受け付けちゃってるんで。。。（現状、abcが存在しても、NICK AbC は成功します）

[paphio-kitsuki]

そうですね。NICKに関しても、もともとabcの人がNICK AbC としても、何も表示されず、abcのままだったことは確認しました。 NICKの対応お願いします。

[weijuan82113]

-> 認証前で実行しても意味がない場合を判定する文も、現在はEventHandlerに記述していますが、この変数を利用したいです。

了解です。 ・マージしてから対応したいですので、レビューの修正をお願いします。 現在は基本のコマンド実行でセグフォーが発生します。確認をお願いします。

join #ch1

・エラーが発生した場合は、EventHandlerのExecuteを回しますので、該当変数に関係ない認識で大丈夫ですか？

[paphio-kitsuki]

はい。その認識で大丈夫です。

[weijuan82113]

再度コードを確認してみました、下記シーケンス図を参考して認識合わせをしたいです。

Drawio

現在動作確認したところセグフォが発生していて、AddNewChannelの処理にchannel名が設定されていないが問題です、 もう一度ロージックを確認して、今日チャートで話した下記問題点を後で確認させていただきたいです。

・フラグを追加してDBのcheckで対象が存在するか確認？ ・パラメータにニックネームがある場合？

[paphio-kitsuki]

修正しました。 今度は挙動も正常であることを確認しました。 所属済みのチャンネルに対するJOINについて... 自身の所属しているチャンネルが最大数に達している場合：エラー それ以外(チャンネルにモードi,k,lが設定されている場合)：特になし だったので、それに合わせましたが、全てのエラーに対して、表示するか何もしないか統一した方が良いかもしれません。

[weijuan82113]

下記を確認しました。

【確認済みテスト】

• エラー：

  • （OK）403 nick1 JOIN : No such channel
  • （OK）461 nick1 JOIN : Not enough parameters.
  • （OK）405 nick1 JOIN : You have joined too many channels

• 動作：

  • （OK）チャンネルに参加
  • （OK）複数チャンネル参加

【未確認テスト】

• ERR_BADCHANNELKEY：（modeでkモードを設定後に確認する）
• ERR_INVITEONLYCHAN：（modeでiモードを設定後に確認する）
• ERR_CHANNELISFULL：（modeでlモードを設定後に確認する）

[paphio-kitsuki]

ちなみに、僕は各modeも確認済みです。