レビュー 2022-02-25

[Reputeless]

2022-0227 実装会 or 3 月の個別 MTG 用メモ

機能

• verbose モードが有効なときのみ Print するようにする
  • メンバ変数 bool m_verbose = true;
• char, int8, uint8, uint16, int16, char32, uint32, int64, uint64 型のサポートを追加できる？
• 図形・カラー型に関する Grid は、以下の型以外を削除すべき（使用頻度が低いと思われるため）
  • Point, Float2, Float3, Float4, Vec2, Vec3, Vec4, Color, ColorF, HSV
• assert(InRange(maxPlayers, 0, 255)); 系の関数、範囲外は即 return して、「範囲外は何もしない」とドキュメントする？

Print << U"SivPhoton::customEventAction(Float3)";
Print << U"playerID: " << playerID;
Print << U"eventCode: " << eventCode;
Print << U"eventContent: " << eventContent;

は PrintEventAction(U"...", playerID, eventCode, eventContent); に置き換えてコードを短縮する？

• Print 系はすべて Print << U"[SivPhoton] ..."; のように SivPhoton 発のメッセージであることがわかるようにする
• bool rejoin は統合時に YesNo クラスにする
• String getName() → String getUserName()

/// @brief データの送信を行います。
/// @tparam T Siv3D系のクラス
/// @param eventCode イベントコード
/// @param value 送信するデータ
template <class T>
void opRaiseEvent(uint8 eventCode, const T& value);

template<class T>
void s3d::SivPhoton::opRaiseEvent(const uint8 eventCode, const T& value)
{
    this->opRaiseEvent<T>(eventCode, value);
}

• これは再帰している？ 丸ごと不要？
• # define NOMINMAX の代わりに # include <Siv3D/Windows/Windows.hpp> に置き換える

# if SIV3D_PLATFORM(WINDOWS)
# if SIV3D_BUILD(DEBUG)
#   pragma comment (lib, "Common-cpp/lib/Common-cpp_vc16_debug_windows_mt_x64")
#   pragma comment (lib, "Photon-cpp/lib/Photon-cpp_vc16_debug_windows_mt_x64")
#   pragma comment (lib, "LoadBalancing-cpp/lib/LoadBalancing-cpp_vc16_debug_windows_mt_x64")
# else
#   pragma comment (lib, "Common-cpp/lib/Common-cpp_vc16_release_windows_mt_x64")
#   pragma comment (lib, "Photon-cpp/lib/Photon-cpp_vc16_release_windows_mt_x64")
#   pragma comment (lib, "LoadBalancing-cpp/lib/LoadBalancing-cpp_vc16_release_windows_mt_x64")
# endif
# endif

• これはヘッダに持ってくる（必要なファイルの名前をユーザが見えるように）
• SivCustomType で、public が先、private があとにくるように
• detail の 2 つの関数は static に
• SivCustomType 引数が 1 つのコンストラクタは explicit に
• SivPhoton, SivCustomType コンストラクタに SIV3D_NODISCARD_CXX20
• SivPhoton, デフォルトコンストラクタを用意できる？
• detail::ToJString(Unicode::Widen(typeid(T).name()));, 変換を多重ではなく 1 回に

const T* hoge = reinterpret_cast<const T*>(pData);
std::memcpy(&m_value, hoge, sizeof(T));

• は std::memcpy(&m_value, pData, sizeof(T)); に短縮

スタイル

• template<> を template <> のように半角スペースを入れてください
• T を Type に置き換えてください
• @brief サーバーといい感じにします。 → @brief サーバーと同期します が良い？
• eventContent が少し長い。content とか data とか？
• for (const auto i : step(length)) は for (size_t i = 0; i < length; ++i) にしてください。Debug ビルドで最適なコードが生成されない可能性があるため

ドキュメント

• defaultRoomName を指定しない場合何が起こるのか

[Reputeless]

2022-0227 実装会メモ

Grid → 全消し //
Array → Point, Float2, Float3, Float4, Vec2, Vec3, Vec4, Color, ColorF, HSV + C++ 基本型以外削除
asser → return + Print <<

userName → 使える文字、文字数について調査（名前の制約について情報提供）

        SivCustomType(const T& value)
            : ExitGames::Common::CustomType<SivCustomType<T, customTypeIndex>, customTypeIndex>{}
            , m_value{ value } {}

void cleanup() {}

~SivCustomType(void) // void いらない