правильная ли реализация, консольная утилита по обработке сетевого файла Опции Подписка на тему Сообщить другу Версия для печати Скачать тему Подписка на этот форум Режимы отображения Древовидный Переключить на: Стандартный Переключить на: Линейный

[rich]

Задача:
Написать консольную утилиту которая получает файл по сети, обрабатывает его неким образом и завершается.
Реализация:
main.cpp:

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

#include <QCoreApplication>
#include <check.h>

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
    check *cck = new check();
    if (cck->can_check){
        cck->do_check();
    }
    QObject::connect(cck, SIGNAL(done()), &a, SLOT(quit()));
    return a.exec();
}

check.cpp:

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

#include "check.h"

check::check(QObject *parent) :
    QObject(parent)
    db = QSqlDatabase::addDatabase("AVAILABLE_DRIVER");
    // Далее коннект к базе
    can_check = db.open();
    if (!can_check) emit done();
}

void check::do_check(){
    fd = new FileDownloader(QUrl("URL_TO_FILE"));
    connect(fd, SIGNAL(downloaded()), SLOT(pars_file()));
}

void check::pars_file(){
    QString file_content = QString(fd->downloadedData());
    //  Анализ файла. Пишем результат в базу, не зря ж коннектились
}

нашёл где-то
filedownloader.cpp:

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

#include "filedownloader.h"

#include "filedownloader.h"

FileDownloader::FileDownloader(QUrl fileUrl, QObject *parent) :
    QObject(parent)
{
    connect(&m_WebCtrl, SIGNAL(finished(QNetworkReply*)),
                SLOT(fileDownloaded(QNetworkReply*)));
    //
    QNetworkRequest request(fileUrl);
    m_WebCtrl.get(request);
    //
}

void FileDownloader::fileDownloaded(QNetworkReply* pReply)
{
    m_DownloadedData = pReply->readAll();
    //emit a signal
    pReply->deleteLater();
    emit downloaded();
}

QByteArray FileDownloader::downloadedData() const
{
    return m_DownloadedData;
}

Так работает.
Не знаю что, но что-то мне не нравится.
А вам?

[lanz]

Из некритичного/отсутствующего.
1.

check *cck = new check();

cck не удаляется. Можно объявить локальной переменной(не указателем).
2. fd не удаляется (или не видно где удаляется)
3. "URL_TO_FILE" hardcoded.

Вообще мне все нравится, но я бы делал скачку файла отдельно, например через curl, а прога обрабатывала уже скачаный файл через pipe или еще как.