テキストボックスにテキストとDBのデータを比べる方法について

Question

前提・実現したいこと

Visual Studio2017のWindows Formアプリケーションにて，アプリを作成中です．
Form画面には，ボタンが2つと，テキストボックスが1つ，DataGridViewが1つ配置されています．
また，SQLiteにてDBを作成しました．テキストボックスに入力された文字列や数字が
DBの中にあるかどうかを判別するプログラムを書きたいです．ボタン1がクリックされると
DBに接続し，テーブルの作成やデータの追加などを行います．そしてボタン2が押されると
それらをDataGridViewに表示します． このボタン2をクリックしたときに，テキストボックスに入力された
テキストと，DBの中のデータと1つでも一致したら，OKとLabelに表示するプログラムを作成したいです．

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

TextBoxのテキストを変数に格納して，SELECT FROM WHERE INの中で検索しようとしたのですが，
変数が認識されません．
ソースコードいろいろと変更した結果中途半端な内容ですが
貼っておきます．

該当のソースコード

sing System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Data.SQLite;
using System.IO;

namespace Database_1018
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
        string path = "test.db";

        private void button1_Click(object sender, EventArgs e)
        {
            // テーブルを作成 (test.dbを新規で作成する場合のみ)
            if (File.Exists(path) == false)
            {
                using (SQLiteConnection con = new SQLiteConnection("Data Source=" + path))
                {
                    // DataBase接続開始
                    con.Open();
                    using (SQLiteCommand cmd = con.CreateCommand())
                    {
                        cmd.CommandText = "CREATE TABLE Lesson (社員ID INTEGER, name NVARCHAR(10), 年齢 INTEGER, 部署名 NVARCHAR(10), パスワード INTEGER)";
                        cmd.ExecuteNonQuery();
                    }
                }

            }

            using (SQLiteConnection con = new SQLiteConnection("Data Source=" + path))
            {
                con.Open();
                using (SQLiteCommand cmd = con.CreateCommand())
                {
                    con.Open();
                    cmd.CommandText = "INSERT INTO Lesson (社員ID,name,年齢,部署名,パスワード) VALUES(1000,'TOM',23,'開発技術部',11111111)";
                    cmd.ExecuteNonQuery();
                    cmd.CommandText = "INSERT INTO Lesson (社員ID,name,年齢,部署名,パスワード) VALUES(2000,'MIKE',23,'開発技術部',22222222)";
                    cmd.ExecuteNonQuery();
                    cmd.CommandText = "INSERT INTO Lesson (社員ID,name,年齢,部署名,パスワード) VALUES(3000,'JOHN',25,'開発技術部',33333333)";
                    cmd.ExecuteNonQuery();
                    cmd.CommandText = "INSERT INTO Lesson (社員ID,name,年齢,部署名,パスワード) VALUES(4000,'EMI',23,'開発技術部',44444444)";
                    cmd.ExecuteNonQuery();
                }
            }
        }
        private DataTable dataTable = new DataTable();

        protected override void OnLoad(EventArgs e)
        {
            dataGridView1.DataSource = dataTable;
            base.OnLoad(e);
        }

        private void button2_Click(object sender, EventArgs e)
        {
            string text1;
            text1 = textBox1.Text;

            using (SQLiteConnection con = new SQLiteConnection("Data Source=" + path))
            {
                con.Open();
                using (SQLiteDataAdapter adapter = new SQLiteDataAdapter("SELECT * FROM Lesson", con))
                adapter.Fill(dataTable);

                try
                {
                    string sql = "select * from lesson where text1";

                    SQLiteCommand com = new SQLiteCommand(sql, con);
                    SQLiteDataReader sdr = com.ExecuteReader();
                }
                finally
                {
                    con.Close();
                }
            }
        }
    }
}

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

Visual Studio 2017 C# Windows Form アプリケーション SQLite

Answer 1

string sql = "select * from lesson where text1";

上記の書き方が意図しているものと違っている気がします。以下のように変更して試してみたらいかがでしょうか。
name列で検索する前提ですが・・・

string sql = "select * from lesson where name =" + text1;

ダブルクォーテーション内の文字列は、特別な場合をのぞいてただの文字列として扱われます。
変数として扱いたい場合は、上記の通り文字列と連結するなどしてやらないといけません。

Comments

[mimi3103]

回答ありがとうございます． さっそくアドバイスいただいたことを実践してみたのですが
以下のようなエラーが発生してしまいました．

①using (SQLiteConnection con = new SQLiteConnection("Data Source=" + path))
using (SQLiteDataAdapter adapter = new SQLiteDataAdapter("SELECT * FROM Lesson WHERE name =" + text1, con))
{
adapter.Fill(dataTable);
}
にて実行した結果adapter.Fill(dataTable); の部分でSystem.Data.SQLite.SQLiteException: 'SQL logic error
incomplete input'
というエラーが発生してしまいました．

[mimi3103]

②string text1;
text1 = textBox1.Text;
try
{
string sql = "select * from lesson where name =" + text1;

SQLiteCommand com = new SQLiteCommand(sql, con);
SQLiteDataReader sdr = com.ExecuteReader();
if (sdr.Read() == true)
{
this.label1.Text = "ok";
}

}
finally
{
con.Close();
}
にて実行をした結果SQLiteDataReader sdr = com.ExecuteReader();の部分でSystem.Data.SQLite.SQLiteException: 'SQL logic error
incomplete input'というエラーが発生してしまいました．

そもそもDB自体がうまく作成されてないということなのでしょうか？
アドバイスお願いしたいです．

[YAmaGNZ]

実行するSQLを確認しましょう。
SQLiteで文字列を比較するときはシングルクォーテーションで囲まないといけません。
DBが絡む処理のデバッグを行う場合、実行されるSQLがどうなっているかは意識しないとできませんので
確認する癖をつけましょう。

[shun_kuwa]

YAmaGNZさんのおっしゃる通りで、私のサンプルにシングルクォーテーションが抜けておりました。
正しくは以下ですね。

string sql = "select * from lesson where name ='" + text1 + "'";

本来はstring.Format()等使ってもう少し見やすく書きたいところですが、問題の本質とは異なるのでここでは割愛させていただきます。

[mimi3103]

お二方，回答ありがとうございました．
そうですね．．．知識不足でした．これから勉強を進めてまいります．

シングルクォーテーションで囲んだところ，無事，求めていた機能を実現することができました．
みなさんありがとうございました！

Answer 2

ボタン２押したときに、ＤＢからdatagridviewにデータを表示後、テキストの内容とＤＢを比較とのことですが、ＤＢは他のユーザーから変更されていることは考えないのでしょうか？もし、変更されているとしたら
ｏｋとなっても、datagridviewには存在しないデータで合致する現象が発生すると思います。
この場合、ＤＢと比較するのではく、datagridviewの内容と比較するほうが確実なのではないでしょうか？

Comments

[mimi3103]

回答ありがとうございます！
今回は他のユーザからの変更は考慮しなくて問題ありませんでした．
すみません，記載しておくべきでした．．．