c# Listの参照わたしでビルドエラー

Question

ご指摘の通り、変数名を変えてみました。

overrideは、ややこしいので、削除しました。
しかし、
コントロールが現在のメソッドを抜ける前に割り当てられる必要があります。
といったビルドエラーになります。
エラーのメッセージが言わんとしていることは理解できますが、
変数定義をどう変えればよいのか、わかりません。

using System;
using System.Collections.Generic;
using System.Windows.Forms;

namespace StudyProject
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            MessageBox.Show("処理を開始します");
            // out を使った場合、変数を初期化不要
            List<string[]> Data = new List<string[]>();
            CsvEdit1 obj1 = new CsvEdit1();
            obj1.ReadCsv(out Data);
            MessageBox.Show("ここでブレークポイントでチェック");
        }
    }

    public class CsvEdit1
    {
        public void ReadCsv(out List<string[]> Data)
        {
            List<string[]> tempData = new List<string[]>();
            try
            {
                // csvファイルを開く
                System.IO.DirectoryInfo dirPath =
                    new System.IO.DirectoryInfo(@"E:/A/");
                System.IO.FileInfo[] files =
                     dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories);
                // 指定フォルダからCSVを取得し、配列に格納する。
                foreach (System.IO.FileInfo filePath in files)
                {
                    using (var readCsv = new System.IO.StreamReader(filePath.FullName))
                    {
                        //ヘッダを読み捨てる。
                        readCsv.ReadLine();
                        // ストリームの末尾まで繰り返す
                        while (!readCsv.EndOfStream)
                        {
                            // ファイルから一行読み込む
                            string line = readCsv.ReadLine();
                            // カンマ区切りで分割して配列に格納する
                            string[] record = line.Split(',');

                            tempData.Add(record);
                        }
                    }
                }
                MessageBox.Show("読み込み完了しました。");
            }
            catch (System.Exception e)
            {
                // ファイルを開くのに失敗したとき
                MessageBox.Show(e.ToString());
            }
        }
    }
}

Answer 1

out いらないんじゃないですか？

これは戻り値みたいなもので、呼び出す関数でなく、呼ばれた関数で初期化して返すものです。

Comments

[KKcambodia]

Zuishinさま

いつもありがとうございます。
outをとってもうまくいきません。
もうすこし試してみます。

Answer 2

out修飾子で引き渡された変数は、メソッドを抜ける前になんらかの割り当てが行われる必要があります。
詳しくは以下を参照ください。

https://msdn.microsoft.com/ja-jp/library/t3c3bfhx.aspx

Comments

[KKcambodia]

y-sakamotoさま

ありがとうございます。
msdnをみてみました。

static void Method(out int i)
{
i = 44;
}
と同じように、
かえてみたのですが、どうもうまくいきません。
out 命令は、配列渡しができないのでしょうか？

public class CsvEdit1
{
public void ReadCsv(out List<string[]> tempData)
{
try
{
// csvファイルを開く
System.IO.DirectoryInfo dirPath =
new System.IO.DirectoryInfo(@"E:/StudyDrive/Desktop/Study/StudyProject/INDATA/");
System.IO.FileInfo[] files =
dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories);
// 指定フォルダからCSVを取得し、配列に格納する。
foreach (System.IO.FileInfo filePath in files)
{
using (var readCsv = new System.IO.StreamReader(filePath.FullName))
{
//ヘッダを読み捨てる。
readCsv.ReadLine();
// ストリームの末尾まで繰り返す
while (!readCsv.EndOfStream)
{
// ファイルから一行読み込む
string line = readCsv.ReadLine();
// カンマ区切りで分割して配列に格納する
string[] record = line.Split(',');

tempData.Add(record);
}
}
}
MessageBox.Show("読み込み完了しました。");
}
catch (System.Exception e)
{
// ファイルを開くのに失敗したとき
MessageBox.Show(e.ToString());
}
}
}

[y-sakamoto]

tempDataに値が割りあたっておりません。
簡素なソースですが、以下のように変更して下さい。

また予断ではありますが、Listはクラスであるため参照渡しされます。
その為、質問者様の例ですとそもそもout修飾子がなくとも、Addした値を呼び出し側のメソッドで参照可能です。

