CodingStyle, Кто как пишет? Опции

[Gloin]

У каждого есть свои предпочтения в стиле кодинга, кто то при вызове функции отделяет переменные от скобок пробелами, кто то нет, кто то ставит фигурную скобку на той же строке что и if, а кто то переносит её вниз — давайте поговорим о своих привычках.
Я намедни решил внимательно прочитать http://lxr.linux.no/linux/Documentation/CodingStyle и от души посмеялся над пассажами в духе вот этого :
«Encoding the type of a function into the name (so-called Hungarian
notation) is brain damaged - the compiler knows the types anyway and can
check those, and it only confuses the programmer. No wonder MicroSoft
makes buggy programs.»

[gamecreator]

я использую стиль Олмана, CamelCase.

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

[Gloin]

Так я с идеей Торвальдса и не спорю, меня его острый язык рассмешил.

Мой стиль:

CODE

/*
ReadWholeFile - считывает весь файл в строку
OUT: BOOLEAN (флаг результата)
*/
function ReadWholeFile($FileName, &$Data) {
$FileSize = @filesize($FileName);
if ($FileSize===false) {
return false;
} // .if
if ($FileSize>0) {
$hFile = fopen($FileName, 'rb');
if (!$hFile) {
return false;
} // .if
if (strlen(($Data = fread($hFile, $FileSize)))!==$FileSize) {
return false;
} // .if
fclose($hFile);
} // .if
else {
$Data = '';
} //.else
return true;
} // .function ReadWholeFile

CODE

PROCEDURE Init;
CONST
C_REWRITE_EXISTING = FALSE;

BEGIN
IF NOT Win.CopyFile(C_FILE_IN, C_FILE_OUT, C_REWRITE_EXISTING) THEN BEGIN
Error('Cannot create output file');
END; // .if
hFileOut:=Utils.FileOpen(C_FILE_OUT, fmOpenWrite OR fmShareExclusive);
IF hFileOut<0 THEN BEGIN
Error('Cannot open output file');
END; // .if
END; // .procedure Init

[Гость_Хмм.._*]

а мне пофиг

[Монца]

Ну и я тогда отпишусь, поскольку пишу на дельфи придерживаюсь стиля разработчиков, примеры можно посмотреть в VCL

[Guevara-chan]

Linux style for comments is the C89 "/* ... */" style.
Don't use C99-style "// ..." comments.

Хм... Попыталась). Вот что пишут:

/* Comment */
^^
Warning(-4104): /* is a *** deprecated *** word (see src\compat\evolve.f)  ok

С "//" проблем никаких.

[Монца]

ну ворнинг и че? ) написано же деприкейт.
А сишный комментарий - это однострочный.

[Guevara-chan]

написано же деприкейт.

Вот именно. Все что "deprecated" могут убрать в любом новом релизе.

[gamecreator]

однострочные коменты рулят. особенно если среда разработки имеет ф-цию комента выделенных строк

[Gloin]

Простите, может изза того что я только проснулся, может по другой причине, но я не понимаю, о чем вы говорите.
В С99 появились однострочные комменты, но по традиции в ядре принято использовать /**/ ещё и звёздочки в начале каждой строки ставить.
А что это за анализатор такой ворнинги на комменты выдаёт?

[Guevara-chan]

[gamecreator]

Linux style for comments is the C89 "/* ... */" style.
Don't use C99-style "// ..." comments.

а вот ту видимо говорится как раз про си

[Guevara-chan]

Нет, тут говорится о комментариях в стиле С. Просто автор явно не в курсе, что они поддерживаются и некоторыми другими языками).

[Gloin]

Статья Торвальдса описывает стиль кода ядра Линукс и форта не касается никоим образом.

[Guevara-chan]

Не только, он еще пытается давать советы как надо писать программы.

Код

1) macros that affect control flow:
#define FOO(x)                                  \
             do {                                    \
                  if (blah(x) < 0)                \
                   return -EBUGGERED;      \
       } while(0)
is a _very_ bad idea.;

Кстати, не соглашусь. Сама использовала подобные констуркции в коде FH и никаких проблем. Куда лучше чем тупая последовательность почти одинаковых условий.

[Gloin]

Ретёрны в макросы лучше не прятать, иначе использующий твой код будет долго удивляться ходу работы получившегося продукта.

[gamecreator]

а я думаю что под плохой идеей он имел ввиду мусор типа do{}while(0);

[Gloin]

Код

Macros with multiple statements should be enclosed in a do - while block:

#define macrofun(a, b, c)                       \
       do {                                    \
               if (a == 5)                     \
                       do_this(b, c);          \
       } while (0)

[gamecreator]

а почему не {}? их же вроде можно ставить просто так