Парсинг строк

[__Heaven__]

Привет, друзья!
Есть файл, который содержит строки типа

*node(363484,232.7625,-113.996,40.69956,0,0,0,0,0)

здесь первая цифра - номер узла, три следующие - его координаты.

Написал код, который считывает строку, но выглядит он коряво на мой взгляд.

Код

C++ (Qt)
    while (!fileStream_.atEnd()){
        QString line(fileStream_.readLine());
        if (line == "END NODES"){
            break;
        }
 
        Q_ASSERT(line.startsWith("*node("));
        QTextStream lineStream(&line, QIODevice::ReadOnly);
 
        lineStream.seek(6);
 
        ParsedNode node;
        lineStream >> node.number;
 
        float x;
        float y;
        float z;
 
        lineStream.seek(lineStream.pos() + 1);
        lineStream >> x;
        lineStream.seek(lineStream.pos() + 1);
        lineStream >> y;
        lineStream.seek(lineStream.pos() + 1);
        lineStream >> z;
 
        node.coordinates = QVector3D(x, y, z);
        nodes << node;
    }
 

В регулярках я не разбирался. Быть может есть более "чистые" способы чтения? Прошу помочь.

[Old]

Написал код, который считывает строку, но выглядит он коряво на мой взгляд.

Да и работать будет только в идиальных условиях. Поставь кто пробелов и все развлится.
Регулярки будут сами проверять, что бы в начале была node и значения разделены запятыми.
А еще есть boost.spirit.

[Bepec]

А ещё есть QString::split

[__Heaven__]

А ещё есть QString::split

Ща бомбить будет, ЕВПОЧЯ

Спирит сложная штука, вроде. Наверное за день не разберусь. Но спасибо. А регулярками как это решить?

[Old]

А регулярками как это решить?

Я не большой специалист по регуляркам, но вот набросок:

[gil9red]

А ещё есть QString::split

А почему мы и нет?

Код

C++ (Qt)
std::tuple<float, float, float> parse_coords(QString text) {
    const QStringList& parts = text.replace("*node(", "").replace(")", "").split(",");
    return std::make_tuple(parts[1].toFloat(), parts[2].toFloat(), parts[3].toFloat());
}
 
QString text = "*node(363484,232.7625,-113.996,40.69956,0,0,0,0,0)";
std::tuple<float, float, float> coord = parse_coords(text);
qDebug() << std::get<0>(coord) << std::get<1>(coord) << std::get<2>(coord);
 

[__Heaven__]

gil9red, классный вариант. Здесь же можно и ликвидацию пробелов добавить и проверку на количество полученных данных.
Old, спасибо за набросок. Для меня он выглядит страшно так как не пользовался ранее регулярками. Думаю, пойду по пути, gil9red.

[kambala]

А регулярками как это решить?

Я не большой специалист по регуляркам, но вот набросок:

я бы так сделал: ссылка (если в координатах дробная часть гарантирована, можно еще упростить)

можно даже не захватывать отдельные результаты, а просто взять и полное совпадение сплитнуть по ,

[Igors]

Ща бомбить будет, ЕВПОЧЯ

Конечно буду Во всех ответах прослеживается прямо какое-то "физиологическое" желание написать короче - нет, даже КАК МОЖНО КОРОЧЕ! Вот это, дескать, крутизна! Не самому там чего-то велосипедить, а тыц готовое - и все дела.

А давайте по-взрослому - не просто так "распарсить", а с полным контролем ошибок и предъявлением юзеру текста ошибки (и где она произошла).  

Код

C++ (Qt)
#include <QStringRef>
#include <QDebug>
 
namespace QStringParser {
 
inline QStringRef BeforeFirst( const QStringRef & src, const QString & splitter )
{
	return src.left(src.indexOf(splitter));
}
 
inline QStringRef AfterFirst( const QStringRef & src, const QString & splitter )
{
	int pos = src.indexOf(splitter);
	iif (pos < 0) return src.right(0);
	return src.mid(pos + splitter.size());
}
 
inline bool ReadInt( QStringRef & ref, const QString & splitter, int & val )
{
	bool ok;
	val = BeforeFirst(ref, splitter).trimmed().toInt(&ok);
	if (!ok) return false;
	ref = AfterFirst(ref, splitter);
	return true;
}
 
inline bool ReadFloat( QStringRef & ref, const QString & splitter, float & val )
{
	bool ok;
	val = BeforeFirst(ref, splitter).trimmed().toFloat(&ok);
	if (!ok) return false;
	ref = AfterFirst(ref, splitter);
	return true;
}
 
}	// namespace
 
