Błąd tablic w programie

Question

Witam piszę program, który nie jest trudny jednak jest w nim błąd, o którym wiem. Błędem są tablice. Próbowałem wielokrotnie zamienić pętle while na for jednak nie wiem jak dobrać petli warunek aby zakończył wpisywanie liter do pojedynczego znaku. '@'

#include <iostream>
#include <cctype>

int main()
{
    std::cout << "Wpisz zdanie" << std::endl;
    char ch, samogloski[6] = {'a', 'e', 'y', 'i', 'o', 'u'},
         spolgloski[17] = {'b', 'c', 'd', 'f', 'g', 'h', 'j', 'k', 'l', 'm', 'n', 'p', 'q', 'r', 's', 't', 'w', 'z'};
    int licz_spol=0, licz_samo=0, inne=0;
    std::cin.get(ch);
    while(ch != '@') // blad tablic ... 
    {
        if(isalpha(samogloski[6])) licz_samo++; //samogloski
	else if(isalpha(spolgloski[17])) licz_spol++;  //spogloski
        else inne++;// inne znaki
	std::cin.get(ch);
    }//while
    std::cout << licz_samo <<" samoglosek\n" << licz_spol << " spolglosek\n" << inne << " wyrazow\n";
    return 0;
}

Proszę o podpowiedź jak zamienić pętle na for.(warunek pętli).

 

Answer 1

Ważnym pytaniem jest co ten program ma robić?

Comments

Użytkownika na wpisać zdanie kończąc znakiem '@' (tylko litery, dlatego w programie znajduje się funkcja isalpha())

Program ma wyliczyć ile jest spółgłosek, ile samogłosek i ile wyrazów liczył program.

---

Jeśli pętla while działa poprawnie (a działa) to warunek pętli for moze byyć identyczny czyli może ona wyglądać tak:

    for(; ch != '@';)

Jesli jednak twierdzisz że nie wyglada to elegancko to zobacz że przed pętlą masz zawsze  std::cin.get(ch); więc wrzućmy to do pętli:

    for(std::cin.get(ch); ch != '@';)
I jeszcze możemy zauważyć że wczytujemy zawsze nowy znak na końcu pętli więc możemy usunąc z tamtąd std::cin.get(ch); i również wkleić je do pętli
 

    for(std::cin.get(ch); ch != '@';std::cin.get(ch))

Błędem przez który się to nie kompiluje jest zbyt mała tablica na spółgłoski. Jest ich 18 w twoich klamrach.
I ostatnim głównym problemem jest to że algorytm wewnątrz pętli jest błędny.

if(isalpha(samogloski[6])) to nie jest sprawdzenie czy wczytany znak jest w tablicy samogłosek.

Answer 2

...
const char * samogloski = "aeyiou";
const char * spolgloski = "bcdfghjklmnpqrstwz";
			 
int licz_spol = 0, licz_samo = 0, inne = 0;
	
while (std::cin.get(ch) && ch != '@')
{        
	if (strchr(samogloski, ch)) ++licz_samo; //samogloski
	else if (strchr(spolgloski, ch)) ++licz_spol; //spogloski
	else ++inne; // inne znaki
} //while

 

Comments

Sprawdza się tylko przy wprowadzaniu pojedynczych znaków.

---

Nie no, działa elegancko. Spróbuj ten kod:

#include <iostream>
#include <cstring>

int main()
{
	const char * samogloski = "aeyiou";
	const char * spolgloski = "bcdfghjklmnpqrstwz";

	int licz_spol = 0, licz_samo = 0, inne = 0;

	char ch;
	while (std::cin.get(ch) && ch != '@')
	{
		if (strchr(samogloski, ch)) ++licz_samo; //samogloski
		else if (strchr(spolgloski, ch)) ++licz_spol; //spogloski
		else ++inne; // inne znaki
	}
	std::cout << "Samo: " << licz_samo << "; Spol: " << licz_spol << "; Inne: " << inne << std::endl;
}

 

---

Tak, mój błąd dodatkowo wprowadziłem ostatnią linię pętli jako std::cin.get(ch);

 

Dziękuję wszystkim za pomoc.