The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Изучение изменения размера кодовой базы Ext4, Btrfs и XFS, opennews (??), 23-Июн-11, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


2. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +2 +/
Сообщение от Битазавр (?), 23-Июн-11, 12:33 
так и правельно делают, чем лучше исходники снабжены коментами, тем легче искать нужное место и легче понять алгоритмы
Ответить | Правка | Наверх | Cообщить модератору

9. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  –16 +/
Сообщение от пельмешка (?), 23-Июн-11, 13:58 
Если есть необходимость комментировать код, то значит этот код плохо написан и его надо переделывать, пока не отпадёт необходимость в комментариях.  
Тоже самое и отладчиков касается. Необходимость в их использовании говорит о запутанном коде.
Ответить | Правка | Наверх | Cообщить модератору

13. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +1 +/
Сообщение от Зилибоба (ok), 23-Июн-11, 14:09 
Сам, то писал хоть раз? =) Суровый челябинский кодер. Если код не документирован, то в первую очередь можно сказать о том, что писатель плевал на других и пишет для себя. Ибо он уверен в том что может написать 100500 строк кода так красиво и лаконично что разобраться не составит и труда. Да - конечно. когда в код чуть сложнее "здравствуй мир" в нем необходимы каменты, иначе ты потратишь много времени на ненужные разборки, типа: "а что это за цикл и нафиг он тут нужен"... А так конечно автор прав. Торвальдс мудак и коментил ядро зазря, или кодер плохой... одно из двух...
Ответить | Правка | Наверх | Cообщить модератору

20. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от пельмешка (?), 23-Июн-11, 14:33 
Не челябинский кодер, а архуский (Дания). Это мнение про комментарии принадлежит некоему Б.Страуструпу.
Ответить | Правка | Наверх | Cообщить модератору

31. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +1 +/
Сообщение от fr0ster (ok), 23-Июн-11, 15:28 
Цитата так же точна, как напев Желтой Субмарины Абрамом по телефону(С)

Что собственно Страустрап по поводу комментов пишет.
http://lib.ru/CPPHB/cpptut.txt
"Можно рекомендовать такой стиль введения комментариев в
программу:

     [1] начинать с комментария каждый файл программы: указать в
         общих чертах, что в ней определяется, дать ссылки на
         справочные руководства, общие идеи по сопровождению
         программы и т.д.;
     [2] снабжать комментарием каждое определение класса или шаблона
         типа;
     [3] комментировать каждую нетривиальную функцию, указав: ее
         назначение, используемый алгоритм (если только он неочевиден)
         и, возможно, предположения об окружении, в котором работает
         функция;
     [4] комментировать определение каждой глобальной переменной;
     [5] давать некоторое число комментариев в тех местах, где
         алгоритм неочевиден или непереносим;
     [6] больше практически ничего."

Кстати говоря Бьярни не ввел бы вообще комментарии в С++ если бы считал их ненужными.
Ну и избыточные комментарии это совершенно другое дело, но против них высказывались многие и это никак не значит "код плохо написан и его надо переделывать, пока не отпадёт необходимость в комментариях"

Ответить | Правка | Наверх | Cообщить модератору

33. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Michael Shigorinemail (ok), 23-Июн-11, 15:30 
> Это мнение про комментарии принадлежит некоему Б.Страуструпу.

Ну и шут с ним и его великом -- если "гура" до седины в бороде дожил, а не осознал, что реализация не всегда прозрачно отражает намерение, и смущает молодняк.

И если комментировать реализацию программисту, а не кодеру, обычно действительно смысла мало (в основном интерфейсы и заковыристые места) -- то задумку бывает очень даже полезно.

Ответить | Правка | К родителю #20 | Наверх | Cообщить модератору

34. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от fr0ster (ok), 23-Июн-11, 15:32 
>> Это мнение про комментарии принадлежит некоему Б.Страуструпу.
> Ну и шут с ним и его великом -- если "гура" до
> седины в бороде дожил, а не осознал, что реализация не всегда
> прозрачно отражает намерение, и смущает молодняк.

