Categories
General

PHP MANDATORY CODE REVIEW Checklist

  1. Solid principles: Ensure that the code follows the SOLID principles of object-oriented design, which include Single Responsibility, Open/Closed, Liskov Substitution, Interface Segregation, and Dependency Inversion principles. These principles promote clean, maintainable, and extensible code.
  2. Type hinting: Use type hints in function and method declarations to specify the expected data types for parameters and return values. Type hinting helps improve code clarity and prevents type-related errors.
  3. TRY CATCH: Implement proper error handling using try-catch blocks. This ensures that exceptions are caught and handled gracefully, preventing crashes and providing meaningful error messages to users or developers.

try { // Code that may throw exceptions } catch (ExceptionType1 $e1) { // Handle ExceptionType1 } catch (ExceptionType2 $e2) { // Handle ExceptionType2 } catch (ExceptionType3 $e3) { // Handle ExceptionType3 }

  1. Properly log exceptions: Log exceptions in a structured and informative way, including the date and time of the exception occurrence, the type of exception, the exception message, the file where it occurred, and the line number. This helps in debugging and tracking issues efficiently.Example log format: [Date and Time] – [Exception Type]: [Exception Message] in [File] on line [Line Number]
  2. Declare strict_types = 1;: Enable strict typing in PHP by declaring strict_types = 1; at the beginning of your PHP scripts. This ensures that type coercion is disabled and helps prevent unexpected type-related issues.
  3. Functional tests: Develop and perform functional tests for your code to verify that it behaves as expected and meets the specified requirements. Functional tests help ensure the correctness of your code.
  4. PHP Doc: Use PHPDoc comments to provide clear and comprehensive documentation for your code. Include descriptions of functions, methods, classes, and their parameters, return types, and any exceptions they may throw. Well-documented code is easier for others to understand and use.
  5. Validate PSR-2 compliance: Ensure that your code adheres to the PSR-2 coding style standard for PHP. PSR-2 provides guidelines for code formatting, naming conventions, and indentation, promoting consistency and readability across projects.
  6. No PHP error messages: Make sure that there are no PHP error messages in your code. PHP errors can disrupt the functionality of your application and should be addressed and resolved.
  7. No PHP Warning messages: Similarly, ensure that there are no PHP warning messages in your code. Warnings can indicate potential issues that need to be fixed to maintain the stability of your application.
  8. No PHP Notices: Eliminate PHP notices from your code. Notices are typically non-fatal issues, but it’s best to address them to prevent unexpected behavior.
  9. No hardcore credentials: Avoid hardcoding sensitive credentials such as usernames, passwords, or API keys directly into your code. Instead, use secure configuration management practices to store and access these credentials securely.
  10. Function no longer than 50 lines: Keep individual functions or methods concise and focused, with a maximum of 50 lines of code. This promotes code readability and maintainability by breaking complex logic into smaller, more manageable pieces.

Leave a Reply