Нужна небольшая помощь

DarksLight2

Знаток
⚖️
📜
  • Золотая медаль
Пртветствую, решил я заново начать учиться новым версиям php и есть просьба, не могли бы вы посмотреть на код, и сказать как можно было лучше его написать, укажите ошибки синтаксиса.

П.С Код писал любой схематически с телефона.

PHP:
<?

    class profile

    {

          global $query = query($mysqli, "SELECT * FROM `users` WHERE `id` = ".$id."");

          global $userData = $query -> fetch_array();

          public function getInformationAboutUser($id)

          {

               return "<img src='urlIconOnline' title='".$userData['login'].">".$user['login']." <br>Name: ".$userData['fullName']."<br>Age: ".$userData['age']."";

           }

           public function       buttonsForGuest($idGuest)

           {

                if($idGuest == $userData)

                {

                      return null;

                }

                else

               {

                      return "<a href=''>Subscribe</a>";

               }

           }

           public function subscribers($idUser)

           {

                 $query = query($mysqli, "SELECT * FROM `subscribers` WHERE `userID` = ".$userData['id']." AND `subscriber` != ".$idUser."");

                 $checking = $query->fetch_array();

                 if($checking)

                 {

                    exit;

                  }

                 else

                 {

                     query($mysqli, "INSERT INTO `subscribers` (`userID`, `subscriber`, `date`) VALUES (".$userData.", ".$idUser.", ".date('d-m-Y').")");
} 
    }
В коде нету ничего сверх истественного так кае это начало, хочу в топике советоваться с вами по поводу php.
 

Insallah

Раздолбай
📜
💎
Короткий открывающий тэг.
Переменные не определены.
В функции getInformationAboutUser($id) параметр $id нигде не используется.

Возможно это вкусовщина, но в return можно возвращать не html код, а переменные, которые потом подставлять куда надо.
Во второй функции нет никакого смысла. if($idGuest != $userData) echo "<a href=''>Subscribe</a>"; и проще и быстрее.
Третья функция не отдаёт выхлоп, если запрос в базу успешно прошёл. Если функция применяется на странице несколько раз и при этом не меняется в процессе подгрузки, можно вынести запрос за её пределы, чтобыеё вызов работал с результатом запроса, а не делал запрос каждый раз.
Качество самих запросов в базу, это твоё личное дело.
 
Последнее редактирование:
Сверху