構造体からの値の取り出しについて

Question

カテゴリリストの構造体クラスを作成しました。

public class CategoryList
{
    public string modelno;
    public string modename;
    public string Category1;
    public string Category2;

    public static void InitCategoryDic(List<int> no,List<string> ListModelNo,List<string> ListModelName,List<string> ListCstrgory1,List<string> ListCstrgory2)
    {
        Dictionary<int,CategoryList> dic = new Dictionary<int,CategoryList> ();
        for (int i = 0; i < no.Count; i++)
        {
            dic.Add (no[i], new CategoryList{modelno = ListModelNo[i], modename=ListModelName[i], Category1 = ListCstrgory1[i], Category2 = ListCstrgory2[i] });
        }
        
    }
        
}

構造体への値のセットについては、調べながらうまく値のセットを行うことができたのですが、
今度、構造体から値を取り出す方法について、教えて頂きたいのですが、
foreach (CategoryList categorylist in ???? ) ←　???の部分はどのように指定すればよいのでしょうか？

public class MainTitle : MonoBehaviour {
    
    void testadd() {

        int cnt=0;
        List<int> Cnt = new List<int>();
        List<string> ListModelNo = new List<string>();
        List<string> ListModelName = new List<string>();
        List<string> ListCategory1 = new List<string>();
        List<string> ListCategory2 = new List<string>();

        for (int i = 0; i < 10; i++)
        {
            cnt +=1;
            Cnt.Add(cnt);
            ListModelNo.Add("ModelNo"+cnt);
            ListModelName.Add("ModelName"+cnt);
            ListCategory1.Add("Cstrgory1"+cnt);
            ListCategory2.Add("Cstrgory2"+cnt);
        }

       CategoryList.InitCategoryDic(Cnt,ListModelNo,ListModelName,ListCategory1,ListCategory2);

    }
        
    public static List<CategoryList> getCategoryList(int no){

        List<CategoryList> result = new List<CategoryList>();
        foreach (CategoryList categorylist in ???? ) {

            if (categorylist.no == no) {
                result.Add (categorylist.ListModelName );
            }
        }
        return result;
    }

}

Answer 1

構造体への値のセットについては、

と言うのは、InitCategoryDic()メソッドの事でしょうか?
ここで、dicという変数に値をセットしていますが、この変数は、メソッド内の変数なので、メソッドの実行が終わった時点で、消えています。従って、この変数の値を他から、参照する事はできません。(Visual Studioだと Warningが出ないか?)

もし、参照するならば、例えば、クラス CategoryList に

  public static Dictionary<int,CategoryList> dic;

の宣言を追加し、こちらに値をセットする必要があります。 (安易なので、あまり推奨できませんが、他の方法は変更が大きい)
そして、

foreach (CategoryList categorylist in ???? ) {

は、以下のようになるのではないかと。(もしかすると、キャストが必要?)

  foreach (KeyValuePair<int, CategoryList> kvp in CategoryList.dic) {
    if (kvp.Key == no) {
      result.Add(kvp.Value.ListModelName);
    }
  }

Comments

[Qoo]

回答ありがとうございます。
まだ理解できていないのですが、

dic.Add (no[i], new CategoryList{modelno = ListModelNo[i], modename=ListModelName[i], Category1 = ListCstrgory1[i], Category2 = ListCstrgory2[i] });

のところで、確かにdicはローカル変数だと思うのですが、new CategoryList・・のところで
Classの構造体の方にも追加されているものだと思っていたのですが、そういうものではないのでしょうか？？

ちなみに今回のテストスクリプトは下記を参考にしました。
このサイトでもdic はローカル変数になっているのですが、クラスの方から値を取得しているような例になっているのですが、こちらも間違いなのでしょうか？？

http://sleepnel.hatenablog.com/entry/2016/07/25/172000

[MMashiro]

リンク先見てみましたが、恐らくあくまで例として出してるだけだと思います。
普通に使うには変数dicを外に出したり、引数に渡すなどする必要があります
new CategoryListしていますがそのnewの結果をローカル変数のdicに入れてしまっているのでdicは関数を抜けた後どこからも参照されてない為消えてしまいます。

Answer 2

前提としてクラスと構造体は別ものです。
で、構造体に値を保存できてないので、もう一度参考資料に戻ってください。
もし、今参照している資料でこれができあがっているなら、別の資料を探してください。

下記に、おそらくこうしたいんじゃないかな、というのを書いておきますが、説明は参考書をあたってください。
(vsで書いてないのでエラーがあったらごめんなさい)

public struct Category //structとclassはちがう
{
  public string ModelNo; //Noがstringっていうのは最悪です
  public string ModelName;
  puglic string Category1; //CategoryならEnumとかにできませんかね。文字っていうのは良くない
  public string Category2; //カテゴリが複数なら配列かタプルにしたほうがよいのでは？
  public Category(string modelNo, string modelName, string cat1, string cat2) //コンストラクタはこういう構文
  {
    ModelNo = modelNo;
    ModelName = modelName;
    Category1 = cat1;
    Category2 = cat2;
  }
}

public class MainTitle : MonoBehaviour {
    private List<Category> _listOfCategory; //リストをプライベートに保持

    void testadd() {
        _listOfCategory = new List<Category>(); //リストを初期化
        foreach(var i in Enumerable.Range(1, 10)) //1から10をiでとる
        {
           _listOfCategory.Add(new Category($"ModelNo{i}",$"ModelName{i}",$"Cstrgory1{i}",$"Cstrgory1{i}"));//構造体をnewして、リストに追加
        }
    }

    public static List<Category> getCategoryList() //全部取る
    {
        return _listOfCategory;
    }
    public static Category getCategoryOfNth(int no) //no番目を取るなら返り値はListではない。おかしいですよね。2番目が3個も4個もあるなんていうのは。
    {
        return _listOfCategory[no - 1];
    }
    public static List<Category> getCategoryByName(int no) //foreachのサンプルのために、名前にnoの数字の文字を含むものを取得するパターンを提示
    {
        List<CategoryList> result = new List<CategoryList>();
        foreach(Category cat in _listOfCategory)
        {
           if(cat.ModelNo.indexOf(no.ToString()) != -1)
           {
              result.Add(cat);
           }
        }
        return result;
    }
}

Comments

[Qoo]

ありがとうございました！

Answer 3

getCategoryListですが、noをDictionaryのキーにしているので、返すCategoryListは必ず1件になるので、List<CategoryList>である必要がないのでは？
また、noもListModelNameもCategoryListのメンバーとして存在していません。
resultがList<CategoryList>なのにAddしているものが異なるなど
???の部分だけでなく、こちらも修正が必要かと思います

Comments

[Qoo]

回答ありがとうございます。
まだ良く理解できていないので、どのように修正して良いのかが理解できていない状況です。
可能でしたら、どのように修正すべきなのか、どんな方法があるのかをアドバイスしていただける方が助かります。

[YAmaGNZ]

クラス外からアクセスする場合publicにしているメンバーにだけアクセスすることが可能です。
ですので、CategoryListクラスの場合はmodelno、modename、Category1、Category2のメンバー変数と
InitCategoryDicメソッドのみとなります。

getCategoryListに関して、私にはこれでどのようなデータが欲しいのか分かりませんので、どのように修正するのが正解なのかが分かりません。
具体的なデータのリストからどのようなデータが欲しいのか整理してみてください。