Variable corrupted!?

[HeavyMetalToast]

Heyho

Ich hab angefangen das Spiel Hangman zu schreiben. Leider funktioniert mein Programm nicht richtig, und mir wird folgende Fehlermeldung mitten im Programm ausgegeben.

Run-Time Check Failure #2 - Stack around the variable 'geraten' was corrupted.

Ich hab mir schon die Finger wund gegoogelt, nur leider finde ich keine Lösung für mein Problem.

Und lasst mich bitte in Frieden mit eurem:"cout/cin ist viel besser"/"int main ist richtig, alles andere ist falsch", ich hab meine Gründe das ich es so verwende. Desweiteren möchte ich auch keine Verbesserungsvorschläge.

 Spoiler

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <time.h>
#include "gconio.h"
#include <windows.h>

int g_erraten=0;
int g_geratenascii;
int g_loesungascii[30];
int g_checking=1;

void fertig(char geraten[]);
int strlaenge(char loesung[]);
void strgets(char loesung[],char geraten[]);
void ersetzen(char eingegeben, char loesung[],char geraten[]);

void main()
{
	char loesung[30];
	char geraten[30];
	char eingabe;
	int laenge;
	int check;
	printf("Gib das Loesungswort ein:\n");
	strgets(loesung,geraten);
	laenge=strlaenge(loesung);
	do{
		system("cls");

		for (int i=0;i<=laenge-1;i++)
			printf("%c",geraten[i]);

		printf("\nDeine Eingabe:");
		scanf("%c",&eingabe);
		g_geratenascii=eingabe;
		ersetzen(eingabe,loesung,geraten);
		fertig(geraten);
	}while (g_checking == 0);
	
	system("cls");
	printf("Glueckwunsch :D Du hast gewonnen.");
	getch();
}

void fertig(char geraten[])
{
	int i=0;
	g_checking=1;
	for (i=0;i<30;i++);
	{
		if (geraten[i]=='_')
			g_checking==0;
	}

}
void ersetzen(char eingegeben,char loesung[],char geraten[])
{
	int x;
	for(int i=0;i<=30;i++)
	{
		if ((g_loesungascii[i]==g_geratenascii)||(g_loesungascii[i]==g_geratenascii+32)||(g_loesungascii[i]==g_geratenascii-32))
		{
			geraten[i]=eingegeben;
			g_erraten++;
		}
	}
}
int strlaenge(char loesung[])
{
	for(int i=0;i<=30;i++)
	{
		if ((loesung[i] == '\0')||(loesung[i] == '\r'))
			return i;
	}
}
void strgets(char loesung[],char geraten[])
{
	for(int i=0;i<=30;i++)
	{
		loesung[i]=getch();
		geraten[i]='o';
		g_loesungascii[i]=loesung[i];
		printf("x");
		if (loesung[i] == '\r')
		{
			geraten[i]='\0';
			return;
		}
	}
}

[Dr. Coxxy]

kann jetzt nicht alles durchsehen, ist nen bissel unübersichtlich, aber:

1. system -> böse -jaja :P

2. Die funktion macht? (doppel == statt zuweisung?)

void fertig(char geraten[])
{
	int i=0;
	g_checking=1;
	for (i=0;i<30;i++);
	{
		if (geraten[i]=='_')
			g_checking==0;
	}

}

3.

for (int i=0;i<=laenge-1;i++) -> for (int i=0; i < laenge; i++)

4.

for(int i=0;i<=30;i++)

access violation... muss < 30 sein...

5. verlieren kann man noch nicht? :P

[HeavyMetalToast]

Originally Posted by Dr. Coxxy

kann jetzt nicht alles durchsehen, ist nen bissel unübersichtlich, aber:

1. system -> böse -jaja :P

2. Die funktion macht? (doppel == statt zuweisung?)

void fertig(char geraten[])
{
	int i=0;
	g_checking=1;
	for (i=0;i<30;i++);
	{
		if (geraten[i]=='_')
			g_checking==0;
	}

}

3.

for (int i=0;i<=laenge-1;i++) -> for (int i=0; i < laenge; i++)

4.

for(int i=0;i<=30;i++)

access violation... muss < 30 sein...

5. verlieren kann man noch nicht? :P

2.) Danke, deswegen war es nach dem ersten mal vorbei
3.) Ist doch nur ein Schönheitsfehler ;D
4.) Danke ;D
5.) Wollte ich gerade einbauen, dann kam der Fehler.

[xNopex]

ich hab meine Gründe das ich es so verwende

Die interessieren mich brennend. Und das meine ich vollkommen unironisch.

[HeavyMetalToast]

Originally Posted by xNopex

Die interessieren mich brennend. Und das meine ich vollkommen unironisch.

Cout/cin benutze ich nicht, da wir in der Schule nicht nur Konsolenanwendungen programmieren, sonder später auch auf Mikrochips. Und mein Professor sagte das wir auf diesen nur mit C programmieren können, also gewöhne ich es mir garnicht anders an, desweiteren ist es besser, das ich es gleich habe wie meine Klassenkameraden, falls ich ihnen was erklären muss, denn die wissen nichtmal das es oben genannte befehle gibt.

void main hab ich mir eben so angewöhnt, und ich kenne keinen Grund warum ich nicht nehmen sollte. In meiner Entwicklungsumgebung (VS2008) funktioniert es einwandfrei.

Genug?

[xNopex]

Dann programmier doch gleich in C. Was du eh schon, vermutlich unbewusst, größtenteils tust.

ich kenne keinen Grund warum ich nicht nehmen sollte.

Weils hässlich ist und nicht portabel.

[MrSm!th]

Wär mir neu, dass man auf Microchips kein C++ programmieren könnte.