Access to the user interface

[leon2225]

No that problem is fixed.
Also the debugger doesn't say anything, the only problem is, that the function openContent() doesn't write the data into itsContent ..

@bool Data::openContent(){
QString fileName = QFileDialog::getOpenFileName(MyFxhnn, tr("Inhalt öffnen"), QString(),
tr("Fxhnn Content Files (*.FXHNNCON)"));
QFile file(fileName);
if(!file.open(QIODevice::ReadOnly)){
QMessageBox::critical(MyFxhnn, tr("Error"), tr("Could not open file"));
return false;
}
QTextStream in(&file);
in >> itsContent;
printItsContent();
file.close();

    if(itsContent != MyFxhnn -> ui -> itsText -> toPlainText()){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Fehler beim Öffnen, Data != Textfeld"));
        return false;
    }



return true;

}@

Leon

[SGaist]

From your code, I wouldn't expect openContent to write anything. You open the selected file in ReadOnly mode and you read from it after that.

[leon2225]

yea I meant that the function has to "write" the data from the stream into the variable ... but I don't know, where there should be the bug :/

[SGaist]

Ok, so you mean that "itsContent" doesn't receive the content of the file ?

Following your code printItsContent just clears the itsText widget content so when you compare the content of itsText to itsContent's value, your test will only succeed if itsContent is empty.

[leon2225]

I've changed that ... a few days ago :D
I'll post the new code .. :
data.h
@#ifndef DATA_H
#define DATA_H

#include <QWidget>
#include "fxhnn.h"
#include "ui_fxhnn.h"
#include <cstdlib>
#include <limits.h>
#include <QString>
#include <QFileDialog>
#include <QFile>
#include <QMessageBox>
#include <QTextStream>
#include <QDataStream>

class Fxhnn;

class Data : public QWidget
{
Q_OBJECT

Fxhnn *MyFxhnn;

public:
explicit Data(Fxhnn*);
~Data();

    //Zugriffe auf den Inhalt
QString getItsContent();
void printItsContent();
void setItsContent( QString newContent);
void readItsContent();
void reset();


    //Speichern der Daten
bool saveKeyAndMeta(bool customFileName = 0);
bool openKeyAndMeta(bool customFileName = 0);
bool saveContent(bool customFileName = 0);
bool openContent(bool customFileName = 0);
bool saveAll(bool customFileName = 0);
bool openAll(bool customFileName = 0);

signals:

private:
qint64 itsKey;
QString itsContent;
ushort itsShuffleLevel;
ushort itsEncryptLevel;
QString itsKeyFilename;
QString itsContentFilename;
};

#endif // DATA_H
@

data.cpp:
@#include "data.h"

Data::Data(Fxhnn * pFxhnn) : MyFxhnn(pFxhnn),itsEncryptLevel(0), itsShuffleLevel(0)
{
itsKey = rand() % LONG_MAX;
setItsEncryptLevel(0);
}

Data::~Data()
{

}

//Zugriffe auf itsContent

void Data::printItsContent(){
MyFxhnn -> ui -> itsText->clear();
MyFxhnn -> ui -> itsText -> setPlainText(itsContent);
setItsEncryptLevel(0);
setItsShuffleLevel(0);
return;
}

void Data::setItsContent(QString newContent){
itsContent = newContent;
return;
}

void Data::readItsContent(){
itsContent = MyFxhnn -> ui -> itsText -> toPlainText();
return;
}

//Zugriffe auf itsKey

qint64 Data::getItsKey(){
return itsKey;
}

void Data::setItsKey(qint64 newKey){
itsKey = newKey;
return;
}

void Data::generateItsKey(){
itsKey = rand() % USHRT_MAX;
return;
}

//Zugriffe auf das Level der Verschlüsslung
void Data::setItsEncryptLevel( short upDownGrade){
itsEncryptLevel = itsEncryptLevel + upDownGrade;

if(itsEncryptLevel <= 0){
    MyFxhnn -> ui -> itsIsEncrypted -> setText("Decrypted");
    MyFxhnn -> ui -> itsIsEncrypted -> setStyleSheet("QLabel {color : #CC0304; }");
}
else{
    if(itsEncryptLevel > 0){
        QString Out = "Encrypt Level ";
        Out.append(QString("%1").arg(itsEncryptLevel));
        MyFxhnn -> ui-> itsIsEncrypted-> setText(Out);
    }
    else
        MyFxhnn -> ui -> itsIsEncrypted -> setText("Encrypted");
    MyFxhnn -> ui -> itsIsEncrypted -> setStyleSheet("QLabel {color : #4ECC39; }");
}

}

ushort Data::getItsEncryptLevel(){
return itsEncryptLevel;
}

//Daten Speichern und öffnen

bool Data::saveKeyAndMeta(bool customFileName){
if(itsKeyFilename.isEmpty() || customFileName)
itsKeyFilename = QFileDialog::getSaveFileName(MyFxhnn, tr("Schlüssel Speichern"), QString(), tr("Fxhnn Key Files (*.FXHNNKEY)"));
if(!itsKeyFilename.isEmpty()){
QFile file(itsKeyFilename);
if(!file.open(QIODevice::WriteOnly)){
QMessageBox::critical(0, tr("Error"), tr("Could not save key"));
return false;
}
QDataStream stream(&file);
stream << itsKey << itsEncryptLevel << itsShuffleLevel;
file.close();
}
return true;
}

bool Data::saveContent(bool customFileName){
if(itsContentFilename.isEmpty() || customFileName)
itsContentFilename = QFileDialog::getSaveFileName(0, tr("Text Speichern"), QString(), tr("Fxhnn Content Files (*.txt)"));

if(!itsContentFilename.isEmpty()){
    QFile file(itsContentFilename);
    if(!file.open(QIODevice::WriteOnly)){
        QMessageBox::critical(0, tr("Error"), tr("Could not save file"));
        return false;
    }
    QTextStream stream(&file);
    stream << itsContent;
    file.close();
}
return true;

}

bool Data::saveAll(bool customFileName){
bool content = saveContent(customFileName);
bool keyAndMeta = saveKeyAndMeta(customFileName);

if(content && keyAndMeta)
    return true;

return false;

}

bool Data::openKeyAndMeta( bool customFileName){
if(itsKeyFilename.isEmpty() || customFileName )
itsKeyFilename = QFileDialog::getOpenFileName(0, tr("Schlüssel Öffnen"), QString(), tr("Fxhnn Key Files (*.FXHNNKEY)"));
if(!itsKeyFilename.isEmpty()){
QFile file(itsKeyFilename);
if(!file.open(QIODevice::ReadOnly)){
QMessageBox::critical(0, tr("Error"), tr("Could not open file"));
return false;
}
QDataStream in(&file);
in >> itsKey >> itsEncryptLevel >> itsShuffleLevel;
file.close();
printItsContent();
}
return true;
}

bool Data::openContent(bool customFileName){
if(itsContentFilename.isEmpty() || customFileName)
itsContentFilename= QFileDialog::getOpenFileName(MyFxhnn, tr("Inhalt öffnen"), QString(), tr("Fxhnn Content Files (*.txt)"));

if(!itsContentFilename.isEmpty()){
    QFile file(itsContentFilename);
    if(!file.open(QIODevice::ReadOnly)){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Could not open file"));
        return false;
    }
    QTextStream in(&file);
    setItsContent(in.readAll());
    printItsContent();
    file.close();

    if((itsContent != MyFxhnn -> ui -> itsText -> toPlainText()) || itsContent == ""){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Fehler beim Öffnen, Data != Textfeld"));
        return false;
    }
}


return true;

}

bool Data::openAll(bool customFileName){
bool content = openContent(customFileName);
bool keyAndMeta = openKeyAndMeta(customFileName);

if(content && keyAndMeta)
    return true;

return false;

}
@

[SGaist]

