funktion

[hund555]

irgendwie werden anscheinend meine Zeichen nicht auf 0 gesetzt, was muss ich da ändern?

kriege als cout falsche ergebnisse

#include<iostream.h>

#include<stdlib.h>

void anzahl(int n, int *anzbuchst, int *anzzeichen, char *buchst)

{

anzbuchst=0;

anzzeichen=0;

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

{

if ( (buchst>= 'A' && buchst<='Z') || (buchst>= 'a' && buchst<='z') )

{

anzbuchst = anzbuchst + 1;

}

anzzeichen = anzzeichen + 1;

}

*anzzeichen;

}

void main(void)

{

char text[]= "Hier steht der Text ";

int gr, anzbuch, anzzei;

gr = strlen(text);

anzahl(gr, &anzbuch, &anzzei, text);

cout << " Anzahl Buchstaben: " << anzbuch << endl;

cout << " Anzahl Zeichen: " << anzzei << endl;

cout << endl;

system("pause");

}

[Guybrush Threepwood]

Eieiei das ist jetzt nicht böse gemeint, aber du solltest dich unbedingt nochmal mit Zeigern beschäftigen.

Innerhalb der Funktion anzahl veränderst du nicht den Inhalt von anzbuch und anzzei sondern den Zeiger der auf sie zeigt.

du müsstest den Zeiger immer dereferenzieren, also so

(*anzbuchst)=0;

(*anzbuchst) = (*anzbuchst) + 1; bzw. (*anzbuchst)++;

Außerdem solltest du nicht iostream.h einbinden, da diese veraltet ist. nimm stattdessen iostream (ohne .h).

Nach dem Standard ist void main (void) falsch, int main() wäre hier richtig.

Da du anscheinend noch Probleme mit Zeigern hast und C++ verwenndest solltest du lieber Referenzen benutzten.

Und die Funktion system() solltest du nicht verwenden, da diese ungeahnte verhaltensweisen an den Tag legen kann.

Ich hab das mal alles in folgendem Programm zusammengefasst, welches meiner Meinung nach besser sein sollte:

#include <iostream>

#include <string.h>

#include <conio.h>


using namespace std;


void anzahl(int n, int& anzbuchst, int& anzzeichen, char* buchst)

{

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

	{

		if ( (buchst[i]>= 'A' && buchst[i]<='Z') || (buchst[i]>= 'a' && buchst[i]<='z') )

		{

			anzbuchst = anzbuchst + 1;

		}

		anzzeichen = anzzeichen + 1;

	}

}


int main()

{

	char text[] = "Hier steht der Text ";


	int gr      = 0,

		anzbuch = 0,

		anzzei  = 0; 


	gr = strlen(text);


	anzahl(gr, anzbuch, anzzei, text);


	cout << " Anzahl Buchstaben: " << anzbuch << endl;


	cout << " Anzahl Zeichen: " << anzzei << endl;




	cout << endl;

	getch();

	return 0;

}

[/PHP]

PS: Wenn du Code Tags verwendest erhöht das die Lesbarkeit enorm

[hund555]

ok danke, alles klar