Sumowanie liczb parzystych

Question

Witam, mam takie zadanie 

Napisz program, który przy użyciu pętli sumuje liczby parzyste z przedziału od x
do y, gdzie wartości x i y są podane przez użytkownika oraz 10 <= x < y <= 100.

Napotkałem błąd podczas sumowania liczb parzystach

#include <iostream>
#include <conio.h>

using namespace std;
/* run this program using the console pauser or add your own getch, system("pause") or input loop */

int main(int argc, char** argv) {
	
	int x=0, y=0, suma=0, sumap=0;
	
		
	 cout << "Podaj liczbe X w przedziale od 11-99: ";
	 cin>>x;
	 
	 if(x<=10 || x>= 100){
	 	
	 	cout << "Podaj liczbe "<<x<<" nie miesci sie w przedziale od 11-99: "<<endl;
	  	cout << "Podaj liczbe X jeszcze raz: ";
	 	cin>>x;
	 	
	 	
	 	while(x<=10 || x>= 100){
	  	
	  	cout << "Podaj liczbe "<<x<<" nie miesci sie w przedziale od 11-99: "<<endl;
	  	cout << "Podaj liczbe X jeszcze raz: ";
	 	cin>>x;
	  	
	 	 }
	 	
	 }
	 
	 cout << "Podaj liczbe Y w przedziale od "<<x<<"-99: ";
	 cin>>y;
	 
	 if(y<=x || y>= 100){
	 	
	 	cout << "Podaj liczbe "<<y<<" nie miesci sie w przedziale od "<<x<<"-99: "<<endl;
	  	cout << "Podaj liczbe Y jeszcze raz: ";
	 	cin>>y;
	 	
	 	
	 	while(y<=x || y>= 100){
	  	
	  	cout << "Podaj liczbe "<<y<<" nie miesci sie w przedziale od "<<x<<"-99: "<<endl;
	  	cout << "Podaj liczbe Y jeszcze raz: ";
	 	cin>>y;
	  	
	 	 }
	 	
	 }
	 
	 //sumowanie liczb z zakresy od x do y podane przez uzytkownika
	 for(int i=x; i<=y; i++)
	 {
	 	suma=suma+i;
	 	
	 }
	 cout<<"suma: "<<suma;
	
	suma=sumap;
	//sumowanie liczb parzystych z zakresy od x do y podane przez uzytkownika
	 if(sumap%2==0){
	 	
		 for(int i=x; i<=y; i++){
	 		
		 	sumap=sumap+i;
		 
		}
		  
	 }
	cout<<"sumap: "<<suma;
	 
	
	
	getch();
	return 0;
}

czy mógłby mnie ktoś naprowadzić jak to zrobić?

 

Answer 1

Strasznie przekombinowałeś. Wykorzystaj wzór na sumę pierwszych N liczb parzystych i własność rzutowania liczb zmiennoprzecinkowych na liczby całkowite:

#include <iostream>

int sumEvenIntegersFromOne(int integer)
{
    int halfInteger;
    halfInteger = int(0.5*integer);

    return halfInteger*(halfInteger + 1);
}

int main()
{
    int lower, upper;

    std::cout << "Podaj dolny przedzial: ";
    std::cin >> lower;
    std::cout << "Podaj gorny przedzial: ";
    std::cin >> upper;
    std::cout << "Suma wynosi: " << sumEvenIntegersFromOne(upper) - sumEvenIntegersFromOne(lower - 1) << '\n';

    return 0;
}

 

Comments

Ok, dzięki.

Twój kod jest o wiele lepszy, ale jeszcze nie miałem czegos takiego jak halfInteger, int sumEvenIntegersFromOne(int integer). 

Więc zrobię po swojemu, ale dzięki za pomoc i tak 

---

To są tylko nazwy, które sam wymyśliłem. Równie dobrze możesz to nazwać a, b, c, choć nie polecam, bo jak wrócisz do kodu za parę tygodni, to nie będziesz wiedział, co te zmienne przechowują i oznaczają.

sumEvenIntegersFromOne to nazwa funkcji, która sumuje naturalne liczby parzyste począwszy od 1. halfInteger to zmienna pomocnicza, która przechowuje połowę wartości ze zmiennej integer. Od biedy możesz te wzory wrzucić do głównej funkcji, a sumEvenIntegersFromOne usunąć całkowicie. Aczkolwiek funkcje są dość prostym zagadnieniem i pewnie lada moment je poznasz.

---

ok, dziękuje za wytłumaczenie ;D

---

Nie chcę się czepiać ale to chyba najgorsze możliwe rozwiązanie.

Sto razy wydajniej jest to zrobić w zwykłej pętli for albo while.

Oczywiście nie można użyć wzoru na sumę ciągu arytmetycznego co byłoby optymalne.

---

Oczywiście nie można użyć wzoru na sumę ciągu arytmetycznego co byłoby optymalne.

Oczywiście tak to jest zrobione. A poza tym uzasadnij, jak już się czepiasz.

---

Bo nie od dziś wiadomo że rekurencja jest złym rozwiązaniem. W niektórych firmach wręcz jest zakaz używania rekurencji. To po pierwsze.

A po drugie to dla kogoś kto nie wie jak zrobić prostą pętlę proponujesz rekurencję co jest o wiele bardziej skomplikowane.

Najprościej to zrobić tak

#include <iostream>
using namespace std;

int main(){
   int x,y,suma=0;
    cout << "Podaj dolny przedział: ";
    cin >> x;
    cout << "Podaj górny przedział: ";
    cin >> y;

    for (int i=x;i<=y;i+=2) {
        suma+=i;
    }

    cout << "Suma wynosi: " << suma << endl;
    return 0;
}

 

---

Jaka rekurencja, o czym Ty piszesz?

Zresztą skąd miałem wiedzieć, że autor wątku nie wie, czym jest funkcja. Nie napisał tego wprost. Twój kod nie zadziała, gdy użytkownik poda nieparzysty x, więc jest błędny.

---

|O przepraszam mój błąd to nie rekurencja. Tak namieszałeś z nazwami funkcji i zmiennych że pomyliłem. Ale w takim razie jest jeszcze gorzej bo tam nie ma wcale pętli ;) A w treści jest napisane że ma być to liczone w pętli.

Słusznie zauważyłeś niedoskonałość mojego kodu, już poprawiłem

#include <iostream>
using namespace std;

int main(){
    int x,y,suma=0;
    cout << "Podaj dolny przedział: ";
    cin >> x;
    cout << "Podaj górny przedział: ";
    cin >> y;
    
    if (x%2==1) x+=1;
    if (y%2==1) y-=1;

    for (int i=x;i<=y;i+=2) {
        suma+=i;
    }

    cout << "Suma wynosi: " << suma << endl;
    return 0;
}

 

---

@Benek lepiej od razu użyć dzielenia dla intów:

int halfInteger = integer / 2;