funkcja strcmp

W9
  • Rejestracja:około 11 lat
  • Ostatnio:około 4 godziny
  • Postów:32
0

Witam mam napisać program korzystający z tablicy char,który będzie odczytywał w pętli słowo po słowie az do wpisania "gotowe".
Potem program pokaze ilośc wpisanych słów(bez "gotowe")
napisałem coś takiego:

Kopiuj
#include <iostream>
#include <string>
#include<conio.h>
#include<cstring>
int main()
{
    using namespace std;
	char word[100];
   // char gotowe =  "gotowe";
	cout<<"Podawaj slowa, (kiedy skonczysz napisz \"gotowe\")\n";
    cin>>word;
	int count=0;
    for(count =0;strcmp(word,"gotowe");++count)
    {
    	cout<<word<<" ";
    	cin>>word;
    
	}	
	cout<<"wpisano "<<count<<" slow"<<endl;
    
    getch();
    return 0;
    
}

1.Nie wiem dlaczego program nie chce się skompilowac, tzn kopiluje sie gdy dodam " ! "przed funkcje strcmp().
2.Wiem ze był takie temat juz na forum : http://4programmers.net/Forum/C_i_C++/224634-pytanie_do_petli_while_tablica_char
,lecz czy da radę to zrobić mniej więcj tak jak wygląda mój kod?
Pozdrawiam

n0name_l
  • Rejestracja:ponad 12 lat
  • Ostatnio:prawie 5 lat
  • Postów:2412
2
Kopiuj
#include <cstring>
#include <cstdio>

int main(void) {
	char buffer[256] = {0}; unsigned counter = 0;
	while((scanf("%255s", buffer) && strcmp(buffer, "gotowe")) && ++counter);
	printf("%u\n", counter);
	return 0;
}
n0name_l
Nie podales po prostu poprawnych danych wejsciowych, czego oczekiwales?
Lucas Darkstorm
Lucas Darkstorm
  • Rejestracja:około 13 lat
  • Ostatnio:około 4 lata
2

Przekombinowałeś, synu.

Kopiuj
#include <iostream>
#include <cstring>
 
using namespace std ;
 
int main()
{
  char word[100] ;
  int count = 0 ;
 
  while (1)
  {
    cin >> word ;

    if (!strcmp(word, "gotowe")  ) break ;

    count++ ;
  }
 
  cout << count << endl ;
}
edytowany 3x, ostatnio: Lucas Darkstorm
n0name_l
Bez przesady z tym break.
Lucas Darkstorm
Lucas Darkstorm
A co złego jest w używaniu breaka?
n0name_l
W samym break nic strasznego nie ma (poza imo zaciemnianiem kodu), natomiast jest cos w uzywaniu break tam, gdzie jest on zupelnie zbedny.
Lucas Darkstorm
Lucas Darkstorm
Rozumiem. Z drugiej strony to jest tak mały programik, że można se na to pozwolić ;).
_13th_Dragon
Zastanów się co się stanie jak przekierujesz do tego programu strumień z pliku a plik się skończy przed "gotowe"
_13th_Dragon
Przecież while((cin&gt;word)&amp;&amp;(strcmp(word,&quot;gotowe&quot;))) ++count; o wiele czytelniejsze i bez niebezpieczeństw
Lucas Darkstorm
Lucas Darkstorm
Wiem, o co Ci chodzi, jednak zakładałem, że użytkownik zawsze wpisze to słowo, aby zakończyć działanie programu.
W9
  • Rejestracja:około 11 lat
  • Ostatnio:około 4 godziny
  • Postów:32
0

n0name: Dzięki , tylko widziałem już to zadanie zrobione w takim stylu, lecz czy da się to zrobić bez scanf , procentów itd?:)Tzn żeby wyglądał tak jak mój lub Lorda Darkstorma ?
Jesli się nie da to czy mógłbys mi poowiedziec po co jest ta linijka? char buffer[256] = {0} oraz scanf("%255s", buffer)
Lord Darkstorm: Twój kod sie nie kompiluje,tak jak wiele moich prób z użyciem linijki:while (strcmp(word, "gotowe") )
Próbowałem już na rózne sposoby,ale kompiluje się tylko jesli przed strcmp daje !.

edytowany 1x, ostatnio: winio94
Lucas Darkstorm
Lucas Darkstorm
Weź teraz skompiluj kod, który jest teraz w moim poście. W tamtym znalazłem mały błąd.
n0name_l
  • Rejestracja:ponad 12 lat
  • Ostatnio:prawie 5 lat
  • Postów:2412