Ой, вот таки не надо сразу гуру дураком делать. :) Сначала посмотрите насколько точна "цытата" оппонента. Страуструп таки знал, что комменты нужны. Непожлобился даже на два вида комментов.

Ответить | Правка | Наверх | Cообщить модератору

36. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Michael Shigorinemail (ok), 23-Июн-11, 15:35 
>> если
> Ой, вот таки не надо сразу гуру дураком делать. :)

if() :)

(а с выражениями действительно надо осторожней -- мало ли кто как поймёт...)

Ответить | Правка | Наверх | Cообщить модератору

69. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от ананим (?), 23-Июн-11, 17:48 
лично я даже и не думал, что выссказывание Страуструпа о необходимости писать код просто настолько, насколько это вообще можно, некоторые воспримут как войну с комментариями.
ату его! Вон там goto! быдлокодер-детектед! :D
и хрен же убедишь, что в этом куске кода готу убрал кучу иф/элсе и тд...
Ответить | Правка | Наверх | Cообщить модератору

107. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от anonymous (??), 23-Июн-11, 23:08 
а вот тут начинает не зватать терминов. комментировать перед функцией — вход, выход, какие подводные камни и применимость — это одно. а комментировать потом код функции — это другое. если первое — оправдано и полезно, то второе — признак быдлокода.
Ответить | Правка | К родителю #33 | Наверх | Cообщить модератору

135. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от ананим (?), 24-Июн-11, 03:45 
>то второе — признак быдлокода.

от быдло-кодера слышу! Этож писец! Теперь что и в структурах не писать какие значения могут принимать члены? И граничные условия? И если реализация не полная, тоже ссылки не давать?
это чё, вот это head -50 /usr/src/linux-3.0-rc4/include/linux/superhyway.h
struct superhyway_vcr_info {
    u8    perr_flags;    /* P-port Error flags */
    u8    merr_flags;    /* Module Error flags */
    u16    mod_vers;    /* Module Version */
    u16    mod_id;        /* Module ID */
    u8    bot_mb;        /* Bottom Memory block */
    u8    top_mb;        /* Top Memory block */
};
уже быдло код? А по мне, так это пример хорошо документированного кода. Или этот
/usr/src/linux-3.0-rc4/drivers/parport/parport_serial.c
static int __devinit netmos_parallel_init(struct pci_dev *dev, struct parport_pc_pci *par, int autoirq, int autodma)
{
    /* the rule described below doesn't hold for this device */
    if (dev->device == PCI_DEVICE_ID_NETMOS_9835 &&
            dev->subsystem_vendor == PCI_VENDOR_ID_IBM &&
            dev->subsystem_device == 0x0299)
        return -ENODEV;
    /*
     * Netmos uses the subdevice ID to indicate the number of parallel
     * and serial ports.  The form is 0x00PS, where <P> is the number of
     * parallel ports and <S> is the number of serial ports.
     */
    par->numports = (dev->subsystem_device & 0xf0) >> 4;
и мне наплевать что об этом думают местные гении. Этот код отлично читается и без документации.

Ответить | Правка | Наверх | Cообщить модератору

21. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Аноним (-), 23-Июн-11, 14:41 
В идеальном случае код должен быть написан так чтобы коментов не требовалось. В реальном, конечно коменты нужны. Но когда их 40% от кода - это уже оверкилл и напрашивается вывод что кто-то перепутал сорец с документацией :)
Ответить | Правка | К родителю #13 | Наверх | Cообщить модератору

25. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Аноним (-), 23-Июн-11, 15:15 
Просто Вы не видели реализацию сложных алгоритмов или структур данных.
Ответить | Правка | Наверх | Cообщить модератору

28. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Аноним (-), 23-Июн-11, 15:22 
> Просто Вы не видели реализацию сложных алгоритмов или структур данных.

Почему же, видел. Просто если хочется что-то именно фундаментально задокументировать - зачем это делать прямо в сорце а не в файлике с документацией?


Ответить | Правка | Наверх | Cообщить модератору

32. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +2 +/
Сообщение от fr0ster (ok), 23-Июн-11, 15:30 
>> Просто Вы не видели реализацию сложных алгоритмов или структур данных.
> Почему же, видел. Просто если хочется что-то именно фундаментально задокументировать -
> зачем это делать прямо в сорце а не в файлике с
> документацией?

