Последние комментарии

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


Боярин

Давай работай, крестьянин, а не в ноутбуки свои играй. Тебе ещё 20 лет кредит за дом выплачивать. Потом возьмешь кредит на дорогой спортивный автомобиль, а чуть позже можно и в могилку отправиться.


Overdrive

Ставьте sonarqube.


Гость

А почему бы не автоматизировать нужный стиль через AStyle? Вы ведь пользуетесь системой контроля версии? Там ведь можно установить pre-commit hook с вызовом astyle и код в нужном стиле будет отправлен в репозиторий.

>Если после оператора сравнения идет только одна строка кода, то в 90% случаев я предпочту не ставить фигурные скобки.
Спорное заявление: при рефакторинге в условие может попасть совсем не та строка которая должна быть там.


Евгений

Надо использовать вменяемые ide, которые автоматически форматируют код.


Megadragon

[quote=owl]Согласен с вами на счёт переносов скобок, сталкивался с таким подходом у некоторых разработчиков, никогда не понимал, как может быть не очевидным факт экономии места, да и в целом это повышает читаемость.


Согласен. Но всё же, иногда в конце длинного if'а открывающая фигурная скобка может «теряться». Хотя сей недостаток отчасти сглаживается подсветкой парных скобок...


owl

Есть рекомендации по оформлению кода, разве нет? Взять тот же PHP, там есть PSR'ы, в которых чёрным по белому написано, как и что писать. Согласен с вами на счёт переносов скобок, сталкивался с таким подходом у некоторых разработчиков, никогда не понимал, как может быть не очевидным факт экономии места, да и в целом это повышает читаемость.


Spider_NET

Фиг знает, если честно. Если идет блок кода и в нем одна строка, то пустоты будут резать глаз. Вот если идет много подряд инструкций, то обычно отбиваю одной пустой строкой:

[pre]

if (something()) {
//Здесь нет смысла ставить пустую строку
}

for (var i=0; i <= 10; i++) {

rambaJamba();
var i++;
rambaJamba2();
rambaJamba();
var i++;
rambaJamba2();
//etc

}

[/pre]

Во втором случае могу разбить цикл пустой строкой для повышения читабильности. В моем примере приведены короткие имена псевдо-функций, поэтому не особо очевиден пруф. В реале тело цикла может быть больше и пустая строка упросит чтения. Тут на вкус и цвет.


Ololo

Есть гидлайны как оформлять код в том или ином языке и нормальные программисты их придерживаются. Видел например как программисту было влом переключать сознание с JavaScript и он делал так же в C#, лично у меня путаница была только с самого начала, потом на автомате пишу в каждом языке по тому как принято и не испытываю неудобств.


Темирхан

В каждом IDE, если не ошибаюсь свои правила? Я не прав?


Megadragon

Что меня ещё бесит, так это явные сравнения с True и False. Например, вместо if (variable == true) можно написать просто if (variable), а вместо if (onemore == false) – if (!onemore).
А C++ (почти уверен, что C# и Java тоже) одинаково воспринимает предикаты вида if (count == 0) и if (!count).


О блоге

Программист, автор нескольких книг серии глазами хакера и просто блогер. Интересуюсь безопасностью, хотя хакером себя не считаю

Обратная связь

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

Пишите мне