Russian Qt Forum

Здравствуйте. Помогите пожалуйста, разобратся с switch! (С++, QT)
Так получилось, что почти всегда пользовался if ... else if в своих программах =). Решил наконец избавиться от этой вредной привычки. Но увы...
Буду благодарен если кто ткнёт носом в ошибку

ошибка компилятора (компилирую в QDevelop + MinGW):
error: switch quantity not an integer

PS. При использовании if else if  работает на ура

В switch нельзя использовать строки (о чем он тебе в ошибке и говорит). Воспользуйся enum.

C++ (Qt)
enum FieldType { specila, day, monthNum, yearLst, ..... };
 
FieldType fld = array[ 1 ];
switch( fld )
{
    case special:
        ....
    case day:
        ....
}
 

Спасибо!

Можно ещё вопрос не по теме... Почему у вас в ответе ситаксис с подсветкой, а у меня нет

C++ (Qt)
int x = 0;
switch (x)
 
case 0: 
 x++;
break;

Выбери стиль в комбике, по стандарту там GeSHi вроде...

Интереса ради хочу задать такой вопрос. В стандартном примере Calculator на Qt в файле calculator.cpp есть функция unaryOperatorClicked().

В ней используется условный переход по if-else if. А вот скажите, возможно ли это как-то переделать, чтоб использовать switch? Проблема та же, что и в первом посте. Какие возможны пути решения данной задачки?

У меня была схожая проблема: при реализации сетевого протокола надо обрабатывать каждый управляющий код отдельно, поэтому в обычном варианте получается большой switch. Здесь можно попробовать использовать табличное отображение в моем случае код выглядел так:
объявление приватного класса и заполнение таблицы методов:

C++ (Qt)
class MServerConnectionPrivate
{
public:
    MServerConnectionPrivate(MServerConnection *value);
    MServer *server;
    quint32 clientId;
    bool sendGreeting();
    bool processPacket(MPacket &packet);
private:
    bool processIdChange(MPacket &packet);
    bool processServerMessage(MPacket &packet);
    bool processServerStatus(MPacket &packet);
    bool processServerIdent(MPacket &packet);
    bool processServerList(MPacket &packet);
    bool processSearchResult(MPacket &packet);
    bool processCallbackRequested(MPacket &packet);
    bool processCallbackFail(MPacket &packet);
    bool processReject(MPacket &packet);
    bool processFoundSources(MPacket &packet);
    bool processFoundSourcesObfuscated(MPacket &packet);
    typedef bool (MServerConnectionPrivate::*ProcessMethodPtr)(MPacket &packet);
    typedef QHash<quint8, ProcessMethodPtr> ProcessMethodsTable;
    ProcessMethodsTable table;
    MServerConnection *q;
};
#ifdef METHOD_TABLE_FILL
#error conflict in macro declaration
#endif 
#define METHOD_TABLE_FILL(_Opcode_) \
    table[Opcode::_Opcode_] = &MServerConnectionPrivate::process##_Opcode_
MServerConnectionPrivate::MServerConnectionPrivate(MServerConnection *value) : q(value), clientId(0)
{
    METHOD_TABLE_FILL(IdChange);
    METHOD_TABLE_FILL(ServerStatus);
    METHOD_TABLE_FILL(ServerMessage);
    METHOD_TABLE_FILL(ServerIdent);
    METHOD_TABLE_FILL(ServerList);
    METHOD_TABLE_FILL(SearchResult);
    METHOD_TABLE_FILL(CallbackRequested);
    METHOD_TABLE_FILL(CallbackFail);
    METHOD_TABLE_FILL(Reject);
    METHOD_TABLE_FILL(FoundSources);
    METHOD_TABLE_FILL(FoundSourcesObfuscated);
}
#undef METHOD_TABLE_FILL

применение:

C++ (Qt)
bool MServerConnectionPrivate::processPacket(MPacket &packet)
{
    ProcessMethodPtr method = table[packet.opcode()];
    if (!method) {
        qFatal("Method not found for opcode = %d", packet.opcode());//TODO:remove after implementing all
        return false;
    }
    qDebug("Found method to process packet, opcode = %d", packet.opcode());
    return (this->*method)(packet);
}

Нужно только определить qHash для типа по которому строится отображение. Хотя есть сомнение, возможно есть решение попроще.

А записать нужные строки в QStringList иль QMap, и делать свитч по индексам (ну или в этой области что-то) это не есть гуд?

... чето подумал, не..лучше так не делать  ;)

всплыла трабла....

в общем свитч нормально работал по enum'ам.
Но теперь значения забиты в конф-файл, и следовательно стали переменны.
Кроме как if... else... нельзя никак заменить систему?

Раньше abonent и phone были enom'ом, а теперь int'ы. Всё, теперь нельзя никак применять такую схему?

почему нельзя? приведи iType к своему перечислению и юзай свич на здоровье.
хотя и так работать должно.

как это привести int к enum'у? 0_o

ну во-первых ругань идет на rs, во-вторых приводится просто

C++ (Qt)
...
enum MyEnum { One = 1, Two, Three };
...
int iType = 3;
MyEnum t = MyEnum(iType);
 
switch (t) {
case One: {...}
            break;
case Two: {...}
            break;
case Three: {...}
            break;
default: {...}
            break;
};
 

покажи лучше код.

ЭЭ... rs это и есть k:

чуть забыл подправить код =)

Вот значения k и должны загружаться из конф-файла, а не браться из enuma. В инструкции case нужно вписывать константу. А у нас значение, вписываемое по ходу программы.

ну все верно компиль ругается, тогда.
в кейсах должны быть константные значения.
читай про switch (http://www.cyberguru.ru/programming/cpp/cpp-velvet-way-page61.html).

так я знаю =)

просто, думал, мож можно как-то обмануть. Просто кейсы удобнее, нежели if else =)

ну в данном случае ифы нужно юзать.

Интересно, а вопрошающие книжки по языку читать пробовали? :)

читали, и даже знаем, но просто проблему с "умными" идеями начальства все делать через кейсы никак им не объяснить, что нельзя. Но они хотят - и все.

=)