Dziwny błąd programu do kodowania plików [C++]

Question

właśnie 10 min temu napisałem sobie szybko prosty koder w C++, który miał za zadanie przyjąć od użytkownika klucz i przemienić wszystkie znaki w pliku o ten klucz (generalnie miał wykonać działanie znak+=klucz), no i wszystko pięknie działa, ale na małych i stworzonych przezemnie plikach. Kiedy chciałem tak zakodować większy plik o rozszerzeniu gif lub inny niż txt, to program albo się wywalił podczas kodowania, albo podczas dekodowania... Generalnie nwm co jest źle, ale stawiam, że to dla tego, że fstream albo nie przyjmuje większych plików, albo pliki te nie są kodowane w ASCII co prowadzi do wywalania się programu (to akurat mało prawdopodobne).

 

Tutaj zamieszczam kod:

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <string>

using namespace std;

unsigned char klucz;

fstream openFile, saveFile;

string line;

int main(int argc, char * argv[])
{
    if(argc!=1)
    {
        if((string)argv[1]=="coding")
        {
            klucz = atoi(argv[2]);
            openFile.open(argv[3], ios::in);
            saveFile.open("coded.ac", ios::out);
            if(openFile.good() && saveFile.good())
            {
                while(getline(openFile, line))
                {
                    for(int a=0; a<=line.length()-1; a++)
                    {
                        saveFile<<(char)((char)line[a]+klucz);
                    }
                    saveFile<<endl;
                }
            }
        }
        if((string)argv[1]=="decoding")
        {
            klucz = atoi(argv[2]);
            saveFile.open(argv[3], ios::out);
            openFile.open(argv[4], ios::in);
            if(openFile.good() && saveFile.good())
            {
                while(getline(openFile, line))
                {
                    for(int a=0; a<=line.length()-1; a++)
                    {
                        saveFile<<(char)((char)line[a]-klucz);
                    }
                    saveFile<<endl;
                }
            }
        }
    }
    openFile.close();
    saveFile.close();
    return 0;
}

Proszę, pomóżcie mi znaleźć błąd! :>

Answer 1

musisz traktować pliki jako binarne - żadnych endl, getline

takie cos powinno działać

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <string>

using namespace std;

unsigned char klucz;


fstream openFile, saveFile;

char c;

int main(int argc, char * argv[])
{
    if(argc!=1)
    {
        if((string)argv[1]=="coding")
        {
            klucz = atoi(argv[2]);
            openFile.open(argv[3], ios::binary|ios::in);
            saveFile.open("coded.ac", ios::binary|ios::out);
            if(openFile.good() && saveFile.good())
            {

                while(openFile.read(&c,1))
                {
       
                  saveFile.put(c+klucz);

      

                }
            }
        }
        if((string)argv[1]=="decoding")
        {
            klucz = atoi(argv[2]);
            saveFile.open(argv[3], ios::binary|ios::out);
            openFile.open(argv[4], ios::binary|ios::in);
            if(openFile.good() && saveFile.good())
            {

                while(openFile.read(&c,1))
                {
                
                   saveFile.put(c-klucz);
   
                }
            }
        }
    }
    openFile.close();
    saveFile.close();
    return 0;
}

 

Comments

OOOO dzięki bro, pomogłeś mi bardzo! :D

Answer 2

Robisz swój plik gif mam rozumieć? Wpisujesz coś w notepadzie i zapisujesz jako *.gif?

Znak += klucz...

Jeśli pobierzesz klucz jako char, to bedziesz mial znak, a nie klucz (chyba, że mówimy tu o czymś innym niż szyfr Cezara)

Comments

no tak, ale zmienne char przechowywane są jako cyfry i nie ma znaczenia czy to int czy char, bo znak+=klucz (gdyby znak=97 i klucz=97), oznacza tyle co 97+=97, czli znak=194. Co prawda to jest znak, ale tylko w momencie wyświetlenia tego na monitorze... w pamięci to jest taka sama liczba jak int, z tym, że ten typ char oznacza, że przy wyświetlaniu ma pokazać znak pod tym numerem, a nie liczbę... :>

---

to musi być coś innego :> przy okazji nie tworzę pliku gif w notatniku tylko, w paincie :)

---

Fakt. Nie zauważyłem tego atoi.

Ale otwierasz plik z rozszerzeniem gif i tu jest błąd. Otwórz go np. notepadem i zobacz co w nim siedzi 

---

no właśnie ja go chcę wczytać do programu i przerobić, ale kiedy go wczytuje i rozpoczynam obróbkę z zapisem do innego pliku, to program nagle z nie wiadomych przyczyn się wysypuje... :D i ja tego nie rozumiem, zaś do małych plików pisanych bezpośrednio w notatniku to jest ok, wszystko śmiga... :D

---

Powtarzam, abys otworzyl plik przed wyslaniem do programu za pomocą notepada++. Plik graficzny != plik textowy

Answer 3

Hmmm, conajmniej ciekawe, dobrze że dodadłeś opis bo trzeba by było sporo rozkminiać, o co chodzi w tym kodzie :P