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

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

新規登録して質問してみよう
ただいま回答率
85.48%
C#

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

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

.NET Framework

.NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

Q&A

解決済

2回答

1280閲覧

(c#)MySQLにあるテーブルをキーワード検索して画面上に出力する方法をお助けください。

LLL

総合スコア2

C#

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

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

.NET Framework

.NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

0グッド

0クリップ

投稿2023/03/18 08:24

編集2023/03/18 09:14

実現したいこと

下のような連絡先をキーワード検索してその情報を画面に出すというものを作りたいです。
また、キーワードに当てはまるものがなければ全ての情報を画面に出したいです。
(デフォルトでは下にあるような全部の情報を画面に出している状態が良いです。)
※下にある情報は偽物です!わかりやすく芸能人の名前にしております。
イメージ説明

前提

※プログラミング初心者です。ネットで色々調べましたがなかなか動作しなかったので質問させていただきます。

使用しているものは以下の通りです。
・visual studio2019( Windows フォーム アプリケーション (.NET Framework) )
・MySQL

現在のコードは以下の通りです。

c#

1using System; 2using System.Collections.Generic; 3using System.ComponentModel; 4using System.Data; 5using System.Drawing; 6using System.Linq; 7using System.Text; 8using System.Threading.Tasks; 9using System.Windows.Forms; 10using MySql.Data.MySqlClient; 11using MySql.Data; 12 13namespace AddressBook 14{ 15 public partial class frmAddressList : Form 16 { 17 public frmAddressList() 18 { 19 InitializeComponent(); 20 } 21 22 private void frmAddressList_Load(object sender, EventArgs e) 23 { 24 string server = "localhost"; 25 string database = "dbaddress"; 26 string user = "root"; 27 string pass = "Maron1207"; 28 string charset = "utf8"; 29 string connectionString = string.Format("Server={0};Database={1};Uid={2};Pwd={3};Charset={4}", server, database, user, pass, charset); 30 31 MySqlConnection con = new MySqlConnection(connectionString); 32 33 DataTable dt = new DataTable(); 34 35 try 36 { 37 con.Open(); 38 MySqlDataAdapter adapter = new MySqlDataAdapter("select * from d_contactinformation", con); 39 adapter.Fill(dt); 40 grdList.DataSource = dt; 41 } 42 finally 43 { 44 con.Close(); 45 } 46 } 47 48 private void btnSearch_Click(object sender, EventArgs e) 49 { 50 string txtkey = txtKeyword.Text; 51 string server = "localhost"; 52 string database = "dbaddress"; 53 string user = "root"; 54 string pass = "Maron1207"; 55 string charset = "utf8"; 56 string connectionString = string.Format("Server={0};Database={1};Uid={2};Pwd={3};Charset={4};", server, database, user, pass, charset); 57 58 MySqlConnection con = new MySqlConnection(connectionString); 59 60 DataTable dt = new DataTable(); 61 62 try 63 { 64 con.Open(); 65 MySqlDataAdapter adapter = new MySqlDataAdapter("select * from d_contactinformation where fullname like '%txtkey%'", con); 66 adapter.Fill(dt); 67 grdList.DataSource = dt; 68 } 69 finally 70 { 71 con.Close(); 72 } 73 } 74 } 75}

frmAddressList_LoadはMySQLと接続させてテーブルを表示させています。(これは動きました)
btnSearch_Clickは記述方法がわからないので仮にfullnameのフィールドに限定して検索をかけようとしましたが、検索ボタンを押しても下のようになります。(記述が間違っているのは承知です)
イメージ説明
イメージ説明

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

visual studio上ではこのように出ます。↓
イメージ説明

補足

説明文に不足があった場合は申し訳ございません。すぐに編集いたします。
初心者で基礎的なこともできていないと思いますが、どうかご教授いただけますと幸いです。

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

fiveHundred

2023/03/18 08:39 編集

ビルドエラーって出ているのに、なんで確認しないんですか? 「問題は見つかりませんでした」とありますが、そうであってもビルド時に今回のようにエラーが出てくることがあります。 そのビルドエラーがどこかに出ているはずなので、それを確認してください。 確認しても分からないのであれば、そのエラー文を追記してください。
fiveHundred

2023/03/18 08:42

なお、「最後に成功したビルドを実行しますか?」で実行しても、そのエラーが出る前の状態で実行されるだけで、変更が反映されていないわけですから、それでいくら実行しても無駄です。
LLL

2023/03/18 09:08

ご回答ありがとうございます。また、ご指摘いただきありがとうございます。 ビルドエラーを確認し直した結果、検索ボタンを押すと画面は変わりましたがデータベースが一つも出てこず、キーワードに引っ掛かりませんでした。 やはり記述が根本的に間違っているのでしょうか?
退会済みユーザー

退会済みユーザー

2023/03/18 09:48 編集

最終的には、MySQL のテーブルのレコード一覧を DataGridView に表示し、ユーザーが DataGridView を見て編集した結果を、元のデータベースのテーブルに反映させるというようなことがしたいのでは?であれば、Visual Studio のデータソース構成ウィザードを使ってアプリを作成してはいかがですか? 一行もコードを書かなくても簡単に作成できます。検索機能はその後実装しては?
LLL

2023/03/19 04:50

ご回答ありがとうございます。 そのような方法もあるのですね! ご教授いただきありがとうございます。参考にさせていただきます。
guest

回答2

0

まず、SQL インジェクション防止のため SQL 文はパラメータ化しましょう。

特に、今回のようなユーザー入力から SQL 文の文字列を組み立てることは絶対やってはいけないことで、そういう SQL 文は議論の対象外と言っても過言ではないと思います。

ただし、今回は、そもそも、検索のため LIKE 句を使った SELECT クエリを投げる必要はないのです。frmAddressList_Load で SELECT クエリを MySQL に投げて全レコードが DataTable に格納されているのだから、その DataTable から検索すればいいのです。

具体的には、DataTable の DefaultView プロパティで DataView を取得し、取得した DataView の RowFilter プロパティにフィルター処理に使用する式を設定します。

以下がサンプルです。Search_Click メソッドのコードを見てください。

C#

1using System; 2using System.Windows.Forms; 3 4namespace WindowsFormsMySql 5{ 6 public partial class Form1 : Form 7 { 8 public Form1() 9 { 10 InitializeComponent(); 11 } 12 13 private void countryBindingNavigatorSaveItem_Click(object sender, EventArgs e) 14 { 15 this.Validate(); 16 this.countryBindingSource.EndEdit(); 17 this.tableAdapterManager.UpdateAll(this.worldDataSet); 18 19 } 20 21 private void Form1_Load(object sender, EventArgs e) 22 { 23 // TODO: このコード行はデータを 'worldDataSet.country' テーブルに読み込みます。 24 // 必要に応じて移動、または削除をしてください。 25 this.countryTableAdapter.Fill(this.worldDataSet.country); 26 27 } 28 29 // ここまでは Visual Studio が自動生成したコード 30 31 // 以下が検索を行うため追加したコード 32 private void Search_Click(object sender, EventArgs e) 33 { 34 if (!String.IsNullOrEmpty(this.textBox1.Text)) 35 { 36 this.worldDataSet.country.DefaultView.RowFilter = 37 "Name LIKE '%" + this.textBox1.Text + "%'"; 38 39 this.countryBindingSource.DataSource = this.worldDataSet.country; 40 // こちらはダメ。結果が DataGridView に反映されない 41 //this.countryBindingSource.ResetBindings(false); 42 } 43 else 44 { 45 // TextBox が空白の場合は元に戻す(全レコード表示) 46 this.worldDataSet.country.DefaultView.RowFilter = ""; 47 } 48 } 49 } 50}

上のコードは MySQL のサンプルデータベース world の country テーブルを使っています。それをベースに Visual Studio のデータソース構成ウィザードを使ってアプリを作成しています。

質問欄下の私のコメントで書きましたが、MySQL のテーブルのレコード一覧を DataGridView に表示し、ユーザーが DataGridView を見て編集した結果を、元のデータベースのテーブルに反映させるアプリが、自力では一行もコードを書かなくても、ウィザードベースで簡単に自動生成されます。

それに、Name 列をあいまい検索するため、Label, TextBox, Button をデザイン画面でドラッグ&ドロップし、Search_Click メソッドを追記したものです。

初期画面は以下のようになります。

イメージ説明

そこから Name 列を ja という文字列であいまい検索した結果が以下です。

イメージ説明

投稿2023/03/19 01:41

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

LLL

2023/03/19 04:55

ご丁寧にありがとうございます。 参考にさせていただきます。
guest

0

ベストアンサー

文字列と変数の違いを理解されていないようです。

"select * from d_contactinformation where fullname like '%txtkey%'"
これは
'%txtkey%'の部分にtxtkeyの変数の内容が入るのではなく
'%txtkey%'という文字列になるだけです。

投稿2023/03/18 10:07

YAmaGNZ

総合スコア10258

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

LLL

2023/03/19 04:57

ご回答いただきありがとうございます。 ご教授いただいたヒントをもとに、うまく作動させることができました。
退会済みユーザー

退会済みユーザー

2023/03/19 05:52 編集

質問者さん> 私が回答で述べた「ユーザー入力から SQL 文の文字列を組み立てることは絶対やってはいけないこと」というのは理解されてますか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問