Usuwanie bialych znakow

Question

Dobry wieczór.

Mam do zrobienia funkcja która usunie wszystkie białe znaki, czyli spacje, tabulacje itp. Oto co zrobiłem:

void UsunBialeZnaki(char *s) {
	int i = 0, j = 0;
	while (s[i]) {
		if (s[i] == ' ' || s[i] == '\t' || s[i] == '\n')
			++i;
		s[j] = s[i];
		++j;
		++i;

	}
	printf("%s", s);
}

void main() {

	 char tablica[100];
	 printf("Podaj dowolny ciag znakow...");
	 scanf("%s", tablica);
	 UsunBialeZnaki(tablica);
}

Problem wygląda następująco: gdy uruchamiam program w konsoli i wpisuje jakiś ciąg znaków np. "Usun biale znaki   ." to wyświetla mi sie "Usun". Co tutaj należy poprawić, bo kod pod względem logiki wydaje się poprawny. Z góry dziękuję za pomoc.

Comments

Uwaga do algorytmu: jak już zastosujesz rady od Crissa, przetestuj sobie napis, w którym występują dwa białe znaki po sobie, np.: "a<spacja><spacja>b".

---

Sprawdzilem i zauważyłem, że:

-usuwa tylko jedna spację, gdy są obok siebie;

-na koncu zdania przypisuje ostatnią litere napisu.

np podaje na wejsciu napis: "Ala ma kota"

zwraca mi "Alamakotaa"

Trochę to dziwne.

---

To dlaczego usuwa tylko jedną spację, to już musisz wymyślić sam czytając swój algorytm. Ale dlaczego zostają literki na końcu postarałem się wytłumaczyć niżej, w komentarzu do odpowiedzi Crissa.

---

Dobra dzięki za pomoc! Już wiem, gdzie mniej więcej szukać problemu i jak go rozwiązać.

Answer 1

Problem nie wynika z algorytmu tylko z obsługi inputu. Do `tablica` wczytujesz tylko pierwsze słowo. Poczytaj w internecie jak sobie z tym poradzić.

Jednak jedna rzecz o której zapomniałeś, to że powinieneś stawiać zero na koniec wynikowego ciągu.

Comments

Stawiać zero to znaczy co?

---

Nie wiem czy o to dokładnie chodziło Criss'owi, ale o bajt zerowy na końcu znaku, taki odnośnik, że to koniec słowa

---

Rozumiem, że zero to chodziło o bajt zerowy \0 (znaczy tak się domyślam). Natomiast bardziej chciałem się dowiedzieć w którym miejscu programu ma być to zero.

---

Jak tworzysz napis "abc" to w pamięci komputera on jest przechowywany jako 'a', 'b', 'c', '\0'. Ten znak na końcu służy do oznaczenia końca napisu. Kiedy wołasz do pracy funkcję printf("%s", str), to ona będzie wypisywać po kolei znaczki z str-a, do póki nie trafi na '\0'.

U Ciebie brak wstawienia tego znaku powoduje błędne działanie programu. Załóżmy, że masz napis 'a', '<spacja>', 'b', '\0'. Wrzucam go do Twojej funkcji usuń białe znaki. Ona działa tak, że zostawi napis w postaci 'a', 'b', 'b', '\0'. Powinieneś po zakończeniu kasowania białych znaków oznaczyć, gdzie teraz kończy się napis (wstawić w to miejsce '\0'). Czyli po pracy Twojej funkcji, napis powinien wyglądać tak: 'a', 'b', '\0'.

Answer 2

Tak nie usuniesz wszystkiego porządnie. Są jeszcze znaki unicode zero-terminated
https://330k.github.io/misc_tools/unicode_steganography.html
oraz znaki unicode o wysokich adresach.
https://www.irongeek.com/i.php?page=security/unicode-steganography-homoglyph-encoder
W obu wypadkach są one obecne w tekście ale nie są widoczne przy wyświetlaniu. Co prawda nie jest to szkodliwe, ale użytkownik może na nich zapisać zakodowaną wiadomość. Jeżeli chcesz utrudnić użytkownikom tajne porozumiewanie się, podawanie linków do stron, to kasuj te znaki, blokuj załączniki.