ChatApp\Search

Complexity

Total Complexity

11

Size/Duplication

Total Lines	75
Duplicated Lines	18.67 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	14	14	1
C	searchItem()	0	52	10

<?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 14 and the first side effect is on line 4.

namespace ChatApp;
require_once (dirname(__DIR__) . '/vendor/autoload.php');
use ChatApp\Time;
use ChatApp\Session;
use Dotenv\Dotenv;
$dotenv = new Dotenv(dirname(__DIR__));
$dotenv->load();

/**
* Search for the user
*/
class Search
{

    protected $connect;
    protected $array;
    protected $obTime;

    public function __construct($sessionId)

This method seems to be duplicated in your project.

    {
        session_id($sessionId);
        @session_start();

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $this->connect = mysqli_connect(
            getenv('DB_HOST'),
            getenv('DB_USER'),
            getenv('DB_PASSWORD'),
            getenv('DB_NAME')
        );
        session_write_close();
        $this->obTime = new Time();
        $this->array = array();
    }

    public function searchItem($suggestion)
    {
        $suggestion = $suggestion->value;
        $flag = 0;
        $userId = Session::get('start');
        if($userId != null && isset($suggestion))
        {
            $suggestion = trim($suggestion);
            if($suggestion != "")
            {
                $query = "SELECT * FROM login where login_id != '$userId' and name like '$suggestion%' ORDER BY name ASC";
                if($result = $this->connect->query($query))
                {
                    if($result->num_rows > 0)
                    {
                        while($row = $result->fetch_assoc())
                        {
                            $check_id = $row["login_id"];
                            $query = "SELECT * from total_message where (user1 = '$check_id' and user2 = '$userId') or (user2 = '$check_id' and user1 = '$userId')";
                            if($result1 = $this->connect->query($query))
                            {
                                if($result1->num_rows > 0)
                                {
                                    $fetch = $result1->fetch_assoc();
                                    $fetch['time'] = $this->obTime->timeConversion($fetch['time']);

                                    $this->array = array_merge($this->array, [['time' => $fetch['time'], 'username' => $row['username'], 'name' => $row['name']]]);
                                    $flag = 1;
                                }
                            }
                        }
                    }
                }
                if($flag != 0)
                {
                    $this->array = array_merge([], ["Search" => $this->array]);
                    return json_encode($this->array);
                }
                else
                    return json_encode(["Search" => "Not Found"]);
            }
            else
            {
                return json_encode(["Search" => "Not Found"]);
            }
        }
        else
        {
            return json_encode(["Search" => "Not Found"]);
        }

    }
}