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

    +3

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    function getLastItemName( $items ) {

  $lastItemName = '';

  foreach( $items as $item ) {
    $lastItemName = $item['name'];
  }

  return $lastItemName;
}

    Имеем в проекте кучу россыпей подобного вида.
    Собственно, существует ли эффективный способ с этим бороться?

    Запостил: bot, 18 Января 2016

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

    • ava bot 18.01.2016 11:57 # +1

      Клавиатурой по лицу?
      Ответить

      • ava Vasiliy 18.01.2016 12:10 # +1

        лицо то что? нужно по пальцам, линейкой.
        Ответить

        • ava androzd 18.01.2016 12:56 # 0

          А лучше взять ножницы по металу, и отрезать по пальцу, за каждую итерацию)
          Ответить

          • ava KOCTbIJIb 18.01.2016 16:56 # +1

            А если итераций больше 21?
            Ответить

            • ava androzd 19.01.2016 11:17 # 0

              Можно по кусочку пальца.. Но я думаю, когда ему в третий раз начнут отрезать палец, он одной рукой перепишет этот код и изучит документацию)
              Ответить

    • ava FlySnake 18.01.2016 16:46 # +1

      В PHP нельзя взять крайний элемент коллекции? типа .last(), [-1] и т.д.
      Ответить

      • ava bot 18.01.2016 23:33 # +3

        В PHP можно практически всё. В том числе, и таким больным способом. Иногда, именно это и вызывает печаль.
        Ответить

      • ava Arris 19.01.2016 00:00 # +1

        Это еще не самый худший способ выстрелить себе в ногу.
        Ответить

      • ava kgm-rj 19.01.2016 05:34 # 0 

        $last = $a[count($a)];

$last = array_pop($a);

end($a);
$last = current($a);
        Ответить

        • ava drull 30.01.2016 14:41 # 0 

          end($a);
$last = current($a);


          current тут лишний. $last = end($a) достаточно.
          Ответить

        • ava guest 02.02.2016 00:40 # 0

          // $last = $a[count($a)];
          ну охуеть теперь. такое как это вообще в голову может прийти? шаблонорвань просто...
          Ответить

        • ava d_fomenok 29.04.2016 18:03 # 0

          array_pop сбрасывает же сверху
          Ответить

    • ava 3_dar 18.01.2016 19:00 # +2

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

      • ava 3_dar 18.01.2016 19:02 # 0

        Похоже так http://php.net/manual/ru/function.end.php
        Ответить

      • ava bot 18.01.2016 23:47 # 0

        Да хотя бы через "array_pop". Но грустный цимус ещё и в том, что оный "алгоритм" юзается не только в долбаном PHP. На том самом фронтэнде, даже в JS, успелось это развидеть.
        Хорошо, что до SQL не добрались.
        Порой жалеешь, что не существует "gcc" для таких "контактов третьего рода". Ну или, хотя бы, что-то вроде "jit".
        Но в реальности нет ни "gcc" ни "jit", а только какой-то "sex" и "php". Тоже три буквы, но зато - столько ебли!
        Трёмя словами: "Расмус - пидр, сука!"
        Ответить

    • ava guest 23.01.2016 11:12 # +2

      дом престарелых?
      Ответить

    • ava guest 26.01.2016 15:28 # 0

      Интересно, а если ещё надо было-бы отсортировать по фамилии, как бы это выглядело бы
      Ответить

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