[Moscow.pm] Комментарии

[Шогин Михаил]

Смысл прост.
Я уехал в отпуск или на больничном.
Мой коллега быстро и легко разобрался с проблемой.
Никому не нравится сидеть и разбираться, что думал человек когда писал код,
ИМХО.
В итоге получаем псевдо код вместе кодом и все мысли на виду.

> # одновременно может работать только один демон
Это для программиста. Админ даже читать не будет. (это не его дело)

Я не утверждаю что это правильно для кого то кроме нас, но мы для себя
выбрали такой подход, и довольны.
Можно сослаться на Алистера Коберна "Быстрая разработка ПО".
"Если практика приносит пользу используйте ее до тех пор пока она не
становится обременительной." (с)

Хотя как альтернативу можно ввести тотальный ревью кода.
В этом случае правда увеличиваются сроки. 


Я думаю что каждый видел код подобный этому )


     for my $month (0..11) {
     my $month_f = sprintf("%02d", $month + 1);
     $cal[$rows]->{cols}->[$cols]->{month} = $MONTHS[$month];
     # make month calendar
     my $dim = Days_in_Month($year_yyyy, $month + 1);
     my ($row, $wd, $d) = (0, 1, 1);
     for my $i (0..41) {
         my $day_f = sprintf "%02d", $d;
         my $dok = $d <= $dim && $wd == Day_of_Week($year_yyyy, $month + 1,
$d);
         my $mess = $sdate{"$day_f.$month_f.$year"};
         $cal[$rows]->{cols}->[$cols]->{days}->[$row]->{wd}->[$wd-1] = {
             day => $dok ? sprintf ("%02d ", $d++) : '&nbsp;',
         mess_id => $dok ? $mess->{MESS_ID} : undef,
         notone => $mess->{NOTONE},
         date => sprintf ("%04d%02d%02d", $year_yyyy, $month+1, $d-1)
         };
         if (++$wd == 8) { 
             $wd = 1;
             $row++;
         }
     }
     if (++$cols == 2) {
         $cols = 0;
             $rows++;
         }
     }

Кому нравится разбираться - пусть разбираются. Я пас, и коллегам своим
такого не желаю ))


-----Original Message-----
From: moscow-pm-bounces+mshogin=gmail.com на pm.org
[mailto:moscow-pm-bounces+mshogin=gmail.com на pm.org] On Behalf Of Andrew
Shitov
Sent: Thursday, March 11, 2010 5:41 PM
To: Moscow.pm group
Subject: Re: [Moscow.pm] Комментарии

> # одновременно может работать только один демон

Михаил, а какой смысл держать такой комментарий в коде, где его,
например, не прочитает администратор, запускающий демон?

-- 
Andrew Shitov
______________________________________________________________________
andy на shitov.ru | http://shitov.ru
--
Moscow.pm mailing list
moscow-pm на pm.org | http://moscow.pm.org