Банальная программа. Но зависает., в чем же дело ? Опции

[kvak]

Здравствуйте. Не могли бы вы посмотреть на исходники во вложении и сказать что в них неправильно. И поправить. Там маленькая программка. Задача в том, чтобы открыть log.txt и отобрать из него вхождения для каждого случая, если они встретятся. Мне кажется, что по идее все должно работать, но что-то я недопонимаю. После компиляции почему-то не хочет выполнять цикл и приложение не отвечает.
Кстати, в с++ можно задавать в цикле for по два инициализатора и два инкримента, типа такого ?
for (int index = 0, int c =0 ; index != -1; index++, c++ )
{
...
}
Если да, то почему-то при компиляции это считалось ошибочным.

Прикрепленные файлы

 PACKAGE.zip ( 7,54 килобайт ) Кол-во скачиваний: 131

 

[Litkevich Yuriy]

Кстати, в с++ можно задавать в цикле for по два инициализатора и два инкримента, типа такого ?
for (int index = 0, int c =0 ; index != -1; index++, c++ )
{
...
}
Если да, то почему-то при компиляции это считалось ошибочным.

самому применять никогда не приходилось, но форма записи подобна тому что есть у Шилдта:

int i, j;
for (i = 0, j =0; i <= j; i++, j++ )
{
...
}

единственная разница - переменные у него объявлены до цикла

[azure]

for (int index = 0, int c =0 ; index != -1; index++, c++ )
{
...
}

Это не код а феерическая фигня. Откройте для себя break; Или объясните, для какой цели используется int index и почему не хватает одного int c.

[BRE]

kvak, Юра правильно подсказал, что определять переменные нужно до цикла, а вот инициализировать можно в нем.

Или объясните, для какой цели используется int index и почему не хватает одного int c.

Да это просто для примера, вопрос то был в другом.
Кстати, часто используемый прием, например:

    char in[ 100 ] = { "Test string" };
    char out[ 100 ] = { 0 };                                                                                                        

    char *pIn;
    char *pOut;
    for( pIn = in, pOut = out; *pIn; ++pIn, ++pOut )
        *pOut = *pIn;

И туда же еще переменную для проверки длины результирующего буфера можно добавить.

Сообщение отредактировал BRE - 5.5.2010, 12:42

[Litkevich Yuriy]

kvak, проблема тут:

            for (int index = 0; index != -1; index++ ) {
            qDebug() << "index =" << index;
            index = Engine.indexOf(vxod[a], index);
            summary[a] += c;
            c++;            
        }

ты управляющую переменную цикла (index) в теле цикла меняешь, это опасно. И в данном случае, скормив программе произвольный файл, она всегда равна НУЛЮ

[Litkevich Yuriy]

index != -1;

цикл прервётся только при не выполнении этого условия

Я сделал так:

Раскрывающийся текст

void MyProg::analize()
{
    QString fileName;
    fileName = fpath->text(); 
    QFile file(fileName);                                                
    if (!file.open(QIODevice::ReadOnly | QIODevice::Text)){
        QMessageBox::warning(this
                             ,tr("Vmaanalize")
                             ,tr("Cannot read file %1:\n%2.")
                                .arg(fileName)
                                .arg(file.errorString())
                            );
        return;
    }


//////////////////////////////start/////////////////////////
    const QString slesh = "/";
    const QString end = "\n";
    const QString equ = " = ";
    QString two;
    

    QTextStream out(stdout);
    
    const int size = vxod.size();     //число инструкций из базы данных
    int cnt = 0;
    while  (!file.atEnd()){
        QString line = file.readLine();
        cnt++;
        out << "[" << cnt << "] line = " << line;
        QApplication::processEvents();
        for (int a = 0; a < size; a++){
            if (line.contains(vxod[a])){
                out << " - OK";
                summary[a]++;
            }
        }
        out << endl << flush;
    }

    for (int a = 0; a < size; a++){ 
        QString result = (vxod[a]);
        two = two.setNum(summary[a]);
        result = result + equ + two + end;
        comit->insertPlainText(result);
    }    

    return;

}

