[РЕШЕНО] константная ссылка в foreach

[deMax]

мне нравится foreach красотой и компактностью, с другой стороны зачем он возвращает ссылку константу?

Код:

    QList<int> ls; ls<<1<<2<<3;
    foreach(const int &item, ls)
        *const_cast<int*>(&item) += 3; // или более жестоко *(int*)&item += 3;
    qDebug()<<ls;

Как то можно сделать подобный код красивее? Без const_cast

[__Heaven__]

c++11

Код

C++ (Qt)
for(int &item: ls){
    item += 3;
}

[deMax]

Спасибо.
А что скажете про C++11? стоит его включить для Qt5, проблем не возникнет?

[__Heaven__]

У меня пока не возникало

[Alex Custov]

использовать const_cast и модифицировать const данные - это UB, насколько я помню. Решение - использовать сишный "итератор" по индексам, или C++11, как написал __Heaven__

[deMax]

Итератор слишком громоздкая запись(для задачи пройтись по всем элементам он точно не нужен).
ub не будет, потому что ссылка не обязана быть константой, а foreach так требует, хотя это неверно.

[__Heaven__]

Ну вам в стандарте 0х всё-таки единственный путь - итераторы.
что такое ub?

[deMax]

ub - Undefined behavior(Неопределённое поведение). В данном случае все будет нормально, даже в бусте он не такой кривой:

Код:

QList<QString> a; BOOST_FOREACH(QString& s, a) s += "s";

[__Heaven__]

Undefined behavior

Тьфу, блин. University at Buffalo

Так используйте тот, что в бусте. Ну или свой определите.

[deMax]

так правильнее:

Код:

    QList<int> ls; ls<<1<<2<<3;
    foreach(const int &item, ls) {
        int &item2 = const_cast<int&>(item); // так, если много раз нужен
        item2 += 2; 
        const_cast<int&>(item) += 3; } // или так

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

p.s. с++11 хорош, надо будет как нибудь перейти.

[__Heaven__]

deMax, уж лучше сделайте на итераторах. Точно наступите на грабли когда-нибудь с const_cast.

[kambala]

так правильнее:

Код:

    QList<int> ls; ls<<1<<2<<3;
    foreach(const int &item, ls) {
        int &item2 = const_cast<int&>(item); // так, если много раз нужен
        item2 += 2; 
        const_cast<int&>(item) += 3; } // или так

то есть это лучше, чем использовать индексы? ок.

[ssoft]

Реализация foreach в Qt несколько криворукая и имеет описанные ограничения, часто уступает в производительности простому for через итераторы или индексы.
Вот в boost реализация оказалась отличной и без описанных ограничений. Но не тащить же еще и boost 

Такое кастование - это зло, которое нужно срочно искоренить из проекта!
К слову из-за того, что реализация QList implicit shared, таким кодом вы корректируете все взаимосвязанные с данным списком данные.

Если есть необходимость изменения данных в массиве QList, то необходимо использовать либо индексы, либо итераторы, либо C11, либо еще что-то, но никогда кастование!

[deMax]

Я где то тесты видел что foreach не уступает в производительности итераторам, да и не такие большие вычисления у меня в foreach.

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

>> К слову из-за того, что реализация QList implicit shared, таким кодом вы корректируете все взаимосвязанные с данным списком данные.
В моем случае будет нормально, хотя спасибо что предупредили. Наверно буду на с++11 переходить.

p.s. только что создал вектор и список на 1E7 элементов, заполнив единицами, int с=1 изначально

Код:

    foreach(const int &i, list1) { count*=i; }

Код:

    QVector<int>::Iterator it1;    for (it1 = list1.begin(); it1 != list1.end(); ++it1)        count*=*it1;

результат: для foreach (вектор 70мс список 200) итераторы (вектор 570 список 470) в режиме отладки
в режиме выпуска все примерно равны(добавил for из с++11):
vector (foreach 150 / for 150 / iterator 190) list (foreach  200 / for 150 / iterator 170) время в мс из QTime::elapsed()

[Old]

Вы ограничены в выборе компилятора? Если нет, то используйте c++11. Для чего использовать конструкцию, которая пытается имитировть for, если есть сам for?