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

    +154

    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
    public static function ShowClaimsByManager($manager_id,$status,$offset_limit,$rows_limit){
        $DBH = Registry::get('DBH');
        $STH = $DBH ->prepare(
            "SELECT * FROM claims,claims_status
             WHERE claims.manager_id = :manager_id
             AND claims.id = claims_status.claim_id
             AND claims_status.status = :statusc
             LIMIT :offsetl, :rowsl" );
        $STH->bindParam(':manager_id', $manager_id, PDO::PARAM_STR);
        $STH->bindParam(':statusc', $status, PDO::PARAM_STR);
        $STH->bindParam(':offsetl', $offset_limit, PDO::PARAM_INT);
        $STH->bindParam(':rowsl', $rows_limit, PDO::PARAM_INT);
        $STH->execute();
        $result = $STH->fetchAll(PDO::FETCH_ASSOC);
        return $result;
    }

    Парень явно любит ctrl+c и ctrl+v
    про $STH->execute($data); не не слышал.

    Запостил: werdan, 04 Февраля 2015

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

    • ava guest 06.02.2015 16:28 # 0

      Да код с PDO вообще не может быть нормальным :)
      Ответить

    • ava guest 03.03.2015 10:51 # 0

      не вижу ничего плохого в этом подходе, иногда удобно зибиндить отдельные параметры с указанием типа валидации (например PDO::PARAM_INT). Это просто другой подход. Здесь же не 100 этих биндов
      Ответить

      • ava bormand 03.03.2015 11:34 # 0

        Ну всё же в данном случае лучше bindValue() поюзать, а не bindParam(). Тут же банальный select, а не извратная хранимка, которая может через аргументы что-нибудь вернуть.
        Ответить

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