artbear писал(а) 24. Сентября 2007 :: 13:54:

А никто до него скорее всего и не додумывался. Просто сначала написали глючный вариант, когда пропущенный параметр равнялся пустому значению (Может быть это даже какая-нибудь обратная совместимость).
Потом поверх этого реализовали значения по умолчанию. Тщательно не протестировали. Ну и оппа.

Артур, ну мы опять возвращаемся к вопросу: "как будет лучше: исправить глюк, или оставить для совместимости?"

И, повторюсь, правильный ответ уже придуман.
Цитата:

fez писал(а) 24. Сентября 2007 :: 14:09:

1) Я лично не знаю, как сделать вот этот режим совместимости с 1С
Т.е. непонятно, как отличить передачу пустого значения от параметра по умолчанию

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

Давно статью читал, умные вещи говорит, согласен.
НО не в данном случае - см. выше про незнание.

artbear писал(а) 24. Сентября 2007 :: 14:26:

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

sedmin писал(а) 24. Сентября 2007 :: 15:04:

Ну, перегнул малось А как еще вас всех на разговор вызывать? Если все время мягко, с улыбочками, то все в пустоту и уйдет - ничего не изменится.

По поводу внесенных правок. А где оценки множества поломанного кода? Где "опросы общественного мнения"? Просто взяли и поменяли. И чужой код поломался. А уж как это выглядит в свете добрых советов радостно ставить себе ночную сборку, сам понимаешь...

Это тебе хорошо - у тебя на каждый чих тест есть. А то и два. А у меня и других пользователей не все так шоколадно. Что-то может гавкнуться непосредственно у клиента. Поэтому про код для широкого пользователя я могу с тем же успехом и у тебя спросить.

Uzhast писал(а) 24. Сентября 2007 :: 15:12:

Кстати, про тесты. Ощущение того, что "на каждый чих тест есть. А то и два." - оно, сцуко, усыпляет.

Не далее, как на прошлой неделе отловил баг.
Мне казалось, я в этом куске все, что можно протестировал. Года три или четыре любое изменение программы в этом месте сопровождалось тестами. Хрен на блюде.
Так обидно было. Уже и не помню, когда последний раз ломал существующий функционал в проводках. Только если что-то новенькое бухи выкапывали. А тут на тебе.

И кстати, вот эта хрень с параметрами по умолчанию, это как раз тот самый тонкий момент, который может быть закопан где-то ОЧЕНЬ глубоко. И даже на покрытом тестами продукте может вылезти БАГ. Просто потому что никому в голову не придет тестировать значения по умолчанию.

sedmin писал(а) 24. Сентября 2007 :: 19:41:

Не совсем верно - я лично сделал настройки только для кода, который меняет поведение 1С и может сказаться на работе системы в целом - например, ТурбоБЛ или Перехватчик ставят свои перехваты на общесистемные методы и соответственно могут повлиять на работу любых других методов!

По теме
fez писал(а) 24. Сентября 2007 :: 15:30:

Не пришло бы, если бы у 1С в этом плане была бы общепринятая совместимость, т.е. как в других языках.
Считаю любое 1С-совместимое использование параметров по умолчанию глючным.
Сам так никогда не делал, потому и не знал, кстати

Я лично за
1) общепринятое поведение, не как у 1С
2) поведение должно соответствовать документации.