Caesar Verschlüsselung. Allgemeine Verschlüsselung.

cooky451

SeppJ schrieb:

cooky451 schrieb:
printf("Please enter shift: ");
    fgets(buffer, sizeof(buffer), stdin); // fgets um anzahl der einzulesenden zeichen kontrollieren zu können
    shift = atoi(buffer); // string zu int
Manchmal frage ich mich ob ich irgendeinen tollen Trick nicht sehe oder ob etwas einfach nur dumm ist. So auch hier. Warum nicht direkt eine Zahl einlesen?

Weil ich scanf() nicht mag?

Don Carsto

Was ist eigendlich an scanf so böse?
Habe das nun schon öfters hier gelesen, daß scanf nicht so toll ist.
Was ist denn so problematisch daran?

Gruß,
DC

cooky451

Don Carsto schrieb:

Was ist eigendlich an scanf so böse?
Habe das nun schon öfters hier gelesen, daß scanf nicht so toll ist.
Was ist denn so problematisch daran?

Gruß,
DC

An und für sich ist scanf() nicht böse.
Nur muss man beim Gebrauch aufpassen.

char buffer[10];
scanf("%s", buffer);

Das wäre zB. ein typischer Fehler. Abgesehen davon dass undefiniertes Verhalten erzeugt wird falls mehr als 9 Zeichen eingegeben werden könnte man auch noch recht einfach einen exploit für ein Programm schreiben das etwas so einliest. (siehe http://de.wikipedia.org/wiki/Pufferüberlauf )

Das gilt aber für alle C-Funktionen (man möge mich schelten falls ich jetzt was falsches schreibe :D). C-Funktionen überwachen nunmal von selbst keine Speicherbereiche. So kann

char buffer[10];
fgets(buffer, 100, stdin);

genauso für einen exploit genutzt werden, nur ist scanf() besonders bei Anfängern die sich hier noch keine Gedanken machen als quasi-Gegenstück zu printf() sehr beliebt und wird somit halt oft "falsch" benutzt. (Viele vergessen beim einlesen von Zahlen auch gerne mal den Adressoperator..)
Dabei wohnt scanf() aber wie schon gesagt das Böse eigentlich nicht inne und kann in dieser Form:

int num;
char buffer[10];
scanf("%i", &num);
scanf("%9s", buffer);

auch problemlos genutzt werden.

Ich mag es halt nur nicht weils sich auf der Konsole komisch verhält aber das ist ne andere Geschichte und auch nur eine persönliche Meinung :xmas1:

Motors-Freak

Hallo
Habe zur Zeit ein kleines Problem. Bin immer noch an meinem verschlüsselungs Programm bei.
Mein Problem Zur Zeit ist, dass ich NICHT in einer IF Anweisung das GETS benutzen kann. Genauso wenig NACH einer IF Anweisung. Das Programm überspringt dann alles und wartet gar nicht auf eine Eingabe, wieso nich? Benutze ich das GETS jedoch vor der ersten IF Anweisung funktioniert es Einwandfrei. In Der IF Anweisung oder danach funktioniert es nicht mehr. Wieso?

Gruß

Don Carsto

Weil deine If Anweisung es verhindert, daß dein Code ausgeführt wird, sprich die if Anweisung ist nicht wahr.

Und ohne Codebeispiel ist es geradezu unmöglich zu sagen warum genau.

Motors-Freak

Die Funktionen kopiere ich jetzt mal nicht mit rein.

void main()
{
	char klar[501];
	char geheim[501];
	char text[501];
	int i;
	int wahl1=0;
	int wahl2=0;
	int a=0;
	int TEST=0;

	printf("========================================\n");
	printf("=	Caesar-Verschluesselung        =\n");
	printf("========================================\n\n");

nochmal:

	printf("Bitte waehlen Sie, ob sie einen Text eingeben wollen[1], oder ob Sie einen Text\naus einer Datei laden moechten[2]: \n");
	scanf("%i",&wahl1);

	if(wahl1==1 || wahl1==2)
	{
		if(wahl1==1)
		{
				printf("Bitte geben sie den Text ein (Max. 500 Zeichen): \n");
				gets(text);
				printf("\nIhr Satz besteht aus %i Zeichen.\n",strlen(text));

		}

		else
		{
			printf("...kommt noch...\n");
		}

	}

	else
	{
		printf("\nE R R O R\n\n");
		goto nochmal;
	}

/*	printf("Bitte geben sie den Text ein (Max. 500 Zeichen): \n");
	gets(text);
	printf("\nIhr Satz besteht aus %i Zeichen.\n",strlen(text));
*/

	printf("\nBitte waehlen sie ob sie Entschluesseln [1] oder Verschluesseln [2] wollen: \n");
	scanf("%i",&wahl2);

	if(wahl2==1 || wahl2==2)
	{
		if(wahl2==1)
		{

			printf("Bitte geben sie den Schluessel ein: \n");
			scanf("%i",&a);
			entschluesseln(a, text);
		}

		else
		{
			printf("Bitte geben sie den Schluessel ein: \n");
			scanf("%i",&a);
			verschluesseln(a, text);
		}
	}

	else
	{
		printf("E R R O R\n\n\n");
		goto nochmal;
	}
}

Don Carsto

if(wahl1==1 || wahl1==2)

Format ändern in:

if((wahl1==1) || (wahl1==2))

Also mehr Klammern.

Motors-Freak

Springt trotzdem drüber, erlaubt also keine Texteingabe...

Don Carsto

Ja er überspringt das gets() weil da noch ein \n im Puffer ist.

Das kannste abfangen indem du ein getchar(); hinterschaltest.

if(wahl1==1)
        {       
                getchar();
                printf("Bitte geben sie den Text ein (Max. 500 Zeichen): \n");
                gets(text);
                printf("\nIhr Satz besteht aus %i Zeichen.\n",strlen(text));

Don Carsto

Oder arbeite auch bei den weiteren Texteingaben mit scanf().

Motors-Freak

Don Carsto schrieb:

Oder arbeite auch bei den weiteren Texteingaben mit scanf().

Aber mit Scanf kann ich doch keine Sätze aufnehmen, da scanf doch nur bis zu einem Leerzeichen Scant oder nicht???

Don Carsto

Stimmt.

itedvo

lösch einfach den buffer... wenn das einlesen übersprungen wird heißt das ja das was im eingabebuffer drinnen ist... is keine schöne art aber ich habs meiner zeit so gelernt:

fflush(stdin);

versuch das mal...

itedvo schrieb:

lösch einfach den buffer... wenn das einlesen übersprungen wird heißt das ja das was im eingabebuffer drinnen ist... is keine schöne art aber ich habs meiner zeit so gelernt:
fflush(stdin);
versuch das mal...

tolle idee

itedvo

:xmas1:
(Notlösungsalarm)
:xmas1:

aber wenn alle hebel brechen kann diese funktion leben retten =b :xmas1: :xmas2:

itedvo schrieb:

:xmas1:
(Notlösungsalarm)
:xmas1:

aber wenn alle hebel brechen kann diese funktion leben retten =b :xmas1: :xmas2:

alter, alter das ist einfach falsch und dein notlösungs späßchen machts leider auch nicht richtig. schau doch mal in die faq, wurde schon tausendmal im forum besprochen.

ich glaub die lösung ging so...

void cb(){
  int c;
  while((c = getchar()) && c != '\n' && c != EOF);
}

denk das war falsch... müsste eher so sein...

void cb(){
  int c;
  while((c = getchar()) != '\n' && c != EOF);
}

Wutz

Don Carsto schrieb:

Stimmt.

Alles Schrott bis hierher.
*scanf ist weitaus flexibler als in eurer Vorstellungswelt. Lest die Spezifikation dazu und trefft keine Entscheidungen aus eurem Halbwissen heraus.
fflush(stdin) ist der nächste Schrott hier im Thread und steht an der Spitze des von Laien hier geposteten Schwachfug.

nachtfeuer

Also, die Aufgabe erscheint mir nicht so ganz trivial. Normalerweise würde es ja reichen, auf jeden Buchstaben die "Verschiebung" draufzuzählen. Wenn man im ASCII-Code auf den Buchstaben z eins draufzählt, kommt aber nicht a heraus (was ja auch cooky451 schon schreibt). Man könnte einen passenden Wert von Buchstaben größer als ... abziehen und so eine ABC-Rotation hinbekommen. Und dann muss man noch Sonderzeichen und Co beachten. Und dann noch die Eingabebedingungen der C-Grammatik gut verstanden haben und schon Problem- und Debuggingerfahrung damit haben...

Assembler ist hier angenehmer, und man kann auch leichter verstehen, was zu tun ist, aber das nur nebenbei. Um eine Rotation hinzubekommen, ist es vielleicht praktisch, das ABC z.B. zweimal in den Speicher zu schreiben, um dann die entsprechende Verschiebung aus der Tabelle auszulesen. Sonderzeichen und Steuerungscodes kann man für den Anfang ausschließen, oder durch eine Markierung ersetzen.

Speichergeschichten und Pointerrechnerei sind allerdings alles andere als absolut Anfängerfreundlich, man sollte also schon paar Stunden C-Grundlagenbüffelei inclusive Debuggingstunden hinter sich haben bevor man sich an diese vermeintlich einfache "Challenge" heranwagt.