Zliczanie liter

Question

Witam.

Robię zliczacz liter, a mianowicie podaje napis i jaka litere ma sprawdzic, a funkcja zlicza ile jest tych liter. Oto, co już zrobiłem:

#define _CRT_SECURE_NO_WARNINGS
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

int ZliczIle(char* s, char c) {
	int liczbaz=0;
	for (int i = 0; s[i] != NULL;i++) {
		if (s[i] == c) 
			liczbaz++;
		
	}
	return liczbaz;
}

 void main() {

	 char napis[100];
	 char znak;
	 printf("Podaj dowolny ciag znakow");
	 scanf("%s", &napis);
	 printf("Podaj dowolny znak");
	 scanf("%c", &znak);
	 int liczba = ZliczIle(&napis, znak);
	 printf("Liczba znakow %c w ciagu znakow %s wynosi %d", napis, znak, liczba);
 }

 

Najbardziej mnie zastanawia, czy ta linijka jest poprawna:

		if (s[i] == c) 

Program wczytuje sie poprawnie, natomiast po podaniu napisu konsola sie "wiesza" i wyskakuje komunikat "Program przestał działać".

Z góry dziękuje za pomoc! :)

Answer 1

Tak, wszystko jest ok. Jedynym błędem jest, że w 24 linii powinno być ZliczIle(napis, znak); (brak & przy "napis"). Właściwie, to nie wiem dlaczego to cię kompiluje w ogóle.

Comments

Nadal nie działa. :/

---

Zauważyłem kolejne: ta sama historia z & w scanf (linia 21).
I w 25 linii argumenty napis oraz znak masz w złej kolejności.

---

O tak, teraz juz lepiej, ale napotkalem inny problem. Otoz wpisuje dowolny ciag znakow, klikam enter i od razu mi przechodzi do wyniku, ile zliczylem wybranych liter, a ze nic nie podalem, to wyskakuje 0, choc nawet nie mialem jak podac tej litery. Czy wczytanie znaku przez scanf to dobry pomysl czy zastapic inna funkcja?

---

Ah, coś zostało w buforze stdin i jest czytane przy kolejnym scanf. Niestety obsługa inputu w c/c++ potrafi być strasznie irytująca... Umieść taki kod po pierwszym scanf (możesz też po każdym):

char c;
while((c = getchar()) != '\n' && c != EOF)
    /* discard */ ;

http://c-faq.com/stdio/stdinflush2.html

---

O dzięki, teraz elegancko działa jak należy! Ale przy tym ostatnim to bym chyba godzinami siedział i myślał, czemu to nie działa :P Wielkie dzięki!