Qtのsplitについてです

Question

sample.txt
の中身が
apple=150
orange=300
grape=200
のときにsample.txtを読み込んで(全読み込み)"\n"で1行ずつsplitした後に、"="でsplitしてパラメータ名が正しければcorrectを表示したいのですが、下記コードだと、grapeがある場合でもgrapeがない場合でもelse if(key != "grape")に引っかかってしまいます。
grapeがある場合、else if(key != "grape")に引っかからないようにするにはどのように修正すべきなのでしょうか。
grapeがない場合やgrapeの綴りが間違っている(例glape)時だけelse if(key != "grape")の処理に入りたいです。

#include "MainWindow.h" 
#include <QApplication>
#include <QDebug>
#include <QFile>
#include <QTextCodec>

int main(int argc, char *argv[])
{
    //QApplication a(argc, argv);
    //MainWindow w;
    //w.show();
    QFile file("C:\Users\Desktop\sample.txt");
    if (!file.open(QIODevice::ReadOnly))
        {
            qDebug() << "can not open file." ;
            return 0;
        }
        QString str;
        QTextStream in(&file);
        str = in.readAll();
        qDebug() << str ;
        QStringList list1 = str.split("\n");
        for (int i=0; i < list1.count(); i++) {
            QString txt = list1[i];
            QStringList list2 = txt.split("=");
            QString key = list2[0];
            QString value = list2[1];
            if(key == "apple"){
                qDebug() << "correct";
            }
            if(key == "grape"){
                qDebug() << "correct";
            }
            else if (key != "grape"){
                qDebug() << "incorrect";
                //break;
            }
            if(key == "orange"){
                qDebug() << "correct";
            }
        }

    file.close();
    //return a.exec();
}

Comments

[yuki23]

「grapeがある場合」とはどういう意味ですか？
手元で実行してみましたが、ちゃんとkeyがgrapeの行では else if (key != "grape"){ の分岐は通りませんでしたが？

[wave_vague]

私が書いたコードだと、keyにappleが入っている際にelse if (key != "grape")の分岐を通ってしまいます。
else if (key != "grape")はsample.txtにgrapeがない場合のみ通ってほしいです。

Answer 1

元々の質問の回答は @tyu_ru_cpp さんが示されているように QSet(や QList や QHash など）でキーの集合を管理して、それに対して確認をするということになると思います。

単に キー=値 形式のファイルを扱いたいのであれば、.ini フォーマットに対応している QSettings を使う方法もあります。

#include <QDebug>
#include <QSettings>

int main(int argc, char **argv)
{
    QSettings settings("./sample.txt", QSettings::IniFormat);

    if (settings.contains("apple"))
        qDebug() << "correct";

    if (settings.contains("grape"))
        qDebug() << "correct";
    else
        qWarning() << "incorrect";

    if (settings.contains("orange"))
        qDebug() << "correct";
    return 0;
}

Answer 2

if (key == "apple"){ // ①
    qDebug() << "correct";
}


if(key == "grape"){ // ②
    qDebug() << "correct";
}
else if (key != "grape") { // ③
    qDebug() << "incorrect";
    //break;
}

keyが"apple"の時に、①のif文を抜けた後③のif文に入ってしまいます。

このようなときには正しい単語の集合を持っておき、keyがそれに含まれるかを調べるのが良いと思います

QSet<QString> fruit = {"apple", "grape", "orange"};

if (fruit.contains(key)) {
    qDebug() << "correct";
} else {
    qDebug() << "incorrect";
}

---

余談ですが、QString("apple=150\norange=300\ngrape=200\n").split("\n");の結果はQList("apple=150","orange=300","grape=200","")の4つになります。

このとき、質問文のコードのように

QStringList list2 = txt.split("=");
QString key = list2[0];
QString value = list2[1];

としてしまうと最後の空文字列に対してsplit("=")した結果の'[1]'を参照しようとするので範囲外参照になってしまいます。

コレを防ぐために、split後のlistのサイズはしっかり調べましょう。また、この問題の直接の対策にはなりませんが、.split("=", QString::SkipEmptyParts)とすると空文字列を飛ばしてくれるので覚えておくと良いと思います

Comments

[wave_vague]

回答ありがとうございます。
あらかじめ、keyがあるかどうかの確認をしたい場合は正しい単語の集合を持てばよろしいのですね。大変参考になりました。

ちなみに、keyがある場合はそのkeyの値をメンバ変数に代入したいのですが、下記方法であっていますでしょうか？

if (key == "apple"){
m_apple = value.toInt();//appleの値である150が代入される
}
if (key == "grape"){
m_grape = value.toInt();//grapeの値である200が代入される
}

[wave_vague]

この場合だとappleの分岐を通った後
else if (key != "grape"){
qDebug() << "incorrect";
}
に引っかかってしまいますよね？

[tyu_ru_cpp]

そのような場合は、
if (key == "apple"){
m_apple = value.toInt();//appleの値である150が代入される
} else if (key == "grape"){
m_grape = value.toInt();//grapeの値である200が代入される
} else if (...) {
...
} else {
qDebug() << "incorrect";
}

といったふうにif-elseを書くのがよいと思います。

[tyu_ru_cpp]

また、連想配列を用いて

QMap<QString, int> fruit;
fruit["apple"] = 0; // デフォルト値
fruit["orange"] = 0; // デフォルト値
fruit["grape"] = 0; // デフォルト値

if (fruit.contains(key)) {
fruit[key] = value.to_int();
} else {
qDebug() << "incorrect";
}

みたいに記述すると拡張性を高くすることができます

[tyu_ru_cpp]

あ、tasukuさんの回答内容と被ってしまった
QSettingsはQMapのキーをQStringにvalueをJSONの値と似たようなものに特殊化して、設定ファイルの入出力にも対応させたような便利なやつです

[wave_vague]

ありがとうございます。if-elseを使う場合は下記コードのようになるという認識であっていますでしょうか。sample.txtに記載されているkeyが正しければvalueをメンバ変数に代入、keyが正しくないおよびkeyがない場合はincorrectを出力する。
for (int i=0; i < list1.count(); i++) {
QString txt = list1[i];
QStringList list2 = txt.split("=");
QString key = list2[0];
QString value = list2[1];

if (key == "apple"){
m_apple = value.toInt();//appleの値である150が代入される
} else if (key == "grape"){
m_grape = value.toInt();//grapeの値である200が代入される
} else if (key == "orange") {
m_orange = value.toInt();//orangeの値である300が代入される
} else {
qDebug() << "incorrect";
}
}

[tyu_ru_cpp]

(valueの中身が整数になっていないようなケースを考慮しなければ)おおよそそんなコードになると思います

valueが正しくパースできたかは.toInt()の第一引数にチェック用のbool変数へのポインタを渡せばできます
詳しくは公式リファレンスをどうぞ　https://doc.qt.io/qt-5/qstring.html#toInt

[wave_vague]

ありがとうございます。大変助かりました。