App\Models\UserProfile

Complexity

Total Complexity

7

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	setUser()	0	4	1
A	setFriends()	0	4	1
A	setBadges()	0	4	1
A	setGroups()	0	8	2
A	setRooms()	0	4	1

<?php

namespace App\Models;

/**
 * Class UserProfile.
 */
class UserProfile
{
    /**
     * User Data.
     *
     * @var User
     */
    public $user;

    /**
     * User Friends.
     *
     * @var UserFriend[]|array
     */
    public $friends = [];

    /**
     * User Badges.
     *
     * @var UserBadge[]|array
     */
    public $badges = [];

    /**
     * User Groups.
     *
     * @var UserGroup[]|array
     */
    public $groups = [];

    /**
     * User Rooms.
     *
     * @var Room[]|array
     */
    public $rooms = [];

    /**
     * UserProfile constructor.
     *
     * @param User $userData
     *
     * @return UserProfile

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

     */
    public function __construct(User $userData)
    {
        $this->setUser($userData);

        $this->setFriends();
        $this->setBadges();
        $this->setGroups();
        $this->setRooms();

        return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }

    /**
     * Set User Data.
     *
     * @param User $userData
     */
    protected function setUser(User $userData)
    {
        $this->user = $userData;
    }

    /**
     * Set User Friends.
     */
    protected function setFriends()
    {
        $this->friends = UserFriend::where('user_one_id', $this->user->uniqueId)->get() ?? [];
    }

    /**
     * Set User Badges.
     */
    protected function setBadges()
    {
        $this->badges = UserBadge::where('user_id', $this->user->uniqueId)->get() ?? [];
    }

    /**
     * Set User Groups.
     */
    protected function setGroups()
    {
        $groups = GroupMember::where('user_id', $this->user->uniqueId)->get() ?? [];

        foreach ($groups as $group) {
            $this->groups[] = $group->guild;
        }
    }

    /**
     * Set User Rooms.
     */
    public function setRooms()
    {
        $this->rooms = Room::where('owner_id', $this->user->uniqueId)->where('state', '!=', 'invisible')->get() ?? [];
    }
}