How to make QByteArray read Full Text file?
-
@ambershark , and @JonB ,thanks sooo much!!! You've been a great help. Yup you were totally right about strings and binary issues. I forgot in my header file, I was using a QHash<QString, int> keywordLoadListOptions, with QString as a type. This is where keywordLoadListOptions come into play. I changed my code to use QTextStream to read the file using file.readLine(). I then used processEvents() inside the while loop to prohibit my GUI from freezing. Some where along the lines my QString within my Qlist keywordLoadListOptions was ending my string. Below is what is currently working. I've heard good and bad things about using processEvents(), so I will keep testing.
QString fileName = QFileDialog::getOpenFileName(this, "Open text file", ""); QFile file(fileName); QFileInfo fi(file.fileName()); QString fileExt = fi.completeSuffix();; QString strings; QString str; if (!file.open(QIODevice::ReadOnly)) { //QMessageBox::warning(this,"...","error in opening keyword file"); return; } QTextStream ts(&file); while (!ts.atEnd()) { QApplication::processEvents(); str = ts.readLine(); *fileList << str; } for (int row = 0; row < fileList->size(); row++) { for (int col = 0; col < 2; col++) { if (col == 0) { ui->tableWidget_Keywords_Queue->setItem(row, col, new QTableWidgetItem(fileList->at(row))); } if (col == 1) { ui->tableWidget_Keywords_Queue->setItem(row, col, new QTableWidgetItem("")); } } } qDebug() << fileList->size(); file.close();hi @tshoats,
processEvents()is a crutch, you should avoid it.if reading your file lasts more then a few milliseconds, you have two options:
- use a QTimer with small timeout and read some lines from the file in the slot. when the timer fires again, read the next lines.
- put the whole file reading in a thread
both will keep your app responsive and you can even have a progress bar.
just my two cent
regards -
@ambershark , and @JonB ,thanks sooo much!!! You've been a great help. Yup you were totally right about strings and binary issues. I forgot in my header file, I was using a QHash<QString, int> keywordLoadListOptions, with QString as a type. This is where keywordLoadListOptions come into play. I changed my code to use QTextStream to read the file using file.readLine(). I then used processEvents() inside the while loop to prohibit my GUI from freezing. Some where along the lines my QString within my Qlist keywordLoadListOptions was ending my string. Below is what is currently working. I've heard good and bad things about using processEvents(), so I will keep testing.
QString fileName = QFileDialog::getOpenFileName(this, "Open text file", ""); QFile file(fileName); QFileInfo fi(file.fileName()); QString fileExt = fi.completeSuffix();; QString strings; QString str; if (!file.open(QIODevice::ReadOnly)) { //QMessageBox::warning(this,"...","error in opening keyword file"); return; } QTextStream ts(&file); while (!ts.atEnd()) { QApplication::processEvents(); str = ts.readLine(); *fileList << str; } for (int row = 0; row < fileList->size(); row++) { for (int col = 0; col < 2; col++) { if (col == 0) { ui->tableWidget_Keywords_Queue->setItem(row, col, new QTableWidgetItem(fileList->at(row))); } if (col == 1) { ui->tableWidget_Keywords_Queue->setItem(row, col, new QTableWidgetItem("")); } } } qDebug() << fileList->size(); file.close();@tshoats
@ambershark is a touch harsh onprocessEvents():) It's lazy and has issues but it's often employed!If you do stick with it, you are calling it for each line read. It needs to be called, but not that often, and it has a performance penalty. Put in a counter and call it every 10 or 100 lines read, enough so that you retain responsiveness but not too often.
If all you are doing with the lines is populating some data structure, and not directly doing anything to the GUI (e.g. not putting them into a widget, you just store them in your read loop and only later put them into your table), @ambershark's suggestion of read in its own thread is not hard to implement and is neatest.
-
hi @tshoats,
processEvents()is a crutch, you should avoid it.if reading your file lasts more then a few milliseconds, you have two options:
- use a QTimer with small timeout and read some lines from the file in the slot. when the timer fires again, read the next lines.
- put the whole file reading in a thread
both will keep your app responsive and you can even have a progress bar.
just my two cent
regards@aha_1980 said in How to make QByteArray read Full Text file?:
processEvents() is a crutch, you should avoid it.
would you be surprised, if I told you, that using processEvents() is an advice I once got from the Qt support team?
-
I still think if you have a better solution, you should avoid it.
you may have good reasons to use it anyway, but you should know what you are doing. therefore I wouldn't recommend it in the forum or elsewhere.
Qt has to stay free or it will die.
-
I still think if you have a better solution, you should avoid it.
you may have good reasons to use it anyway, but you should know what you are doing. therefore I wouldn't recommend it in the forum or elsewhere.
-
@tshoats
@ambershark is a touch harsh onprocessEvents():) It's lazy and has issues but it's often employed!If you do stick with it, you are calling it for each line read. It needs to be called, but not that often, and it has a performance penalty. Put in a counter and call it every 10 or 100 lines read, enough so that you retain responsiveness but not too often.
If all you are doing with the lines is populating some data structure, and not directly doing anything to the GUI (e.g. not putting them into a widget, you just store them in your read loop and only later put them into your table), @ambershark's suggestion of read in its own thread is not hard to implement and is neatest.
@JonB said in How to make QByteArray read Full Text file?:
@tshoats
@ambershark is a touch harsh on processEvents() :) It's lazy and has issues but it's often employed!Well to be fair that wasn't me that said that. ;) However harsh or not I do agree with @aha_1980. I think if it lasts long enough to freeze your GUI, you are much better offloading it to a thread.
Qt makes this super easy with
QObject::moveToThread().I have nothing against processEvents, but I wouldn't really use it personally. :)
My L-GPL'd C++ Logger github.com/ambershark-mike/sharklog
-
@JonB said in How to make QByteArray read Full Text file?:
@tshoats
@ambershark is a touch harsh on processEvents() :) It's lazy and has issues but it's often employed!Well to be fair that wasn't me that said that. ;) However harsh or not I do agree with @aha_1980. I think if it lasts long enough to freeze your GUI, you are much better offloading it to a thread.
Qt makes this super easy with
QObject::moveToThread().I have nothing against processEvents, but I wouldn't really use it personally. :)
@ambershark
Whoops, sorry, it was @aha_1980. It's your faults for having the same starting character in your name when this forum auto-completes ;-) -
@ambershark
Whoops, sorry, it was @aha_1980. It's your faults for having the same starting character in your name when this forum auto-completes ;-)@JonB Lol totally, I take full responsibility for that and apologize. ;)
-
Java has Files.readFileContent() and even Files.readAllLines(), similar .NET has File.ReadAllText(). Why does the QT/CPP doesn't provide a simple way to do this?
@stokito said in How to make QByteArray read Full Text file?:
Java has Files.readFileContent() and even Files.readAllLines(), similar .NET has File.ReadAllText(). Why does the QT/CPP doesn't provide a simple way to do this?
Hi,
It has: QFile::readAll.