Затем, что в какой то момент вы рискуете вместо пояснения темных мест реализации алгоритма встретить запись типа "Подробности смотри в тетрадке у Чжаня".

Ответить | Правка | Наверх | Cообщить модератору

80. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Аноним (-), 23-Июн-11, 18:28 
> Затем, что в какой то момент вы рискуете вместо пояснения темных мест
> реализации алгоритма встретить запись типа "Подробности смотри в тетрадке у Чжаня".

Если код пишут товарисчи с таким подходом - им лучше всего вообще не пользоваться ни в коем случае. Так, во избежание. Какими вы только пинками не стройте дураков в стойло, умнее они от пинков не становятся.

Ответить | Правка | Наверх | Cообщить модератору

91. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от fr0steremail (ok), 23-Июн-11, 20:49 
>> Затем, что в какой то момент вы рискуете вместо пояснения темных мест
>> реализации алгоритма встретить запись типа "Подробности смотри в тетрадке у Чжаня".
> Если код пишут товарисчи с таким подходом - им лучше всего вообще
> не пользоваться ни в коем случае. Так, во избежание. Какими вы
> только пинками не стройте дураков в стойло, умнее они от пинков
> не становятся.

Лучше то лучше, но иногда выбор небогат. Здесь Родос, здесь и прыгай :)

Ответить | Правка | Наверх | Cообщить модератору

129. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от Аноним (-), 24-Июн-11, 01:32 
> Лучше то лучше, но иногда выбор небогат. Здесь Родос, здесь и прыгай :)

Ну тогда прыгайте, чо. Только так и убиться можно. Умные люди обычно стараются прыгать так чтобы не убиться. А дураки попадают на darwinawards, остальным в назидание...

Ответить | Правка | Наверх | Cообщить модератору

181. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от fr0ster (??), 24-Июн-11, 13:36 
Вот умные люди и комментируют свой код.
Буквально каждый чих.
Что бы потом не вспоминать почему в этом мести код такой, а не эдакий.
Ответить | Правка | Наверх | Cообщить модератору

182. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от anonymous (??), 24-Июн-11, 13:41 
> Вот умные люди и комментируют свой код.
> Буквально каждый чих.
> Что бы потом не вспоминать почему в этом мести код такой, а
> не эдакий.

если это надо специально вспоминать, то это называется «хак». хаки, конечно, нужны — но лучше их изолировать куда-нибудь в обёртки и потом трогать поменьше.

Ответить | Правка | Наверх | Cообщить модератору

184. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от fr0ster (??), 24-Июн-11, 15:04 
Это не хак, это требования контролирующих органов. Ну вот положено считать так, а не эдак. В теме только консультант ставящий задачу, ибо госорганы чуть ли не каждый квартал придумывают новые требования к отчетности. Ну и через полгода не вспомнить, почему округление сделано таким диким образом. А всего лишь, чтоб по алгоритму от ПФР было.
Ответить | Правка | Наверх | Cообщить модератору

185. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/
Сообщение от anonymous (??), 24-Июн-11, 15:12 
> Это не хак, это требования контролирующих органов. Ну вот положено считать так,
> а не эдак. В теме только консультант ставящий задачу, ибо госорганы
> чуть ли не каждый квартал придумывают новые требования к отчетности. Ну
> и через полгода не вспомнить, почему округление сделано таким диким образом.
> А всего лишь, чтоб по алгоритму от ПФР было.

по-моему, ты только что описал грязный хак.

Ответить | Правка | К родителю #184 | Наверх | Cообщить модератору

39. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +2 +/
Сообщение от segoonemail (ok), 23-Июн-11, 16:23 
Чтобы не было рассинхронизации документации и кода.
Ответить | Правка | К родителю #28 | Наверх | Cообщить модератору

109. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  –4 +/
Сообщение от anonymous (??), 23-Июн-11, 23:16 
> Чтобы не было рассинхронизации документации и кода.

ты риальне веришь, что при наличии этого в одном файле вероятность десинка меньше? O_O

Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру