Показать сообщение отдельно
Старый 30.12.2017, 02:16   #13  
skuull is offline
skuull
Участник
Most Valuable Professional
Лучший по профессии 2014
 
699 / 752 (27) +++++++
Регистрация: 08.03.2013
Адрес: ХЗ
Цитата:
Сообщение от sukhanchik Посмотреть сообщение
Замечание корректное, но ... null reference exception никогда не будет словлено, т.к. панель кнопок всегда привязана к исходной форме. Т.е. здесь просто лишняя проверка на formRun, которая в целом никому не мешает.
Ну как никогда... Вы написали класс который принимает FormRun т.е. кто угодно может передать туда любой FormRun у которого может и не быть caller и напрямую вызвать generateFullURL, т.к. он публичный. Так же вы его назвали "полноценный". Это дает мне основание думать, что это не какой-то там примерчик, а целый класс, бери и используй

Выкладывая код на всеобщее обозрение надо быть готовым к тому что вам скажут про construct с параметрами который не best practice и про прочие мелочи