[kierownica]

Uczę się c++ i chciałem napisać program sumujący kolejne wartości tablicy.
Czy wie ktoś dlaczego wynik pokazuje mi zawsze 0? Tak jak by nie widziało mojej pętli.
Pozdrawiam :)

#programowanie

[Narkan]

@kierownica: kod zawsze na pastebina wrzucaj

[kierownica]

http://pastebin.com/uLNfyavV

[I.....u]

@kierownica: Nie możesz zrobić

int mojatablica[k]

Żeby tak stworzyć tablicę musisz podać stałą, a nie zmienną

[arek-niziolek]

@kierownica: for(n; n>0; n--)
Wydaje mi się, że zerujesz tutaj zmienną n, a potem robisz pętlę for(int i=0 ; i<n; i++)

[kierownica]

@IchBinKrzychu dzięki zmieniłem:

int main()
{
int mojatablica[5];
cout << "-------" << endl;
wczytywanie(mojatablica, 5);
return 0;
}

Ale cały czas wynik wychodzi mi 0 w funkcji.

[k.....v]

@kierownica: odpowiedź arka jest poprawna :)

[MakontufTRZY]

@kierownica: program musi z góry wiedzieć ile pamięci przypisać do tablicy. Jeśli chcesz tablicę dynamiczną to musisz to zrobić za pomocą wskaźników.

[kierownica]

@arek-niziolek dzieki wielkie :))

[G.....g]

@kierownica: korzystaj z std::accumulate
http://en.cppreference.com/w/cpp/algorithm/accumulate

[F.....s]

@IchBinKrzychu: @MakontufTRZY: Technicznie może, któryś z standardów C na to pozwalał i bywa że kompilator przymknie na to oko.

Variable-length automatic arrays are allowed in ISO C99, and as an extension GCC accepts them in C90 mode and in C++.

@kierownica: Zawsze używaj flag kompilatora typu -Wall oraz -pedantic. Dzięki temu kompilator będzie bardziej surowy i łatwiej będzie wychwycić błąd.

[Freezerx]

Tak w ogóle to chyba niepotrzebnie w tym przykładu funkcja wczytywanie jest jako int. Myślę, że spokojnie można by było zrobić void wczytywanie i usunąć return n, który zwraca wartość wprowadzaną do funkcji.

[KrzaQ2]

@Frondeus: Przecieƶ wyraźnie z tego cytatu wynika, ƶe VLA dodano w C99, a o C++ nie ma mowy. Ponadto w C11 ich obsługa przestała być obowiązkowa.

[F.....s]

@KrzaQ2:

and as an extension GCC accepts them in C90 mode and in C++

Innymi słowy kompilator może na to pozwolić mimo że nie jest to zgodne z standardem C++.

Nie mówię że jest to prawidłowe, tylko co się stanie jeśli kolega spróbuje uruchomić kompilator i nie doda odpowiednich flag.

[matthosu]

@kierownica: Jak już zauważył Arek po drugim forze zmienna jest równa 0
Dlatego trzecia pętla nie wywołuje się ani razu. Więc wynik musi być zerem skoro nic do niego nie dodajesz
Napraw to poprzez dodatkową zmienną w drugim forze, chyba najprościej w ten sposób.

[marcinzwwa]

@kierownica: unikaj jezyka polskiego, nawyki łap od samego poczatku