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

[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]

Как правильно или где правильно удалять fd?

Используйте deleteLater. С обычным delete объект удаляется, а потом управление возвращается в fileDownloaded уже удаленного объекта.

Можно, конечно, скрестить ужа с ежом (bash с QT).

Не вижу в этом ничего предосудительного

но хочеться только средствами QT

Но зачем?

[Iron Bug]

Можно, конечно, скрестить ужа с ежом (bash с QT).

Не вижу в этом ничего предосудительного

но хочеться только средствами QT

Но зачем?

если кроссплатформа не нужна, то, возможно, и незачем. а так, баш есть далеко не на всех системах.

[rich]

Можно, конечно, скрестить ужа с ежом (bash с QT).

Не вижу в этом ничего предосудительного

но хочеться только средствами QT

Но зачем?

если кроссплатформа не нужна, то, возможно, и незачем. а так, баш есть далеко не на всех системах.

Как, у вас ещё нет баша?

Уменьшено до 61% 669 x 312 (4.2 килобайт)

[Iron Bug]

Как, у вас ещё нет баша?

юзеры выбирают то, что им удобнее и привычнее. они ради твоей поделки точно не будут ставить какой-нить cygwin или unix utils. они просто не станут пользоваться поделкой.
кроссплатформенность означает не только код, но и нативность в рамках той оси, на которой работает программа. использование родных интерфейсов и стандартных утилит.