advancedコマンドをリファクタリング

- `load_character_data()`関数を追加し、キャラクターデータ読み込み処理を分離
- `find_move_and_images()`関数を導入し、技情報と画像データの読み込みロジックを抽出
- `create_advanced_embeds()`関数を作成し、埋め込みメッセージ生成処理を分離
- エラーハンドリングを改善し、関数の責務を明確に分割
- コードの可読性と保守性を向上

Author: eda3

src/commands/frames/advanced.rs

@@ -12,118 +12,103 @@ use std::{fs, string::String};
 /// デフォルト画像URL
 const IMAGE_DEFAULT: &str = "https://www.dustloop.com/wiki/images/5/54/GGST_Logo_Sparkly.png";
 
-/// 指定されたキャラクターの技データを詳細に表示する非同期コマンド
-///
-/// # 概要
-/// 入力されたキャラクター名（または愛称）と技名（入力またはエイリアス）から、
-/// キャラクター情報ファイルおよび画像リンクファイルを読み込み、  
-/// 対象の技データを抽出し、Discord用の埋め込みメッセージとして表示する。
+/// キャラクターデータを読み込む関数
 ///
 /// # 引数
-/// * `ctx` - コマンド実行コンテキスト（Discordとの通信に利用）
-/// * `character` - キャラクター名または愛称（2文字以上必須）
-/// * `character_move` - 技名、入力、またはエイリアス（2文字以上必須）
+/// * `character` - ユーザーが入力したキャラクター名
+/// * `ctx` - コマンドコンテキスト
 ///
 /// # 戻り値
