Suma liczb o numerach parzystych i nieparzystych

[Linkj]

Witam, mam problem z zadaniem: "Napisz funkcje, która znajduje sumę elementów o numerach parzystych oraz sumę elementów o numerach nieparzystych pewnej tablicy"

To moje wypociny, problem w tym, ze dla nieparzystych sume pokazuje ogromna, a dla parzystych rowną 0, niezaleznie od wprowadzonych liczb

Kod: Zaznacz cały

# include <stdio.h>

int main()
{
   int n, i, x;
   int sumaP=0;
   int sumaNP=0;
   printf ("Podaj ilosc liczb: ");
   scanf ("%d", &n);
   int liczby[n];
   for (i=0; i<n; i++)
   {
      printf ("\nPodaj %d liczbe:", i+1);
      scanf ("%d", &liczby[i]);
   }
   for (i=0; i<n; i++);
   {
      x=(i+1)%2;
      if (x=0)
      {
         sumaP=sumaP+liczby[i];
      }
      else
      {
         sumaNP=sumaNP+liczby[i];
      }
   }
   printf ("\n\n__________________________________________ \n\n");
   printf ("Suma liczb parzystych wynosi: %d \n", sumaP);
   printf ("Suma liczb nieparzystych wynosi: %d", sumaNP);
   getch ();
   return 0;
}


[xxSlayeRxx]

1. to jest c# a nie cpp ;D

po 2
if (x=0)

= jest rpzypisaniem,
== jest porownaniem

[Linkj]

A co tu jest charakterystycznego z cpp? Program ma być w c#. Dzięki za ==, poprawione, tylko teraz dla parzystych wyświetla ogromną liczbe, a dla nieparzystych 0, ;_;

[Linkj]

Nieważne, właśnie się dowiedziałem że Ci C# to nie to samo, pomimo tego ktoś pomoże??

[xxSlayeRxx]

for (i=0; i<n; i++); <- usun ;

x=(i+1)%2; -< czemu tu jest i+1 a nie samo i?

[Linkj]

W zadaniu chodzi o liczby o określonych numerach (parzyste/nieparzyste) => jest i+1, bo zakładam, że numery liczb zaczynają się od 1 (pierwsza, druga, trzecia...)

Wrzuciłem drugą pętle do pierwszej jak poradziłeś i działa ) , chociaż nie wiem dlaczego nie chciało chodzić z dwoma pętlami (pomimo faktu, że była niepotrzebna)
Dzięki za pomoc!

[Eldier]

Slayerowi chodziło żebyś usunął średnik na końcu fora. Ale jak przeniosłeś do jednej też będzie dobrze. Jak chcesz koniecznie od 1 to zrób:

Kod: Zaznacz cały

for (i=1; i<=n; i++)


[Linkj]

Pomyłka wyszła na dobre, bo jest krócej
a jak zmienię na i=1; i<=n to będę musiał przekombinować liczba[i], w nich numery idą od 0, z ciekawości sprawdziłem liczba[i-1] nie działa,
Dzięki jeszcze raz za poświęcony czas

[Eldier]

Myślałem, że jak przerzuciłeś do jednej pętli to wyrzuciłeś tablice.
Jeżeli nie korzystasz dalej z tych liczb i chcesz mieć tylko sumę liczb parzystych i nieparzystych to zrób tak:

Kod: Zaznacz cały

   int liczba;
   for (i=1; i<=n; i++)
   {
      printf ("\nPodaj %d liczbe:", i);
      scanf ("%d", &liczba);
      if (liczba%2==0)
      {
         sumaP=sumaP+liczba;
      }
      else
      {
         sumaNP=sumaNP+liczba;
      }

   }

Zaoszczędzisz w ten sposób trochę pamięci oraz operacji.
W przypadku tak małego programu nie zauważysz różnicy, ale od razu staraj się w miarę optymalnie pisać.

Pozdrawiam

Ps.
Kod pisany z palca, mogą być jakieś drobne błędy.

[xxSlayeRxx]

Eldier nei zrozumiales zadania

liczby[1] NP
liczby[2] P
liczby[3] NP
liczby[4] P
liczby[5] NP
liczby[6] P

wystarczy zamienic if (liczba%2==0) na if (i%2==0)

[Eldier]

Szczerze mówiąc nawet go nie czytałem ;p myślałem że chodzi o sumę liczb parzystych i nieparzystych.