using namespace QStringParser;
 
bool Error( const QStringRef & ref, QString & err, const QString & txt )
{
	err = "\nError at position " + QString::number(ref.position()) + ": " + txt +
		  "\n" + ref.trimmed().toString(); 
	return false;
}
 
bool Parse( const QString & srcStr, int & number, float val[3], QString & err )
{
	static const QString prefix("node");
	static const QString comma(",");
 
	QStringRef src(&srcStr);
	if (!src.startsWith(prefix)) 
		return Error(src, err, "string must start with 'node'");
 
	src = AfterFirst(QStringRef(&srcStr), prefix).trimmed();
	if (!src.startsWith('(')) 
		return Error(src, err, "'(' expected");
	src = src.mid(1);
 
	if (!ReadInt(src, comma, number)) 
		return Error(src, err, "integer value expected");
 
	for (int i = 0; i < 3; ++i) 
		if (!ReadFloat(src, comma, val[i])) 
			return Error(src, err, "float value expected");
 
	return true;
}
 
int main(int argc, char **argv)
{
	int nodeNum;
	float val[3] = { 0 };
 
	QString err;
	QString test("node  ( 363484   ,232.7625,    -113.996, 40.69x956,0,0,0,0,0)");
	bool ok = Parse(test, nodeNum, val, err);
	qDebug() << ok << nodeNum << val[0] << val[1] << val[2];
	if (!ok) qDebug() << err;
	return 0;
}

Не надо бояться что (в первый раз) придется потратиться на полезные ф-ции, очень быстро это окупится

[Old]

А давайте по-взрослому - не просто так "распарсить", а с полным контролем ошибок и предъявлением юзеру текста ошибки (и где она произошла).  

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

[Bepec]

Распарсить строку и написать класс, делающий это это разные вещи.

PS преждевременная оптимизация, так помоему называется то, что демонстрирует Igors.

[Igors]

Конечно недаром придумали "регулярки", и недаром они так популярны. Да и спырыт мне очень понравился. Но "в большинстве случаев это избыточно", все можно решить куда проще написав неск ф-ций "по руке" и закинув их в утили. С оптимизацией проблем нет - ну юзаем другой Qt класс, и все. И чего ото козырять туплями, заряжать на строку целый QTextStream и.т.п ?

В общем налицо какой-то нестерпимый зуд чем-то непременно "воспользоваться", и ни в коем случае не писать самому. Если его унять - все сразу станет нормально, задачка-то детская. Просто-напросто не хватает терпения и аккуратности (после жратвы букваря)

Да, по скорости там нет желающих посравнивать? 

[Old]

Распарсит три числа задачка действительно детская. Были бы вместо чисел номера телефонов в любых видах или паспортные данные, вы бы велосипед еще долго писали (и не факт, что доделали бы). А пользователи регулярок уже давно бы задачу решали.

[__Heaven__]

Поставь кто пробелов и все развлится.

Как в воду глядите Полдня потратил на отладку из-за этого. Предполагал, что формат без пробелов.

В итоге реализовал так:

Код

C++ (Qt)
    while (!fileStream_.atEnd()){
        QString line(fileStream_.readLine());
        if (line == endNodesGroupString){
            break;
        }
 
        if (line.startsWith(nodeIndicatorString)){
            const QStringList &lineParts = line.remove(nodeIndicatorString).remove(" ").remove(")").split(',');
            constexpr int partCountRequried = 4;
            if (lineParts.count() < partCountRequried){
                qCritical() << "Got bad node line";
            }
            else{
 
                ParsedNode node;
                node.number = lineParts.at(0).toFloat();
 
                for (int k = 0; k < nodeDimension; ++k){
                    node.coordinates[k] = lineParts.at(k + 1).toFloat();
                }
                nodes << node;
            }
        }
    }
 

Пришлось заменить хранение координат с QVector3D на std::array<float, 3>.
Критикуем, улучшаем (читабельность), пожалуйста
Думаю, блок else заменить на метод readNode(QStringList) и  добавить к qCritical строку ошибки. Номер линии как-то лениво считать. Или есть какие средства QTextStream позволяющие выводить номер линии?

[kambala]

ага, а потом вместо пробелов попадутся табы и опять все развалится а в регулярках есть \s*