gestor-inventarios-it icon indicating copy to clipboard operation
gestor-inventarios-it copied to clipboard

Published 20 hours ago • verified publisher icon wcadena

Refactor 13 lines occurring 2 times in 2 files: PuestoController.php, UserController.php

Open wcadena opened this issue 5 years ago • 0 comments

I've selected for refactoring 13 lines of code which are duplicated in 2 file(s) (1, 2). Addressing this will make our codebase more maintainable and improve Better Code Hub's Write Code Once guideline rating! 👍

Here's the gist of this guideline:

  • Definition 📖 Do not copy code.
  • Why❓ When code is copied, bugs need to be fixed in multiple places. This is both inefficient and a source of regression bugs.
  • How 🔧 Avoid duplication by never copy/pasting blocks of code and reduce duplication by extracting shared code, either to a new unit or introduce a superclass if the language permits.

You can find more info about this guideline in Building Maintainable Software. 📖

ℹ️ To know how many other refactoring candidates need addressing to get a guideline compliant, select some by clicking on the 🔲 next to them. The risk profile below the candidates signals (✅) when it's enough! 🏁

Good luck and happy coding! :shipit: :sparkles: :100:

wcadena avatar Nov 30 '18 13:11 wcadena