1

Tak, da sie. Wystarczy zamienic scanf na cin i printf na cout.

Kopiuj
#include <cstring>
#include <iostream>
#include <iomanip>
 
int main(void) {
    char buffer[256] = {0}; unsigned counter = 0;
    while((std::cin >> std::setw(sizeof(buffer)) >> buffer && strcmp(buffer, "gotowe")) && ++counter);
    std::cout << counter << '\n';
    return 0;
}
edytowany 1x, ostatnio: n0name_l
Zobacz pozostałe 6 komentarzy
vpiotr
@_13th_Dragon: napiszę to dla nowicjuszy, bo obaj znamy swoją różnicę zdań w tym temacie: jeden wiersz w którym jest zawarty kod z kilku innych prawie zawsze NIE będzie czytelniejszy. Tylko w wypadku gdy będzie po prostu krótszy ma szansę być czytelniejszy - o ile nie zmusza do dłuższej analizy (bo np. wykonuje konwersje w locie albo mocno korzysta z obowiązującej kolejności wykonania zamiast nawiasów).
vpiotr
@n0name_l: tak sobie pomyślałem, że powinno być chyba setw(sizeof(buffer) - 1)?
_13th_Dragon
@vpiotr, a propos setw - to trochę inaczej działa niż dla cin. A propos czytelności - jeżeli mówisz o czytelności dla początkujących koderów to dla nich każdy kod - nie czytelny, jeżeli mówisz o czytelności dla przynajmniej średnio zaawansowanych to jeden wiersz zawsze czytelniejszy. Jeżeli z tym zdaniem się nie zgadzasz to się zgodzisz kiedy będziesz średnio zaawansowanym. Oczywiście są nieliczne wyjątki, jako przykład - nie chodzi o jeden wiersz który nie mieści się na ekranie.
vpiotr
@_13th_Dragon: dla mnie pisanie nieczytelnego kodu nie świadczy o wysokim zaawansowaniu autora. Jest granica między kodem zwięzłym a nieczytelnym - bardzo łatwo ją przekroczyć jeśli się łączy w jednym wyrażeniu wczytanie z warunkiem i inkrementacją - w jednej linijce nie poprawia czytelności, ale tutaj autor jest tego świadomy. Dla mnie jeśli już jednolinijkowce to takie jak poniżej - z count_if. Koniec offtopa.
_13th_Dragon
@vpiotr, w tym poście akurat (++counter) w warunku jest nieistotny, więc wepchnięcie go do warunku na siłę - zmniejsza czytelność. Ale nie jest to efekt zapisu jednolinijkowego. Dla mnie również pisanie nieczytelnego kodu nie świadczy o wysokim zaawansowaniu autora - tylko że nieczytelność dla mnie to np pisanie w 6 wierszach z błędem zamiast jednego wiersza bez błędu.
_13th_Dragon
  • Rejestracja:prawie 20 lat
  • Ostatnio:11 dni
2

Może zrobię jeszcze całkiem c++ wersje:

Kopiuj
#include <iostream>
#include <string> 
using namespace std ;
 
int main()
  {
   size_t count=0;
   for(string word;(cin>word)&&(word!="gotowe");++count) {}
   cout<<count<<endl;
   return 0;
  }

Wykonuję programy na zamówienie, pisać na Priv.
Asm/C/C++/Pascal/Delphi/Java/C#/PHP/JS oraz inne języki.
edytowany 1x, ostatnio: _13th_Dragon
Zobacz pozostałe 22 komentarze
Sopelek
Rzeczywiście. Wydawało mi się, że dostawałem takie warningi wcześniej.
Lucas Darkstorm
Lucas Darkstorm
@_13th_Dragon, int main() { unsigned int x = -5 ; }. Taki najprostszy przykład.
n0name_l
@Lord Darkstorm Bo C++ zaklada, ze programista jest swiadomy swoich czynow, ma to pewne plusy wydajnosciowe. Jesli deklarujesz interfejs, ktory potrzebuje zmiennej bez znakowej, to programista ma wiedziec, ze jak przekaze tam cos innego, to mozliwe jest odpalenie rakiet w kierunku chile.
_13th_Dragon
Powiedziałbym nawet "Bo C++ niesłusznie zakłada, ze programista zawsze jest inteligentny :D
Lucas Darkstorm
Lucas Darkstorm
Wiem o tym panowie :). Pisząc w C++ uzyskuje się szybszy program, bo operacje zawarte w kodzie nie są w żaden sposób sprawdzane, jednak z tym wiążą się właśnie różnego typu UB, na które programista musi uważać. Za to istnieją właśnie języki, które mają wszystko podefiniowane, lecz odbija się to wówczas na wydajności. Mówiąc krótko, coś za coś.
n0name_l
  • Rejestracja:ponad 12 lat
  • Ostatnio:prawie 5 lat
  • Postów:2412
