1. Perl / Говнокод #4638

    −116

    1. 1
    map { push @{ $cgih->{add_result}->{errors} }, 'НЕ добавлено: ' . $_->{title} } @failed_comps;

    Видимо теперь использовать for это уже не по-пацански!

    Запостил: antonfin, 15 Ноября 2010

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

    • ava Мистер Хэнки 15.11.2010 20:00 # +1

      пиши в одну строку @ будь плохим парнем
      PS: push внутри map - сильно.
      Ответить

    • ava Oleg_quadro 15.11.2010 22:35 # 0

      Плюсую, хотя бы переводы строк поставил бы.
      Ответить

    • ava Lure Of Chaos 16.11.2010 01:19 # 0

      ну зачем нам for, если это перл!
      Ответить

    • ava fey 17.11.2010 08:21 # 0

      а где собственно говнокод ?
      Ответить

      • ava antonfin 17.11.2010 09:42 # +1

        В первом комментарии же написано: "push внутри map - сильно.".
        Данный код можно было написать так:

        push @{ $cgih->{add_result}->{errors} }, 'НЕ добавлено: ' . $_->{title} for @failed_comps;
        Ответить

        • ava Мистер Хэнки 17.11.2010 09:57 # 0

          или так (для того чтобы было понятно Arigato и прочим людям, которых в детстве покусал перловик-маньяк :))
          my $error_pool = $cgih->{add_result}->{errors};
foreach my $comp (@failed_comps) {
   my $error_message = 'НЕ добавлено: ' . $comp->{title};
   push @$error_pool, $error_message;
}
          Ответить

        • ava krushi 17.11.2010 15:52 # +4

          Или так: 
          push @{ $cgih->{add_result}->{errors} }, map{  "НЕ добавлено:  $_->{title}" } @failed_comps
          Ответить

    • ava fey 17.11.2010 19:57 # 0

      Ну кроме как варианта через foreach, мне кажется остальные способы, в принципе монопеснесуальны, читаемости сильно не прибавляют, единственное, что push вызывается один раз.
      Вариант krushi мне понравился
      push @{ $cgih->{add_result}->{errors} }, map{  "НЕ добавлено:  $_->{title}" } @failed_comps

      Но единственное непонятно , чем же так плох вариант push внутри map ?
      Ответить

      • ava Dummy00001 18.11.2010 00:38 # +3

        потому что не читабельно.

        ванила синтакс мапа это `@output = map { conversion; } @input` на что как бы имя оператора очень сильно и намекает.
        Ответить

      • ava krushi 18.11.2010 01:25 # +3

        Абсолютно согласен с Dummy00001. Map используется как конвеер: взял список, перелопатил, передал дальше. Push внутри map нарушает эту идиому. Зачем использовать гаечный ключ для забивания гвоздей, если рядом лежит молоток?
        Ответить

    • ava guest 01.01.2011 18:59 # −2

      perl сосёт у С++са
      Ответить

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