Znaki zamiast liczby.

Question

Witam, napisałem program, który ma funkcję przeglądającą łańcuch w poszukiwaniu podanego znaku. Funkcja posiada dwa parametry pierwszy
typu wskaźnik do char, drugi typu char i zwraca ilość wystąpień tego znaku w łańcuchu. Jeżeli znak nie wystąpi
wypisać odpowiedni komunikat.

Coś jest nie tak bo zamiast zwracać liczbę ilości wystąpienia znaku to zwraca jakieś buźki uśmiechnięte albo serduszka. Co skopałem? 

I jak sprawdzić, czy dany znak wystąpił w łańcuchu?

PS. WIEM, ŻE NAZWY FUNKCJI I ZMIENNYCH POWINNY BYĆ PO ANGIELSKU. 

#include <iostream>
#include <cstdlib>
#include <string>
#include <stdio.h>


int dlugosc(char lancuch[])
{
	int index;

	for (index = 0; lancuch[index] != '\0'; ++index);

	return index;
}


char szukajZnaku(char* wsk, char ilosc)
{
	char znak;
	std::cout << "\nPodaj znak: ";
	std::cin >> znak;
	ilosc = 0;

	for (int i = 0; i < strlen(wsk); i++)
	{
		if (wsk[i] == znak)

		{
			ilosc++;
		}
	}

	return ilosc;
}

int main()
{
	char line[100];
	while (1)
	{
		std::cout << "Podaj lancuch: ";

		if (fgets(line, sizeof(line), stdin))
		{
			char* p = strchr(line, '\n');
			if (p) *p = 0;
		}
		std::cout << "Dlugosc wynosi: " << dlugosc(line) << " ";
		break;
	}

	char ile = 0;
	std::cout << szukajZnaku(line, ile);

	std::cin.ignore();
	std::cin.get();
	return 0;
}

 

Answer 1

char nie służy generalne na licznik, po zadeklarowaniu jako int działa.

Comments

To samo mówiłem Pani Doktor na studiach.... No, ale cóż. Uparła się na typ 'char' i ma on zwracać ilość... Jest jakiś sposób na zrobienie tego z 'charem' (jakaś konwersja z char na int) czy nie bardzo?

---

static_cast<int>()

---

@Kacperek_code,

Uparła się na typ 'char' i ma on zwracać ilość

Typ char ma reprezentację 1 bajtową, czyli maksymalna liczba jaką można przedstawić przy pomocy niego to 256. Co jeśli w danym ciągu znakowym jakiś element będzie występował więcej razy niż 256?

---

Dobre pytanie

Answer 2

Zwracasz char więc zamiast liczby dostaniesz odpowiadający tej liczbie znak w ASCII. Dlaczego podajesz do funkcji szukajZnaku argument ilość i do tego w charze? Ogólnie to jeśli wolałbyś pracować na stringach to masz do dyspozycji funkcję  std::count, która może zliczyć ilość wystąpień jakiegoś znaku.

Comments

Funkcja posiada dwa parametry pierwszy
typu wskaźnik do char, drugi typu char i zwraca ilość wystąpień tego znaku w łańcuchu.

---

Uwierz mi, że to nie mój wymysł.

---

Wierzę, po prostu nie zauważyłem komentarza pod inną odpowiedzią 

Answer 3

Użyj std::string i funkcji std::count() a kod ulegnie kilkukrotnemu zmniejszeniu:
 

#include <iostream>
#include <algorithm>

using namespace std;

int main()
{
    string text; char sign;
    cout << "Enter text" << endl; cin >> text;
    cout << "Enter sign" << endl; cin >> sign;

    auto number = count(cbegin(text),cend(text),sign);
    cout << (number == 0 ? "Does not exist":to_string(number));
}

 

Comments

Chyba lepiej użyć:  

getline(cin, text);

zamiast

cin >> text;

 

---

To zależy czy w łańcuchu tekstowym mogą występować spacje. Jeżeli tak, to rzeczywiście trzeba użyć getline.