البرمجة

تحسين شيفرة Python: إصلاح الأخطاء وتحسين الأداء

18/02/2024
4 2 دقائق

في البداية، يظهر أن لديك بعض الأخطاء في الشيفرة البرمجية الخاصة بك. سأقوم بتحليلها وتوفير التوجيه اللازم لإصلاحها:

أولاً وقبل كل شيء، يبدو أنك قد استخدمت خاصية return بشكل غير صحيحة داخل دالة print_menu(). الدالة return يجب أن تستخدم لإرجاع قيمة واحدة فقط، ولكن في حالتك، يمكنك استخدام تعليمة print بدلاً من return لطباعة قائمة الخيارات. يمكنك تحسين الأمور عن طريق تغيير الكود إلى ما يلي:

python
def print_menu():
    print '1. Translate a string to numbers'
    print '2. Translate numbers to a string'
    print '3. Quit'

ثانياً، عند تحديد القائمة num_ltr و ltr_num، يجب عليك استخدام list بدلاً من dict. يمكنك تعديل جزء الكود كما يلي:

python
num_ltr = [None] + list(string.ascii_lowercase)
ltr_num = {ltr: num for num, ltr in enumerate(string.ascii_lowercase, start=1)}

وبالتالي، لن تحتاج إلى دورة num يدويًا.

ثالثًا، في دالة ltr_to_num، يجب أن تقوم بفحص إذا كان الحرف في ltr_num قبل الطباعة. يمكنك تحسين هذه الدالة كالتالي:

python
def ltr_to_num(s, ltr_num):
    for char in s:
        if char in ltr_num:
            print(ltr_num[char])
    print()

رابعًا، في دالة num_to_ltr، هناك مشكلة في كيفية استخدامك لـ else، يمكنك تصحيحها كما يلي:

python
def num_to_ltr(num_ltr, s):
    num_list = s.split()
    sentence = ""
    for num in num_list:
        if num.isdigit():
            sentence += num_ltr[int(num)]
        else:
            sentence += num
    print(sentence)

وأخيرًا، يجب أن يكون لديك علامة == بدلاً من = عند مقارنة قيم user_choice في الجمل الشرطية. يمكنك تصحيحها كما يلي:

python
if user_choice == 1:
    # الشيفرة
elif user_choice == 2:
    # الشيفرة
elif user_choice == 3:
    # الشيفرة

بإجمال، هذه توجيهات لتحسين الشيفرة البرمجية الخاصة بك وتصحيح الأخطاء الشائعة. يمكنك استخدام هذه التحسينات لضمان عمل البرنامج بشكل صحيح وفعّال.

المزيد من المعلومات

بالطبع، دعني أضيف المزيد من المعلومات وأوسع تحليلي للشيفرة البرمجية:

أولاً، فيما يتعلق بتعريف num_ltr و ltr_num، استخدام قائمتين لربط الأحرف بالأرقام والعكس يعد فكرة جيدة، ولكن يجب أن تتأكد من أن حجم القائمتين يكون كافيًا لتغطية جميع الحالات. في حالتك، يجب أن تكون القائمتين بحجم 27، وليس 26، لضمان تضمين الصفر كعنصر فارغ. يمكنك تعديل الكود على النحو التالي:

python
num_ltr = [None] + list(string.ascii_lowercase)
ltr_num = {ltr: num for num, ltr in enumerate(string.ascii_lowercase, start=1)}

ثانيًا، في دالة print_menu()، يمكنك تحسين الشكل العام للقائمة عن طريق دمجها في سلسلة نصية واحدة، مما يزيد من قراءتها وجاذبيتها. يمكنك تعديلها كما يلي:

python
def print_menu():
    return '\n'.join(['1. Translate a string to numbers',
                      '2. Translate numbers to a string',
                      '3. Quit'])

ثالثًا، في دالة ltr_to_num()، قمت بطباعة مسافة فارغة بعد الحلقة الفورية. يمكنك حذف هذا السطر إذا لم تكن هناك حاجة إليه.

رابعًا، في دالة num_to_ltr()، لاحظت أنك تقوم بفحص num باستخدام isdigit() ومن ثم تقوم بتحويله إلى int قبل استخدامه كمفتاح في num_ltr. يمكنك تحسين هذا الجزء عن طريق إجراء الفحص بشكل مباشر في القائمة. يمكنك تعديل هذا الجزء كما يلي:

python
def num_to_ltr(num_ltr, s):
    num_list = s.split()
    sentence = ""
    for num in num_list:
        if num.isdigit() and 1 <= int(num) <= 26:
            sentence += num_ltr[int(num)]
        else:
            sentence += num
    print(sentence)

باختصار، هذه هي بعض التحسينات التي يمكن إجراؤها على الشيفرة البرمجية لتحسين فهمها وجعلها أكثر فعالية.

الوسوم
18/02/2024
4 2 دقائق
جميع الحقوق محفوظة مركز حلول تكنولوجيا المعلومات
زر الذهاب إلى الأعلى