Есть некоторое условие Х, но оно встречается редко, т.е. описывает частный случай.

Как правильно писать:

Если НЕ Х Тогда
 Код1;
Иначе
 Код2;
КонецЕсли;

Или:

Если Х Тогда
 Код2;
Иначе
 Код1;
КонецЕсли;


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

Или все же общий - в конце? Или как-то по другому?

(0) Лучше второй вариант - при наличии Иначе изврат делать в условии общее Не

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

я смотрю по смыслу Код1 и Код2, если код1 основной, а код2 дополнительный, то делаю чтобы первый был код1

а если они равнозначны, то первым пишу тут чтобы без "Не"

(0) извращенец

второй - читабельней. пожалей тех кто будет за тобой косяки править

(0)
"Суть вопроса в том, чтобы более часто используемый код описывался сначала, а частные случаи - позже.
Или все же общий - в конце?"

Без разницы в общем случае.
Частные случаи можно подобрать как под один так и под другой вариант.

Обсуждения, имхо, не стОит.

(0) сейчас производительность процессоров такова, что никто инверсию и не заметит в твоем копрокоде )))

(6) я порой думаю о красоте кода. Не хотелось бы описывать редкие исключения раньше по ходу модуля, чем общий случай. Как-то оно бросается в глаза.

(7) Я не о скорости вопрос задал, а о красоте

(6)+1
не ассемблере пишем.

лучше так

Если Х Тогда
 Код2;
Иначе
 Код1;
КонецЕсли;
ибо можно делать так
Если Х Тогда
 Код2;
ИначеЕсли Х Тогда
 Код3;
ИначеЕсли Y Тогда
 Код4;
ИначеЕсли Z Тогда
 Код5;
Иначе

И вот еще варианты: "не(а=б)", "a<>b". Что лучше в смысле понимания?

а вообще лучше сделать так:
class Object
calculate(){Код2}

class Object_1
override calculate(){Код1}

(13) где Object_1 порожден от Object

(12) по стандарту 1с a<>b

(9) кому нафик нужна красота? главное - чтобы работало правильно и быстро

+16 пользователю глубоко нас-рать на красоту кода...

(16) я олд-скул, мне нужна красота в коде

(18) такими скулами похвастать может каждый, и каждой школы свои тараканы в голове...

(18) Если тебе нужна красота, то: Отрицательные условия читаются гораздо хуже

(0) Правильно писать в три ветки. Третья - для условия, которое не встречается никогда.

(18) и это - объясни , в чем прелесть отрицания?

(0) ты ошибаешься, если думаешь, что тебя это каким-то образом спасет

Уникальность условия можно подчеркнуть не разделением основного потока алгоритма, а внесением изменения в основной поток (круто загнул) :)

...
Четыре = 2 х 2;

Если Гений_1С = Истина Тогда
  Четыре = 5;
КонецЕсли;
...

(24)
Это 1С. В нем
"Если Условие" и "Если Условие = Истина" - немного разные вещи...

(25) чем же?

(0) под условие Иначе могут разные вещи попадать, поэтому все зависит от условия и вариантов Иначе

(26) наличием составных типов напр

отрицание НЕ вообще никогда не надо писать.
Код должен быть легким чтоб его читать как книжку слева на право, а не посмотреть что справа а потом проотрицать слева.

(26)
Если в Условие - Неопределено, то
"Если Условие Тогда" - вылетит с ошибкой "Преобразование значения к типу Булево не может быть выполнено"
а
"Если Условие = Истина Тогда" - без ошибок перейдет в Иначе.

(29) не вижу смысла такой категоричности

(28) если у ваши условия могут вернуть не только булево, но и ссылку на план счетов, или там Неопределено, для такого кода название  существуют очень давно

(32) и к (29) тоже относится

(31) и к (32) тоже относится ;)

(33) равно как и к 34 8)

(24) лишний прогон кода не есть хорошо для скорости

(31) Не можно использовать, но по другому.
Если у вас есть отрицание в услоии, нужно писать так.

ТоварИсключительный = НЕ Товар.ВидТовара = "Бук";
Если ТоварИсключительный Тогда
...
КонецЕсли;

Тогда Не не замыливает глаз

(35) Я запутался в ваших goto, шеф :)
Думаю меня вы поняли

И этот копрокод красив?
ТоварИсключительный = НЕ Товар.ВидТовара = "Бук";

функция сравнитьРезультат(условие, резИстина, резЛожь)

(40) Такая функция уже есть "?"

(37) очешуенно читабельно...

(39) Условие должно вычисляться отдельно и сохраняться в переменную с читаемым названием, в названии которой нет НЕ

(0)Если можешь не писать - не пиши.
Сказал как то один уважаемый человек. И я, глядя на тебя, полностью с ним согласен.

(44) ты не художник

(45) ну если запрос к базе занимает 99.99% времени, а всё остальное 0,01%, то вы наверно сильно чего-то накурились, что ищете резервы скорости в присваивании значения булевой переменной.

А еще функции (особенно булевские) в принципе не должны иметь императивного названия! Функция на то и функция, что не является процедурой...

Просто убивают такие конструкции в типовых:
"Если ПроверитьЗначенияРеквизита(Реквизит) тогда"

Это кошмар какой-то..

Так, конечно же лучше:

Если Х Тогда
 Код2;
Иначе
 Код1;
КонецЕсли;

так

// Если не редкая >|<опа, делаем, как обычно
Если НЕ Х Тогда
 Код1;
Иначе
 Код2;
КонецЕсли;

(48) Что посоветуешь для этого кода?
http://s52.radikal.ru/i137/1207/77/6ea15989e208.png

(50) к (51)

(50) ctrl-a + delete ^)

(50) Впечатляет. Только глВозвестиВСтепеньЧисло некорректно реализовано.

?(,,)

вот ? сделали, а ++ нет. все таки 1с делали обкуренные.

(50) это мой ночной кошмар.

(50) Это просто конгениально. Напоминает мне пору писания на с++.

Обычно короткие ветки условия я располагаю спереди, чтобы читать код было проще.

(58) вот то то и оно, что часто используемый код обычно короче, чем редко. ;-)

длинных веток условий вообще не должно существовать.

+(60) И циклов. И вообще процедур.

Серега, у тебя какое то очередное обострение?
Какая нафик красота кода? Код должен быть работоспособен и быстр...

А код в (50) может быть и неплохой. Тока раскидать по строкам.

(62) я привык писать красиво

(64) красиво это как в (39)

(64) Что значит "красиво"?
Оптимально(по разным критериям) - может быть.
Читаемо (то же могут быть разные "стандарты" и точки зрения)- может быть.
Красиво? - что это?

Не ну я понимаю, иногда можно назвать "красивым" какое-то удачное(иногда, может, нестандартное) решение, алгоритм и тп.
Сабж же - это с какой стороны к монитору бантик привесить.

(0) "... а кому свиной хрящик"

(64) За последние 3 часа успел, или когда?

(64) а чем не нравится : =?(X,Код2,Код1);

(54) +1, не заметил)))

(66) "Красиво" - это когда достигнут баланс между читаемостью (поддерживаемостью) и производительностью(затратами ресурсов).