Ｃ＃、ora-00933エラーについて

質問

ora-00933エラーについて

ボタンを押したらグリッドビューにSQL文で取得したデータが表示されるプログラムを作っていますが

private void button1_Click(object sender, EventArgs e)
{
string strSQL = string.Empty;
OleDbDataAdapter oraAdp;
strSQL = "select Kounyurireki.Hiduke,SyohinHyo.Syouhinmei,SyouhinHyo.Tanka,Kounyurireki.Suryou from SyohinHyo,Kounyurireki where Kounyurireki.Syohinbago = SyohinHyo.Syouhinbangou and between'"+ dateTimePicker1.Text + "and" + dateTimePicker2.Text+"'";
oraAdp = new OleDbDataAdapter(strSQL, this.m_oDbConn);
oraAdp.Fill(this.m_dsGridBind,"MEISAI");
if (this.m_dsGridBind.Tables["MEISAI"].Rows.Count == 0)
{
MessageBox.Show("該当データがありません");
return;
}
else
{
this.dataGridView1.DataSource = this.m_dsGridBind;
this.dataGridView1.DataMember = "MEISAI";
}
}
を実行したところoraAdp.Fill(this.m_dsGridBind,"MEISAI");でora-00933：式がありませんと出ます。
どこがいけないのでしょうか？？教えていただきたいです。

回答No.4

bin-chan · ベストアンサー

＃３は、betweenが対象とする列（Kounyurireki.Hiduke？）の記述が欠けたままなので
適宜修正してください。

回答No.3

bin-chan

> ora-00905：キーワードが見つかりませんと
「解決した」とは「betweenの後ろに列を追加した結果、正常動作に至った」ですか？
その後別な問題が生じた？正常動作に至ってないでしょ。

ora-00905は、対を成すキーワードが見つからないといっている。
bewteenには範囲のfromとtoを示すためのandが必要だが、それが欠けてる。

今回の場合、betweenの後に出てくる and は"'and'"が正しい。
構文はbewtten <列名> <範囲from> and <範囲to>だけど、
文字列である範囲from・範囲toを括るシングルクォートの記述が足りない。
誤）between<列名>'2010/10/01and2010/11/30'
正）between <列名> '2010/10/01' and '2010/11/30'

strSQL を編集し終わった段階でデバッグ表示させ、SQL*Plusにでも貼り付けて
試験する習慣つけたほうが良いと思います。

また無駄なようでも、strSQLの編集を分けて書くと可読性と移植性が向上します。
＃１の通り、適宜半角スペースを入れること。特にカンマの次。ピリオドとの誤読が減らせます。
strSQL = "select Kounyurireki.Hiduke, SyohinHyo.Syouhinmei, SyouhinHyo.Tanka, Kounyurireki.Suryou "
strSQL = strSQL & "from SyohinHyo, Kounyurireki "
strSQL = strSQL & "where Kounyurireki.Syohinbago = SyohinHyo.Syouhinbangou "
strSQL = strSQL & "and between '"+ dateTimePicker1.Text + "' and '" + dateTimePicker2.Text+"'";

aliasも研究してね。記述しやすいので間違いが少なくなりますよ。
strSQL = "select K.Hiduke, S.Syouhinmei, S.Tanka, K.Suryou "
strSQL = strSQL & "from SyohinHyo S, Kounyurireki K "
strSQL = strSQL & "where K.Syohinbago = S.Syouhinbangou "
strSQL = strSQL & "and between '"+ dateTimePicker1.Text + "' and '" + dateTimePicker2.Text+"'";

回答No.2

bin-chan

投稿が二重になってたらすみません。

SQL文に誤りがあります。

where Kounyurireki.Syohinbago = SyohinHyo.Syouhinbangou
and between'"+ dateTimePicker1.Text + "and" + dateTimePicker2.Text+"'";

where Kounyurireki.Syohinbago = SyohinHyo.Syouhinbangou までは良いのですが、
その次の and とbetween の間に列が記述されていませんよ。（Kounyurireki.Hidukeかな？）

補足

解決しました！！がora-00905：キーワードが見つかりませんと出たのですがどうすればよいでしょうか？質問とは別ですが回答していただければとてもありがたいです。

回答No.1

bin-chan

SQL文のWhereで、betweenがおかしいように思います。
(Betweenの前に列名が無い）

カンマの後ろに半角スペース入れる習慣つけると読みやすいですよ。