Проблема с консольным приложением, Какой-то неуловимый глюк, всю голову уже сломал... Опции

[x-8973]

Всем бобра!
Второй день бьюсь с консольным приложением, пытаясь найти и искоренить странный глюк. Приложение должно запросить у пользователя два имени файлов, обработать первый из них и записать результаты во второй. Проблема состоит в том, что сразу после ввода первого имени файла программа вываливается в критическую ошибку "память не может быть written". Если закомментировать весь код, оставив только ввод имен файлов, то прорабатывают оба ввода и прога останавливается. Ничего понять не могу. Может, здесь кто поможет?
Код main.cpp:

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

#include <QtCore/QCoreApplication>
#include "cut-off_proc.h"

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
    printf("Enter name of file 1: ");
    scanf("%s", ch1);
    fileName1 = QString::fromAscii(ch1);
    printf("Enter name of file 2: ");
    scanf("%s", ch2);
    fileName2 = QString::fromAscii(ch2);
    /*file1.setFileName(fileName1);
    if (file1.open(QIODevice::ReadOnly))
    {
        QTextStream t(&file1);
        while (!t.atEnd())
        {
            str = t.readLine();
            if ((str.section(" ", 0, 0).compare("#"))&&(!str.isEmpty()))
            {
                number = str.section(" ", 0, 0);
                strCount = str.section(" ", 1, 1).toInt(&ok);
                for (int i = 0; i < strCount; i++)
                {
                    str = t.readLine();
                    if (!str.compare("fail"))
                    {
                        list1.clear();
                        list2.append(number+" fail");
                        break;
                        //вернуться в начало цикла while
                    }
                    if (!str.compare("restart"))
                    {
                        list1.clear();
                        break;
                        //вернуться в начало цикла while
                    }
                    list1.append(str.section(" ", 1, 1).toInt(&ok));
                } //end of "for"
                if (list1.isEmpty())
                    continue;
                mainTimeMS = list1.at(list1.count()-1)-list1.at(0);
                //calculating of time mm:ss.ms
                int min=0, sec=0, msec=0;
                QString min_s, sec_s, msec_s;
                sec =  mainTimeMS / 1000;
                msec = mainTimeMS - sec * 1000;
                min = sec / 60;
                sec = sec - min * 60;
                if (min < 10)
                    min_s = "0";
                if (sec < 10)
                    sec_s = "0";
                if (msec < 10)
                    msec_s = "0";
                if (msec < 100)
                    msec_s.append("0");
                min_s.append(QString::number(min));
                sec_s.append(QString::number(sec));
                msec_s.append(QString::number(msec));
                mainTimeMSMS = min_s+":"+sec_s+"."+msec_s;
                //end of calculating
                list2.append(number+" "+mainTimeMSMS+" "+QString::number(mainTimeMS));
                list1.clear();
            } //end of "if"
        }
        file1.close();
    }
    else
        printf("CRITICAL ERROR: Couldn't open file %s", fileName1.toAscii().data());
    if (!list2.isEmpty())
    {
        file2.setFileName(fileName2);
        if (file2.open(QIODevice::Append))
        {
            QTextStream t2(&file2);
            for (int i = 0; i < list2.count(); i++)
            {
                printf("%s", list2.at(i).toAscii().data());
                t2<<list2.at(i)<<"\n";
            }
            file2.close();
        }
        else
            printf("CRITICAL ERROR: Couldn't open file %s", fileName2.toAscii().data());
    }*/
    return a.exec();
}

Код cut-off_proc.h:

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

#ifndef CUTOFF_PROC_H
#define CUTOFF_PROC_H

#endif // CUTOFF_PROC_H
#include <QFile>
#include <QTextStream>
#include <QString>
#include <QList>
#include <iostream>

QString str;
QString number, mainTimeMSMS;
int strCount, mainTimeMS;
bool ok;
QFile file1, file2;
QList<int> list1;
QList<QString> list2;
QString fileName1, fileName2;
char *ch1, *ch2;

Вот скрин ошибки:

Эскизы прикрепленных изображений

 

[Iron Bug]

char *ch1, *ch2;

 scanf("%s", ch1);

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

[x-8973]

char *ch1, *ch2;

 scanf("%s", ch1);

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

Вот этот код прекрасно работает:

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

#include <QtCore/QCoreApplication>
#include <QFile>
#include <QString>
#include <iostream>
#include <stdio.h>
#include <QTextStream>
using namespace std;

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
    QString filename;
    char *aa;
    printf("Enter filename: ");
    scanf("%s", aa);
    filename = QString::fromAscii(aa);
    QFile file;
    file.setFileName(filename);
    if (file.open(QIODevice::ReadOnly))
    {
        QTextStream t(&file);
        while (!t.atEnd())
        {
            printf("%s\n", t.readLine(0).toAscii().data());
        }
    }
    printf("%s", filename.toAscii().data());
    return a.exec();
}

Ладно, согласен, тут косяк. Но я же не знаю, какой длины имя файла введет пользователь. Соответственно, я не знаю, сколько памяти выделять.

Да, проблема оказалась в этом... Огромная глупость с моей стороне, это ведь еще на первом курсе проходят...
Еще один глупый вопрос: а сколько же все-таки нужно выделять памяти под имя файла? В Дельфи есть константа MAX_LEN, если мне память не изменяет. Может, и в QT что-то похожее есть?

[lanz]

printf ("Enter the length of file name:");

А если серьезно то лучше использовать QTextStream:

QTextStream str (stdin);

fileName1 = str.readLine ();

[Wlad]

В Дельфи есть константа MAX_LEN, если мне память не изменяет. Может, и в QT что-то похожее есть?

Qt здесь не причем. Есть виндовая переменная MAX_PATH (определена в файле windef.h) равная 260 символам. Это и есть максимальная длина файла с путем к нему. Виндовые функции, работающие с файлами проверяют имя файла на эту длину Это ограничение можно объехать, но это другой вопрос.

[Iron Bug]

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

[x-8973]

lanz, благодарствую)
Еще вопрос. Подумал, что раз уж юзать потоки, то юзать их и на выводе данных. Написал

QTextStream qcout(stdout);
qcout << "Enter name of file 1: ";

При запуске ничего не вывелось.
В принципе, не критично, printf в этом плане отлично справляется) Просто интересно, можно ли поток и для вывода данных использовать)

[Алексей1153]

Вот этот код прекрасно работает:

повезло )))

Но я же не знаю, какой длины имя файла введет пользователь. Соответственно, я не знаю, сколько памяти выделять.

это в общем случае вполне решабельно: считывать по 1 символу и накапливать в буфере

А инициализировать переменные всё равно нужно

[lanz]

QTextStream outstr (stdout);
outstr << "Hello";
outstr.flush ();

Буфер приходится чистить вручную.