PDA

View Full Version : این روش insert کردن صحیح است؟


whiterabbit
سه شنبه 02 مهر 1392, 16:49 عصر
دوستان من با این روش اطلاعات و ذخیره میکنم ایا مشکلی داره؟
cmd.CommandText = "Insert Into [Contact] ([Time], [Name], [Email], [Subject], [Contact], [Code]) Values (N'" + DateandTime.ToString() + "' , N'" + txtName.Text + "' , N'" + txtEmail.Text + "' , N'" + lstSubject.Text + "' , N'" + txtContact.Text + "', N'" + ResultCode.ToString() + "')";

برای یک پروژه معمولی
fakhravari
سه شنبه 02 مهر 1392, 17:35 عصر
:بامزه: خوب چرا پارامتريش نميكني.
4 تا خط بيشتر نيستا.
merlin425
سه شنبه 02 مهر 1392, 18:00 عصر
اگه فیلد های بانکتو nvarchar تعریف کردی دیگه نیاز نیست قیل از مقادیر N بزاری ولی در کل اره این شکلیم میشه ولی عذابه این کارو نکن برو سراغ EF
whiterabbit
سه شنبه 02 مهر 1392, 19:16 عصر
یه پروژه ای رو بخش های مختلفش و اینطوری نوشتم
الان اگه بخوام برگردم از اول پارامتری بنویسم واقعا عذاب اور هستش

یکی از بچه ها از لحاظ امنیتی گفت ایراد داره
merlin425
سه شنبه 02 مهر 1392, 21:51 عصر
یکی از بچه ها از لحاظ امنیتی گفت ایراد داره

اگه تحت وب باشه اره مشکل زا میشه
sql injection رو Search کن
whiterabbit
چهارشنبه 03 مهر 1392, 01:34 صبح
با تشکر از شما
میشه یک روش ساده و امنوارد کردن اطلاعات تو دیتا بیس و بگید؟ مثال
mehdi.mousavi
چهارشنبه 03 مهر 1392, 03:42 صبح
سلام.
خیر این روش صحیح نیست. شما حتما باید از Parametric Command (http://msdn.microsoft.com/en-us/library/yy6y35y8.aspx) ها استفاده کنید.
برای مشاهده نمونه به لینک فوق (تقریبا انتهای صفحه، مثال C#) مراجعه کنید.

موفق باشید.
mahdidabaghi
چهارشنبه 03 مهر 1392, 09:19 صبح
دوستان من با این روش اطلاعات و ذخیره میکنم ایا مشکلی داره؟
cmd.CommandText = "Insert Into [Contact] ([Time], [Name], [Email], [Subject], [Contact], [Code]) Values (N'" + DateandTime.ToString() + "' , N'" + txtName.Text + "' , N'" + txtEmail.Text + "' , N'" + lstSubject.Text + "' , N'" + txtContact.Text + "', N'" + ResultCode.ToString() + "')";

برای یک پروژه معمولی
با سلام
پیشنهاد بنده هم استفاده از Store procedure هست. که از لحاظ سرعت و امنیت نسبت به روش شما بهینه تره ...
Good Luck
fakhravari
پنج شنبه 04 مهر 1392, 17:44 عصر
فرقي نداره بين استور و پارامتر ها .
فقط سرعت بيشتري داره استور ها ، از نظر امنيتي گفتم :متفکر: