Zufällige Zeichen angezeigt, beim drucken array von Zeichen

Dies ist mein problem. Ich lese eine Datei im binary-Modus anfügen von bytes zu einem int-array und Druck-die Werte nach. Mein problem ist, wenn ich cout meine Ergebnisse, zufällige Zeichen sind befestigt, in den Strom.

comp.txt:

this text is a testt1

main.cpp:

#include <iostream>
#include <fstream>
#include <time.h>


using namespace std;

void read(ifstream& stream, unsigned int buf[], int size)
{
    for(int i = 0; i < size; ++i)
    {
        unsigned char temp[4] = {'\0', '\0', '\0', '\0'};
        stream.read((char*)temp, 4);
        cout << "Temp: " << temp << '\n';
        buf[i] = *((int*)temp);     
        cout << "Read: " << buf[i] << endl;
        memset(temp, '\0', 4);
    }
}

int main()
{

    //open file
    ifstream f;
    f.open("comp.txt", ios::binary);
    cout << "File opened. " << endl;

    //get size
    f.seekg(0, ios::end);
    int l = f.tellg();
    int length = (l / 4) + 1;
    f.seekg(0, ios::beg);
    cout << "Size found: L: " << l << " Length: " << length << endl;

    //allocate byte arrays
    unsigned int* buf = new unsigned int[length];
    memset(buf, '\0', 4*length);
    //unsigned short* key = new unsigned short[length];
    cout << "Preparing to read..." << endl;

    //read byte into short
    cout << "Reading..." << endl;
    read(f, buf, length);
    f.close();
    delete[] buf;
    cin.ignore(1000, 10);
    return 0;
}

Ausgabe:

C:\Users\daminkz\Desktop>encrypt
File opened.
Size found: L: 21 Length: 6
Preparing to read...
Reading...
Temp: this
Read: 1936287860
Temp:  tex☺
Read: 2019914784
Temp: t is☻
Read: 1936269428
Temp:  a t♥
Read: 1948279072
Temp: estt♦
Read: 1953788773
Temp: 1
Read: 49

Dinge zu beachten:

temp ist nur 4 bytes, aber 5 bytes gedruckt

Ein Rat: verlassen Sie sich nicht auf int 32 bit breit. #include <stdint.h> und verwenden uint32_t.

InformationsquelleAutor Charles Ray | 2011-01-14

c++

6

Beim Lesen in temp, überschreiben Sie alle 4 Charaktere mit Daten, so dass Sie ohne einen NULL-terminator. cout.operator<<(char*) erwartet, dass eine null-terminierte Zeichenfolge, so dass es den Druck so viele Zeichen, wie er kann, bis es erreicht ein null-terminator. Machen temp 5 Zeichen lang sein, alle '\0', sondern um die Anzahl der bytes, die gelesen, bei der 4 wird das problem zu lindern.
- Wird das nicht zu Problemen führen, wenn ich Rücksicht temp " und " cast to buf[i]?
- einfach nicht konvertieren vier chars die einen int-Wert von typecasting Zeiger, es ist eine Schlechte Sache. Verwenden Sie so etwas wie buf[i] = temp[0] + (temp[1] << 8) + (temp[2] << 16) + (temp[3] << 24).
- Es wird wahrscheinlich keine Probleme verursachen, aber wie ulidtko sagte, Sie fast sicher nicht wollen, das zu tun. Wenn Sie Lesen Ganzzahlen in derselben endianness, wie das system es sich handelt, Lesen Sie in der int-direkt oder verwenden Sie bitweise Operationen um die bytes der integer-manuell. Ich witheld Kommentar auf die Allgemeine ästhetik von dem, was Sie tun, aber es gibt bessere Möglichkeiten, Lesen Sie in binäre Daten.
- Pflege, genau zu erklären, warum es schlecht ist?
- Ray, es ist schlimm, weil es ist nicht tragbar. Sie haben keine Garantie für sizeof(int) an das Betriebssystem, das Sie gehen, um ausgeführt werden, und Gießen vier bytes, wie, die verursachen können, schwer fassbare Probleme. Dies ist auch nicht endian-portable. Schließlich (es gibt auch andere Gründe, aber die, die ich erwähnt habe sollten überzeugend genug, es nicht zu verwenden), diese Art ist nicht sehr sauber und wird härter sein für andere Entwickler, um durch zu kommen.
- Es ist schlecht, weil aus der Abhängigkeit zu viel Plattform-spezifische Dinge, wie die endian von zahlen, die auf Ihrem system, und die byte-Ausrichtung von Speicher, und wahrscheinlich einige andere nette Sachen. Obwohl, wenn Sie planen, um Sie überhaupt verwenden Ihr Programm auf nur einem einzigen computer, das alles spielt keine Rolle und ist einfach nur nervig Schlagworte Wert, ignoriert zu werden.
- Danke für die Beschreibung. Mehr Kritik ist willkommen. Es ist eigentlich nicht sauber zu sein, jedoch als ich bin, zu versuchen, meine eigene Verschlüsselungsmethode.
- Zu versuchen, Ihre eigenen Verschlüsselungs-Methode ist eine weitere Flagge, die Sie sollten nie, nie, nie tun, wenn Sie code schreiben, der ist eigentlich gedacht, um Ihre Daten zu schützen. Wenn Sie gerade dicking herum, dann mit allen Mitteln, aber wenn Sie wollen starke crypto-es ist nie eine gute Idee, Rollen Sie Ihre eigenen.
- sauber, leicht wartbar Quelle ist immer gut. ;) Wenn Sie halten Ihre Quellen zu schmutzig ist, bekommen Sie Fehler, und Sie haben echte Probleme loszuwerden, die Tiere.
- Dies ist ein Lern-Beispiel. Wenn wollte ich eigentlich verschlüsseln, sowas würde ich implementieren Sie AES-256.
- Dein Beispiel funktionierte, abgesehen von Sie nicht typecasting die gesamte Anweisung als int. Danke.
InformationsquelleAutor yan
2

Du ausgeben temp als eine null-terminierte Zeichenfolge, aber es ist nicht null-terminiert ist, so dass Sie drucken aus 4 Zeichen temp und unbekannte Anzahl von Müll-Zeichen, bis Sie versehentlich in eine 0.

InformationsquelleAutor MK.
1

Du bist nicht null beendet den Puffer, die Sie drucken wollen.

InformationsquelleAutor Fred Larson
1

Wenn Sie an einem char-array zu cout es wird umgewandelt in einen char-Zeiger, in anderen Worten, was normalerweise als ein "C-string". Dies bedeutet, dass finden Sie am Ende der Zeichenfolge, die output-routing der Suche nach dem ersten 0x00 NUL-Zeichen.

Array ist 4 chars jedoch, und 4 Zeichen aus der Datei gelesen werden. Wenn keiner dieser Punkte ist eine NULL, dann wird die Ausgabe Lesen Sie chars aus dem Gedächtnis, bis er eine findet und das sind die seltsamen chars Sie beobachten. Beachten Sie, dass eine weitere mögliche Folge ist ein segmentation Fault, weil Sie nicht in der Lage sein sollen, gehen herum und Lesen Sie den Speicher, dass Sie nicht explizit zuordnen.

Als Lösung könnte man erklären das array von 5 chars, statt der 4, sodass die fünfte immer auf NUL zu werden, beenden Sie die Ausgabe.

InformationsquelleAutor 6502

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.