AnkitJain\RegistrationModule\Validate

Complexity

Total Complexity

7

Size/Duplication

Total Lines	59
Duplicated Lines	35.59 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	validateEmailInDb()	10	10	3
A	validateUsernameInDb()	10	10	3

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 25 and the first side effect is on line 14.

/**
 * Validate Class Doc Comment
 *
 * PHP version 5
 *
 * @category PHP
 * @package  Registration-Module
 * @author   Ankit Jain <ankitjain28may77@gmail.com>
 * @license  The MIT License (MIT)
 * @link     https://github.com/ankitjain28may/registration-module
 */
namespace AnkitJain\RegistrationModule;
require_once dirname(__DIR__) . '/config/database.php';

/**
 * For validation the Email and Username in DB.
 *
 * @category PHP
 * @package  Registration-Module
 * @author   Ankit Jain <ankitjain28may77@gmail.com>
 * @license  The MIT License (MIT)
 * @link     https://github.com/ankitjain28may/registration-module
 */
class Validate
{
    /*
    |--------------------------------------------------------------------------
    | Validate Class
    |--------------------------------------------------------------------------
    |
    | For validation the Email and Username in DB.
    |
    */

    protected $connect;

    /**
     * Create a new controller instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()
    {
        $this->connect = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_NAME);
    }

    /**
     * For checking whether the credentials are empty or not
     *
     * @param string $email Contains the email for checking
     *
     * @return 0 | 1

The doc-type 0 could not be parsed: Unknown type name "0" at position 0. (view supported doc-types)

     */
    public function validateEmailInDb($email)

This method seems to be duplicated in your project.

    {
        $query = "SELECT login_id FROM login WHERE email = '$email'";
        if ($result = $this->connect->query($query)) {
            if ($result->num_rows > 0) {
                return 1;
            }
            return 0;
        }
    }

    /**
     * For checking whether the credentials are empty or not
     *
     * @param string $username Contains the username for checking
     *
     * @return 0 | 1

The doc-type 0 could not be parsed: Unknown type name "0" at position 0. (view supported doc-types)

     */
    public function validateUsernameInDb($username)

This method seems to be duplicated in your project.

    {
        $query = "SELECT login_id FROM login WHERE username = '$username'";
        if ($result = $this->connect->query($query)) {
            if ($result->num_rows > 0) {
                return 1;
            }
            return 0;
        }
    }
}