За последние 24 часа нас посетили 7432 программиста и 488 роботов. Сейчас ищут 164 программиста ...

является ли плохим стилем такого рода дублирование?

Тема в разделе "PHP для новичков", создана пользователем memfiss, 13 дек 2011.

  1. memfiss

    memfiss Активный пользователь

    С нами с:
    14 ноя 2011
    Сообщения:
    19
    Симпатии:
    0
    PHP:
    1.  
    2. <?php
    3. public static function SetLike($id , $sid , $is)
    4. {
    5. $sql = Sql::getInstance('horo');
    6. if (!self::IsInTable($id , $sid))
    7.  $sql -> run("insert into FacebookAppLikes(fbID,liked,sid) values('".addslashes($id)."','".$is."','".substr($sid,-3)."')");
    8. else
    9.  $sql -> run("update FacebookAppLikes set liked = '".$is."' where fbID = '".addslashes($id)."' and sid = '".substr($sid,-3)."'");
    10. }
    11.  
    12. public static function LikeIt($id , $sid)
    13. {
    14. self::SetLike($id,$sid,'1');
    15. }
    16.  
    17. public static function DislikeIt($id , $sid)
    18. {
    19. self::SetLike($id,$sid,'0');
    20. }
    21.  
    22.  
     
  2. [vs]

    [vs] Суперстар
    Команда форума Модератор

    С нами с:
    27 сен 2007
    Сообщения:
    10.449
    Симпатии:
    581
    Нет. Плохом стилем является использование addslashes и отсутствие `обратных апострофов` у имен полей таблиц. Да и операторы SQL принято писать заглавными.