質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.62%

  • C#

    6889questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

このC#コードでエラーになる理由がわかりません

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 395

odmdom

score 70

下記のTextData2をnewしたときに言語テキストをセットし、GetList()で言語を指定して取得という処理を行いたいのですが、switchのreturn data箇所でコンテキストが存在しませんというエラーがでてしまいます。
実際にはfruit以外のcaseも設定しますがここでは省いています。
また、GetList()では引数によって結果的にどの型のデータがはいるか明確でないので、このようなメソッドを作ること自体がおかしいでしょうか。

public class Fruit
{
    public string lang { get; set; }
    public string apple { get; set; }
    public string banana { get; set; }
    public string grape { get; set; }
}


public class TextData2
{
    public List<Fruit> fruit_list;

    public void Main(string page)
    {
    dynamic data;
        switch (page)
        {
            case "fruit":
                fruit_list = new List<Fruit>()
                    {
                        new Fruit(){lang = "ja", apple = "りんご", banana = "ばなな", grape="ぶどう"},
                        new Fruit(){lang = "en", apple = "apple", banana = "banana", grape="grape"},
                    };
                break;


        }
    }

    public dynamic GetList(string lang, string page)
    {
     dynamic data;
        switch (page)
        {
            case "fruit":
                data = (from l in fruit_list
                            where l.lang == lang
                            select l).FirstOrDefault();
                break;

        }

        return data;
    }
}

