The OpenNET Project / Index page

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

форумы
правила/FAQ
поиск
регистрация
вход
слежка
RSS


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

Оглавление

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

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


49. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +2 +/–
Сообщение от PereresusNeVlezaetBuggyemail (ok), 23-Июн-11, 17:07 
> Чем больше код, тем больше ошибок.
> Чем меньше комментариев, тем сложнее его поддерживать и больше ошибок при рефакторинге.

Чем меньше _полезных_ комментариев. И чем больше бесполезных. Индусское комментирование в стиле:

/*
 * prod_average_price - функция для формирования средней цены.
 *
 * Входные параметры:
 * struct elm *a - указатель на массив данных
 * int c - количество элементов в массиве
 *
 * Возвращаемое значение:
 * double - средняя цена продуктов в массиве.
 */
double
prod_average_price(struct elm *a, int c) {
        int i;    // счётчик
        double s;    // сумма цен продуктов
       for (i = 0, s = 0; i < c; i++)
               // прибавляем цену каждого продукта к значению s
               s += a[i].price;
        // разделим на количество продуктов
        return (s / c);
}

— тоже маразм, который лишь отвлекает и заставляет тратить силы на его поддержание; бюрократия без необходимости.
Ответить | Правка | К родителю #26 | Наверх | Cообщить модератору

53. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/–
Сообщение от fr0ster (ok), 23-Июн-11, 17:18 
Маразм, но иногда алгоритм не так прозрачен, как кажется (возьмите алгоритм округления начисленного налога при сдаче отчетности в ПФР реализованный в чекере от БухСофта и по которому ПФР проверяет принимаемую отчетность), иногда имена переменных определены не разработчиком данного кода (В саповских отчетах приходится использовать имена придуманные индусами из вальдорфа, далеко не всегда по ним можно сказать для чего объект назначен, а абапера сведущего в данном модуле может и не быть под рукой). Вот и получается, без подобных комментариев ну никак. И главное слезть с кактуса нельзя, кушать надо всем, надо семью одеть накормить :)
Ответить | Правка | Наверх | Cообщить модератору

55. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +1 +/–
Сообщение от PereresusNeVlezaetBuggyemail (ok), 23-Июн-11, 17:21 
> Маразм, но иногда алгоритм не так прозрачен, как кажется (возьмите алгоритм округления
> начисленного налога при сдаче отчетности в ПФР реализованный в чекере от
> БухСофта и по которому ПФР проверяет принимаемую отчетность), иногда имена переменных
> определены не разработчиком данного кода (В саповских отчетах приходится использовать
> имена придуманные индусами из вальдорфа, далеко не всегда по ним можно
> сказать для чего объект назначен, а абапера сведущего в данном модуле
> может и не быть под рукой). Вот и получается, без подобных
> комментариев ну никак. И главное слезть с кактуса нельзя, кушать надо
> всем, надо семью одеть накормить :)

Я и не спорю, что иногда это, к сожалению, нужно. Речь была как раз о том, что в тривиальных, очевидных ситуациях подобные комментарии — зло. Сорцы индийских CMS-ок видели когда-нибудь? Это ж зачастую полный атас, 90% комментариев описывают только сами операции, но не к чему это приводит, или что затрагивает — а именно эта информация в комментариях востребована больше всего.

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

60. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +1 +/–
Сообщение от fr0ster (ok), 23-Июн-11, 17:29 
> Я и не спорю, что иногда это, к сожалению, нужно. Речь была
> как раз о том, что в тривиальных, очевидных ситуациях подобные комментарии

Жаль только тривиальные случаи это вовремя не обнаруженные необычности.

> — зло. Сорцы индийских CMS-ок видели когда-нибудь? Это ж зачастую полный
> атас, 90% комментариев описывают только сами операции, но не к чему
> это приводит, или что затрагивает — а именно эта информация в
> комментариях востребована больше всего.

Индусский код видел. Не в ЦМС, но ЕРП. Насчет атаса согласен. И проблема не столько в комментах, сколько в прочем коде.


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

64. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +1 +/–
Сообщение от PereresusNeVlezaetBuggyemail (ok), 23-Июн-11, 17:34 
>> Я и не спорю, что иногда это, к сожалению, нужно. Речь была
>> как раз о том, что в тривиальных, очевидных ситуациях подобные комментарии
> Жаль только тривиальные случаи это вовремя не обнаруженные необычности.

Дык на то голова программисту и дана, чтобы отличать одно от другого... Чем проще код, чем он предсказуемее, тем легче его поддерживать. :)

>> — зло. Сорцы индийских CMS-ок видели когда-нибудь? Это ж зачастую полный
>> атас, 90% комментариев описывают только сами операции, но не к чему
>> это приводит, или что затрагивает — а именно эта информация в
>> комментариях востребована больше всего.
> Индусский код видел. Не в ЦМС, но ЕРП. Насчет атаса согласен. И
> проблема не столько в комментах, сколько в прочем коде.

Угу. В коде ДНК...

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

66. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/–
Сообщение от fr0ster (ok), 23-Июн-11, 17:42 
>> Жаль только тривиальные случаи это вовремя не обнаруженные необычности.
> Дык на то голова программисту и дана, чтобы отличать одно от другого...
> Чем проще код, чем он предсказуемее, тем легче его поддерживать. :)

Если архитектура запутанна, то медицина бессильна. Весь код не может быть ясным на 100%, если архитектура "прости господи", "туши свет, кидай гранату" и "святых выноси".

>> Индусский код видел. Не в ЦМС, но ЕРП. Насчет атаса согласен. И
>> проблема не столько в комментах, сколько в прочем коде.
> Угу. В коде ДНК...

Не столько кодеров, сколько архитекторов и администраторов. И руководства.
Это если не брать кодинг4фан. :)

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

97. "Изучение изменения размера кодовой базы Ext4, Btrfs и XFS"  +/–
Сообщение от PereresusNeVlezaetBuggyemail (ok), 23-Июн-11, 21:23 
>>> Жаль только тривиальные случаи это вовремя не обнаруженные необычности.
>> Дык на то голова программисту и дана, чтобы отличать одно от другого...
>> Чем проще код, чем он предсказуемее, тем легче его поддерживать. :)
> Если архитектура запутанна, то медицина бессильна. Весь код не может быть ясным
> на 100%, если архитектура "прости господи", "туши свет, кидай гранату" и
> "святых выноси".

Дык вы ж сами и привели примеры, когда внешние обстоятельства заставляют городить что-то сложное. А в особо тяжёлых случаях, когда, например, делается реверс-инжиниринг таинственной железки или софтины, код вообще трогать стрёмно: от безобидной смены порядка операций чтения в ходе рефакторинга Тот-Кого-Нельзя-Вызывать может просто перестать работать...

>>> Индусский код видел. Не в ЦМС, но ЕРП. Насчет атаса согласен. И
>>> проблема не столько в комментах, сколько в прочем коде.
>> Угу. В коде ДНК...
> Не столько кодеров, сколько архитекторов и администраторов. И руководства.
> Это если не брать кодинг4фан. :)

Угу. Рыба с головы гниёт.

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

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

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




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

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