[G.....4]

Robię zadania z informatora z informatyki na maturę. Zrobiłem program. Działa. Ale raz liczy i działa, a raz obliczy wszystko co ma obliczyć i się wysypuje. Mogę kod pokazać i wytłumaczyć bardziej jak znajdzie się jakiś ekspert. #cpp #gorzkiezale

[a.....r]

@Grzesiek4: Spróbuj może ściągnąć Visual Studio Express (jest za darmo).

Przy okazji przypuszczam, że wiem o co chodzi z tym -35: ten plik pary.txt ma windowsowe zakończenia linii (

\r\n

), a ja mam system nie-windowsowy gdzie jest tylko

\n

. Stąd program próbuje traktować

\r

jako cyfrę.

[G.....4]

@ghostface: Na maturze będzie Code::Blocks... Mogłem jeszcze wybrać Dev C++, ale to co wybrałem to wiem że było najlepsze z możliwych :P

@almafater: Raz u mnie w szkole byli studenci z politechniki Białostockiej i rozdawali klucze do "StudentLive" i "DreamSpark". Ważne rok, ale nie wiem czy od użycia czy od wygenerowania. Do tej pory nie używałem tego

[ghostface]

@Grzesiek4: no to co, że będzie - trudno. Jak się przygotowujesz lepiej jednak używać lepszych i nowocześniejszych narzędzi. Albo sobie przynajmniej debuggera pod to code::blocks podepnij.

[Rincewind]

@Grzesiek4: Potwornie przekombinowany ten kod, po co bawisz się w czytanie znak po znaku i ręczne parsowanie? Nie lepiej czytać do

std::string

operatorem

>>

?

Anyway, można dużo krócej:

#include <fstream>

#include <string>

#include <iterator>

#include <algorithm>

int convert(std::string s) {

    int res = 0, pow = 1;

    try {

        std::for_each(s.rbegin(), s.rend(), [&](char c) {

            int num = c - '0';

            if (num < 0 || num >= 3)

                throw "error";

[Rincewind]

Ale ten parser w komentarzach smutny. :/

[G.....4]

@Rincewind: Weź może na tutaj wklej, bo jakoś nie widzę tu nic :P Wybacz, ale jestem początkujący i piszę cały kod ręcznie praktycznie

@ghostface: To wypluwa debuger

[Rincewind]

@Grzesiek4: Proszzz: http://pastebin.pl/69100

[Rincewind]

@Grzesiek4: Jeśli korzystasz z MinGW (np. dla C::B) to do opcji kompilatora musisz dopisać

-std=c++0x

żeby się skompilowało.

[G.....4]

@Rincewind:

C:\Users\grzegorzu\Desktop\trojki2\main.cpp | 9 | error: expected primary-expression before '[' token|

Coś z 9 linią nie tak, dopisane tamte coś.

[Rincewind]

@Grzesiek4: Ech, czyli masz starą wersję kompilatora, jeszcze nie wspiera features z C++11. Funkcję

convert()

musisz zmienić na taką:

int convert(std::string s) {

int res = 0, pow = 1;

std::string::constreverseiterator i = s.rbegin();

while (i != s.rend()) {

int num = *i++ - '0';

if (num < 0 || num >= 3)

return -1;

res += num * pow;

pow *= 3;

}

return res;

}

Wstawiłbym na

[Rincewind]

Tam gdzie jest kursywa powinno być (bez spacji) const reverse iterator

[G.....4]

@Rincewind: Dopóki nie ma tych śmiesznych znaczków to nie ma problemu. Ściągałem wczoraj ze strony producenta. Ale może jak się bardziej wgłębię w to, to się pobawię.

[G.....4]

@Rincewind: Już nie mam siły do tego :P nie zapisuje nic w

out.txt

. Dam se spokój i jednak w coś pogram.

[Rincewind]

Ale tworzy w ogóle

out.txt

? Bo jeśli tak, to zamknij plik przed wyjściem z programu, może nie opróżnia bufora.

[G.....4]

@Rincewind: Tworzy, ale plik jest pusty. Dla mnie z kolei ten kod jest przekombinowany i nie wiem o co chodzi tutaj, myślałem że w kodzie jest zamknięcie pliku. Jeszcze wiele przede mną widzę ;)

[Rincewind]

@Grzesiek4: Tak naprawdę ważna jest tylko funkcja

convert

– pobiera liczbę zapisaną trójkowo jako

string

i zwraca przekonwertowaną na

int

lub -1 gdy wejście jest nieprawidłowe (np. wystąpi cyfra 5).

Możesz w pętli wczytywać do stringa kolejne liczby trójkowe przez file >> str; Nie musisz wtedy przejmować się spacjami i końcami linii, STLowe IO samo to za Ciebie załatwi. W kolejnych wczytaniach (operatorem >>) będziesz miał pobrane całe ciągi znaków jako