Vigenere-Chiffre falsche Ausgabe

Question

Ich muss Vigenere-Chiffre programmieren, aber meine Ausgabe sieht ein bisschen anders aus.

Eingang: Po Treti raz sa ohlasi Der Schlüssel: euhwa

Ausgang: TI ANEXC YWZ WU VDLEMP Was ich bekommen habe: TI ANEDM LHV SK SBSWSS

Könnten Sie bitte Hilf mir herauszufinden, warum funktioniert es nicht richtig?

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <ctype.h> 

char* vigenere_encrypt(const char* key, const char* text) 
{ 
    if(key==NULL || text==NULL) 
     return NULL; 

    int i,k,t,j=0; 
    t = strlen(text); 
    k = strlen(key); 

    char* copy=(char*)malloc(strlen(text)+1); 
    char* enc=(char*)malloc(strlen(text)+1); 
    char* copykey=(char*)malloc(strlen(key)+1); 

    for (i=0;i<k;i++) 
    { 
     copykey[i]=toupper(key[i]); 
    } 

    for (i=0;i<k;i++) 
    { 
     if(!(isalpha(copykey[i]))) 
     { 
      free(copy); 
      free(copykey); 
      free(enc); 
      return NULL; 
     } 
    } 

    for (i=0;i<=t;i++) 
    { 
     copy[i]=toupper(text[i]); 
    } 

    for (i=0;i<=t;i++) 
    { 
     if (isupper(copy[i])) 
     { 
      enc[i]=(copy[i]+copykey[j])%26+'A'; 
      j++; 
      if (j>k) 
       j=0; 
     } 
     else enc[i]=copy[i]; 
    } 

    free(copy); 
    free(copykey); 
    return enc; 
} 

int main() 
{ 
    char* encrypted; 
    encrypted = vigenere_encrypt("euhwa","Po treti raz sa ohlasi!"); 
    printf("%s\n",encrypted); 
    free(encrypted); 
} 

Answer 1

Die Frage ist, wie Sie behandeln das:

"Hey, ich den Schlüssel beendet haben können j, bringen wieder auf Null"

Jetzt ist der Code, hübsch chaotisch, nicht für die doppelte Kopie/Paste (es kann passieren), aber weil es könnte (und sollte) ein wenig optimiert werden ..

Wie auch immer die Lösung für Ihr Problem ist einfach:

sollten Sie j auf 0 setzen, wenn j>= von k ist. Wichtig ist der equal Teil, da Sie möchten, dass es 0 ist, wenn Sie die Länge des Schlüssels erreichen. Die Art, wie Sie es tun (nur Testen für greater than) bedeutet, dass, wenn Sie das Ende des Schlüssels erreichen Sie eine zusätzliche Schleife, die einen ungültigen copykey Wert verwendet. Wenn der Schlüssel Länge gleich 5 ist, sollten Sie bei copykey[4] aber die extra Schleife tut copykey[5] die ungültig ist .. und (un) zum Glück ist es nicht segfault Sie zur Hölle.

for (i=0;i<=t;i++) 
{ 
    if (isupper(copy[i])) 
    { 
     enc[i]=(copy[i]+copykey[j])%26+'A'; 
     j++; 
     if (j >= k) 
      j=0; 
    } 
    else enc[i]=copy[i]; 
}