スコープ外での変数や配列の利用について

Question

実現したいこと

・スコープ外で行数がわからないものを配列に格納する方法はどうしたらいいのかわかりません。
・こういう場合はリストを使った方がいいのでしょうか？
・最終的には日付と会社名が一致すれば、内容が表示できるようにしたいです。
・まだまだ不慣れなので、色々プログラムの効率のいい考え方等も教えていただけると助かります。

発生している問題・エラーメッセージ

whie内で作成した配列がwhile外では使えない

前提

・C#を勉強がてら、手探りで個人的に会社の打ち合わせメモアプリを作成しています。
・data.csvの中には「日付,会社名,内容」をカンマ区切り出力できるように保存しています。
・まだ慣れていないのでコードを考えてから、それぞれクラスに考えていこうと思っています。

・プログラムの順番は
①streamを使ってcsvをカンマ区切りで配列に格納
配列fields　[0]日付　[1]会社名　[2]内容
②fields配列をさらに配列に格納？
③「日付」と「会社名」を入力し一致したら「内容」が表示される
という順番で考えています。

該当のソースコード

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.IO;
using System.Runtime.Remoting.Messaging;

namespace login
{
    internal class Program
    {
        static void Main(string[] args)
        {
            try
            {
                StreamReader s = new StreamReader("C:\\Users\\gonta\\Desktop\\data.csv", Encoding.GetEncoding("Shift-JIS"));
                int count = 0;
                while (!s.EndOfStream)
                {
                    string line = s.ReadLine();
                    string[] fields = line.Split(',');
                    for (int i = 0; i > fields.Length; i++)
                    {
                        csv[count] = fields[i];
                    }
                    count++;
                }
                s.Dispose();

                string date = Console.ReadLine();
                string name = Console.ReadLine();

                string msg = "";
                foreach (string lines in csv)
                {
                    if (lines[0] == date && lines[1] == name)
                    {
                        msg = lines[2];
                    }
                    else if(lines[0] == date)
                    {
                        msg = "指定した日にちに指定の会社のメモが存在しません。";
                    }
                }
                if( msg != "")
                {
                    Console.WriteLine(msg);
                }
                else
                {
                    Console.WriteLine("指定した日にちにメモはありません。");
                }
            }
            catch(System.Exception e)
            {
                // ファイルを開くのに失敗したとき
                System.Console.WriteLine(e.Message);
            }
        }
    }
}

補足情報（FW/ツールのバージョンなど）

visual studio2022

Comments

[Zuishin]

スコープが何を指すのかわからないけど、とりあえず参考にしているものが古すぎます。

[episteme]

> ・こういう場合はリストを使った方がいいのでしょうか？

やってみてはいかが?
# きっとうまくいく

[otn]

実現したいことに書いてあることの意味ががよくわかりませんが、
> whie内で作成した配列がwhile外では使えない
の1行だけを見て言うと、while外で使いたい変数はwhile外で宣言すれば良いだけでは？

Answer 1

・スコープ外で行数がわからないものを配列に格納する方法はどうしたらいいのかわかりません。

string[][] csv = new string[/* ここをどうすればいいかわからない */][];ということですね？

・こういう場合はリストを使った方がいいのでしょうか？

StreamReaderを使う縛りなら、リストを使った方がいいでしょう。

using System;
using System.Collections.Generic;
using System.IO;
using System.Text;

namespace Qbk5ysjl4t6nvtm
{
    internal class Program
    {
        static void Main()
        {
            List<string[]> csv = new List<string[]>();

            try
            {
                //string filePath = @"C:\Users\gonta\Desktop\data.csv";
                string desktopPath = Environment.GetFolderPath(Environment.SpecialFolder.DesktopDirectory);
                string filePath = Path.Combine(desktopPath, "data.csv");

                //Dummy(filePath);

                using (StreamReader s = new StreamReader(filePath, Encoding.GetEncoding("Shift-JIS")))
                {
                    while (!s.EndOfStream)
                    {
                        string line = s.ReadLine();
                        string[] fields = line.Split(',');
                        csv.Add(fields);
                    }
                }
            }
            catch (Exception e)
            {
                Console.WriteLine(e.Message);
                Console.ReadKey();
                return;
            }


            while (true)
            {
                Console.Write("日付を入力（yyyyMMdd） >");
                string date = Console.ReadLine();

                Console.Write("会社名を入力 >");
                string name = Console.ReadLine();

                bool found = false;
                foreach (string[] fields in csv)
                {
                    if (fields[0] == date && fields[1] == name)
                    {
                        Console.WriteLine(fields[2]);
                        found = true;
                    }
                }

                if (!found)
                {
                    Console.WriteLine("指定した日にちに指定の会社のメモが存在しません。");
                }

                Console.WriteLine();
            }
        }

