Ключевые слова Qt/C++, Где посмотреть? Опции

[ufna]

Считаю все равно философию goto устаревшей и слишком узкой )

[igor_bogomolov]

Считаю все равно философию goto устаревшей и слишком узкой )

Рискну выложить на суд общественности такой код (написан не мной, но в проекте используется). Считаю использование goto в данном случае оправданным

int HostGraph::createKey()
{
    int iKey=0;
    forever {
        m1:
        ++iKey;
        foreach(HostItem* na, m_lsHosts) {
            if (na->m_iSelfKey == iKey) goto m1;
        }
        foreach(FolderItem* fa, m_lsFolders) {
            if (fa->m_iSelfKey == iKey) goto m1;
        }
        return iKey;
    }
}

[Алексей1153]

не знаю, что делают forever и foreach

но ВСЕГДА можно обойтись без goto

если foreach - это перебор, то проще сделать

if(... find(...) )continue;

соответственно, forever - заменить на while(1){}

[kwisp]

igor_bogomolov,
я согласеня с Алексей1153,
m_lsHosts, m_lsFolders - рискну предположить, что контейнеры, тогда циклы

foreach(HostItem* na, m_lsHosts) {
            if (na->m_iSelfKey == iKey) goto m1;
}

foreach(FolderItem* fa, m_lsFolders) {
            if (fa->m_iSelfKey == iKey) goto m1;
}

ни к чему
тут goto даже всей своей сволочной натуры не показывает он заменяет continue и всё!

читал у Лафоре и еще где-то что оправдать использование goto можно лишь при большушем уровне вложенности если требуется выйти на сразу несколько уровней вверх. Сам не пробовал и не собираюсь такого городить

[Алексей1153]

если требуется выйти на сразу несколько уровней вверх. Сам не пробовал и не собираюсь такого городить

такое встречается редковато, но выход я находил в выделении кода в функцию, из которой можно выйти return. Все нужные параметры передаются по ссылке - даже думать практически не надо, как это сделать Заодно и автоматически разгружается загромождённый код и разделяются сущности

[igor_bogomolov]

тут goto даже всей своей сволочной натуры не показывает он заменяет continue и всё!

continue здесь не подойдёт

Алгоритм использует последовательный перебор целых чисел начиная с нуля и проверяет его уникальность последовательным сравнением ключей (na->m_iSelfKey, fa->m_iSelfKey) в контейнерах объектов хостов (m_lsHosts) и папок (m_lsFolders). Если число уникально, то алгоритм останавливается.

но ВСЕГДА можно обойтись без goto

С этим я не спорю. Я думал у вас разговор о том, когда использование goto может быть оправданным. В выше приведенном примере оно мне кажется оправданным

[Алексей1153]

В выше приведенном примере оно мне кажется оправданным

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

[kwisp]

continue здесь не подойдёт

подойдёт, буду спорить. если убрать цикл foreach с заменой на find всё подходит великолепно

какой тип контейнера используется?

Алгоритм использует последовательный перебор целых чисел начиная с нуля и проверяет его уникальность последовательным сравнением ключей (na->m_iSelfKey, fa->m_iSelfKey) в контейнерах объектов хостов (m_lsHosts) и папок (m_lsFolders). Если число уникально, то алгоритм останавливается.

т.е. если числа нет в обоих контейнерах то останавливается.

[Sokoloff]

Вот так и знал, что начнется спор про goto, самого подмывало написать, но удержался. У человека же после ваших споров в голове каша будет.

QMainWindow, большинство современных авторов считают что goto вредно, потому что с его помощью легко наделать логических ошибок в коде. Иногда в отдельных случаях его применение дает выгоду, но это скорее исключение из правила.
Поэтому на начальных этапах изучения языка считай что оператора goto нет. Со временем и опытом придет понимание где его можно использовать.

[igor_bogomolov]

подойдёт, буду спорить.

Я говорю про конкретную реализацию алгоритма, описание которого привел после. Там continue не подойдет

т.е. если числа нет в обоих контейнерах то останавливается.

Ага. А если есть хоть в одном, выходит по goto и начинает поиск для следущего числа

какой тип контейнера используется?

QList<HostItem*> m_lsHosts;

если убрать цикл foreach с заменой на find всё подходит великолепно

Контейнер хранит указатели, поэтому qFind не подойдет. Представь что ты не можешь изменить HostItem и FolderItem. И не можешь использовать find. Попробуй изменить алгоритм только внутри функции HostGraph::createKey(). Сможешь написать его короче и красивее? Я только это имел в виду

QMainWindow, большинство современных авторов считают что goto вредно, потому что с его помощью легко наделать логических ошибок в коде. Иногда в отдельных случаях его применение дает выгоду, но это скорее исключение из правила.
Поэтому на начальных этапах изучения языка считай что оператора goto нет. Со временем и опытом придет понимание где его можно использовать.

+500