Folgende Aufgabe möchte ich lösen: Ich habe einen Text als Input und möchte in diesem Text nun alle Buchstaben a zählen. Nun habe ich mal "ganz klein" begonnen und möchte entscheiden, wenn ich einen Buchstaben eingebe, ob dieser ein a ist oder nicht. Wenns kein Element im Input hat, dann möchte ich, dass das Programm "hallo" ausgibt. Dazu habe ich auch mal ein Programm geschrieben, voilà:

#include <iostream>
#include <ios>

int main()
{
  //Eingabe
  std::cin>>std::noskipws;

  int i=0;
  char letter = 'a';
  int counter=0;

  char*const t = new char [1];

    if(std::cin>>t[i])
    {
      if (t[i]==letter)
	{
	  ++counter;
    	}
    }
    else std::cout<<"hallo"; //Kein Input

  std::cout<<letter<<":"<<counter<<"of"<<i+1<<".\n";

 delete [] t;

return 0;
}

So. Nun zum Problem: Wenn ich nichts als Input "reingebe", dann gibt mir das Programm nicht "hallo" aus.... Warum ist mir ein Rätsel...Wisst ihrs?

Vielleicht ein kleiner Hinweis: Ich bin ein absoluter Anfänger. Vor zwei Monaten wusste ich nicht einmal was C++ ist...

Liebe Grüsse
Rungo

Meinst du nicht, dass es etwas übertrieben ist, wegen einem Buchstaben auf dynamische Speicherverwaltung zurückzugreifen?

Du weisst halt nicht, wann std::cin>>t[i] false liefert. Bei einem Enter müsste in t[i] '\n' stehen, bei einem C-d (Linux) oder Strg-Z (imho Windows) hingegen müsste hallo ausgegeben werden.

Also. Ich habe mal den Tipp mit dem else in die Tat umgesetzt und dann mal keinen Input reingegeben und natürlich kam das "hallo" auch sofort raus. Das Problem ist, wenn ich nun "b" eingebe kommt das "hallo" auch raus, was ich nicht will. Und die Antwort (sei mir nicht böse) von dir, gnuro, verstehe ich nicht wirklich...

Gruss
Rungo

std::cin>>t[i];

	if (t[i]==letter)
	{
		++counter;
	}
	else if(t[i] == '\n')
	{
		 std::cout<<"hallo"; //Kein Input
	}

#include <iostream>
#include <string>

int main()
{
    std::string input;
    unsigned int counter = 0;
    std::cin >> input;

    if(input.empty()) cout << "hallo\n";
    else
    {
        for(size_t i = 0; i < input.size(); ++i) if(input[i] == 'a') ++counter;
        cout << "Sie haben " << counter << " mal den Buchstaben 'a' eingegeben";
    }
}

#include <iostream>
#include <ios>

int main()
{
  std::cin>>std::noskipws;

  char*const t = new char [1];
  for(char letter='a';letter<='z';++letter) 
 {
   int i=0;
   int counter=0;
   for(;;++i)
     {
       if(std::cin>>t[i])
	 {
	   if (t[i]==letter)
	     {
	       ++counter;
	     }
	   else if(t[i]=='\n')break; //Input zu Ende
	 }
     }
  std::cout<<letter<<":"<<counter<<"of"<<i<<".\n";

}  

 delete [] t; 
 return 0;
}

Das Problem: Für a gibts den richtigen Wert aus, für den Rest des Alphabets nicht mehr... Was ist falsch?

Gruss
Rungo

size_t inputSize = input.size( );
for(size_t i = 0; i < inputSize; ++i)
{
    if(input[i] == 'a') ++counter;
}

#include <iostream>
#include <string>

int main()
{
    std::string input;
    unsigned int counter = 0;
    std::cin >> input;

    if(input.empty()) cout << "hallo\n";
    else
    {
        for(char letter = 'a'; letter <= 'z'; ++letter)
        {
            for(size_t i = 0; i < input.size(); ++i) if(input[i] == letter) ++counter;
            cout << "Sie haben " << counter << " mal den Buchstaben '" << letter << "' eingegeben";
        }
    }
}

#include <string>
#include <iostream>
#include <algorithm>

int main()
{
  std::string line;
  std::getline(std::cin, line);

  if(line.empty())
  {
    std::cout << "Hallo?\n";
  }
  else
  {
    int count = std::count(line.begin(), line.end(), 'a');
    std::cout << "Sie haben " << count << " mal den Buchstaben a eingegeben.\n"
  }
}

@Rungo,
Gerade noch in der Vorschau gesehen. Du kannst aus dem Stream nur einmal lesen. Was du einmal gelesen hast, ist auch eingelesen. Oder meinst du ein anderes Problem? Dann präzisiere die Problembeschreibung bitte.

Grüssli

oder noch besser die Größe vor der Schleife berechnen, damit .size( ) nur einmal aufgerufen wird. Wer weiß, wie lang der Text wird...

Ein Kompiler wird das höchstwahrscheinlich selber wegoptimieren. Das sind unnötige Mikrooptimierungen.

Grüssli

#include <iostream>
#include <string>

#define CHAR_MAX_VALUE (255)

int main()
{
    std::string input;
    unsigned int letter_statistics[CHAR_MAX_VALUE];

    //initialisieren mit 0
    for(size_t i = 0; i < CHAR_MAX_VALUE; ++i) letter_statistics[i] = 0;

    std::cin >> std::noskipws >> input;

    if(input.empty()) cout << "hallo\n";
    else
    {
        for(size_t i = 0; i < input.size(); ++i) ++letter_statistics[(int)input[i]];
        for(size_t i = 0; i < CHAR_MAX_VALUE; ++i) cout << (char)i << ": " << letter_statistics[i] << "\n";
    }
}