        private static void Dummy(string filePath)
        {
            string csv = @"
20230423,ほげ株式会社,ほげほげ
20230423,株式会社ふが,ふがふが
".Trim();
            File.WriteAllText(filePath, csv, Encoding.GetEncoding("Shift-JIS"));
        }
    }
}

List<T> クラス (System.Collections.Generic) | Microsoft Learn

Environment.GetFolderPath メソッド (System) | Microsoft Learn

Environment.SpecialFolder 列挙型 (System) | Microsoft Learn

Path.Combine メソッド (System.IO) | Microsoft Learn

using ステートメント - 破棄可能なオブジェクトが正しく使用されるようにする | Microsoft Learn

---

StreamReaderにこだらないのであれば、配列にすることも可能です。

using System;
using System.IO;
using System.Linq;
using System.Text;

namespace Qbk5ysjl4t6nvtm
{
    internal class Program
    {
        static void Main()
        {
            string[][] csv;

            try
            {
                //string filePath = @"C:\Users\gonta\Desktop\data.csv";
                string desktopPath = Environment.GetFolderPath(Environment.SpecialFolder.DesktopDirectory);
                string filePath = Path.Combine(desktopPath, "data.csv");

                //Dummy(filePath);

                csv = File.ReadLines(filePath, Encoding.GetEncoding("Shift-JIS"))
                          .Select(line => line.Split(','))
                          .ToArray();
            }
            catch (Exception e)
            {
                Console.WriteLine(e.Message);
                Console.ReadKey();
                return;
            }


            while (true)
            {
                Console.Write("日付を入力（yyyyMMdd） >");
                string date = Console.ReadLine();

                Console.Write("会社名を入力 >");
                string name = Console.ReadLine();

                bool found = false;
                foreach (string[] fields in csv)
                {
                    if (fields[0] == date && fields[1] == name)
                    {
                        Console.WriteLine(fields[2]);
                        found = true;
                    }
                }

                if (!found)
                {
                    Console.WriteLine("指定した日にちに指定の会社のメモが存在しません。");
                }

                Console.WriteLine();
            }
        }

        private static void Dummy(string filePath)
        {
            string csv = @"
20230423,ほげ株式会社,ほげほげ
20230423,株式会社ふが,ふがふが
".Trim();
            File.WriteAllText(filePath, csv, Encoding.GetEncoding("Shift-JIS"));
        }
    }
}

File.ReadLines メソッド (System.IO) | Microsoft Learn

Enumerable.Select メソッド (System.Linq) | Microsoft Learn

Enumerable.ToArray<TSource>(IEnumerable<TSource>) メソッド (System.Linq) | Microsoft Learn

Comments

[gonta]

ベストアンサーにさせていただきました。

私のわかりにくい意図を理解していただき、2種類のわかりやすいコードを書いていただきありがとうございます。
3日間悩んでいた事が無事に実装できました。

リストの中に配列を格納できること、またstram readerを使用しない方法で配列にする方法もあるなんて、色々調べても出てこなかったので、とても勉強になりました。

本当にありがとうございました。

Answer 2

こういう場合はリストを使った方がいいのでしょうか？

そうです。問題は、スコープは関係なくて、今のコードでは while ループが終わるまで行数が分からないので配列の初期化ができないことです。なので、List<string[]> を使いましょう。

以下の一行、

var csv = new List<string[]>();

を Main メソッド内の最初の行に追加して、while ループの中の、

for (int i = 0; i > fields.Length; i++)
{
   csv[count] = fields[i];
}

を、

csv.Add(fields);

に代えてはいかがですか？

foreach (string lines in csv) 以下の処理も、上記に合わせて変更する必要がありますので忘れずに。

どうしても配列を使いたいということであれば、File.ReadAllLines メソッドを使って全行読んでしまうという手もあります。

Comments

[gonta]

回答ありがとうございます。

私のわかりにくい質問の意図を理解していただきありがとうございます。
配列は要素数を明確にしないと使えない事が今回のことで私自身に叩き込まれました。
Listに変更することで無事に実装することができました。

Listの使い方を、わかりやすく解説していただきありがとうございました。