Вход на сайт
Вопросы по C#
975 просмотров
Перейти к просмотру всей ветки
в ответ Mahone 12.11.13 18:15
Вижу - <class Program>.
Не вижу - <class TISBN>, <class TISBNChar>.
Вижу - <if (totalDigits != 12)>
Не вижу - быстрой проверки длинны введенной строки - смотри какие проперти есть у массива.
Не вижу - смысла в числе 12. У меня в коде нет каких-либо цифровых и текстовых констант - полиси не разрешает и весьма правильно - сейчас работаю с кодом где это не соблюдается - головная боль постоянна...
Вижу - <if (char.IsDigit(digitAsChar) == true)>
Не вижу - почему должна выполнятся повторная проверка.
То, что тебе приходится писать довольно громоздкий и сложный для чтения код есть следствие того, что у тебя нет <TISBN> и <TISBNChar>. Подумай как это организовать.
По общему коду - у тебя смешаны хранение данных, ввод данных и какие-то действия с данными. В коде который будет "хорошим" должно быть четкое разделение где, что и как делается. Даже в том случае когда вроде бы можно совместить пару вычислений - все же стоит иметь их отдельно - меньше головной боли будет...
Вопросик на засыпку: <ISBN>, если Я правильно помню, имеет более одного формата. Как будешь решать проблему, если у тебя будут данные с двух внешних источников и в разных форматах?
<P.S.> Цель, как всегда, - научить, а не сделать вместо... научить включает в себя многое, в т.ч. заставить думать над тем как сделать... иногда даже в пустую...
Не вижу - <class TISBN>, <class TISBNChar>.
Вижу - <if (totalDigits != 12)>
Не вижу - быстрой проверки длинны введенной строки - смотри какие проперти есть у массива.
Не вижу - смысла в числе 12. У меня в коде нет каких-либо цифровых и текстовых констант - полиси не разрешает и весьма правильно - сейчас работаю с кодом где это не соблюдается - головная боль постоянна...
Вижу - <if (char.IsDigit(digitAsChar) == true)>
Не вижу - почему должна выполнятся повторная проверка.
То, что тебе приходится писать довольно громоздкий и сложный для чтения код есть следствие того, что у тебя нет <TISBN> и <TISBNChar>. Подумай как это организовать.
По общему коду - у тебя смешаны хранение данных, ввод данных и какие-то действия с данными. В коде который будет "хорошим" должно быть четкое разделение где, что и как делается. Даже в том случае когда вроде бы можно совместить пару вычислений - все же стоит иметь их отдельно - меньше головной боли будет...
Вопросик на засыпку: <ISBN>, если Я правильно помню, имеет более одного формата. Как будешь решать проблему, если у тебя будут данные с двух внешних источников и в разных форматах?
<P.S.> Цель, как всегда, - научить, а не сделать вместо... научить включает в себя многое, в т.ч. заставить думать над тем как сделать... иногда даже в пустую...