الفرق بين المراجعتين ل"Refactoring/replace parameter with method call"
اذهب إلى التنقل
اذهب إلى البحث
Khaled-yassin (نقاش | مساهمات) (أنشأ الصفحة ب'تبديل المعاملات باستدعاءات التوابع (Replace Parameter with Method Call) == المشكلة == استدعاء تابع استعلام وت...') |
Khaled-yassin (نقاش | مساهمات) ط |
||
| سطر 1: | سطر 1: | ||
| − | تبديل المعاملات باستدعاءات التوابع (Replace Parameter with Method Call) | + | <noinclude>{{DISPLAYTITLE: تبديل المعاملات باستدعاءات التوابع (Replace Parameter with Method Call)}}</noinclude> |
| − | |||
== المشكلة == | == المشكلة == | ||
استدعاء تابع استعلام وتمرير نتائجه كمعاملات لتابع آخر، في حين أنه يمكن لهذا التابع استدعاء الاستعلام مباشرة. | استدعاء تابع استعلام وتمرير نتائجه كمعاملات لتابع آخر، في حين أنه يمكن لهذا التابع استدعاء الاستعلام مباشرة. | ||
| سطر 10: | سطر 9: | ||
==== قبل إعادة التصميم ==== | ==== قبل إعادة التصميم ==== | ||
| − | في لغة Java: | + | في لغة Java:<syntaxhighlight lang="java"> |
| − | + | int basePrice = quantity * itemPrice; | |
| − | في لغة C#: | + | double seasonDiscount = this.getSeasonalDiscount(); |
| − | + | double fees = this.getFees(); | |
| − | في لغة PHP: | + | double finalPrice = discountedPrice(basePrice, seasonDiscount, fees); |
| − | + | </syntaxhighlight>في لغة C#:<syntaxhighlight lang="c#"> | |
| − | في لغة Python: | + | int basePrice = quantity * itemPrice; |
| + | double seasonDiscount = this.GetSeasonalDiscount(); | ||
| + | double fees = this.GetFees(); | ||
| + | double finalPrice = DiscountedPrice(basePrice, seasonDiscount, fees); | ||
| + | </syntaxhighlight>في لغة PHP:<syntaxhighlight lang="php"> | ||
| + | $basePrice = $this->quantity * $this->itemPrice; | ||
| + | $seasonDiscount = $this->getSeasonalDiscount(); | ||
| + | $fees = $this->getFees(); | ||
| + | $finalPrice = $this->discountedPrice($basePrice, $seasonDiscount, $fees); | ||
| + | </syntaxhighlight>في لغة Python:<syntaxhighlight lang="python"> | ||
| + | basePrice = quantity * itemPrice | ||
| + | seasonalDiscount = this.getSeasonalDiscount() | ||
| + | fees = this.getFees() | ||
| + | finalPrice = discountedPrice(basePrice, seasonalDiscount, fees) | ||
| + | </syntaxhighlight> | ||
==== بعد إعادة التصميم ==== | ==== بعد إعادة التصميم ==== | ||
| − | في لغة Java: | + | في لغة Java:<syntaxhighlight lang="java"> |
| − | + | int basePrice = quantity * itemPrice; | |
| − | + | double finalPrice = discountedPrice(basePrice); | |
| − | |||
| − | |||
| − | في لغة Python: | + | </syntaxhighlight>في لغة C#:<syntaxhighlight lang="c#"> |
| + | int basePrice = quantity * itemPrice; | ||
| + | double finalPrice = DiscountedPrice(basePrice); | ||
| + | </syntaxhighlight>في لغة PHP:<syntaxhighlight lang="php"> | ||
| + | $basePrice = $this->quantity * $this->itemPrice; | ||
| + | $finalPrice = $this->discountedPrice($basePrice); | ||
| + | </syntaxhighlight>في لغة Python:<syntaxhighlight lang="python"> | ||
| + | basePrice = quantity * itemPrice | ||
| + | finalPrice = discountedPrice(basePrice) | ||
| + | </syntaxhighlight> | ||
== لم إعادة التصميم؟ == | == لم إعادة التصميم؟ == | ||
| سطر 38: | سطر 58: | ||
== آلية الحل == | == آلية الحل == | ||
# تأكد من أن شيفرة الحصول على القيمة لا تستخدم معاملات من التابع الحالي، لأنها لن تكون متاحة من داخل التوابع أخرى. إذا كان الأمر كذلك، فلن يكون من الممكن نقل الشيفرة البرمجية. | # تأكد من أن شيفرة الحصول على القيمة لا تستخدم معاملات من التابع الحالي، لأنها لن تكون متاحة من داخل التوابع أخرى. إذا كان الأمر كذلك، فلن يكون من الممكن نقل الشيفرة البرمجية. | ||
| − | # إذا كانت الشيفرة البرمجية ذات الصلة أكثر تعقيدًا من استدعاء تابع أو دالة منفردَين، استخدم استخراج التوابع لعزل هذه الشيفرة البرمجية في تابع جديد وجعل الاستدعاء بسيطًا. | + | # إذا كانت الشيفرة البرمجية ذات الصلة أكثر تعقيدًا من استدعاء تابع أو دالة منفردَين، استخدم [[Refactoring/extract method|استخراج التوابع]] لعزل هذه الشيفرة البرمجية في تابع جديد وجعل الاستدعاء بسيطًا. |
# استبدل جميع المراجع إلى المعامل المراد استبداله بالاستدعاءات إلى التابع الذي يحصل على القيمة داخل شيفرة التابع الرئيسي. | # استبدل جميع المراجع إلى المعامل المراد استبداله بالاستدعاءات إلى التابع الذي يحصل على القيمة داخل شيفرة التابع الرئيسي. | ||
| − | # استخدم إزالة | + | # استخدم [[Refactoring/extract method|إزالة المعاملات]] لإزالة المعامل غير المستخدم الآن. |
== مصادر == | == مصادر == | ||
| − | * صفحة توثيق تبديل المعاملات باستدعاءات التوابع في موقع refactoring.guru. | + | * [https://refactoring.guru/replace-parameter-with-method-call صفحة توثيق تبديل المعاملات باستدعاءات التوابع في موقع refactoring.guru]. |
| + | [[تصنيف:Refactoring]] | ||
| + | [[تصنيف:Refactoring Techniques]] | ||
| + | [[تصنيف:Simplifying Method Calls]] | ||
مراجعة 09:29، 5 فبراير 2019
المشكلة
استدعاء تابع استعلام وتمرير نتائجه كمعاملات لتابع آخر، في حين أنه يمكن لهذا التابع استدعاء الاستعلام مباشرة.
الحل
بدلًا من تمرير القيمة من خلال المعامل، حاول وضع استدعاء الاستعلام داخل متن التابع.
مثال
قبل إعادة التصميم
في لغة 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)
لم إعادة التصميم؟
من الصعب فهم قائمة طويلة من المعاملات. إضافة إلى ذلك، غالبًا ما تشبه استدعاءات مثل هذه التوابع سلسلةً من الشلالات، ذات حسابات قيم ملتوية ومتكررة والتي يصعب تتبعها مع وجوب تمريرها إلى التابع. لذا إذا كان يمكن حساب قيمة المعامل بمساعدة تابع ما، فافعل ذلك داخل التابع نفسه وتخلص من المعامل.
فوائد تطبيق الحل
- التخلص من المعاملات غير الضرورية وتبسيط استدعاءات التابع. غالبًا ما لا تُنشأ هذه المعاملات لاستخدام المشروع في وضعه الحالي، ولكن لاحتياجات المستقبل التي قد لا تأتي أبدًا.
مساوئ تطبيق الحل
- قد تحتاج إلى المعامل في الغد القريب لتلبية احتياجاتٍ أخرى... الأمر الذي يجعلك تعيد كتابة التابع.
آلية الحل
- تأكد من أن شيفرة الحصول على القيمة لا تستخدم معاملات من التابع الحالي، لأنها لن تكون متاحة من داخل التوابع أخرى. إذا كان الأمر كذلك، فلن يكون من الممكن نقل الشيفرة البرمجية.
- إذا كانت الشيفرة البرمجية ذات الصلة أكثر تعقيدًا من استدعاء تابع أو دالة منفردَين، استخدم استخراج التوابع لعزل هذه الشيفرة البرمجية في تابع جديد وجعل الاستدعاء بسيطًا.
- استبدل جميع المراجع إلى المعامل المراد استبداله بالاستدعاءات إلى التابع الذي يحصل على القيمة داخل شيفرة التابع الرئيسي.
- استخدم إزالة المعاملات لإزالة المعامل غير المستخدم الآن.