اشکال در اجرای دستور

[mehdi4467]

سلام
نمی دونم چرا خط زیر syntax error میده؟

string text = "insert into check (ch_bankid,ch_series,ch_number)values('" +
bank_id + "','" +
seriesNumberTextBox.Text + "','" +
i + "')";

[Programmer 1]

bank_id آيا يك تكست باكسه اگر آره بايد بنويسيد : bank_id.Text و همچنين i هم به همين صورت .
لطفا وقتي سوال مي كنيد كمي هم توضيح بيشتري در موردش بديد تا هم خودتون معطل نشيد و هم پاسخ دهنده سردرگم نشه ، مثلا اينجا معلوم نيست i يك متغيره كه فبلا مقدار گرفته يا نام يك تكست باكس و يا هر چيز ديگه اي . بازم اگر مشكل رفع نشد در خدمت هستم .

با تشكر /.

[hamid67fathi]

دوست عزيز احتمالاً به خاطره تک کتيشن ( ' ) باشه که اطراف bank_id و i گذاشتيد.

[mehdi4467]

bank_id آيا يك تكست باكسه اگر آره بايد بنويسيد : bank_id.Text و همچنين i هم به همين صورت .
لطفا وقتي سوال مي كنيد كمي هم توضيح بيشتري در موردش بديد تا هم خودتون معطل نشيد و هم پاسخ دهنده سردرگم نشه ، مثلا اينجا معلوم نيست i يك متغيره كه فبلا مقدار گرفته يا نام يك تكست باكس و يا هر چيز ديگه اي . بازم اگر مشكل رفع نشد در خدمت هستم .

با تشكر /.




ممنون
خطوط قبلی به این شکله:


int i;
string bank_id_sql_command = "Select bank_id from bank where bank_name = '" + accountNumbercomboBox.Text + "'";
string a1 = "provider=microsoft.jet.oledb.4.0;data source=bank.mdb";
OleDbConnection con1 = new OleDbConnection(a1);
OleDbCommand cmd5 = new OleDbCommand(bank_id_sql_command, con1);
con1.Open();
int bank_id = Convert.ToInt32(cmd5.ExecuteScalar());
con1.Close();

for (i = Convert.ToInt32(fisrtNumberTextBox.Text); i < Convert.ToInt32(fisrtNumberTextBox.Text) - 1 + Convert.ToInt32(NumberOfPaperCmboBox.Text); i++)
{
Class1 c2 = new Class1();

string text = "insert into check (ch_bankid,ch_series,ch_number)values('" +
bank_id + "','" +
seriesNumberTextBox.Text + "','" +
i + "')";
c2.insert(text);

که کلاس C1 مربوط به احرای کوئری هست که درست کار میکنه.

[mehdi4467]

دوست عزيز احتمالاً به خاطره تک کتيشن ( ' ) باشه که اطراف bank_id و i گذاشتيد.

نه متاسفانه. تک کوتیشن ها رو برداشتم ولی فرقی نکرد.

[cheshm6k]

سلام .

bank_idو i رو به string تبدیل کنید واگه نوع این دو فیلد در جدول number هست نباید کوتیشن بذاری.

موفق باشید.

[mehdi4467]

سلام .

bank_idو i رو به string تبدیل کنید واگه نوع این دو فیلد در جدول number هست نباید کوتیشن بذاری.

موفق باشید.

جواب نداد. به string تبدیل کردم و چه با تک کوتیشن و چه بی تک کوتیشن جواب نداد.
خیلی عجیبه من همین دستور رو در جای دیگه و برای جدول دیگری از دیتا بیس استفاده کرده ام و جواب داده . کد زیر رو ببینید. این کد جواب داده است.


string text = "insert into trakonesh (tr_bankid,tr_date,tr_date_miladi,tr_time,tr_date_ time,tr_sanadno,tr_ghabzno,tr_checkno,tr_tozihat,t r_bedehkar,tr_bestankar)values('" +
bank_id + "','" +
date1 + "','" +
dd + "','" +
time1 + "','" +
date_time + "','" +
sanadNumberTextBox.Text + "','" +
ghabzNumberTextBox.Text + "','" +
checkNumberTextBox.Text + "','" +
tozihatTextBox.Text + "','" +
bedehkarValueTextBox.Text + "','" +
bestankarValueTextBox.Text + "')";


یک سوالی من جدول check رو تازه اضاقه کرده ام. نمی تونه از این باشه؟

[mahdi87_gh]

دوست عزیز این روش کدنویسی شما اشکال داره. شما بهتره از پارامترها استفاده کنید یا حداقل در نوشتن این جور دستورات از String.Format استفاده کنید

[mehdi4467]

دوست عزیز این روش کدنویسی شما اشکال داره. شما بهتره از پارامترها استفاده کنید یا حداقل در نوشتن این جور دستورات از String.Format استفاده کنید

می تونید توضیح بیشتری بدهید؟

[cheshm6k]

یک سوالی من جدول check رو تازه اضاقه کرده ام. نمی تونه از این باشه؟

برنامه اجرا میشه ؟

[mehdi4467]

برنامه اجرا میشه ؟

آره و برای اینکه مطمئن بشم جدول check رو پیدا می کنه در عبارت sql نام check رو به check1 تغییر دادم این بار خطا عوض شد و گفت این جدول وجود نداره. از اینجا مطمئن شده جدول رو پیدا می کنه.

[mahdi87_gh]

می تونید توضیح بیشتری بدهید؟ برای نوشتن این دستورات بهتر است همیشه از گارامترها استفاده کنید. در سایت نمونه های فراوانی وجود داره که بگردید پیدا می کنید. اما در مورد فرمت بشکل زیر عمل کنید:

string.Format("Insert Into Table1 (field1,field2) Values ('{0}','{1}'", var1, var2);
var1 بجای 0 قرار میگیره و var2 بجای 1 اینطوری از نوشتن 50 تا += خلاص میشین

[cheshm6k]

اینجا رو ببین :

http://www.barnamenevis.org/forum/showpost.php?p=1085848&postcount=7

یه نمونه از کار با پارامترهاست.

[MoS4k1]

متغیرهاتو به صورت local تعریف کن

[mehdi4467]

سلام
ممنون از راهنمايي هاتون ولي مشكل من فكر نمي كنم به موارد بالا مربوط باشه. درسته اين روش نوشتن اشتباهه ولي اين مشكل مربوط به ايت قضايا نميشه. من كد را مي گذارم اگر ممكنه يك نگاهي كنيد.

تشكر

[علیرضا حسن زاده]

دوست عزیز من برنامه شمارو مطالعه کردم ایراد برنامه شما از نام جدول هست احتمالا عبارت Check یه کلمه کلیدی در اکسس هست و اگه اون رو به Check2 یا Chek تغییر نام بدید برنامتون درست کار میکنه
** یه موردی هم در مورد دیتا بیستون چرا برای هر برگ چک یک رکورد اضافه می کنید این به نظر من اشتباهه و در زمان اجرا میشه اون رو محاسبه کرد**

[mehdi4467]

دوست عزیز من برنامه شمارو مطالعه کردم ایراد برنامه شما از نام جدول هست احتمالا عبارت Check یه کلمه کلیدی در اکسس هست و اگه اون رو به Check2 یا Chek تغییر نام بدید برنامتون درست کار میکنه
دست شما درد نكنه كار كرد. ولي جالبه من براي اينكه مطمئن بشم دستور درسته مستقيم تو access اجرا كرده بودم و كار مي كرد.


** یه موردی هم در مورد دیتا بیستون چرا برای هر برگ چک یک رکورد اضافه می کنید این به نظر من اشتباهه و در زمان اجرا میشه اون رو محاسبه کرد**

من در اين مورد فكر مي كنم اگر برگه هاي چك رو ذخيره كنم بهتره. چون بعد قرار يكسري مقايسه بشه و موارد ديگري كه ممكنه محاسبات رو پيچيده كنه و سنگين. البته نمي دونم اين حرفم چقدر درسته.

[علیرضا حسن زاده]

می تونی نام جدول رو به صورت [Check] بنویسی و مشکل حل بشه ولی به کار بردن نام های کلیدی معمولا مشکل ساز میشه