[c++][qt] Как уйти от if else?

учи про std::map( вероятно в Qt - QMap ) + указатели на функции/методы

П.С. а за

...
else if(e.attribute(«get») == «notification_number»
...
else if(e.attribute(«get») == «assured_address_index»)
...

надо руки отрывать

Это даже неэффективно. Можно все свести к отображению (map) строки на функцию двух аргументов (fontFamily, fontSize). Сначала где-то создаем map, а потом проверяем, нет ли нужной функции обработки в этом map по заданному атрибуту. Будет работать, если все можно свести к общему набору аргументов (как в твоем примере).

Да, аргументов может быть больше. Е тоже напрашивается.

OH SHI~~~~~.
мсье не в курсе про switch()/case ?

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

> мсье не в курсе про switch()/case ?

Со строками? Разве что через какой-нибудь qHash(const QString&).

Осиль фабрику и полиморфизм.

http://antiifcampaign.com

switch?

== перегружен, надеюсь? Или ты просто так строки сравниваешь?

>== перегружен, надеюсь? а разве он не определён в std::string?

Как самый дебильный вариант, ну кроме свитча на строки.

int index = getStatement()->metaObject()->indexOfMethod(QMetaObject::normalizedSignature("get"+e.attribute("get")+"()"));
QMetaMethod method = getStatement()->metaObject()->method(index);
drawElement(e, method.invoke(getStatement()), fontFamily, fontSize);

сумасшедшие вы все тут

QMap + boost::function

А то. :)

Пусть будет хоть какое-то использование меты.
А тащить буст ради свитча сомнительное удовольствие. :)

А где указано, что это std::string?

В qtassistante

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

QMap + boost::function

Таскать с собой помимо тяжелой кути еще и тяжелый буст? Увольте

s/возможный/возможные

На самом деле, не принципиально какую мапу.

>учи про std::map( вероятно в Qt - QMap ) + указатели на функции/методы

спасибо, подохдит!

Меня как-то просили как раз избавиться от свича на несколько экранов)

Предыдущие ораторы насоветовали фигни.

Правильно так: всякие statement_serial сделай свойствами statement'а. Для этого в определении класса Statement пишешь такое

Q_PROPERTY(ТИП statement_serial READ getStatementSerial WRITE setStatementSerial);

Часть WRITE setStatementSerial можно и пропустить, если свойство предназначено только для чтения.

После этого твоя гроздь if'ов превращается
const QString prop = e.attribute(«get»);
drawElement(e, getStatement()->property(prop), fontFamily, fontSize);

Кроме того, при таком подходе даром получаешь интеграцию своих Statement'ов с QtScript и QtWebKit в том смысле, что нужные свойства можно будет дергать из javascript'а.

Switch case же. Я аж чуть чаем не подавился...

>Я аж чуть чаем не подавился...

Я тоже

а потом будут говорить, что функциональные языки не нужны.

> Таскать с собой помимо тяжелой кути еще и тяжелый буст? Увольте

boost::function тяжел?

Не тяжёл, но здесь ненужен.

#include <string>
#include <cstdlib>
#include <map>
#include <iostream>
using std::string;

/* начало эмуляции API ======================================= */

int myrand(int from, int to)
{
    return (float(rand())/RAND_MAX)*(to-from)+from;
}
#define LEN(x) (sizeof(x)/sizeof(*x))

class SomeData {};

class Statement
{
public:
    SomeData getStatementSerial() { return SomeData(); }
    SomeData getStatementNumber() { return SomeData(); }
    SomeData getNotificationSerial() { return SomeData(); }
    SomeData getNotificationNumber() { return SomeData(); }
    SomeData getAssuredFio() { return SomeData(); }
    SomeData getAssuredAddressIndex() { return SomeData(); }
    SomeData getAssuredAddress() { return SomeData(); }
    SomeData getAssuredPhone() { return SomeData(); }
    SomeData getApartmentAddressIndex() { return SomeData(); }
    SomeData getApartmentAddress() { return SomeData(); }
};

Statement * getStatement() { static Statement s; return &s;} 

class SomeE
{
    static char const * const ATTRIBUTES[];
public:
    string attribute(const string & which);
};

char const * const SomeE::ATTRIBUTES[] = {
    "statement_serial",
    "statement_number",
    "notification_serial",
    "notification_number",
    "assured_fio",
    "assured_address_index",
    "assured_address",
    "assured_phone",
    "apartment_address_index",
    "apartment_address",
};

string SomeE::attribute(const string & )
{
    return ATTRIBUTES[myrand(0,LEN(ATTRIBUTES))];
}

class SomeFontFamily {};
class SomeFontSize {};
SomeE e;
SomeFontFamily fontFamily;
SomeFontSize fontSize;

void drawElement(SomeE, SomeData, SomeFontFamily, SomeFontSize) {}

/* конец эмуляции API ======================================= */



int main()
{
    // составляем карту:
    typedef SomeData (Statement::*attrib_getter)();
    typedef std::pair <char const * const, attrib_getter> ATTR_DEF;
    ATTR_DEF ATTRIBUTES[] = {
    //можно убрать эти "ATTR_DEF" и и заменить круглые скобки на фигурные, но ворнинги появляются
        ATTR_DEF("statement_serial",        &Statement::getStatementSerial        ),
        ATTR_DEF("statement_number",        &Statement::getStatementNumber        ),
        ATTR_DEF("notification_serial",        &Statement::getNotificationSerial    ),
        ATTR_DEF("notification_number",        &Statement::getNotificationNumber    ),
        ATTR_DEF("assured_fio",            &Statement::getAssuredFio        ),
        ATTR_DEF("assured_address_index",    &Statement::getAssuredAddressIndex    ),
        ATTR_DEF("assured_address",        &Statement::getAssuredAddress        ),
        ATTR_DEF("assured_phone",        &Statement::getAssuredPhone        ),
        ATTR_DEF("apartment_address_index",    &Statement::getApartmentAddressIndex    ),
        ATTR_DEF("apartment_address",        &Statement::getApartmentAddress        ),
    };
    //инициализируем карту массивом пар:
    std::map<string, attrib_getter> ATTR_MAP(ATTRIBUTES,ATTRIBUTES+LEN(ATTRIBUTES));
    //используем:
    string attribute = e.attribute("get");
    attrib_getter getter = ATTR_MAP[attribute];
    SomeData statement = (getStatement()->*getter)();
    drawElement(e, statement, fontFamily, fontSize);
    std::cout << attribute << std::endl;
    //то-же самое одной строкой:
    drawElement(e, (getStatement()->*ATTR_MAP[e.attribute("get")])(), fontFamily, fontSize);
}

Максров. фи

> (getStatement()->*ATTR_MAP[«garbage»])()

BOOM

Какой только херотой не страдают когда нормальных макросов нет.
Словари! С указатели на функции!! Иерархии классов!

А потом еще возникают вопросы, че ж это все программы сложнее хелловордов, написанные на плюсах, так тормозят, несмотря на охрененные оптимизирующие компиляторы?

проходи мимо - тут еды нет

*с указателями на функции

А что, на питоне они летают?

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

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

State machines или конечные автоматы ещё никто не советовал?
На QML или Ragel, bison...

>Максров Что?

PS аноним прав, надо заассертить getter.

На практике, эти свитчи часто плодятся. Поддерживать уже пяток свичей - совершенно не интересно.

> словарь с коллбэками для замены фактически статического свитча это трындец. При чем тут питон?

Статический switch быстро делается только на целочисленных переменных. Или есть в этом сомнения?