"PHP" / Говнокод #17577 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 werdan, (Updated )

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

  • ava guest # Ссылка на оригинал 0

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

  • ava guest # Ссылка на оригинал 0

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

    • ava bormand # Ссылка на оригинал 0 ⇈

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

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

Где здесь C++, guest?!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