توحيد التعبير الشرطي (Consolidate Conditional Expression)
المشكلة
وجود عدة شروط تؤدي إلى نفس النتيجة أو الإجراء.
الحل
توحيد جميع هذه الشروط في تعبير وحيد.
مثال
قبل إعادة التصميم
وجود عدة شروط يتم التحقق منها في الشيفرة:
في لغة Java:
double disabilityAmount() {
if (seniority < 2) {
return 0;
}
if (monthsDisabled > 12) {
return 0;
}
if (isPartTime) {
return 0;
}
// حساب مقدار العجز
//...
}
في لغة C#:
double DisabilityAmount()
{
if (seniority < 2)
{
return 0;
}
if (monthsDisabled > 12)
{
return 0;
}
if (isPartTime)
{
return 0;
}
// حساب مقدار العجز
//...
}
في لغة PHP:
function disabilityAmount() {
if ($this->seniority < 2) return 0;
if ($this->monthsDisabled > 12) return 0;
if ($this->isPartTime) return 0;
// حساب مقدار العجز
...
في لغة Python:
def disabilityAmount():
if seniority < 2:
return 0
if monthsDisabled > 12:
return 0
if isPartTime:
return 0
# حساب مقدار العجز
#...
بعد إعادة التصميم
إنشاء دالة باسم ()isNotEligableForDisability تضم جميع الشروط السابقة واستعمال بنية شرطية وحيدة للتحقق من القيمة التي تعيدها:
في لغة Java:
double disabilityAmount() {
if (isNotEligableForDisability()) {
return 0;
}
// حساب مقدار العجز
//...
}
في لغة C#:
double DisabilityAmount()
{
if (IsNotEligableForDisability())
{
return 0;
}
// حساب مقدار العجز
//...
}
في لغة PHP:
function disabilityAmount() {
if ($this->isNotEligableForDisability()) return 0;
// حساب مقدار العجز
...
في لغة Python:
def disabilityAmount():
if isNotEligableForDisability():
return 0
# حساب مقدار العجز
#...
لم إعادة التصميم؟
تحتوي الشيفرة البرمجية على العديد من العوامل المتناوبة التي تنفذ إجراءات متطابقة. ومن غير الواضح لماذا تنقسم هذه العوامل.
الغرض الرئيسي من التوحيد هو استخراج الشروط إلى توابع منفصلة لزيادة الوضوح.
فوائد تطبيق الحل
- إزالة تكرار الشيفرة البرمجية لتدفق التحكم، إذ يساعد الجمع بين العديد من الشروط التي لها نفس "الوجهة" على إظهار أنك تجري اختبارً واحدًا معقدًا فقط يقود إلى إجراء واحد.
- بتوحيد جميع العوامل، يمكن الآن عزل هذا التعبير المُعقَّد في تابع جديد له اسم يفسر الغرض من الشرط.
آلية الحل
قبل إعادة التصميم، تأكد من أن الشروط لا تملك أي "آثار جانبية" أو تُعدِّل شيئًا خلاف ذلك، بدلًا من مجرد إعادة القيم. قد تختبئ الآثار الجانبية في الشيفرة البرمجية المُنفَّذة داخل العامل نفسه، مثل عند إضافة شيئًا ما لمتغير استنادًا على نتائج الشَرطية.
- وحِّد الشرطية في تعبير واحد باستخدام
andوor. وكقاعدة عامة عند توحيد:- تُضم الشرطيات المتداخلة باستخدام
and. - تُضم الشرطيات المتتابعة باستخدام
or.
- تُضم الشرطيات المتداخلة باستخدام
- نفِّذ استخراج التوابع (Extract Methods) على شروط العامل واعطِه اسمًا يعكس الغرض من التعبير.