Cos zle dziala...

Question

#include <iostream>
#include <string>

bool szukaj( std::string & tekst, std::string wyraz1, std::string wyraz2 )
{
    size_t fraza=tekst.find (wyraz1);
    size_t fraza2=tekst.find (wyraz2);
    if (fraza||fraza2==std::string::npos )
    {
       return false;
    }
    return true;

}

void wypiszWynik( bool czyZnaleziono )
{
    if( czyZnaleziono )
         std::cout << "Znaleziono" << std::endl;
    else
         std::cout << "Nie znaleziono" << std::endl;

}

int main()
{
    std::string napis = "Zadanie domowe z kursu C++ (http://cpp0x.pl) - najlepszy kurs C++ w Internecie!";
    wypiszWynik( szukaj( napis, "ada", "kurs" ) );
    wypiszWynik( szukaj( napis, "ada", "taki" ) );
    wypiszWynik( szukaj( napis, "C++", "cpp0x" ) );
    wypiszWynik( szukaj( napis, "C#", "cpp0x" ) );
    return 0;
}

Program ma wyrzucać odpowiedz:

Znaleziono
Nie znaleziono
Znaleziono
Nie znaleziono

A to napisane przeze mnie wszedzie odpowiada nie znaleziono...

Answer 1

if(fraza||fraza2==std::string::npos ) { /* ... */ }

Źle zaprojektowany warunek. Powinno być tak:

if(fraza == std::string::npos || fraza2 == std::string::npos) { /* ... */ }

Musisz zrobić dwa osobne porównania :)

#EDIT Ponadto Twoją funkcję można znacznie uprościć:

bool szukaj(const std::string& text, const std::string& arg1, const std::string& arg2) {
  return text.find(arg1) != std::string::npos && text.find(arg2) != std::string::npos;
}

Comments

Dzieki za podpowiedz :)

Pozdrawiam

Answer 2

fraza||fraza2==std::string::npos

Ten warunek jest nieprawidłowy. Porównaj frazy osobno, każdą. Nie wiem jak w C++ ale w Javie porównywanie Stringów poprzez == jest niebezpieczne.

Comments

Bez obaw, jest bezpieczne :)

---

Nawyk z praktyk  godzina szukania błędu w kodzie Javy i... switch dla Stringa który de facto używa '=='. Jestem już uczulony 

---

To w Javie switch działa na stringach? O.o Ojj, już dawno nie programowałem w tym języku xd

---

Zrobiłem switch dla stringa co okazało się ogromnym błędem. tylko equals