Den ubrydelige kryptering!

hvad starter dit j med første gang ? nul? Så skal du også sætte j=0 hvis (j<kodeLen)

upz - j bliver initialiseret med verdien 1 længere opppe i koden...

Hej Emil jeg troede du vidste det hele :)

jf: http://www.eksperten.dk/spm/72780

Nok om det ..

chr = (char)(((int)chr)-((int)Kryptkode[j])); - der er ikke nogen grund til at typecaste til hhv. char og int ...  chr = chr - Kryptkode[j]; - giver samme resultat (næsten) - i sidste ende betyder det ikke noget.

chr = (char)(((int)chr)%255) - denne sætning har ingen effekt ... en char er default signed dvs. -127 til 127. Modulo division returnerer en rest - 127/255 = 0, rest = 127 (127%255=127).

Dit problem ligger (tror jeg) i enten krypteringsTekst.Length() eller i kodelen - prøv at skrive -1 bag ved dem begge.

Hvis j er global eller derhenad skal du huske at sætte den tilbage til 1 inden du starter med dekryptering.

Starter en ansistreng i streng[1]?

>>dejbjerg

Det troede jeg også - det gik bare op for mig at det kun var i teorien :)~

Ja - en AnsiString starter (iriterende nok) på 1 - det er også derfor at jeg ikke skriver -1 foran krypteringsTekst.Length() (hvilket havde været korrekt at gøre hvis der var tale om en char streng)...

Jeg har løst problemet med længden af teksten - ved at ændre initialiseringen af krypteringsTekst fra:
AnsiString krypteringsTekst=tekst->Text;
til:
AnsiString krypteringsTekst=AnsiString(tekst->Text);

Nu er der bare det, at hvis jeg krypterer en tekst flere gange, så forsvinder en del tegn...

Jeg har mistanke om at den der AnsiString kun vil tage ansi tegn (fornuftigt tænkt ikk :)~ ), og derfor sier de tegn fra, der ikk er almindelige i en tekst - dem der dukker op under en kryptering...

Hvad jeg så har brug for, er et alternativ til min AnsiString, eller en måde at bypasse dens kræsne opfattelse af tegn...

E-)mil
http://eej.dk
http://fotx.net

#include <cstring>
#include <string>

char str[] = \"dette er noget tekst\";

eller

int MAX = 40;
char str[MAX];
cin.getline(str, MAX); // eller cin.getline(str, MAX, \'\\n\'; (læser indtil \'\\n\' dvs. retur)

eller du kan studere char* , men der er jeg ikke til meget hjælp.

Jeg selv bruger filer -


/*Et meget simpelt krypteringsprogram. Læser filerne textFile.txt og keyFile.txt char for char og lægger de to char sammen i en tredie char, som skrives til filen codeFile.txt. Derefter dekrypteres til deFile.txt. Brugeren skal selv oprette filerne textFile.txt og keyFile.txt */

// includes
#include <iostream>
#include <fstream>

// namespace
using namespace std;

// function declarations
int encrypt();
int decrypt();

// main
int main()
{
encrypt();
decrypt();
return 0;
}

