Zum Inhalt springen
View in the app

A better way to browse. Learn more.

Fachinformatiker.de

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

Empfohlene Antworten

Veröffentlicht

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");

}

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 :)

Erstelle ein Konto oder melde dich an, um einen Kommentar zu schreiben.

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.