Suma w jezyku C.

Question

Witam!

Jestem studentem pierwszego roku informatyki, i dopiero zaczynam przygode z programowaniem. Mam problem z jednym, dosc prostym zadaniem - mam napisac program, ktory bedzie obliczal sume podanej liczby naturalnej. Dla liczby mniejszej od 0 program ma prosic o podanie innej liczby (nie musi wyswietlac komunikatu) Napisalem cos takiego:

#include <stdio.h>
#include <math.h>
int main(){
    int n, suma, i;
    scanf("%d", &n);
    i=0;
    while(n<0){
        scanf("%d", &n);
    }
    for(suma=0; i!=(n+1); i++){
        suma=suma+i;
    }
    printf("%d", suma);
    
}

Program ogolnie dziala - co by nie wpisac ladnie wszystko sumuje. Problem jest tylko w tym, ze zadania sa sprawdzane automatycznie - sa automatycznie poddawane kilku testom. Moj program nie zdaje jednego testu - prawidlowy wynik to 6, a moj program wylicza 0. Niestety nie wiem co to za test, i nie wiem co jest nie tak w tym kodzie.

 

PS: Z gory przepraszam za brak Polskich znakow, pisze z anglojezycznego linuxa.

Comments

bedzie obliczal sume podanej liczby naturalnej

Co to znaczy?

---

Dobrze, podaje zatem przyklad zeby bylo jasniej

Σ5 = 1+2+3+4+5 = 15. Takie dzialanie ma rozwiazywac moja petla. Liczba naturalna jest 5, i moja petla ma wyznaczyc jej Σ.

---

6 = 1+2+3 sprawdzałeś ręcznie co twój program piszę jak się poda 3 na wejście?

Dla testów możesz instrukcję print przenieść, a nawet lepiej, skopiować do wnętrza pętli - zobaczysz jakie są kolejne sumy.

Troche dziwnie napisałeś kod

scanf();
while (n<0) {
    scanf();
}

Jeśli instrukcje mają się wykonać co najmniej raz bardziej naturalne jest wykorzystanie pętli do-while

do {
    scanf();
} while (n<0)

Zerowanie zmiennych i i suma masz tak bardzo dziwnie. Zmienną sterującą pętli zwykle inicjuje się na wejściu pętli, często ją się tam nawet deklaruje.

Z kolei suma - wypadałoby ją zainicjować wyraźnie przed pętlą. Oczywiście tak jak jest powinno działać, ale jakoś dziwnie ten kod wygląda.

Można też wyzerować obie zmienne w inicjalizacji pętli

for (suma = 0, i = 0; ...

PS. Suma liczb od 1 do N włącznie to N*(N+1)/2 - łatwiej będzie sprawdzać.

---

Troszke zmienilem kod, ale niestety, nadal nie dziala jak trzeba. Teraz wyglada tak:

#include <stdio.h>
#include <math.h>
int main(){
    double n, suma;
    scanf("%lf", &n);
    while(n<0){
        scanf("%lf", &n);
    }
    suma=n*((n+1)/2);
  
    printf("%.lf", suma);     
}

Zmienilem int na double bo po zmianie wczesniejszego kodu wyskakiwal dodatkowy blad. Kombinowalem tez z nawiasami we wzorze, niestety bezskutecznie. Teraz jest znow to samo co poprzednio:

Probowalem recznie wpisywac 3, program wylicza dobrze, czyli 6. 

PS: Troche celowo nie uzylem petli do-while, bo to tresc nastepnego zadania- trzeba przerobic ten kod z wykorzystaniem petli do-while :) 

---

Trochę niepotrzebne użyłeś nawiasów. Jeśli n jest nieparzyste to n+1 jest parzyste i odwrotnie. Stąd iloczyn n * (n+1) jest zawsze parzysty więc dzieli się przez 2.

Jeśli dzielisz jeden z tych czynników to dostajesz ułamek i trzeba użyć liczb zmiennoprzecinkowych.

Dopisz może '\n' w printfie - tak zwyczajowo program powinien kończyć swoje wyniki nową linią. Nie wiadomo dokładnie jak działają te testy.

---

Probowalem tez bez tego dodatkowego nawiasu, efekt jest ten sam. I z \n tez to samo. Zaczynam podejrzewac, ze to w zadaniu jest blad, nie wiem czemu ciagle w tym tescie wychodzi 0 zamiast 6.

Answer 1

bedzie obliczal sume podanej liczby naturalnej

Zamiast pętli możesz użyć wzoru n(n+1)/2 i zamiast liniowej mieć stałą złożoność obliczeniową.

Answer 2

Możesz jasniej napisać co twoja pętla for robi? :) 

  for(suma=0; i!=(n+1); i++){
        suma=suma+i;
    }

zwróć uwagę na uzycie sumy.