Refactoring/replace parameter with method call

المشكلة

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

الحل

بدلًا من تمرير القيمة من خلال المعامل، حاول وضع استدعاء الاستعلام داخل متن التابع.

مثال

قبل إعادة التصميم

في لغة Java:

int basePrice = quantity * itemPrice;
double seasonDiscount = this.getSeasonalDiscount();
double fees = this.getFees();
double finalPrice = discountedPrice(basePrice, seasonDiscount, fees);

في لغة C#‎:

int basePrice = quantity * itemPrice;
double seasonDiscount = this.GetSeasonalDiscount();
double fees = this.GetFees();
double finalPrice = DiscountedPrice(basePrice, seasonDiscount, fees);

في لغة PHP:

$basePrice = $this->quantity * $this->itemPrice;
$seasonDiscount = $this->getSeasonalDiscount();
$fees = $this->getFees();
$finalPrice = $this->discountedPrice($basePrice, $seasonDiscount, $fees);

في لغة Python:

basePrice = quantity * itemPrice
seasonalDiscount = this.getSeasonalDiscount()
fees = this.getFees()
finalPrice = discountedPrice(basePrice, seasonalDiscount, fees)

بعد إعادة التصميم

في لغة Java:

int basePrice = quantity * itemPrice;
double finalPrice = discountedPrice(basePrice);

في لغة C#‎:

int basePrice = quantity * itemPrice;
double finalPrice = DiscountedPrice(basePrice);

في لغة PHP:

$basePrice = $this->quantity * $this->itemPrice;
$finalPrice = $this->discountedPrice($basePrice);

في لغة Python:

basePrice = quantity * itemPrice
finalPrice = discountedPrice(basePrice)

لم إعادة التصميم؟

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

فوائد تطبيق الحل

• التخلص من المعاملات غير الضرورية وتبسيط استدعاءات التابع. غالبًا ما لا تُنشأ هذه المعاملات لاستخدام المشروع في وضعه الحالي، ولكن لاحتياجات المستقبل التي قد لا تأتي أبدًا.

مساوئ تطبيق الحل

• قد تحتاج إلى المعامل في الغد القريب لتلبية احتياجاتٍ أخرى... الأمر الذي يجعلك تعيد كتابة التابع.

آلية الحل

1. تأكد من أن شيفرة الحصول على القيمة لا تستخدم معاملات من التابع الحالي، لأنها لن تكون متاحة من داخل التوابع أخرى. إذا كان الأمر كذلك، فلن يكون من الممكن نقل الشيفرة البرمجية.
2. إذا كانت الشيفرة البرمجية ذات الصلة أكثر تعقيدًا من استدعاء تابع أو دالة منفردَين، استخدم استخراج التوابع لعزل هذه الشيفرة البرمجية في تابع جديد وجعل الاستدعاء بسيطًا.
3. استبدل جميع المراجع إلى المعامل المراد استبداله بالاستدعاءات إلى التابع الذي يحصل على القيمة داخل شيفرة التابع الرئيسي.
4. استخدم إزالة المعاملات لإزالة المعامل غير المستخدم الآن.

مصادر

• صفحة توثيق تبديل المعاملات باستدعاءات التوابع في موقع refactoring.guru.