-/// 正常終了時は `Ok(())` を返し、エラー発生時は `Err(Error)` を返す。
-///
-/// # 例
-/// ```rust,no_run
-/// # async fn example(ctx: Context<'_>) -> Result<(), Error> {
-/// advanced(ctx, "Sol".to_string(), "236H".to_string()).await?;
-/// # Ok(()) }
-/// ```
-///
-/// # 注意
-/// この関数は非同期関数であるため、呼び出し時に `.await` が必要。
-#[poise::command(prefix_command, slash_command)]
-pub async fn advanced(
-    ctx: Context<'_>,
-    #[min_length = 2]
-    #[description = "キャラクター名または愛称"]
-    character: String,
-    #[min_length = 2]
-    #[rename = "move"]
-    #[description = "技名、入力、またはエイリアス"]
-    character_move: String,
-) -> Result<(), AppError> {
-    // コマンド引数の表示　引数確認用
-    println!(
-        "{}",
-        ("Command Args: '".to_owned() + &character + ", " + &character_move + "'").purple()
-    );
-
-    // 埋め込み画像用変数の初期化　初期値はデフォルト画像URL
-    let mut embed_image = IMAGE_DEFAULT.to_string();
-
-    // 入力チェックの実施　各種前提条件チェック
-    if (check::adaptive_check(
-        ctx,
-        check::CheckOptions {
-            data_folder: true,
-            nicknames_json: true,
-            character_folders: true,
-            character_jsons: true,
-            character_images: true,
-        },
-    )
-    .await)
-        .is_err()
-    {
-        return Ok(());
-    }
-
+/// 成功時は正式なキャラクター名、失敗時はエラー
+async fn load_character_data(character: &str, ctx: &Context<'_>) -> Result<String, AppError> {
     // キャラクター名の正規化　入力に基づく正式名称の取得
-    let character_arg_altered = match find::find_character(&character).await {
+    let character_arg_altered = match find::find_character(&character.to_string()).await {
         Ok(character_arg_altered) => character_arg_altered, // 正式名称取得
         Err(err) => {
             ctx.say(err.to_string()).await?; // エラーメッセージ送信
             println!("{}", ("Error: ".to_owned() + &err.to_string()).red());
-            return Ok(()); // 処理中断
+            return Err(AppError::CharacterNotFound(err.to_string())); // 処理中断
         }
     };
 
+    Ok(character_arg_altered)
+}
+
+/// 技情報と画像データを読み込む関数
+///
+/// # 引数
+/// * `character_arg_altered` - 正式なキャラクター名
+/// * `character_move` - ユーザーが入力した技名
+/// * `ctx` - コマンドコンテキスト
+///
+/// # 戻り値
+/// 成功時は (技情報, 画像URL) のタプル、失敗時はエラー
+async fn find_move_and_images(
+    character_arg_altered: &str,
+    character_move: &str,
+    ctx: &Context<'_>,
+) -> Result<(MoveInfo, String), AppError> {
     // JSONファイルパスの組み立て　キャラクター情報ファイルのパス生成
     let char_file_path =
-        "data/".to_owned() + &character_arg_altered + "/" + &character_arg_altered + ".json";
+        "data/".to_owned() + character_arg_altered + "/" + character_arg_altered + ".json";
     // JSONファイルの読み込み　キャラクター情報の取得
     let char_file_data = fs::read_to_string(char_file_path)
-        .expect(&("\nFailed to read '".to_owned() + &character_arg_altered + ".json" + "' file."));
+        .expect(&("\nFailed to read '".to_owned() + character_arg_altered + ".json" + "' file."));
 
     // JSON文字列のデシリアライズ　技情報の構造体へ変換
     let moves_info = serde_json::from_str::<Vec<MoveInfo>>(&char_file_data).unwrap();
 
     // JSON読み込み成功の表示　確認メッセージ出力
     println!(
         "{}",
-        ("Successfully read '".to_owned() + &character_arg_altered + ".json' file.").green()
+        ("Successfully read '".to_owned() + character_arg_altered + ".json' file.").green()
     );
 
     // 画像リンク用JSONファイルのパス組み立て　画像情報ファイルの指定
     let image_links = fs::read_to_string(
-        "data/".to_owned() + &character_arg_altered + "/images.json",
+        "data/".to_owned() + character_arg_altered + "/images.json",
     )
     .expect(
-        &("\nFailed to read 'data/".to_owned() + &character_arg_altered + "'/images.json' file."),
+        &("\nFailed to read 'data/".to_owned() + character_arg_altered + "'/images.json' file."),
     );
 
     // 画像リンクJSONのデシリアライズ　画像リンク情報の構造体へ変換
     let image_links = serde_json::from_str::<Vec<ImageLinks>>(&image_links).unwrap();
+
+    // 技インデックス検索
+    let move_index = match find::find_move_index(
+        &character_arg_altered.to_string(),
+        character_move.to_string(),
+        &moves_info,
+    )
+    .await
+    {
+        Ok(index) => index,
+        Err(err) => {
+            ctx.say(err.to_string() + "\nView the moves of a character by executing `/moves`.")
+                .await?;
+            println!("{}", ("Error: ".to_owned() + &err.to_string()).red());
+            return Err(AppError::MoveNotFound(err.to_string()));
+        }
+    };
+
     // 対象技情報の取得　入力に対応する技データの抽出
-    let move_info = &moves_info
-        [find::find_move_index(&character_arg_altered, character_move, &moves_info).await?];
+    let move_info = moves_info[move_index].clone();
 
     // 対象技の読み込み成功の表示　確認メッセージ出力
     println!(
         "{}",
         ("Successfully read move '".to_owned()
-            + &move_info.input.to_string()
+            + &move_info.input
             + "' in '"
-            + &character_arg_altered
+            + character_arg_altered
             + ".json' file.")
             .green()
     );
 
+    // デフォルト画像URL設定
+    let mut embed_image = IMAGE_DEFAULT.to_string();
+
     // 画像リンクの探索　対象技の画像リンクを検索
     for img_links in image_links {
         // 対象技の入力と画像情報の入力が一致し、画像リンクが存在する場合
@@ -133,22 +118,38 @@ pub async fn advanced(
         }
     }
 
+    Ok((move_info, embed_image))
+}
+
+/// 技情報の詳細な埋め込みメッセージを作成する関数
+///
+/// # 引数
+/// * `move_info` - 技情報
+/// * `embed_image` - 埋め込む画像のURL
+/// * `character_arg_altered` - 正式なキャラクター名
+///
+/// # 戻り値
+/// 埋め込みメッセージのベクター
+fn create_advanced_embeds(
+    move_info: &MoveInfo,
+    embed_image: &str,
+    character_arg_altered: &str,
+) -> Vec<CreateEmbed> {
     // 埋め込みメッセージ群生成用ベクターの初期化
     let mut vec_embeds = Vec::new();
     // 埋め込みタイトルの作成　キャラクター名と技名を組み合わせたタイトル
     let embed_title = "__**".to_owned() + &move_info.input + "**__";
     // 埋め込みURLの作成　Dustloop Wiki のキャラクター概要ページURL生成
-    let embed_url =
-        "https://dustloop.com/w/GGST/".to_owned() + &character_arg_altered + "#Overview";
+    let embed_url = "https://dustloop.com/w/GGST/".to_owned() + character_arg_altered + "#Overview";
     // 埋め込みフッターの作成　技に関するキャプションを利用
     let embed_footer = CreateEmbedFooter::new(&move_info.caption);
 
     // 埋め込みメッセージの生成　技データの各パラメータをフィールドとして追加
     let embed = CreateEmbed::new()
         .color(EMBED_COLOR) // 埋め込みカラー設定
-        .title(&embed_title) // タイトル設定
-        .url(&embed_url) // URL設定
-        .image(&embed_image) // 画像リンク設定
+        .title(embed_title) // タイトル設定
+        .url(embed_url) // URL設定
+        .image(embed_image) // 画像リンク設定
         .fields(vec![
             (
                 "ダメージ",
@@ -235,6 +236,67 @@ pub async fn advanced(
         vec_embeds.push(embed2); // ベクターに追加
     }
 
+    vec_embeds
+}
+
+/// 技の詳細情報を埋め込みメッセージで表示するコマンド
+///
+/// # 引数
+/// * `ctx` - コマンドコンテキスト
+/// * `character` - キャラクター名または愛称
+/// * `character_move` - 技名、入力、またはエイリアス
+///
+/// # 戻り値
+/// 処理結果 `Result<(), AppError>`
+#[poise::command(prefix_command, slash_command)]
+pub async fn advanced(
+    ctx: Context<'_>,
+    #[min_length = 2]
+    #[description = "キャラクター名または愛称"]
+    character: String,
+    #[min_length = 2]
+    #[rename = "move"]
+    #[description = "技名、入力、またはエイリアス"]
+    character_move: String,
+) -> Result<(), AppError> {
+    // コマンド引数の表示　引数確認用
+    println!(
+        "{}",
+        ("Command Args: '".to_owned() + &character + ", " + &character_move + "'").purple()
+    );
+
+    // 入力チェックの実施　各種前提条件チェック
+    if (check::adaptive_check(
+        ctx,
+        check::CheckOptions {
+            data_folder: true,
+            nicknames_json: true,
+            character_folders: true,
+            character_jsons: true,
+            character_images: true,
+        },
+    )
+    .await)
+        .is_err()
+    {
+        return Ok(());
+    }
+
+    // キャラクターデータ読み込み
+    let Ok(character_arg_altered) = load_character_data(&character, &ctx).await else {
+        return Ok(());
+    };
+
+    // 技情報と画像データ読み込み
+    let Ok((move_info, embed_image)) =
+        find_move_and_images(&character_arg_altered, &character_move, &ctx).await
+    else {
+        return Ok(());
+    };
+
+    // 埋め込みメッセージ作成
+    let vec_embeds = create_advanced_embeds(&move_info, &embed_image, &character_arg_altered);
+
     // 返信メッセージ用オブジェクト生成　送信用オブジェクトの初期化
     let mut reply = poise::CreateReply::default();
     // 生成した埋め込みメッセージ群を返信オブジェクトに追加