نوشته شده توسط
majid_3ma
با سلام. فرمودید نظر و انتقاد چشم.
1- به program کد زیر رو اضافه کنید تا از چندبار اجرا شدن برنامه خودداری بشه.
استفاده از:
بعداز :
کد :
{
bool ok;
Mutex m = new Mutex(true, "تقویم پاییزی", out ok);
if (!ok)
{
return;
}
و بعد ادامه ماجرا:
Application.EnableVisualStyles();
2- بعداز لود بنظر من اگه لیست یاداوری ها خالی باشه نباید اون فرم هم لود بشه. چون کمی گیج کننده است.
3- اگه به نرم افزار هایی مثل office ، photoshop و امثالهم توجه کرده باشید می بینید که در گزینه اول منو قالبا گزینه های داخل منو کارهایی رو انجام میدن که در راستای ایجاد اطلاعات و ذخیره و... پیش میره. در حالی که نرم افزار شما در منوی اول و گزینه اول نمایش اطلاعات داره.!
4- هنگام ایجاد اطلاعات: اگه شرطی گذاشتید که با درست بودن شرط اطلاعات ذخیره بشه و در غیر اینصورت هیچ عملی انجام نده و فقط یک پیغام ظاهر بشه باید در نقض شرط بعداز show کردن پیغام مورد نظر یه return بزارید تا از ادامه اجرای کدهای بعدی جلوگیری بشه.
مثال: در منوی افزودن اشخاص حقیقی اگه فقط تاریخ رو وارد کنیم با نشان دادن پیغام های مورد نظر در آخر اطلاعات پوچ در دیتابیس ذخیره میشه!!!
5- از طرفی شما بجای استفاده از showdialog در هنگام فرخوانی فرم ها نوشتید
بجای اون بهتره بنویسید:
AddPeople.ShowDialog();
تا از اجرای چندباره فرم مذکور جلوگیری بشه.
6- ویرایش اطلاعات بدرستی عمل نمیکنه.
اگه میخوایید فرمی رو ویرایش کنید بهتره حداقل دو گزینه بنام های ویرایش و ذخیره بزارید که پس از انتخاب شخص مورد نظر و کلیک روی گزینه ویرایش مثلا تکست باکس ها فعال بشن و کاربر بتونه تغییرات رو انجام بده و بعد با کلیک روی ذخیره اطلاعات ویرایش شده در دیتابیس ذخیره بشه.
7- در برنامه های کوچک بهتره از دیتابیس اکسس استفاده بشه تا SQL البته این نظر شخصی منه
8- بقیه گزینه ها رو هم وقت نکردم چک کنم. ایشالا بعداز رفع این نقطه نظرات به بقیه هم خواهیم رسید.
در کل واسه اول راهید و خوبه.
موفق باشید.