15-10-18, 01:19 AM
اسمح لي عندي بعض الملاحظات
تم استخدام الداتا سيت كمصدر للبيانات
لماذا لا يتم استخدام BindingSouce ايضا قلما تجد داتا سيت من غيرها ستوفر عليك المزيد من الوقت والجهد مثلا
سيكون كافيا ان تكتب مثل
كما سيمكنك ربط الادوات مرئيا ايضا .
ايضا هناك انشاء لكائن الاتصال اكثر من مرة مرة في النموذج الاول ومرة في نموذج الاضافة وسيكون في التعديل وما الى ذلك مثل
لماذا لا يكون عاما على مستوى المشروع
وايضا لم اشاهد امر التخلص منه بعد استخدامه ، اي قتله وازالته من الذاكرة .
ايضا طالما تم استخدام الداتا سيت لماذا الاضافة والتعديل بطريقة غير مباشرة
طالما لدينا داتاسيت يمكن استخدامها للتعديل والاضافة مباشرة عن طريقها .
وايضا هنا لم يتم استخدام البارمترات لتمرير البيانات وهذا فيه خطر على قاعدة البيانات وسلامة البيانات .
ولا اعرف ما الذي فاتني ايضا هذه ملاحظاتي على السريع
تم استخدام الداتا سيت كمصدر للبيانات
كود :
TextBox1.DataBindings.Add("Text", DataSet1, "Table1.Name")
لماذا لا يتم استخدام BindingSouce ايضا قلما تجد داتا سيت من غيرها ستوفر عليك المزيد من الوقت والجهد مثلا
كود :
Me.BindingContext(DataSet1, "table1").Position = 0
ChangePosition()
سيكون كافيا ان تكتب مثل
كود :
BindingSouce.MoveFirst
كما سيمكنك ربط الادوات مرئيا ايضا .
ايضا هناك انشاء لكائن الاتصال اكثر من مرة مرة في النموذج الاول ومرة في نموذج الاضافة وسيكون في التعديل وما الى ذلك مثل
كود :
Dim Conn As New OleDbConnection(ConStr)
لماذا لا يكون عاما على مستوى المشروع
وايضا لم اشاهد امر التخلص منه بعد استخدامه ، اي قتله وازالته من الذاكرة .
ايضا طالما تم استخدام الداتا سيت لماذا الاضافة والتعديل بطريقة غير مباشرة
كود :
SavInto.CommandText = "UPDATE Table1 SET name = '" & Trim(TextBox1.Text) & "' ,City = '" & _
Trim(TextBox2.Text) & "' , mobile= '" & Trim(TextBox3.Text) & "' WHERE mobile ='" & n & "'"
طالما لدينا داتاسيت يمكن استخدامها للتعديل والاضافة مباشرة عن طريقها .
وايضا هنا لم يتم استخدام البارمترات لتمرير البيانات وهذا فيه خطر على قاعدة البيانات وسلامة البيانات .
ولا اعرف ما الذي فاتني ايضا هذه ملاحظاتي على السريع