Comments on: Чтение и запись CSV файлов в Java (comma separated values file format)

By: ognivo777

ognivo777 — Wed, 06 Aug 2008 13:36:42 +0000

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

By: patrick

patrick — Sun, 20 Jul 2008 18:08:26 +0000

я не пойму 2 вещи
зачем делать внутренними классы Writer и Reader
и
зачем делать свои исключения копиями стандартных, да еще и называть их также…а как же повторное использование кода?

By: Крым

Крым — Tue, 15 Apr 2008 23:03:15 +0000

Спасибо за коды, возможно пригодиться, завтра дам компьютерику на фирме.
А вот нащёт лицензирования вы зря так. Изучайте если можете создавать подобные доработки!
Напишу завтра отзыв своего хакера, он вечно чем то недоволен в Excel, может как раз то что ему нужно.

By: c0nst

c0nst — Tue, 08 Apr 2008 16:30:14 +0000

Стандартное поведение библиотеки рассчитано на чтение и запись CSV файлов, которые понимает Excel.

By: Alexander

Alexander — Tue, 08 Apr 2008 08:53:57 +0000

Здравствуйте! А подскажите, пожалуйста, если я все 4 параметра подставлю по умолчанию, получится Excel или Pure CSV?

By: Samolisov Pavel

Samolisov Pavel — Sat, 05 Apr 2008 13:23:38 +0000

Хм… я сторонник несколько иной стратегии использования исключений. Хотя может быть я не понимаю всего смысла. Предположим что возникнет рантайм-эксепшн в обычном однопоточном приложении. Оно не прибьется?

Кстати согласен с вами что для ошибки парсера надо использовать другое исключение. Все-таки чтение файла и парсинг - разные операции. По поводу использования IllegalStateException - я думаю лучше написать свои типы сообщений, их и пробрасывать. Код использующий библиотеку будет читабельнее.

By: c0nst

c0nst — Fri, 04 Apr 2008 16:48:56 +0000

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

На счет лицензирования: К сожалению эту область я еще не изучал.

На счет Exception: Мне больше нравится использовать unchecked exceptions, но только в том случае, если ситуация действительно исключительная и не может быть обработана пользователем класса в большинстве случаев.

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

Единственное, на счет чего я сомневаюсь - то что для ошибок невалидности формата я использовал тот же Exception, что и для обертки IOException.
Наверное для ошибок формата лучше бросать IllegalStateException с соответствующим текстом.
Ну а IOException я оборачиваю в RuntimeException для того, чтобы пользователь моего класса не матюкался добавляя в своих методах некрасивые конструкции типа throws или try/catch/finally :).
Если программист посчитает нужным ловить Csv.Exception, он напишет соответствующий код. Но ловить ее надо только в случае чтения/записи CSV через сеть, так как только в этом случае можно попробовать корректно решить проблему - переконнектится и дальше совершить чтение/запись.

By: Samolisov Pavel

Samolisov Pavel — Fri, 04 Apr 2008 15:26:11 +0000

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

З.Ы. Понравился файл с лицензией ) Имхо можно было бы выбрать что-нибудь стандартное вроде BSDL. Впрочем ваш код - ваше право.

З.Ы.Ы. Спасибо за библиотеку.