'''C#
public void RunOutMethod()
{
List<string[]> data;
OutMethod(out data);
}

public void OutMethod(out List<string[]> result)
{
result = new List<string[]>();
result.Add(new[] { "test" });
}
'''

[y-sakamoto]

上でお話しました、参照渡しについて簡単に検証するソースを書きましたので記載しておきます。
意味を理解した上で用件にマッチするほうをお使いいただければと思います。

public void RunReferenceMethod()
{
List<string[]> data = new List<string[]>();
ReferenceMethod(data);

MessageBox.Show(data[0][0]); //メッセージボックスに「test」と表示されます
}

public void ReferenceMethod(List<string[]> result)
{
result.Add(new[] { "test" });
}

[KKcambodia]

y-sakamotoさま

ありがとうございます。

理解できました。

tempData = new List<string[]>();
をtryの上で定義することにより、
class内で、tempDataが有効になり、
うまくoutで引き渡しできました。

変数tempDataの定義場所を理解していなかったようです。

大変勉強になりました。
ありがとうございます。

Answer 3

引数名と新しく定義している変数の名称が重複しております。
具体的には以下の部分となります

using System;
using System.Collections.Generic;
using System.Windows.Forms;

namespace StudyProject
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            MessageBox.Show("処理を開始します");
            // out を使った場合、変数を初期化不要
            List<string[]> stArrayData = new List<string[]>();
            CsvEdit1 obj1 = new CsvEdit1();
            obj1.ReadCsv(out stArrayData);
            MessageBox.Show("ここでブレークポイントでチェック");
        }
    }

    public class CsvEdit1 : CsvEdit
    {
        public override ReadCsv(out List<string[]> stArrayData)        ......ここと
        {
            List<string[]> stArrayData = new List<string[]>();         ......ここです
            try
            {
                // csvファイルを開く
                System.IO.DirectoryInfo dirPath =
                    new System.IO.DirectoryInfo(@"E:/StudyProject/input/");
                System.IO.FileInfo[] files =
                     dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories);
                // 指定フォルダからCSVを取得し、配列に格納する。
                foreach (System.IO.FileInfo filePath in files)
                {
                    using (var readCsv = new System.IO.StreamReader(filePath.FullName))
                    {
                        //ヘッダを読み捨てる。
                        readCsv.ReadLine();
                        // ストリームの末尾まで繰り返す
                        while (!readCsv.EndOfStream)
                        {
                            // ファイルから一行読み込む
                            string line = readCsv.ReadLine();
                            // カンマ区切りで分割して配列に格納する
                            string[] record = line.Split(',');

                            stArrayData.Add(record);
                        }
                    }
                }
                MessageBox.Show("読み込み完了しました。");
            }
            catch (System.Exception e)
            {
                // ファイルを開くのに失敗したとき
                MessageBox.Show(e.ToString());
            }
        }
    }

    public class CsvEdit
    {
        public virtual void ReadCsv()
        {
            List<string[]> stArrayData = new List<string[]>();
            try
            {
                // csvファイルを開く
                System.IO.DirectoryInfo dirPath =
                    new System.IO.DirectoryInfo(@"/");
                System.IO.FileInfo[] files =
                     dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories);
                // 指定フォルダからCSVを取得し、配列に格納する。
                foreach (System.IO.FileInfo filePath in files)
                {
                    using (var readCsv = new System.IO.StreamReader(filePath.FullName))
                    {
                        //ヘッダを読み捨てる。
                        readCsv.ReadLine();
                        // ストリームの末尾まで繰り返す
                        while (!readCsv.EndOfStream)
                        {
                            // ファイルから一行読み込む
                            string line = readCsv.ReadLine();
                            // カンマ区切りで分割して配列に格納する
                            string[] record = line.Split(',');

                            stArrayData.Add(record);
                        }
                    }
                }
                MessageBox.Show("読み込み完了しました。");
            }
            catch (System.Exception e)
            {
                // ファイルを開くのに失敗したとき
                MessageBox.Show(e.ToString());
            }
        }
    }
}