Krångel med return värde i C!

[ZerQ]

Hej

Jag har i skolarbetet gjort en miniräknare som skall använda funktioner men har problem med returvärdet ifrån en funktion. Se koden nedan:

Kod: Markera allt

char felkontroll(cOperator)
{
	/* Nu kontrollerar vi om användaren har anget rätt räknesätt i raden ovan, om det 
	är fel så frågar programmet efter ett korrekt operator. Denna funktion håller kvar 
	användaren så länge som det är fel tecken i inmatningen*/
	while (TRUE){
		if ((cOperator == '+') || (cOperator == '-') || (cOperator == '*') || (cOperator == '/'))
			return;
		else{
			printf("Du har angett felaktigt r\x84knes\x84tt, korrekta \x84r: +, -, *, /\n");
			printf("Ange korrekt r\x84knes\x84tt och avsluta med enter!\n");
			scanf("%c", &cOperator);
			fflush(stdin);
			return cOperator;}}
}

Detta är en nekel felkontroll som skall kolla om användaren har skrivit korrekt tecken i uträkningen, kruxet är att den fungerar alldeles utmärkt tills jag har skrivit in fel värde och else satsen skall åtgärda detta, variabeln cOperator får värdet som jag skriver in men returvärden ifrån funktionen blir samma som skrevs in som fel ifrån början!

Om jag exempelvis har skrivit 5 h 5, då reagerar felkontrollen och vill att jag matar in rätt, skriver då + men får tillbaka h.......

Vart har jag gjort fel?

[gasol]

Det första felet du gör är att din funktion är definerad att returnera ett char värde men om operatorn stämmer så returnerar du inget alls utan bara kör return; utan något retur värde.

Det andra felet som finns i koden är att oavsett om cOperator är en giltig operator eller inte så kommer du att returnera den i else satsen.
Vad är det för mening att ha while loopen om du kommer att returnera i första iterationen?

Kod: Markera allt

			printf("Du har angett felaktigt r\x84knes\x84tt, korrekta \x84r: +, -, *, /\n");
			printf("Ange korrekt r\x84knes\x84tt och avsluta med enter!\n");
			scanf("%c", &cOperator);
			fflush(stdin);
			return cOperator; <---- den här vet du inte om den är giltig eller inte så du kan inte returnera den!

edit: Ett tips är att alltid kompilera med -Wall flaggan i gcc och alltid fixxa ALLA varningar du får av gcc, i princip alla varningar man får är fel!

[Smygis]

Jag skulle skriva såhär:

Kod: Markera allt

bool felkontroll(cOperator)
{
	while (true){
		if ((cOperator == '+') || (cOperator == '-') || 
		      (cOperator == '*') || (cOperator == '/'))
			return true;
		else
			return false;
	}
}

och i 'main' ha:

Kod: Markera allt

do{
	//inmatning
while(!felkontroll(variabel));

men det är bara jag.

Men du, Sätt måsarna rätt för helvt.!
return cOperator;}}
ser ut som skit.

[gasol]

Loopen i felkontroll är helt onödig du kommer ju att returnera i första iterationen!

Jag skulle skriva såhär:

Kod: Markera allt

bool felkontroll(cOperator)
{
	while (true){
		if ((cOperator == '+') || (cOperator == '-') || 
		      (cOperator == '*') || (cOperator == '/'))
			return true;
		else
			return false;
	}
}

och i 'main' ha:

Kod: Markera allt

do{
	//inmatning
while(!felkontroll(variabel));

men det är bara jag.

Men du, Sätt måsarna rätt för helvt.!
return cOperator;}}
ser ut som skit.

[Smygis]

Loopen i felkontroll är helt onödig du kommer ju att returnera i första iterationen!

Blä, Jag började ändra på hans sedan sket jag i det. Glömde ta bort loopen :/

[per9000]

[...]

Kod: Markera allt

char felkontroll(cOperator)
{
/* ... */
			scanf("%c", &cOperator);
/* ... */
}

[...]

Har du rätt typ här? Du tar in en char? Om jag inte har fel så antas det vara en int om man inte anger typ. Detta kanske ger dig problem när du läser in &cOperator sen också? Du vet: pekare/referens/värde och hela den historien.

/Per

[ZerQ]

Hej igen och tack för svaren, vad det gäller måsvingarna så är jag inte så konsekvent än på hur jag vill ha det i koden 

Det andra har jag gjort om så jag slapp skicka in det till funktionen och istället gjorde kontrollen direkt i main funktionen. Vart mycket enklare då att få till det riktigt.