C++, problem z zadaniem Kalkulator 1/Calc

pytanie zadane 16 marca 2017 w SPOJ przez Konrad Kurczyński Użytkownik (580 p.)
edycja 17 marca 2017 przez Konrad Kurczyński


string zestawdanych;
float wynik;
cin>>zestawdanych;
if (zestawdanych[0] = "+")
{
 wynik = zestawdanych[2]+zestawdanych[4]
{
(...)

Błąd wyskakuje przy powyższym kodzie, adres zadania: http://pl.spoj.com/problems/CALC/

2	komentarz 16 marca 2017 przez tangarr Mędrzec (154,860 p.) i== <-- serio?

komentarz 17 marca 2017 przez Konrad Kurczyński Użytkownik (580 p.)

Dziękuję, wyelminowało to jeden z błędów, literówka...

komentarz 17 marca 2017 przez niezalogowany

Zła kategoria + łamiesz regulamin dotyczący zadań ze SPOJ. Popraw wszystko inaczej będę zmuszony ukryć to pytanie

1 odpowiedź

odpowiedź 16 marca 2017 przez mokrowski Mędrzec (155,460 p.)

Zapisałeś zmienną ilośćprob oraz inne z użyciem znaków spoza ASCII. Język C++ wymaga aby tego typu nazwy nie zawierały znaków narodowych.

Następną sprawą jest to że nie podajesz które zadanie ze SPOJ'a. Ludzie mają zgadywać aby Ci pomóc?

komentarz 17 marca 2017 przez Konrad Kurczyński Użytkownik (580 p.)
edycja 17 marca 2017 przez Konrad Kurczyński

Napisałem Zadanie to kalkulator, poprawiłem to według twoich zaleceń, jednak to nic nie zmienia (w innych programach (pisanych w Xcode) mogłem używać polskich znaków, jeżeli były na spoja to je eliminowałem przed wrzuceniem, aczkolwiek ten jest źle zrobiony na etapie Xcode)

komentarz 17 marca 2017 przez mokrowski Mędrzec (155,460 p.)

Podawaj link do zadania. Nikt nie będzie domyślał się "który kalkulator, jaki flamaster jakie liczby pierwsze" (jest to częsty powód dlaczego nikt nie odpowiada).
Każde IDE pracujące w systemie dopuszcza wpisywanie znaków alfabetów spoza ASCII. Robi się to przecież w komentarzach, robi się to w komunikatach wpisywanych. Rodzaj IDE jest nieistotny (ja także pracuję w XCode). Zasada jest prosta. Nie używaj znaków spoza ASCII w nazwach funkcji/klas/zmiennych itp. i się do tego przyzwyczaj.
Popatrz na for. Jak inkrementujesz zmienną i? Na marginesie popraw na : ++i a nie na i++. Dobre nawyki warto wyrabiać.
Popatrz na zestawdanych. w którym miejscu definiujesz że jest tablicą?
Dlaczego stosujesz zmienne globalne?

komentarz 17 marca 2017 przez Evelek Nałogowiec (28,960 p.)

Popatrz na for. Jak inkrementujesz zmienną i? Na marginesie popraw na : ++i a nie na i++. Dobre nawyki warto wyrabiać. - O! I to mnie zastanawia czasami. Czytałem dużo na temat ++i vs. i++, różne fora, komentarze ze Stackoverflow itp. I dużo osób twierdziło, że ++i jest "wydajniejsze" cokolwiek by to miało znaczyć. Różne kody z Assemblera widziałem, gdzie ludzie pokazywali, że niezależnie czy to jest preinkrementacja czy postinkrementacja to kod jest taki sam. Pytanie więc do Ciebie mokrowski, czy można to jakoś uzasadnić i wybrać "lepszą" inkrementację?

komentarz 17 marca 2017 przez mokrowski Mędrzec (155,460 p.)

Standard języka C++ nie definiuje jak ma być generowany kod asemblera. Może być generowany mniej lub bardziej optymalnie. I bywa naprawdę różnie. Za to standard definiuje operator pre/post oraz pre/post dekrementacji. W przypadku operatorów post, jest tam przesyłany int. Taka jest prawdziwa sygnatura tej funkcji/metody. Dość że budzi to znane kontrowersje. Ale to przyzwyczajenie łatwo zmienić i IMHO (jeszcze) warto.

komentarz 17 marca 2017 przez Konrad Kurczyński Użytkownik (580 p.)
edycja 17 marca 2017 przez Konrad Kurczyński

Dziękuje za odpowiedź, a czy ze zmiennej string nie mogę odczytać jednego znaku(np. wynik = zestawdanych[2] + zestawdanych[4])? Przecież ona jest jakby tablicą.