3

W C++ to sens by mialo uzyc tego C++ :)

Kopiuj
#include <iostream>
#include <iterator>
#include <algorithm>
#include <string>
using namespace std;

int main() {
	cout << count_if(istream_iterator<string>(cin),
					 istream_iterator<string>(),
					 [](const string& w) { return w != "gotowe"; });
	return 0;
}
Zobacz pozostałe 2 komentarze
vpiotr
return w != "gotowe" ?
n0name_l
A to bardzo proste i duzo bardziej przejrzyste niz wersja z c-stringa. Mowi policz ilosc wszystkich stringow w strumieniu roznych od &quot;gotowe&quot;, doslownie. Gdyby nie smieszna skladnia z tymi iteratorami to by pewnie to ladniej wygladalo.
n0name_l
@vpiotr a co z tym nie tak?
W9
Łoł, dzieki za wytłumaczenie, na pewno przydatna rzecz, tylko ta składnia troszkę odstrasza... :) Tylko że, czy to w ogóle działa, bo u mnie wpisanie "gotowe" nic nie powoduje
vpiotr
@n0name_l: w sumie nic, po prostu mam uczulenie na porównywanie (char *), ale tutaj jest string i pewnie dlatego działa.
W9
  • Rejestracja:około 11 lat
  • Ostatnio:około 4 godziny
  • Postów:32
0

ohoho ile postów... niech ja to ogarne :D

vpiotr
  • Rejestracja:prawie 14 lat
  • Ostatnio:prawie 3 lata
1

Po przeróbce oryginalnego programu

...wyszedł mi program prawie identyczny z postu Lorda Darkstorm...

edytowany 1x, ostatnio: vpiotr
W9
@vpiotr niestety jak już wyżej pisałem takie cuś się nie kompiluje, robiłem bardzo podobnie kilak razy, lecz za każdym razem się zawiesza :) Więc albo to jest źle albo to wina Deva... :)
vpiotr
trochę lepsza wersja: http://ideone.com/54kLFB
vpiotr
setw też się przyda: http://ideone.com/ovnf9t
W9
  • Rejestracja:około 11 lat
  • Ostatnio:około 4 godziny
  • Postów:32
0

Dzięki wszystkich, przetrawię wszystkie sposoby które zostały tutaj pokazane. Pozdrawiam

_13th_Dragon
  • Rejestracja:prawie 20 lat
  • Ostatnio:11 dni
1

To może takie coś:

Kopiuj
#include <cstring>
#include <iostream>
#include <iomanip>
using namespace std;
 
int main()
  {
   char buffer[256];
   size_t counter=0;
   while(((cin>>setw(sizeof(buffer))>>buffer)&&(strcmp(buffer,"gotowe")))) ++counter;
   cout<<counter<<endl;
   return 0;
  }

setw nie pozwoli wczytać do buffer więcej niż 256 znaków

EDIT: po zmianach wyszło dokładnie to co u @n0name_l tu: http://4programmers.net/Forum/C_i_C++/233025-funkcja_strcmp?p=1030590#id1030590


Wykonuję programy na zamówienie, pisać na Priv.
Asm/C/C++/Pascal/Delphi/Java/C#/PHP/JS oraz inne języki.
edytowany 1x, ostatnio: _13th_Dragon
W9
  • Rejestracja:około 11 lat
  • Ostatnio:około 4 godziny
  • Postów:32
0

Orłem to może ja jeszcze nie jestem, lecz ty też nie;) Twój kod nie działa, temat do zamknięcia:)

W9
EDit kto wywalił post gośćabcd czy jak mu było? :D
msm
Sam usunął.
gośćabc
http://ideone.com/05Tbuc to jest ten kod niezmieniony. Twój kod na bank nie działa a mój owszem; post został usuniety i lekcja wyciągnięta, nie próbować pomagać więcej niż to potrzeba bo Cię "płocie" zjedzą :D

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.