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

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
  17. 17
  18. 18
/**
 * @param $parentId
 * @param $parentType
 * @return array
 */
public static function getReports( $parentId, $parentType )
{
    $redis = Redis::connection();
    $r = array(
        'inappropriate'=> (int) $redis->scard(__CLASS__.":report:".$parentId.":".$parentType.":inappropriate"),
        'copyright'=>(int) $redis->scard(__CLASS__.":report:".$parentId.":".$parentType.":copyright"),
        'harassment'=>(int) $redis->scard(__CLASS__.":report:".$parentId.":".$parentType.":harassment")
    );

    $r['all'] = $r['inappropriate']+$r['copyright']+$r['harassment'];

    return $r;
}

Попробуйте разобртаться без поллитры

Запостил: keltanas keltanas, (Updated )

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

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

    А что тут непонятного? Именование ключей в редисе, конечно, попахивает, но а в остальном особого криминала нет.
    Ответить

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

      Ну во-первых с первого раза не очень понятно, что вообще за массив вернет эта функция. Не заглядывая внутрь это вообще не реально. Да и сам массив формируется неочевидно.

      Ну и во-вторых, если мы вдруг решим перенести метод в другой класс (ну или класс переименовать), то не можем увидеть старые данные =)
      А ноги этого растут из $logger->debug(__CLASS__, [some data]); Автор строк выше очень понравились встроенные константы.
      Ответить

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

Помни, guest, за тобой могут следить!

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

    ↑
    8
    ↓