استفاده بد از تکنولوژی

گاهی پیش میاد که توی کدهایی که میخونم از برنامه نویسهای دیگه مشکلات زیر رو توشون میبینم:

1.از یک Framework استفاده شده ولی از تمامی قابلیتهاش استفاده نشده و گاهی میبینیم قابلیتی که توی فریمورک به خوبی و استاندارد پیاده سازی شده رو استفاده نکرده و خود شخص اون قابلیت رو کد زده

2.کتابخانه یا فریمورکی نصب شده و قابلیتی داره ولی از اون استفاده نشده و برای اون قابلیت کتابخانه دیگری نصب شده

دلیل این اشتباه اینه که برنامه نویس مستندات اون فریمورک و کتابخانه رو نخونده. کد زیر مثالی از حالت اوله. ببینید در ادامه یک فرم جنگو Django میبینیم:

class ServiceForm(ModelForm):
    def __init__(self, *args, **kwargs):
        super(ServiceFormPart3, self).__init__(*args, **kwargs)
        self.fields['service_account'].empty_label = ' '
        self.fields['technician'].empty_label = ' '
        self.fields['service_manager'].empty_label = ' '
        

    class Meta:
        model = Service
        fields = [
            "service_account", "technician", "service_manager"...
        ]

class ServiceForm(ModelForm):

def __init__(self, *args, **kwargs):

super(ServiceFormPart3, self).__init__(*args, **kwargs)

self.fields['service_account'].empty_label = ' '

self.fields['technician'].empty_label = ' '

self.fields['service_manager'].empty_label = ' '

class Meta:

model = Service

fields = [

"service_account", "technician", "service_manager"...

]

و کد زیر view هست که توی اون از این فرم استفاده شده:

def service(request, service_id):
    svc = Service.objects.get(pk=service_id)
    update_versions(svc)
    context = {
        'id': service_id,
        'service': svc,
        'form1': ServiceForm(instance=svc),
    }
    if request.method == "POST":
        name = request.POST.get("name", None)
        if not name:
            messages.error(request, '...', extra_tags='danger')
            return redirect(f'/man/service/{service_id}/')
        branch = request.POST.get("branch", None)
        if not branch:
            messages.error(request, 'ا...', extra_tags='danger')
            return redirect(f'/man/service/{service_id}/')
        office = request.POST.get("office", None)
        if service_account == '':
            service_account = None
        technician = request.POST.get("technician", None)
        if technician == '':
            technician = None
        service_manager = request.POST.get("service_manager", None)
        if service_manager == '':
            service_manager = None
        

        svc.service_account_id = service_account
        svc.technician_id = technician
        svc.service_manager_id = service_manager
        svc.save()
        messages.success(request, 'اطلاعات سرویس با موفقیت به روز شد')
        messages.info(request, json.dumps({
            'service_id': f'service_{service_id}',
        }))
        return redirect(f'/man/service/')
    else:
        return render(request, "man/service/service.html", context)

def service(request, service_id):

svc = Service.objects.get(pk=service_id)

update_versions(svc)

context = {

'id': service_id,

'service': svc,

'form1': ServiceForm(instance=svc),

}

if request.method == "POST":

name = request.POST.get("name", None)

if not name:

messages.error(request, '...', extra_tags='danger')

return redirect(f'/man/service/{service_id}/')

branch = request.POST.get("branch", None)

if not branch:

messages.error(request, 'ا...', extra_tags='danger')

return redirect(f'/man/service/{service_id}/')

office = request.POST.get("office", None)

if service_account == '':

service_account = None

technician = request.POST.get("technician", None)

if technician == '':

technician = None

service_manager = request.POST.get("service_manager", None)

if service_manager == '':

service_manager = None

svc.service_account_id = service_account

svc.technician_id = technician

svc.service_manager_id = service_manager

svc.save()

messages.success(request, 'اطلاعات سرویس با موفقیت به روز شد')

messages.info(request, json.dumps({

'service_id': f'service_{service_id}',

}))

return redirect(f'/man/service/')

else:

return render(request, "man/service/service.html", context)

مشکل کجاست؟ برنامه نویس اومده مقدار فیلدهایی که از template ارسال شده POST با استفاده از request.POST.get گرفته شده. بعد برنامه نویس اومده field validation (اعتبارسنجی فیلدها) رو انجام داده. بعد اگر توی ورودی فیلد ایرادی وجود داشت اومده یک پیام ارور به کاربر نشون داده. تمامی این کاری که برنامه نویس براش کد زده توی django.forms.Formپیاده سازی شده. به بهترین شکل توی جنگو پیاده سازی شده. در حالی که با تست این view مشکلات خیلی زیادی هست.

با استفاده از ServiceForm.is_valid() تمامی فیلدها validate و اعتبار سنجی میشن و اگر ایرادی داشته باشه همونجا فیلد قرمز رنگ میشه و ارور مرتبط نشان داده میشه اما در این حالت اگر یک فیلد required توی فرم خالی بمونه ارور مرتبط به کاربر نشان داده نمیشه. مشکل دوم اینه که مثلا اگر یک فیلد ایمیل یا پسورد باشه ایمیل با روش های استاندارد اعتبارسنجی میشه و پسورد هم همینطور.

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

همینطور که توی این لینک میبینید فرم جنگو چند لایه صحت سنجی داره.

همیشه اگر از فریمورک یا کتابخانه ای استفاده میکنید حتما تمامی مستنداتش رو بخونید.

دیدگاهتان را بنویسید لغو پاسخ