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

    −11

    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
    /**
 * @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, 05 Октября 2015

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

    • ava Stallman 06.10.2015 12:18 # +1

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

      • ava keltanas 06.10.2015 13:13 # +1

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

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

    • ava Fike 07.10.2015 10:51 # 0

      оче много харассмента
      Ответить

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