C, parse comma-separated line

[metaclass]

Кошерен ли такой код для разбиения строки, разделенной запятыми на отдельные строки: http://ideone.com/rhNXq ?

PS: strtok не признаю по идеологическим причинам - она меняет анализируемую строку, что меня крайне пугает, хотя в моем конкретном случае это не мешает.

PPS: безотносительно к strtok, в функции было минимум две ошибки - отсутствие проверки на выход за пределы буфера и попадание eol в последнюю строку (девайсина, с которой приходит строка, терминирует ее \r)

Comments

[blacklion.livejournal.com]

man strsep strtok

[metaclass.livejournal.com]

Про них какие-то неприятные вещи в man написаны, меня идея модифицировать строку ради парсинга крайне пугает.

[dair-spb.livejournal.com]

Как предлагаешь парсить потенциальный гигабайт?

[gds.livejournal.com]

итератами, чо.

[gds.livejournal.com]

ну да, и не через strхуйню.

[blacklion.livejournal.com]

mmap()? :)
Решение топикстартера тоже особо на подкачку не рассчитано

[blacklion.livejournal.com]

Есть ещё strcspn() и strspn() и strpbrk()

[http://users.livejournal.com/_slw/]

use pcre, luck!

[artureg.livejournal.com]

а split в этой дремучести не определён?

[fas-tm.livejournal.com]

в С нет строк как таковых. есть последовательность символов.

[artureg.livejournal.com]

я знаю, это ужасно

[theiced.livejournal.com]

luke*

[theiced.livejournal.com]

сплит? ты дурак?

[fas-tm.livejournal.com]

fuck*
ближе по смыслу :)

[metaclass.livejournal.com]

А возвращать он что будет?:)

[avnik.livejournal.com]

Кстати -- а куда бы посмотреть на предмет "загрузить большой файл с radix tree mmap'ом и поискать там чего"? (не в плане супер-реализации, а скорее чего попроще, чтобы понять как такое работает)

[sorhed.livejournal.com]

Вы так говорите, будто последовательность символов нельзя сплитить.

[lionet.livejournal.com]

Вектор. Так, например. http://libstrfunc.cvs.sourceforge.net/viewvc/libstrfunc/libstrfunc/sf_core/split.c?content-type=text%2Fplain

[sorhed.livejournal.com]

(ггг, тут был комент, в котором я был дебилом, извините).

[theiced.livejournal.com]

а например память оно зачем будет ТАК жратъ?

[smalgin.livejournal.com]

Как уже предложили выше (lionet), используйте готовую библиотеку строк. Чем раньше вы начнете ее использовать, тем меньше времени потратите на переписывание потом.

Если вам нужно просто разобрать CSV, то и для разбора оного есть готовые библиотеки с хорошей историей.

В строках можно даже с добрыми намерениями нагородить 'one off' и buffer overruns (как вы показательно его вставили в первой же функции сверху)

Еще стандартный совет из сети - не использовать арифметику указателей без сильной нужды. operator[] and indexes do the same job just fine.

Будет весело, гарантирую.

[metaclass.livejournal.com]

А где там overrun?

[metaclass.livejournal.com]

Холерный вибрион, я ж не проверяю в цикле выход за пределы буфера.

[w00dy.livejournal.com]

strsep и strtok разруливают ситуацию с экранированием? Нет? Тогда в печку их.

[abbra]

Knuth, Donald. E. [1998]. The Art of Computer Programming, Volume 3, Sorting and Searching. Addison-Wesley, Reading, Massachusetts.