C, parse comma-separated line

[metaclass]

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

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

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

Comments

[metaclass.livejournal.com]

У меня там не только первое вхождение, а перебирает все вхождения. Т.е.я специально source_start модифицирую.

Проверки параметров и выход из цикла, да, надо исправить.
А вот насчет возвращаемых значений - почему именно так: 0 и -1?
Я, когда писал, ориентировался на что-то вроде: 0 (false) - завершение работы, 1 (true) - продолжение.
Стандартные же true и false вроде только в C99 появились.

[slonik-v-domene.livejournal.com]

http://metaclass.livejournal.com/680788.html?thread=11874900#t11874900

[plumqqz.livejournal.com]

"Как всякий истинный ученый, он был немногословен..."

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

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

[lionet.livejournal.com]

Условия задачи уже известны, да.

[slonik-v-domene.livejournal.com]

1) Потому, что ожидаемое поведение -1 - ошибка, а 0 и больше - успех.
2) true и false не существуют до тех пор, пока не существует типа bool. Тот, кто указывает #define true .... - тот является клиническим мудаком.
3) если надо найти все последовательности, следует просто вызывать функцию в цикле

[slonik-v-domene.livejournal.com]

1) Твоя идея говно.
2) Если ты без троллинга, в здравом уме и трезвой памяти считаешь что для такой задачи нужна библиотека PCRE ТОЛЬКО НА ТОМ ОСНОВАНИИ ЧТО ОНА УЖЕ ЕСТЬ, ты - клинический идиот. Ничего личного.

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

1) моя идея охуенна
2.1) не, если мы точно и абсолютно уверены (готовы заложить голову), что никогда ничего ни в этом месте ни в другом близкого к pcre не понадобится -- то да, идея оверкил.
2.2) в жизни все будет не как в 2.1. а сначала мы захотим выкидывать завершающие пробелы. потом сравнивать со всякими CONNECT. и т.д. и т.п.

[slonik-v-domene.livejournal.com]

Все. Разговор можно далее не продолжать.
Кратко:

1) пошел нахуй
2.1) пошел нахуй со своим оверинжинирингом
2.2) пошел нахуй с идеей использования регулярных выражений при обработке нерегулярных языков

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

разумеется строки, разделенные запятыми задются регулрной грамматикой.
это же csv с экранированием и кавычками.
вот теперь можешь идти нахуй.

[slonik-v-domene.livejournal.com]

>... задются регулрной грамматикой...

Идешь нахуй ты, т.к. про расширяемость и прочую хуиту ты начал первым. Что там и как расширять чтобы csv не стал вдруг нерегулярным - внешнее ограничение к твоей "бесконечной расширяемости". Потому как потом ВНЕЗАПНО выяснится что внутри строк переменные нужны, анэскейпинг из octal, unicode, hex, dec, control cheracters или еще что похожее, а, вот беда, PCRE с этой задачей справиться не может.

Для парзинга строк csv никакого PCRE не то, чтобы не надо, а вредно и бессмысленно. Оверинжиниринг там, где не требуется - признак программиста-мудака. Короче, с тобой все понятно. Продолжай рассказывать всем дальше про свою perl-травму головного мозга, хе-хе.

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

о да, будем руками закатывать солнце вручную, а вдруг у нас будет csv, а не просто запятые?
нахуй, нахуй.
мне мое время дорого, а оргазма от заката вручную не испытываю.

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

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

[berezovsky.livejournal.com]

главное опиатные наркотики не пробовать

[plumqqz.livejournal.com]

Господа, обратитесь наконец к мануалу:
http://ru.wikipedia.org/wiki/%D0%9F%D0%BE%D0%BB%D0%B5_%28%D0%BF%D0%BE%D0%B5%D0%B4%D0%B8%D0%BD%D0%BE%D0%BA%29

А логомахия хуями - это, знаете, какой-то базар с цыганами, но без медведей.

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

издвеваешься?
это же ни какашками не покидаться, ни в мочой побрызгаться.

а что считается за палицу?

[plumqqz.livejournal.com]

это же ни какашками не покидаться, ни в мочой побрызгаться.

Ребят, если вам хочется мочой побрызгаться, то это я даже не знаю какое извращение.

а что считается за палицу?

Ножка от офисного стола, скажем,

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

я только защищаюсь.

[plumqqz.livejournal.com]

Так все только защищаются; например, нигде нет министерств нападения и везде только министерства обороны.

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

кажется есть где-то

[plumqqz.livejournal.com]

Ну, исключение только подтверждает.
Кстати, а где?

[nicka-startcev.livejournal.com]

только не забыть скорость замерить.
Плюсявые потоки заметно медленнее нормального ввода/вывода.
В опенскаде это увеличивает время рендеринга почти вдвое.

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

не помню, но кажется встеречал упоминание. но зуба не дам

[jdevelop.livejournal.com]

хаскель же