[sylwke3100]

Mam taki problem. Otóż w tym kodzie https://gist.github.com/sylwke3100/b928e098d7fa1865f8c0 Nie działa mi referencja przez co "Text" nie jest zmieniany przez żadną metodę z żadnej klasy (oprócz tej w której jest). Jak temu zaradzić ?

#programowanie #cpp

[KrzaQ2]

@sylwke3100: Jeśli chcesz trzymać referencję do tekstu w klasie to trzymaj referencję, a nie kopię.

http://ideone.com/GPSESg

[sylwke3100]

@KrzaQ2: Ja nie chciałem się getami i temu podobnym bawić. Raczej chodziło mi żeby w klasach wywoływanych przez CristallGrammar była możliwość modyfikacji obiektu.

[KrzaQ2]

@sylwke3100: Nie musisz bawić się ƶadnymi getami. Ale jeśli chcesz trzymać referencję to trzymaj referencję. Tylko wtedy musisz uwaƶać, ƶeby Ci obiekt nie znikł zanim przestaniesz się z nim bawić.

[sylwke3100]

@KrzaQ2: Dalej mi jednak nie modyfikuje. Zobacz gista

[KrzaQ2]

@sylwke3100: Na pewno trzymasz referencj​ę?

w moim przykładzie istotna jest 18. linia:

string& ref;

[sylwke3100]

@KrzaQ2: GIsta zobacz

[KrzaQ2]

@sylwke3100: W klasie

Grammar

masz kopię - ale zakładam ƶe to jest poprawne (tylko ja bym wymusił podawanie nazwy konstruktorem, ale to zupełnie inna sprawa).

W klasie

SingleGrammar

masz kopię - tutaj moƶliwe, ƶe chciałeś referencję?

btw, nie wiedziałem, ƶe gisty mają rewizje, fajnie.

btw2, omg przesłanianie nazw. Jakie to nieczytelne.

[sylwke3100]

@KrzaQ2: Czyli co mam w kazde klasie robić zmienną w taki sposób jak podałeś ? myślałem że to trochę inaczej działa.

[KrzaQ2]

@sylwke3100: Teraz jak się wczytałem to raczej nie. Czy na pewno podajesz tam stringa, który będzie sparsowany? Moƶe

isRule(T)

zwraca

false

?

[sylwke3100]

@KrzaQ2: W isRule to ma zwrócić true lub false, parser ma za zadanie przetworzyć i usunąć ten znaleziony ciąg i tu nie działa referencja

[KrzaQ2]

@sylwke3100: Ale jeśli

isRule

zwróci false to nawet nie wywołasz

parse

, więc nie będzie co miało zmieniać stringa.

[sylwke3100]

@KrzaQ2: Do parse przechodzi bo mi wywala co usuwa ale erase nie działa (to znaczy podejrzewam że działa tylko lokalnie)

[KrzaQ2]

@sylwke3100: Jak na moje to działa

Od razu polecam lekturę i poczytanie o forwarddeklaracjach.

[sylwke3100]

@KrzaQ2: http://ideone.com/3ihfcI Niekoniecznie

[KrzaQ2]

@sylwke3100: Miałem na myśli zmianę zawartości zewnętrznego stringa przez referencję. Nad logiką musisz popracować sam ;)

[sylwke3100]

@KrzaQ2: Ale nie zauważyłeś że on nie usunął cyfr mimo że zostały zdefiniowanie ?

[KrzaQ2]

@sylwke3100: http://ideone.com/gA4ebW Dzięki nazywaniu wszystkich zmiennych "Text" nie zorientowałeś się, ƶe

bool SingleGrammar::isRule(std::string &Text)

{

return (Text.substr(0, this-> Text.length()) == Text );

}

Sprawdzasz czy pierwsze

n

znaków podanego tekstu jest równe jemu samemu - czyli czy ten tekst ma dokładnie

n

znaków (gdzie

n

to długość rule'a).

Po poprawieniu: http://ideone.com/Q7wSL8

[sylwke3100]

@KrzaQ2: Slingle Grammar jest ok. Problem jest w NumbersGrammar

[KrzaQ2]

@sylwke3100: Kliknij linka, zobacz.

[sylwke3100]

@KrzaQ2: Dzięki za pomoc. Teraz się dopiero zorientowałem o co chodzi w tym błędzie :D

[sylwke3100]

@KrzaQ2: Jedyny problem to teraz jak zapisać wyjście tak żeby użytkownik dostawał np w numbers/floatnumbers int/double a w text string lub jakoś tak zorganizować żeby mógł sbie odczytywać na zasadzie (w sensie żeby takie pola miał do odczytu).

{

nazwa,

typ (w sensie czy to number/fnumber/text)

zmiennaokreslonegotypu

}

[KrzaQ2]

@sylwke3100: variant