C# テキストボックス・コンボボックスの値をDBに登録

Question

テキストボックスの値をDBに登録したい

Visual Sutudio C#　DBはOracle

研修でDBに登録されているNoの最大値＋１、テキストボックス、コンボボックスに入力された
値をボタンクリックするとDBに登録するという課題をしております。

エラーは出ていないのですが、DBに登録されません

string sql = "INSERT INTO FRUITS (NO,NAME,SEASON) VALUES ((SELECT MAX(NO) + 1 FROM FRUITS)," + name.Text + "," + seasonList.SelectedValue + ")";

上記のsql分の+ nameText.Text +と+ seasonList.SelectedValue +を
"INSERT INTO FRUITS(NO,NAME,SEASON) VALUES ((SELECT MAX(NO) + 1 FROM FRUITS),’桃’,3)";
など、元から書いて実行すると登録されるので、この部分が問題だと思うのですが
エラーも出ないのでどう間違っているのか見当もつかず困っています。

コンボボックスの値は
DataTable season = new DataTable();

    season.Columns.Add("code", Type.GetType("System.Int32"));
    season.Columns.Add("season", Type.GetType("System.String"));

    season.Rows.Add("0", "");
    season.Rows.Add("1", "春");
    season.Rows.Add("2", "夏");
    season.Rows.Add("3", "秋");
    season.Rows.Add("4", "冬");

    seasonList.DataSource = season;
    seasonList.DataTextField = "season";
    seasonList.DataValueField = "code";
    seasonList.DataBind();

で格納しています。

初心者のため説明下手・説明不足があるかと思いますが、ヒントを頂けると嬉しいです…。

Comments

[YAmaGNZ]

作成したsqlの中身は確認しましたか？
それとは別に、自分でSQLを文字列操作で構築するのではなく、パラメータクエリを使用したほうがよろしいかと思います。

[sasamichan]

コメントありがとうございます！
sqlの中身とはどう確認すればよいのでしょうか？
パラメータクエリも初めて聞きました…
ありがとうございます、調べてみます！

[YAmaGNZ]

sqlに設定しているところの次の行ででもブレイクポイントを設定して変数の中身を見ればよろしいかと

[sasamichan]

変数の中身確認いたしました！
テキストボックスの値は問題なく取得しているみたいですが
コンボボックスの値が何を選択しても0しか入らない状態でした。
もう少し試行錯誤してみます。ありがとうございます！

[YAmaGNZ]

そこも重要ですが、実行できるSQLとの差異は分かりましたか？

[sasamichan]

まだ全然わからずです…

[YAmaGNZ]

sqlに格納されているSQL文を別のツール等で直接実行するなりして確かめてみてください。
プログラムで構築したSQLが正しいSQLなのか、別のツールで確認することもデバッグする上で重要な手順のひとつとなります。

[sasamichan]

そうなんですね！教えていただきありがとうございます！
やってみたところ + name.Text + の前後に「’」がなく登録されないことがわかりました！
ありがとうございます(^^)
まだコンボボックスの値が正しく取れないのでもう少し頑張ってみます！

[退会済みユーザー]

研修の課題に関する質問は講師に聞くべきで、ここで聞くべき話ではないと思いますが？

[YAmaGNZ]

また、ASP.NET等の環境を追記したほうが正しい回答が得られるかと思います。

[sasamichan]

研修ではありますが教えて頂いてる方が時短勤務の方で15時前には帰ってしまわれるのでここで質問し課題を進めている次第です。

[sasamichan]

環境なども一体自分が何を使っているかもあまりハッキリ理解していない段階でそこまで頭が回らずでした…。
アドバイスありがとうございます。

[退会済みユーザー]

> 研修ではありますが教えて頂いてる方が時短勤務の方で15時前には帰ってしまわれるのでここで質問し課題を進めている次第です。

それは理由になりません。一度 https://teratail.com/help/question-tips を読んでください。ここは「質問と回答を通してお互いに知識や情報を交換・共有する場所です」ということです。

何を作っているかも書かない、開発環境も書かない、何を聞いたら良いのかも分からない生徒に、講師が face-to-face で何度もやりとりをして、何を聞きたいのかを探って、回答してもらうというようなことを期待しているとすれば、ここは適当な場所ではないと思います。

[sasamichan]

ご指摘頂きありがとうございます。
知識を付け、情報交換が出来るよう精進して参りたい所存です。

何度もやり取りして頂き、手取り足取り教えて頂くつもりは
ありませんでしたが親切にして頂いたので
つい甘えてしまいました。
不快に感じられたなら申し訳ございません。

Answer 1

sql文のname.Textの前後に「'」を追加で文字列にし、DBに登録できるようになり、
デザインからコンボボックスのプロパティのAutoPostBackをTrueに変更し
Page_Lode部分に下記のコードを追加することで
コンボボックスの値が正常に取得できるようになりました。

if (IsPostBack == true)
        {
            return;
        }

アドバイスくださった方ありがとうございました。

Comments

[Zuishin]

それでもできるかもしれませんが、会社のレベルにもよりますけど、現実にはパラメータが使えないと不合格になります。実用にならないので。パラメータについては講師に聞いてください。