Hilfe: Caesar Verschlüssung in C++

EOP

gast12987897 schrieb:

Ich finde es schade, dass hier dermaßen auf dem TE herumgehackt wird.

Das habe ich nicht - im Gegenteil, ich habe ihm mehrere helfende Hände gereicht.

swapper

EOP schrieb:

Und das gleiche nochmal für Großbuchstaben?

nö, das programm macht einfach gnadenlos großbuchstaben zu kleinbuchstaben (oder umgekehrt). normalerweise ist groß oder klein dem caesar unbekannt. ziffern und satzzeichen etc. auch nicht. den ollen römern stellte sich das problem einfach nicht.

EOP

swapper schrieb:

EOP schrieb:

Und das gleiche nochmal für Großbuchstaben?

nö, das programm macht einfach gnadenlos großbuchstaben zu kleinbuchstaben (oder umgekehrt). normalerweise ist groß oder klein dem caesar unbekannt. ziffern und satzzeichen etc. auch nicht. den ollen römern stellte sich das problem einfach nicht.

PHP str_rot13 berücksichtigt auch GROß und klein.

SvenJungwird

Hallo,

zuerst einmal danke an alle.

Es funktioniert, bzw. geht in die richtige richtung.
Eingabe: abc def xyz!ABC XYZ?
Verschiebe um 3:

Ausgabe: def ghi abc$DEF []B

Sollte aber sein:

def ghi abc!DEF ABC?

____
2. Beispiel:
Eingabe: Abc Zyx!
Verschiebe um 55

Ausgabe: Def ]ba$

Sollte aber sein:

Def Cba!

Ich bin so nah am Ziel! Die Frage ist nur, wieso stehen dort sonderzeichen, obwohl ich die mit IF ausgeschlossen habe? Normalerweise sollten es buchstaben ergeben, wieso ergeben es Sonderzeichen?

#include <iostream>
#include <string>

using namespace std;
int main() {

	string text;
	int v;
	string geheim;
	cout << "Bitte geben Sie den zu verschluesselnden Text ein: ";
	getline(cin, text);

	cout << "Bitte geben Sie die Anzahl Verschiebepositionen ein (als positive ganze Zahl): ";
	cin >> v;
	v %= 26;
	for (int i = 0; i < text.length(); i++) {

		if(text[i] >= 65 && text[i] <= 122) {

			geheim += static_cast<unsigned>(text[i]) + v;

			if (static_cast<unsigned>(geheim[i]) > 'z') {
				geheim[i] = static_cast<unsigned>(geheim[i]) - (v / 26 + 1) * 26;
			}

		}
		else {
			geheim += text[i];
		}

	}

	cout << geheim << endl;
	system("pause");

}

swapper

EOP schrieb:

swapper schrieb:

EOP schrieb:

Und das gleiche nochmal für Großbuchstaben?

nö, das programm macht einfach gnadenlos großbuchstaben zu kleinbuchstaben (oder umgekehrt). normalerweise ist groß oder klein dem caesar unbekannt. ziffern und satzzeichen etc. auch nicht. den ollen römern stellte sich das problem einfach nicht.

PHP str_rot13 berücksichtigt auch GROß und klein.

um das zu erreichen, kann sich der code ja merken, ob das zeichen vorher groß war und es dann wieder groß machen. dafür ist bei rot13 der verschiebefaktor festgenagelt.

beispiel:

/**
 * rotate one char like cesar did. (but case preserving) ;)
 * @param c input char
 * @param v steps to rotate
 * @return rotated char (if it is a letter)
 */
char rot (char c, int v)
{
    static char* arr = "abcdefghijklmnopqrstuvwxyz";
    if (isalpha(c))
    {
        char low = tolower(c);
        char out = arr[(low-'a'+v)%26];
        return low == c ? out : toupper(out);
    }
    return c; // not a letter
}

EOP

SvenJungwird schrieb:

Es funktioniert, bzw. geht in die richtige richtung.
Eingabe: abc def xyz!ABC XYZ?
Verschiebe um 3:

Ausgabe: def ghi abc$DEF []B

Sollte aber sein:

