style.txt 2.41 KB
Newer Older
Mike Radyuk's avatar
Mike Radyuk committed
1 2
== Требования по оформлению кода ==

3 4
* постарайтесь не вносить без обсуждения разнобой стилей,
  если есть предметные пожелания по коррекции текущего --
5
  пишите в devel-distro@ или мне (mike@), обсудим;
6 7 8 9 10 11 12 13

* перед тем, как делать существенные переработки уже имеющегося
  кода -- опять же опишите проблему, идею и предполагаемый результат,
  порой могут выясниться непредвиденные последствия;

* документируйте на русском (README) или английском (README.en) языке
  то, что написали или изменили.

Mike Radyuk's avatar
Mike Radyuk committed
14 15 16
[float]
=== рекомендации ===

17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32
* трезво относитесь ко входным данным и не пренебрегайте кавычками:
  название дистрибутива с пробелом или получение текста ошибки вместо
  ожидаемого вывода команды могут привести к сложнодиагностируемым
  ошибкам; вместе с тем в ряде случаев, где требуется пословная
  обработка значений переменных или раскрытие шаблонов shell (*?[])
  -- кавычки могут оказаться лишними;

* пользуйтесь if [ ... ]; then ...; fi вместо [ ... ] && ...:
  это кажется более громоздким, но текст оказывается более читаемым
  и в т.ч. расширяемым, поскольку между then/else/fi можно спокойно
  добавлять строки; с && проще забыть { ... } или оставить ненулевой
  код возврата при неудаче теста, когда он считается несущественным;

* предпочтительно применение $() вместо `` (особенно при вложенности);

* постарайтесь не вылазить за 80 колонок.
33

Mike Radyuk's avatar
Mike Radyuk committed
34 35 36 37
[float]

=== ссылки ===

38 39
* https://lists.altlinux.org/mailman/listinfo/devel-distro
  (подписка по приглашению)