App\Http\Controllers\CodeCount

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getTotalPoints()	0	3	1
A	setRank()	0	8	2
A	getCodeCount()	0	5	1
A	setCodeCount()	0	5	1

<?php
    /**
     * Created by PhpStorm.
     * User: caspa
     * Date: 31.05.2018
     * Time: 11:39.
     */

namespace App\Http\Controllers;

use Illuminate\Support\Facades\DB;

    class CodeCount extends Controller
    {
        /**
         * Anzahl codes in DB hinterlegen.
         *
         * @param $codeCount
         *
         * @return int
         */
        public static function setCodeCount($codeCount)
        {
            DB::table('game_admin')->insert(['code_count' => $codeCount, 'total_points' => $codeCount]);

            return intval($codeCount);
        }

        /**
         * Anzahl codes aus DB holen und ausgeben.
         *
         * @return mixed
         */
        public function getCodeCount()
        {
            $codeCount = DB::table('game_admin')->select('code_count')->first()->code_count;

            return $codeCount;
        }

        /**
         * Anzahl codes aus DB holen und ausgeben.
         *
         * @return mixed
         */
        public static function getTotalPoints()
        {
            return $totalPoints = DB::table('game_admin')->select('total_points')->first();

The assignment to $totalPoints is dead and can be removed.

        }

        public static function setRank()
        {
            $totalPoints = self::getTotalPoints();
            $rankObj = DB::select("SELECT * FROM users WHERE total_points = $totalPoints ORDER BY TIMEDIFF(start, end) DESC;");

            for ($i = 0; $i < count($rankObj); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                DB::table('users')->where('name_gen', $rankObj[$i]->name_gen)->update(['rank' => (++$i)]);
                $i--;
            }
        }
    }