So is it working as you expect it to ?

[leon2225]

No it isn't ... it still have the problem that it couldn't read the data from the files.

[SGaist]

Rather than
@
QTextStream in(&file);
setItsContent(in.readAll());@

Why not just:
@
setItsContent(file.readAll());
@

In any case, are you sure that your file is not empty ?

[leon2225]

That doesn't work... And the file isn't empty

[SGaist]

Did you check what file.readAll() contains ?

since setItsContent only assign the parameter value to your itsContent member variable why not just do:

@itsContent = file.readAll()@

?

By the way, you should use const QString& in your setItsContent function, that will avoid unnecessary copies.

[leon2225]

Okey ehm file.readAll() return the right content of the file, the problem is, that it doesn't convert it into QString. readAll() gives back a QByteArray wich normaly could be converted in QString with for example:
@itsContent = QString(file.readAll());@

But it doesn't ...

here is the function again:
@bool Data::openContent(bool customFileName){
if(itsContentFilename.isEmpty() || customFileName)
itsContentFilename= QFileDialog::getOpenFileName(MyFxhnn, tr("Inhalt öffnen"), QString(), tr("Fxhnn Content Files (*.txt)"));

if(!itsContentFilename.isEmpty()){
    QFile file(itsContentFilename);
    if(!file.open(QIODevice::ReadOnly)){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Could not open file"));
        return false;
    }
    QTextStream in(&file);
    QMessageBox::information(MyFxhnn, "Content of readAll(", file.readAll());
    itsContent = QString(file.readAll());
    QMessageBox::information(MyFxhnn, "Content of itsContent", QString(file.readAll()));
    printItsContent();
    file.close();

    if((itsContent != MyFxhnn -> ui -> itsText -> toPlainText()) || itsContent == ""){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Fehler beim Öffnen, Data != Textfeld"));
        return false;
    }
}


return true;

}@

The Message "Content of readAll()" gives back the right content the other message return nothing an the last error message appears.

[leon2225]

Now it's working a 'bit'.
It is able to read the file and print it into the output, BUT it doesn't save onicode ... and I need unicode so we have to find an other solution :/ :D

[leon2225]

Now it's working a 'bit'.
It is able to read the file and print it into the output, BUT it doesn't save onicode ... and I need unicode so we have to find an other solution :/ :D
@bool Data::openContent(bool customFileName){
if(itsContentFilename.isEmpty() || customFileName)
itsContentFilename= QFileDialog::getOpenFileName(MyFxhnn, tr("Inhalt öffnen"), QString(), tr("Fxhnn Content Files (*.txt)"));

if(!itsContentFilename.isEmpty()){
    QFile file(itsContentFilename);
    if(!file.open(QIODevice::ReadOnly)){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Could not open file"));
        return false;
    }
    //QTextStream in(&file);
    //QMessageBox::information(MyFxhnn, "Inhalt von readAll()", file.readAll());
    QByteArray tempString = file.readAll();
    itsContent = tempString;
    QMessageBox::information(MyFxhnn, "Inhalt von itsContent", itsContent);
    printItsContent();
    file.close();

    if((itsContent != MyFxhnn -> ui -> itsText -> toPlainText()) || itsContent == ""){
        QMessageBox::critical(MyFxhnn, tr("Error"), tr("Fehler beim Öffnen, Data != Textfeld"));
        return false;
    }
}


return true;

}@

and here are the printItsContent():
@void Data::printItsContent(){
//MyFxhnn -> ui -> itsText->clear(); <- If I uncomment this line it clear the output text, but it can't set it again
MyFxhnn -> ui -> itsText -> setPlainText(itsContent);
setItsEncryptLevel(0);
setItsShuffleLevel(0);
return;
}@

[SGaist]

That's the kind of detail that's good to share.

Then it seems that you should use QTextCodec

[leon2225]

OK!
now it works, but only with textstream, but that's enough.

Thank you!! :)

[SGaist]

What do you mean by "only with textstream" ?