npgsqlで名前は存在しませんとエラーが出る

Question

前提

PCはwindows11、
データベースの管理にpgadmin4、
コードの作成にVisual Studio 2022を使用しています。

実現したいこと

asp.net mvcにてNpgsqlを使用して、pgadmin4で作成したデータベースに接続して値を取得したいです。

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

SELECT文で取得した値を food_nm という変数に代入し、それをViewData["food"]に代入しようとしたところ、
CS0103 現在のコンテキストに'food_nm'という名前は存在しません
とエラーが表示されてしまいました。

該当のソースコード

public void sql_test(int food_number)
{
    string ip_address = "127.0.0.1";
    string port = "5432";
    string user_id = "postgres";
    string password = "pgadmin4のパスワード";
    string db_name = "データベース名";

    string constr = @"Server = " + ip_address +
                    ";Port = " + port +
                    ";Username = " + user_id +
                    ";Password = " + password +
                    ";Database = " + db_name;

    NpgsqlConnection conn = new NpgsqlConnection(constr);

    conn.Open();

    string sql = @"SELECT food FROM test-table WHERE number = '" + food_number + "'";
    NpgsqlCommand cmd = new NpgsqlCommand(sql, conn);

    using (NpgsqlDataReader dr = cmd.ExecuteReader())
    {
        while (dr.Read())
        {
            string food_nm = dr["food"] == DBNull.Value ? string.Empty : (string)dr["food"];
        }
    }

    ViewData["food"] = food_nm;

    conn.Close();
}

試したこと

mvcやnpgsqlには触り始めたばかりなので、いまいちどこが間違っているのか見当が付けられなかったので質問させていただきました。エラーの原因となっている個所を教えていただきたいです。

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

Comments

[YAmaGNZ]

{}の中で宣言しているからスコープ外になっているだけでは？

[退会済みユーザー]

> mvcやnpgsqlには触り始めたばかりなので

そんなこと以前に C# に基本的な文法を学んでください？　基本のキが分かってないということを自覚しましょう。

[退会済みユーザー]

質問者さん、無言ですが、回答したのでそれに対するフィードバックを返してください。役に立った/立たなかったぐらいはすぐに返せるのでは？ 役に立たなかったならどこがダメかを書くとより期待に近い回答が出てくるかも。解決したなら解決に役立った回答にベストアンサーをつけてクローズしてください。とにかく無言は NG です。

Answer 1

質問のコードの、

string food_nm = dr["food"] == DBNull.Value ? string.Empty : (string)dr["food"];

・・・中略・・・

ViewData["food"] = food_nm; 

を以下のようにすれば

ViewData["food"] = dr["food"] == DBNull.Value ? string.Empty : (string)dr["food"];

・・・中略・・・

// 以下はコメントアウト
//ViewData["food"] = food_nm; 

少なくとも「CS0103 現在のコンテキストに'food_nm'という名前は存在しません」というエラーは出なくなるはずです。

ただし、それだけで質問者さんがやりたいことができるかどうかは分かりませんが。他にもいろいろ問題がありそうですので。例えば、

(1) dr["food"] == DBNull.Value という式が成り立つのか？

(2) 複数のレコードで number が同じ値ということはないのか？

(3) SQL 文をパラメータ化してないので SQL インジェクションの恐れがありそう。

(4) conn.Close(); は途中で例外が起きた場合も必ず実行されなければならないがそうなってない。

Answer 2

ローカル変数は、定義したブロック内｛｝でしか有効ではありません。
その変数を、もちっと外のブロックで定義するようにしましょう