1. PHP / Говнокод #9728

    +157

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    class UserAPI
{
   function GetUserName($UserId)
   {
      $SqlQuery = mysql_query("SELECT UserName FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['UserName'];
   }
     
   function GetUserSurname($UserId)
   {
      $SqlQuery = mysql_query("SELECT UserSurname FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['UserSurname'];
   }
   
   function GetUserPhoto($UserId)
   {
      $SqlQuery = mysql_query("SELECT UserPhoto FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['UserPhoto'];
   }
   
   function GetUserCountry($UserId)
   {
      $SqlQuery = mysql_query("SELECT country FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['country'];
   }
}

    О да!

    Запостил: Govnisti_Diavol, 22 Марта 2012

    Комментарии (8) RSS

    • ava eth0 22.03.2012 13:17 # −1

      На каждое поле каждой таблицы по своей функции. Всё правильно сделал.
      Можно будет в будущем обойтись без б-гомерзких джойнов, через GetUserSomething(GetUserByLogin("qwerty" ))
      Ответить

      • ava rat4 22.03.2012 13:48 # −1

        И без ORM'ов!
        Ответить

      • ava ling 22.03.2012 17:37 # −1

        А не лучше ли написать getUserData($id, $fields), чтобы не теребить базу, если надо несколько полей?
        Ответить

        • ava Lure Of Chaos 22.03.2012 18:49 # −1

          Кэп нашептывает, что лучше всего написать функцию, возвращающую целый обьект, а там уже делайте, что хотите.
          Совсем хорошо бы заюзать ОРМ, чтобы на том же обьекте можно было и манипулировать данными
          Ответить

    • ava PSIAlt 22.03.2012 21:30 # −1

      Настораживает название UserAPI....
      Ответить

    • ava guest8 09.04.2019 12:52 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий