Prosty problem - moje błędne rozwiązanie :D

Question

Musimy znaleźć linię w której jest najwięcej różnych liter.

Napisałem więc poniższy kod. Moje pytanie brzmi dlaczego w linijce z błędem ( zaznaczona komentarzem) nie jestem w stanie porównać 2 obiektów typu string ( czy są identyczne). Jest to oczywiście błąd tylko ja nie wiem na czym polega :D

#include <iostream>
#include <string>
#include <fstream>
int main()
{
    using namespace std;
    string linijka;
    int points = 0;
    int rekord = 0;
    bool znaleziono;
    string zwyciezca;
    string tablica[] = {"q","w","e","r","t","y","u","i","o","p","a","s","d","f","g","h","j","k","l","z","m","x","c","v","b","n","m"};
    fstream plik("przyklad.txt");
    while(getline(plik,linijka))
    {
        points = 0;
       for(int i = 0; i < sizeof(tablica)/sizeof(*tablica); i++)
        {
            znaleziono = false;
            for(int x = 0; x < linijka.length(); x++)
            {
                if(tablica[i] == linijka[x] ) // BŁĄD BŁĄD  BŁĄD  BŁĄD
 
                {
                    if(!znaleziono)
                    {
                    points++;
                    }
                }

            }
        }
        if(points > rekord)
        {
            rekord = points;
            zwyciezca = linijka;
        }
    }


    return 0;

}

 

Answer 1

 jestem w stanie porównać 2 obiektów typu string ( czy są identyczne). Jest to oczywiście błąd tylko ja nie wiem na czym polega :D

Nie porównujesz stringów.

Porównujesz i-ty element tablicy stringów (czyli string) z x-tym elementem linijki (charem).

Ogólnie nie ma powodu trzymać zbiór jedno-znakowych stringów na chary, jak może być jeden zbiór charów (tablica/vector charów lub po prostu std::string).

Comments

Dzięki, zrozumiałem 

Answer 2

bo porównujesz stringa z charem.

 if(tablica[i] == linijka[x] )

"tablica" to tablica stringow, więc odwołując się do 

tablica[i]

odwołujesz się do stringa z tej tablicy, a odwołując sie do

linijka[x]

odwołujesz się do chara ze stringa linijka o indexie x

Comments

Dzięki, zrozumiałem