// the encrypt function
int encrypt()
{
    char cht, chk, chc;
    cht = chk = chc = 0;

ifstream keyFile(\"keyFile.txt\");
ifstream textFile(\"textFile.txt\");

ofstream outFile(\"codeFile.txt\");
   
    while (textFile)
    {
   
        keyFile.seekg(0, ios::end);   
        int endpos = keyFile.tellg();
        keyFile.seekg(0, ios::beg);
       
        for (int i = 0; i < endpos; i++)
        {
            textFile.get(cht);
       
            if (textFile.eof())
                return 0;
           
            keyFile.seekg(i);
            keyFile.get(chk);
       
            if (i == endpos - 1)
                i = -1;

            chc = cht + chk;
            outFile.put(chc);
        }   
    }

return 0;
}

// the decrypt function
int decrypt()
{
    char chc, chk, chd;
    chc = chk = chd = 0;

ifstream keyFile(\"keyFile.txt\");
ifstream codeFile(\"codeFile.txt\");

ofstream outFile(\"deFile.txt\");

    while (codeFile)
    {   
        keyFile.seekg(0, ios::end);   
        int endpos = keyFile.tellg();
        keyFile.seekg(0, ios::beg);
       
        for (int i = 0; i < endpos; i++)
        {
            codeFile.get(chc);
       
            if (codeFile.eof())
                return 0;
            keyFile.seekg(i);
            keyFile.get(chk);
       
            if (i == endpos - 1)
                i = -1;

            chd = chc - chk;
            outFile.put(chd);
        }
    }       
           
return 0;   
}

Der er temmelig mange medlemsfunktioner i string og cstring, men der er du nok nødt til at kigge i en bog :)

Kan jeg ikk bare lave en:
char str[strlen(tekst->Text.c_str())]=tekst->Text.c_str();
i stedet for den der AnsiString?

E-)mil

Hej Emil.

Det kan jeg ikke svare på.

Hvorfor bruger du alle de snørklede -> operatorer?

Hvis jeg skulle lave det med strenge ville jeg gøre noget ala detteher:


#include <iostream>
#include <string>

using namespace std;

const int MAX = 255;

void main()
{
    char s1[MAX] = \"dette er den tekst, der skal kodes\";
    char s2[MAX], s3[MAX];
    char endstring = \'\\0\';
    char text, key, kod;

    cout << \"Indtast koden \";
    cin.get(s2, MAX, \'\\n\');
   
    cout << endl << s1 << endl;
   
    for (int i = 0, j = 0; i < strlen(s1); i++, j++)
    {
        if (j == strlen(s2))
            j=0;
       
        text = s1[i];
        key = s2[j];
        kod = text + key;

        s3[i] = kod;

        cout << s2[j];

        if(i == strlen(s1)-1)
            s3[i+1] = endstring;
       
    }
    cout << endl ;
    cout << s3 << endl;

   
    for (i = 0, j = 0; i < strlen(s3); i++, j++)
    {
        if (j == strlen(s2))
            j=0;
       
        kod = s3[i];
        key = s2[j];
        text = kod - key;

        s3[i] = text;

        if(i == strlen(s1)-1)
            s3[i+1] = endstring;
       
    }
   
    cout << s3 << endl << endl;
}

Det tror jeg nu godt du kan.

Jeg vil hellere prøve med
char str[strlen(tekst->Text.c_str())]=tekst->Text.c_str();
da jeg ikke kan lide MAX delen i din kode :)

E-)mil
Jeg tester det når jeg får fri idag

Det hjalp ikke...

Er der ikke noget med at en strenge-erklæring ala
char str[], kun tager const argumenter.

følgende giver vistnok en fejl:

char s1[] = \"noget tekst\";
char s2[strlen(s1)];

Men du kan bruge:

string s1, s2;

uden at behøve at angive indhold eller størrelse.

Jeg bryder mig bare ikke om at arbejde med medlemsfuktionerne i string. Jeg synes de er bøvlede. En string kan konverteres til char s[], hvis man vil det - men det kan jeg se at du godt ved ..Text.c_str();..

Har du prøvet: char str[] = tekst->Text.c_str();

Hej Emil

Hvis du får tid så prøv evt. at besøge
http://www.eskimo.com/~weidai/cryptlib.html

NB! Hvis du downloader, skal du ikke bruge eksempelprogrammet - det har fucked up i Visual C++. Jeg kan ikke engang compilere det mest simple HelloWorld i MFC.

Geninstallerer .... :-X

Nøjes med at studere/bruge filerne.

>>dejbjerg
Jeg vil nu helst have mit eget til at funke :)
Du bruger VC++?! Det jeg laver er i Borland Builder...
Ja det har jeg prøvet - det kunne jeg ikke få til at virke.
Det jeg tror er, at AnsiString str=AnsiString(KrypteretTekst); fjerner alle ikke tal / bogstaver eller alm. tegn...
Fejlen opstår jo først når man prøver at kryptere anden gang (krypterer krypteringen)...

E-)mil

Hved er forskellen på en String og en AnsiString?

Det skulle ikke have nogen indflydelse hvilken compiler der bruges.

Hvis du kan fortælle mig hvad en AnsiString er kan jeg måske svare. Jeg har prøvet at slå den op, men kunne ikke finde en header fil.

Hmmm - du har ikk borland builder? Der er ret meget om *AnsiString objektet i hjælpen - AnsiString er et stort objekt med mange funktioner...

Hvorfor bruger du den, er den et must?

AnsiString bliver brugt overalt i Borlan Builder - derfor ville det være nemmeste med den :)

[Det er ikk sundt at være så sent oppe på en hverdag *gg*]

E-)mil

Tak for svarene dejbjerg - kikker på den der C løsning, når jeg kommer hjem fra ferie :)

E-)mil
http://eej.dk
http://fotx.net/emileej

Det var så lidt.
Jeg er ked af at jeg ikke kunne være til mere hjælp med dit specifikke ansistreng-problem.