Refactoring/replace parameter with method call

المشكلة

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

الحل

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

مثال

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

تخزين القيمة التي يعيدها كلٌّ من التابعين ()getSeasonalDiscount و ()getFees في متغير ثم تمريرها إلى التابع ()discountedPrice:

في لغة 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)

في لغة TypeScript:

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

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

استدعاء التابعين السابقين ضمن التابع ()discountedPrice نفسه:

في لغة 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)

في TypeScript:

let basePrice = quantity * itemPrice;
let finalPrice = discountedPrice(basePrice);

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

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

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

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

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

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

آلية الحل

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

انظر أيضًا

• القائمة الطويلة للمعاملات (Long Parameter List).
• البيانات المُجمَّعة (Data Clumps).
• ازالة توابع الإعدادات (Remove Setting Method)
• التوابع الطويلة (Long Methods).

مصادر

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