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 написаны, меня идея модифицировать строку ради парсинга крайне пугает.

[blacklion.livejournal.com]

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

[dair-spb.livejournal.com]

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

[w00dy.livejournal.com]

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

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

use pcre, luck!

[theiced.livejournal.com]

luke*

[inhate.livejournal.com]

Еще блин один...

Some people, when confronted with a problem, think "I know, I'll use regular expressions." Now they have two problems.

Вышеприведенную цитату надо калёны железом на жопах студентов профильных факультетов выжигать.

[slonik-v-domene.livejournal.com]

Ваша идея - говно.

[artureg.livejournal.com]

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

[fas-tm.livejournal.com]

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

[theiced.livejournal.com]

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

[metaclass.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]

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

[blackyblack.livejournal.com]

1. В коде косяки - работать не будет.
2. В идеале стоило бы разбить код на две функции: поиск символа в строке и разбиение строки по номеру символа.
3. \r в результате за ошибку не считаем. Откуда нам знать, что она тебе не нужна :)

В итоге, ждём исправленную версию.

[metaclass.livejournal.com]

Проблема в том, что работало. Пока я не стал использовать ее вывод для дальнейшего парсинга, и то при этом в первую очередь сломалось на \r в последней строке.

А зачем разбивать на две строки, по моему так понятнее?

[plumqqz.livejournal.com]

Интересно даже (я комментарии не читал пока), сколько раз предложили strtok_r?

[plumqqz.livejournal.com]

Ебануться.

[slonik-v-domene.livejournal.com]

strtok_r здесь излишен

[slonik-v-domene.livejournal.com]

Код кривой, аффтара - отлучить от C и перевести в программисты PHP.

[metaclass.livejournal.com]

Не, не, мы уж как нибудь на clojure да F#, с PHP связываться не желаю.

[esil0x.livejournal.com]

Вы всё продолжаете себя мучать? Первый и третий пункт рецепта как и в прошлом посте, меняется только второй:

#include <boost/tokenizer.hpp>
#include <iostream>

const std::string str("aaaa,ssdsd asdsd,ffffff");

int main() {
    typedef boost::tokenizer< boost::char_separator<char> > splitter;

    boost::char_separator<char> sep(",");
    splitter spl(str, sep);

    for (splitter::const_iterator it = spl.begin(), end = spl.end(); it != end; ++it) {
        std::cout << *it << "\n";
    }
}

[metaclass.livejournal.com]

Да, надо будет С++ тоже попробовать, на следующей потусторонней задаче.