Was hilft ist die Bibliothek:

#include <cctype>

...

for(  ...  )    if( islower( input[ i } ) ++counter;

@wxSkip: Du hast 2 Schleifen verschachtelt, das heißt, das Ding läuft für jeden Buchstaben 26x durch.

Was hilft ist die Bibliothek:

#include <cctype>

...

for(  ...  )    if( islower( input[ i } ) ++counter;

Ich wollte auch, dass das 26 mal durchläuft, so sah das zumindest bei Rungo aus.

Ich wollte auch, dass das 26 mal durchläuft, so sah das zumindest bei Rungo aus.

Du hast recht, hab mir da nur die neue "Aufgabenstellung" durchgelesen und war im Kopf noch bei der Einmal-über-alle-Input-Buchstaben-Schleife. Diese reicht ja auch aus, wenn man für jeden Buchstaben nur prüft, ob er zwischen 'a' und 'z' liegt (oder kurz eben islower( ) ).

wxSkip schrieb:

Ich wollte auch, dass das 26 mal durchläuft, so sah das zumindest bei Rungo aus.

Du hast recht, hab mir da nur die neue "Aufgabenstellung" durchgelesen und war im Kopf noch bei der Einmal-über-alle-Input-Buchstaben-Schleife. Diese reicht ja auch aus, wenn man für jeden Buchstaben nur prüft, ob er zwischen 'a' und 'z' liegt (oder kurz eben islower( ) ).

Hm, geht es um das Zählen von allen a-z (Gesamtsumme) oder um jeden Buchstaben einzeln? Im letzteren Fall ist die Statistiklösung natürlich richtig... Ich geh jetzt lieber was essen... Weiter viel Spaß!

Ich melde mich auch wieder einmal zu Wort. Leider verstehe ich nicht viel von euren Vorschlägen und es ist mir bewusst, dass ich für jeden Buchstaben nochmals den ganzen Input einlese, was nicht sonderlich elegannt ist, klar. Wie könnte ich mein Programm (siehe den letzen meiner Beiträge) in dieser Form trotzdem zum laufen kriegen? Ich bin einfach daran interessiert, wo mein Fehler liegt.

Liebe Grüsse
Rungo

Was soll das?

Naja. Ich konstruiere mir ein array der Länge 1, in das ich nach und nach die Werte einlese... (Erklärung eines Anfängers). Evtl könnte man das natürlich anders lösen, aber soweit ich das richtig sehe, funktionierts...

Ja, aber es ist einfach unleserlich. Nimm lieber einen stinknormalen char.
Wie ich schon geschrieben habe, liegt der Fehler daran, dass du in jeder Schleife neu die Buchstaben einliest. Was genau verstehst du an meinen beiden Programmen nicht?

Du benötigst irgendwas von string input und so. Irgenwie kann ich schon erahnen, wie es funktioniert, obwohl ich soetwas noch nie gesehen habe, allerdings darf ich das auch noch gar nicht benutzen, da dies eine Aufgabe für die Schule ist. Wieso funktionnierts denn nicht, wenn ich in jeder Schleife neu einlese?

OK. Dürft ihr char-Arrays, also C-Strings verwenden?
Was gibt std::cin>>t[i] zurück?
Wenn es tatsächlich so funktioniert wie du es willst, liest du Zeichen bis zu einem Zeilenumbruch ein und überprüfst sie auf dem Buchstaben a. Die nächste Zeile überprüfst du auf den Buchstaben b usw.

- Achtung: Grossbuchstaben kannst du dir selber was noch dazudenken
- Und schau dir mal die Ascii-Tabelle an

#include <iostream>

int main()
{
	char input[500];
	const char *seek = input;
	int cntAll = 0;
	// ein Array, weil ja jeder Buchstabe gezaehlt werden soll
	int cntLetter[26];

	// Array mit 0 vorfuellen, damit man irgendwo mit zaehlen beginnen kann
	for(int i=0; i<26; ++i)
		cntLetter[i] = 0;

	// Text einlesen (inc Leerzeichen)
	std::cin.getline(input, 500);

	// nochmal anschauen ;)
	std::cout << input << std::endl;

	// solange loopen, bis das Terminierungszeichen \0 erreicht ist
	while(*seek)
	{
		// Buchstabe nochmal ausgeben zum pruefen
		std::cout << *seek << std::endl;

		// Jeden Buchstaben zaehlen
		cntAll++;

		// Spagetti-Methode
// ENTWEDER
		if(*seek == 'a')
			cntLetter[0]++;
		if(*seek == 'b')
			cntLetter[1]++;
// ODER
		// Ansonsten geht auch
		// 'b'-'a' = 98-97 = 1 ... somit cntLetter[1] erhoehen
		// !!Achtung index noch pruefen von 0-25
		int index = *seek - 'a';
		cntLetter[index]++;

		// Zeiger auf den naechsten Buchstaben
		seek++;
	}

	// Ausgabe des Zaehlergebnisses
	for(int i=0; i<26; ++i)
		// Buchstabe 'a' hat im Ascii-Zeichensatz die 97 ... drum 97+0=a / 97+1=b usw.
		std::cout << static_cast<char>(i+'a') << ":" << cntLetter[i] << std::endl;
}