また、取得する側のコードのvar textdata2 = new TextData2("fruit");の箇所では下記画像のようなエラーがでてしまいますが原因が分からない状況です。
![イメージ説明(0fe89f2c8c50c3da414f04bff7ffba25.png)

//取得側
var textdata2 = new TextData2("fruit");
var fruit_list = textdata2.GetList(lang, "fruit");

追記
イメージ説明

追記2
色々助言をいただいて下記コードに書き直しました。
しかしまだ、下記のエラーがでてしまいます。
fruit_listはコンストラクタでセットしているはずなのですがなぜエラーがでてしまうのでしょうか。

イメージ説明

   public class TextData2
    {
        public string Page { get; set; }

        public List<Fruit> fruit_list;

        public TextData2()
        {
            string page = Page;
            switch (page)
            {
                case "fruit":
                    fruit_list = new List<Fruit>()
                    {
                        new Fruit(){Lang = "ja", Apple = "りんご", Banana = "ばなな", Grape="ぶどう"},
                        new Fruit(){Lang = "en", Apple = "apple", Banana = "banana", Grape="grape"},
                    };
                    break;
            }
        }

        public dynamic GetList(string lang)
        {
            dynamic data = new List<string>();
            string page = Page;
            switch (page)
            {
                case "fruit":
                    data = (from f in fruit_list
                            where f.Lang == lang
                                select f).FirstOrDefault();
                    break;

            }
            return data;
        }
    }

取得側のコード

var textdata2 = new TextData2() { Page = "fruit"};
var fruit_text = textdata2.GetList(lang);
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

checkベストアンサー

+3

switchのreturn data箇所でコンテキストが存在しませんというエラーがでてしまいます。

switchの後の中括弧がスコープを構築するので、その中で宣言したdataの有効範囲はこのブロックの中に限られてしまいます。「switchに入る前にdataを宣言する」、あるいは「switchの中から直接returnする」などの処置が必要です。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/15 09:41

    ありがとうございます。質問のコードを修正しました。
    コードを修正したところ今度は"return data;のところで未割当のローカル変数が使用されました"
    というエラーが出るようになりました。
    dataはメソッドの引数によって型が変わるのでdynamicとしています。

    キャンセル

  • 2018/05/15 09:44

    switchで1つもヒットしなかった場合に、代入されないままとなるのでエラーとなります。

    キャンセル

  • 2018/05/15 10:06

    どのように書くのが一般的でしょうか?

    キャンセル

  • 2018/05/15 10:09

    まず、選択肢が1つしかないのにswitchを使う時点で、「一般的」ではありません。

    そして、page != "fruit"だった場合に「どうしたいか」で、どう書くべきかは決まります。

    キャンセル

  • 2018/05/15 10:22

    質問に書きましたが、これは仮のコードなのでfruit以外を省いています。
    他にanimalやcarなどがあると思ってください。
    fruitのようにそれぞれにヒットしたときに、それぞれのプロパティから値を返したいと思っています。

    キャンセル

  • 2018/05/15 10:26

    では、「1つもヒットしなかった場合」はどう動作するのが適切でしょうか。

    キャンセル

  • 2018/05/15 10:46

    そうですね。
    空のリストを返そうかと思います。

    キャンセル

  • 2018/05/15 11:02

    switchにdefault:を作って、そこで空のリストを代入するようにしましょう(宣言時に代入してもいいですが)。

    キャンセル

  • 2018/05/15 11:26

    追記に修正したコードを記載しました。
    画像のようなエラーがでますがどのように宣言するとよいのでしょうか。

    キャンセル

  • 2018/05/15 11:29

    varで宣言した場合、型推論により型が固定されてしまいます。本当に多様な型を扱う必要があるのなら、dynamicで宣言しましょう。

    キャンセル

  • 2018/05/15 14:15

    ありがとうございます。
    いただいたご助言を基に追記にコードを修正したのですが、まだエラーがでてしまっております。
    何かご助言いただけますと助かります。

    キャンセル

  • 2018/05/15 15:00

    どうやら
    var textdata2 = new TextData2() { Page = "fruit"};
    でPageプロパティにfruitをセットしているはずなのですが、
    コンストラクタでpageを取得できていません。
    コンストラクタ内ではnew時に初期化したデータは取得できないのでしょうか。

    キャンセル

+1

また、取得する側のコードのvar textdata2 = new TextData2("fruit");の箇所では下記画像のようなエラーがでてしまいますが原因が分からない状況です。

そこに出ているとおりです。
TextData2で、引数一つを取るコンストラクタが定義されていません

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/15 14:16

    ありがとうございます。
    こちらの引数の件は解決できました。

    追記に修正コードを記載しましたが、新たなエラーがでてしまっておりますのでご助言いただけますと助かります。

    キャンセル

  • 2018/05/15 15:34

    どう解決してるんでしょうか。
    コンストラクタを記述したソースを提示してください。

    > fruit_listはコンストラクタでセットしているはずなのですがなぜエラーがでてしまうのでしょうか。

    セットしている(はずの)コンストラクタも見当たりません

    キャンセル

  • 2018/05/15 15:41 編集

    ああ、ソースを追記してるんですね
    ViualStudioを使っているなら、ブレークポイントを設定して、コードを実行途中で止め、各変数の値を参照できます。、また、1行ごとに実行して各変数の変化を参照することもできます。
    闇雲に実行してうまくいかないと言うより、そういうデバッグ機能を使ってご自分で動作検証をなさってください。
    「VisualStudio ブレークポイント」などでぐぐると、そこらへんの解説が出てきます

    キャンセル

  • 2018/05/15 16:57

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

    キャンセル

+1

var textdata2 = new TextData2() { Page = "fruit"};


この書き方は初期化子と言って、雑に展開すると

var textdata2 = new TextData2();
textdata2.Page = "fruit";


こう書いたのと同じ動作をします。
つまりコンストラクタが値設定前に走ります。

コンストラクタで値が必要なのであれば、

        public TextData2()
        {
            string page = Page;


こうではなく

        public TextData2(string page)
        {
            Page = page;


こうしたうえで、初期化子ではなく、

var textdata2 = new TextData2("fruit");


こうしてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/15 16:58

    ありがとうございます。
    やはりそういうことなのですね。
    仰る通りの方法で期待通りの処理を行うことができました!

    キャンセル

0

皆様のおかげで最終的に以下のコードで実装できました。
ありがとうございます。
一番長くお付き合いいただいたmaisumakun様をBAとさせていただきました。

   public class TextData2
    {
        public string Page { get; set; }

        public List<Fruit> fruit_list;

        public TextData2(string page)
        {
            Page= page;
            switch (page)
            {
                case "fruit":
                    fruit_list = new List<Fruit>()
                    {
                        new Fruit(){Lang = "ja", Apple = "りんご", Banana = "ばなな", Grape="ぶどう"},
                        new Fruit(){Lang = "en", Apple = "apple", Banana = "banana", Grape="grape"},
                    };
                    break;
            }
        }

        public dynamic GetList(string lang)
        {
            dynamic data = new List<string>();
            string page = Page;
            switch (page)
            {
                case "fruit":
                    data = (from f in fruit_list
                            where f.Lang == lang
                                select f).FirstOrDefault();
                    break;

            }
            return data;
        }
    }

取得側のコード

var lang = "ja";
var textdata2 = new TextData2("fruit");
var fruit_text = textdata2.GetList(lang);

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/15 17:44

    string Page= page;
    これでは意味ないです。
    Page= page;
    です。
    public string Page { get; set; }
    ここに入れるという事です。

    キャンセル

  • 2018/05/15 17:48

    あ、すみませんstringが残っておりましたね;;
    ご指摘ありがとうございます!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C#

    6889questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。