нет смысла проверять всю строку, т.к. в асемблере одна строка - одна инструкция

А еслиб и было то можно было бы сделать так:

Раскрывающийся текст

for (int a = 0; a < size; a++){
    int i = line.count(vxod[a])
    if (i){
        out << " - OK";
        summary[a] += i;
    }
}

[kvak]

Спасибо всем за грамотные ответы. Теперь все встало на свои места для меня, новичка. Как же я не увидел, что есть функции contains и count. Но с другой стороны их тоже кто-то придумал. А подскажите еще, что дает эта функция:
QApplication::processEvents();
И это
out << " - OK";
Как потом можно полезно использовать объект out ?
И еще немного не понял куда попадает qDebug() << "index =" << index;
Я имею ввиду, где я смогу увидеть qDebug() и что такое qDebug() ?________________________________________________________________________________

p.s. А я все-таки заставил работать это дело сам Мож не идеально, но зато работает. 50.000 строк обработало за несколько секунд.

Раскрывающийся текст

void MyProg::analize()
{
QString fileName;
fileName = fpath->text();
QFile file(fileName);
if (!file.open(QIODevice::ReadOnly | QIODevice::Text)) {
QMessageBox::warning(this, tr("Vmaanalize"),
tr("Cannot read file %1:\n%2.")
.arg(fileName)
.arg(file.errorString()));
return;
}

///////////////////////////start parser///////////////////////////////

while (!file.atEnd()) {
QByteArray line = file.readLine();
start(line);
}
/////////////////////////////////////////////////////////////////
for (int a = 0; a < 40; a++)
{
QString two;
QString equ = " = ";
QString end = "\n";
QString result = (vxod[a]);
two.setNum(summary[a]);
result = result + equ + two + end;
comit->insertPlainText(result);
}
}
//////////////////////////////parser///////////////////////////////
void MyProg::start(QByteArray line)
{
QByteArray temp = line;
char *lima = temp.data();
int size = 0;
global = 0;
while (size < 40){
parser(lima);
global++;
size++;}
}

void MyProg::parser(char* mypointer)
{
QString temp;
QString temp2 = vxod[global];
int lee = temp2.length();
while (*mypointer)
{
temp.clear();
for (int length = 0; length < lee && *mypointer; length++) {
temp.append(*mypointer);
mypointer++; }
if (!*mypointer)
return;
mypointer -= (lee - 1);
int x = QString::compare(temp, temp2, Qt::CaseInsensitive);
if (x == 0)
summary[global]++;
}
}

[Litkevich Yuriy]

QApplication::processEvents();

обрабатывает события поступающие в программу из ОСьки (например перетаскивание окна). Такие вещи надо делать в длинных циклах. Иначе программа не будет реагировать на внешние события.

out << " - OK";

печатает в стандартный поток вывода строчку " - ОК"

Как потом можно полезно использовать объект out ?

ну как тебе вздумается так и используй

И еще немного не понял куда попадает qDebug() << "index =" << index;
Я имею ввиду, где я смогу увидеть qDebug() и что такое qDebug() ?

qDebug() - поток отладки, сделаный в Qt для удобства. Реально он печатет в стандартный поток ошибок. Но в него можно передавать не только строки, но и множество классов Qt. Например:

QPoint point(10, 20);
qDebug() << "point = " << point;

П.С.
Оформляй исходники в сообщениях так, как написано здесь

[kvak]

Пнятненько.
Так все-таки есть разница в этих примерах или нету ? Я нем могу привести примера, где бы была видна разница.

1.
for (int index = 0; index < 100; index++ ) 
2. 
int index;
for (index = 0; index < 100; index++ )

[BRE]

Здесь разница в том, что в первом случае переменная index после тела цикла видна (доступна) не будет, а во втором будет.