def ghi abc!DEF ABC?

Gibt bei meiner Funktion dein erwartetes Ergebnis aus.
Funktioniert mit -3 dann auch rückwärts.
Alles kontrolliert.

Über die Unsinnigkeit des Faktors 55 müssen wir uns nicht weiter unterhalten bevor du nicht den Aufbau der ASCII-Tabelle verinnerlicht hast.

Die Caesar-Verschlüsseling ist nicht für Zahlen oder gar Sonderzeichen gedacht.

swapper

EOP schrieb:

Über die Unsinnigkeit des Faktors 55 müssen wir uns nicht weiter unterhalten bevor du nicht den Aufbau der ASCII-Tabelle verinnerlicht hast.

einspruch: ne caesar-chiffre sollte mit jedem beliebigen rotationsfaktor klar kommen und auch die ascii-table nicht unbedingt als in stein gemeißelt ansehen.

EOP

swapper schrieb:

EOP schrieb:

Über die Unsinnigkeit des Faktors 55 müssen wir uns nicht weiter unterhalten bevor du nicht den Aufbau der ASCII-Tabelle verinnerlicht hast.

einspruch: ne caesar-chiffre sollte mit jedem beliebigen rotationsfaktor klar kommen und auch die ascii-table nicht unbedingt als in stein gemeißelt ansehen.

Wohin schiebe ich dann denn z.B. eine '0' (ASCII #48)?

swapper

EOP schrieb:

swapper schrieb:

EOP schrieb:

Über die Unsinnigkeit des Faktors 55 müssen wir uns nicht weiter unterhalten bevor du nicht den Aufbau der ASCII-Tabelle verinnerlicht hast.

einspruch: ne caesar-chiffre sollte mit jedem beliebigen rotationsfaktor klar kommen und auch die ascii-table nicht unbedingt als in stein gemeißelt ansehen.

Wohin schiebe ich dann denn z.B. eine '0' (ASCII #48)?

die ist vom caesar nicht betroffen, da sie kein buchstabe ist.

Das ist doch auch einfacher möglich oder nicht ?

EOP

Gast267282 schrieb:

Das ist doch auch einfacher möglich oder nicht ?

Na dann hau raus.

Er muss nur
C++:
if (static_cast<unsigned>(geheim[i]) > 'Z') {
geheim[i] = static_cast<unsigned>(geheim[i]) - (v / 26 + 1) * 26;
}

prüfen. Dann stimmt der vordere Teil aber nicht.

SvenJungwird

Hallo Gast, Hallo @all.

Dieser Thread ist ja fast mit Antworten explodiert.

Ich habe, dank euch einiges dazu gelernt, dafür vielen Dank.

Die Lösung ist:

#include <iostream>
#include <string>

using namespace std;
int main() {

	string text;
	int v;
	string geheim;
	cout << "Bitte geben Sie den zu verschluesselnden Text ein: ";
	getline(cin, text);

	cout << "Bitte geben Sie die Anzahl Verschiebepositionen ein (als positive ganze Zahl): ";
	cin >> v;
	v %= 26;
	for (int i = 0; i < text.length(); i++) {

		if(text[i] >= 65 && text[i] <= 90) {

			geheim += static_cast<unsigned>(text[i]) + v;

			if (static_cast<unsigned>(geheim[i]) > 'Z') {
				geheim[i] = static_cast<unsigned>(geheim[i]) - (v / 26 + 1) * 26;
			}

		}
		else if (text[i] >= 97 && text[i] <= 122) {
			geheim += static_cast<unsigned>(text[i]) + v;

			if (static_cast<unsigned>(geheim[i]) > 'z') {
				geheim[i] = static_cast<unsigned>(geheim[i]) - (v / 26 + 1) * 26;
			}

		}
		else {
			geheim += text[i];
		}

	}

	cout << geheim << endl;
	system("pause");

}

vielen Dank!

Belli

Wie ich oben schon schrob:
Wenn Du

v %= 26;

machst, dann kann

(v / 26 + 1) * 26;

einfach durch

ersetzt werden, denn dann ist

v / 26 + 1

immer gleich 1.