How to write a switch statement for strings in Qt?

Question

I need to create the equivalent of a switch/case statement for strings in C++ with Qt. I believe that the simplest way is something like this (pseudo code)

enum colours { red, green, blue };
QString array[] colour_names = { "red", "green", "blue" };
switch (color_names[user_string]) {
  case red: answer="Chose red";
  case green: answer="Chose green";
  case blue: answer="Chose blue";
  other: answer="Invalid choice";
}

But this doesn't take advantage of some of the features of Qt. I've read about QStringList's (to find the position of the string in a list of strings), and std:map (see How to easily map c++ enums to strings which I don't fully understand).

Is there a better way to do a switch on strings?

You cannot use strings in a switch statement. You haven't told us why you need a swicth statement. The most simple solution to a lookup based on a string is to use a map. In Qt it would mean using a QMap. — user2672165, Sep 29 '13 at 19:37
You should use the llvm::StringSwitch: http://llvm.org/docs/doxygen/html/StringSwitch_8h_source.html — Industrial-antidepressant, Sep 29 '13 at 19:41
I found a suggestion on another site to use a QStringList of colors, use IndexOf() in the switch, and then use the enum value in the case statements — TSG, Sep 30 '13 at 16:58
You can do it with `QMetaEnum` and `QMetaObject` like in my answer [here](http://stackoverflow.com/a/25873662/995714) — phuclv, Sep 16 '14 at 16:14

Kuba hasn't forgotten Monica · Accepted Answer · 2013-09-29T19:57:07.617

The only way to use switch() with strings is to use an integer-valued hash of a string. You'll need to precompute hashes of the strings you're comparing against. This is the approach taken within qmake for reading visual studio project files, for example.

Important Caveats:

If you care about hash collisions with some other strings, then you'll need to compare the string within the case. This is still cheaper than doing (N/2) string comparisons, though.
qHash was reworked for QT 5 and the hashes are different from Qt 4.
Do not forget the break statement within your switch. Your example code missed that, and also had nonsensical switch value!

Your code would look like the following:

#include <cstdio>
#include <QTextStream>

int main(int, char **)
{
#if QT_VERSION < QT_VERSION_CHECK(5,0,0)
    static const uint red_hash = 30900;
    static const uint green_hash = 7244734;
    static const uint blue_hash = 431029;
#else
    static const uint red_hash = 112785;
    static const uint green_hash = 98619139;
    static const uint blue_hash = 3027034;
#endif

    QTextStream in(stdin), out(stdout);
    out << "Enter color: " << flush;
    const QString color = in.readLine();
    out << "Hash=" << qHash(color) << endl;

    QString answer;
    switch (qHash(color)) {
    case red_hash:
        answer="Chose red";
        break;
    case green_hash:
        answer="Chose green";
        break;
    case blue_hash:
        answer="Chose blue";
        break;
    default:
        answer="Chose something else";
        break;
    }
    out << answer << endl;
}

OK cool answer. How did you get your hash constants? (Do you have to run a simple program once to print them all out?) — TSG, Oct 12 '13 at 14:53
Yes, you can write a little program that will pick them out for you from source code. Maybe I'll add it to the question later. — Kuba hasn't forgotten Monica, Oct 12 '13 at 18:34

score 1 · Answer 2 · answered Dec 07 '16 at 01:47

1

QStringList menuitems;
menuitems << "about" << "history";

switch(menuitems.indexOf(QString menuId)){
case 0:
    MBAbout();
    break;
case 1:
    MBHistory();
    break;
}

answered Dec 07 '16 at 01:47

sonichy

1,370
2
14
17

score 0 · Answer 3 · answered Sep 30 '13 at 16:58

0

I found a suggestion on another site to use a QStringList of colors, use IndexOf() in the switch, and then use the enum value in the case statements

answered Sep 30 '13 at 16:58

TSG

4,242
9
61
121

How to write a switch statement for strings in Qt?

3 Answers3

Linked

Related