[Pr0s_TaK]

Jest na nocnej jakiś #programista15k ? Może zerknąć ?

"Napisz program który wczytuje kolejne liczby ciągu, aż do wprowadzenia liczby ujemnej (która nie należy do ciągu) a następnie wypisze:
a: średnią liczb w ciągu
b: liczbę nieparzystych wyrazów ciągu

"
O ile udało mi się wypisać liczbę nieparzystych wyrazów ciągu to nie mogę wypisać odpowiedniej średniej, do sumy liczona jest cały czas ta ostatnia liczba ujemna i wszystko psuje. Oto co uzyskałem do tej pory.

#include
using namespace std;

int main()
{
int a;
int licznik = -1;
int licznikniepa = 0;
int suma = 0;
int licznikpar = 0;

do {
cout << "podaj liczbe ciagu: " ;
cin >> a;
licznik++;
if (a%2!=0){
licznikniepa++;
suma -=a; // chciałem odjąć ten wyraz od sumy.
}
else {
licznikpar++;

}
suma+=a;
}
while (a>0);
cout << "suma liczb w ciagu wynosi: " << suma << endl;
cout << "licznik: " << licznik << endl ;
cout << "srednia to: " << (suma/(float)licznik) << endl;
cout << "nieparzyste: " << licznikniepa << endl;
cout << "parzyste: " << licznikpar << endl;
return 0;
}
#programowanie

[tatagimpera]

@Pr0s_TaK ta źle jest wszystko

[l.....e]

@Pr0s_TaK: skup sie

[6.....6]

@Pr0s_TaK w tym roku nie będzie już religii w tym temacie lub rekomendacji dodatku do zasiłku rodzinnego i opiekuńczego w Polsce jest organizowanie spotkań z historią kredytową dla dorosłych ale nie wiem czy to nie jest tak samo jak w domu rodziców w przypadku zakupu pasującej do plakatu oprawy LED telewizory 3D hardware w tym temacie

[l.....e]

@666Krzysiek666: a ok

[wytrzzeszcz]

@Pr0s_TaK nocna jest

[l.....e]

do sumy liczona jest cały czas ta ostatnia liczba ujemna

@Pr0s_TaK: no to tu jest bug pogrzebion

[B.....u]

@Pr0s_TaK: nie prościej przy dodawaniu sumy dodać warunek, że a>0? Dopiero wtedy by dodało

[Pr0s_TaK]

@Bridu: No w sumie to tak, dzięki wielkie już jest git :P

[p.....k]

@Pr0s_TaK:

while((cin>>a), a>=0) {
licznikpar+=a%2;
suma+=a;
}

proszsz

[KrzaQ2]

@powaznyczlowiek: Ignorujesz wynik wczytania, więc trochę słabo.

[p.....k]

@KrzaQ2: kolega to chyba #programista1k ( ͡° ͜ʖ ͡°)

[KrzaQ2]

@powaznyczlowiek: bardziej mi wygląda na #programista0k, chyba że 1k to aluzja do jakiegoś stypendium... Tak czy inaczej, podawanie błędnych rozwiązań jest meh.

[p.....k]

@KrzaQ2: ehh jest druga w nocy i nie mam na to siły ale niech stracę... Czemu to rozwiązanie jest błędne? ( ͡° ͜ʖ ͡°)

[KrzaQ2]

@powaznyczlowiek: Bo ignorujesz wynik wczytywania. Jak podana zostanie liczba spoza zakresu albo jakieś literki to masz nieskończoną pętlę. Ponadto, powinieneś dodawać do zmiennej licznik_niepar.

[p.....k]

@KrzaQ2: ok zdecydowanie jesteś #programista1k XD może do tego jeszcze pełna walidacja i graficzne ramki z komunikatami? XDD nawet w treści zadania jest napisane że podawane bd liczby, a nazwa zmiennej to tylko nazwa - to jest tylko draft rozwiązania a nie kompilowalny kod ( ͡~ ͜ʖ ͡°)

Branoc!

[KrzaQ2]

może do tego jeszcze pełna walidacja

@powaznyczlowiek:

Biorąc pod uwagę jak proste to jest - tak.

i graficzne ramki z komunikatami?

@powaznyczlowiek: Nie, lepiej nie nastawiać się z góry na interaktywny terminal.

w treści zadania jest napisane że podawane bd liczby

@powaznyczlowiek: Gdzie?

to jest tylko draft rozwiązania a nie kompilowalny kod

@powaznyczlowiek: To jest kompilowalny kod ze złymi praktykami, który przedstawiasz newbowi jako dobry.

Dobranoc, programisto 15k.

[rotflolmaomgeez]

@KrzaQ2 kod od @powaznyczlowiek jest jak najbardziej ok, to zadanie na prostą pętlę i oswojenie się z językiem a nie grubą walidację danych.

[KrzaQ2]

@rotflolmaomgeez sprawdzenie wyniku operatora >> to nie jest gruba walidacja danych. Oswajanie się ze złymi praktykami nie jest ok.