01/10/2018, 17:36

Sum from 1 to n in c++

#include <iostream>
using namespace std;
int summax(int n){
int sum=1,i=1;
while(i<=n){
i+=1;
sum+=i;	
}	
cout<<"sum is: "<<sum<<endl;	
}

int main(){
	int n;
	cout<<"enter integer n: ";
	cin>>n;	
	summax(n);
return 0;
}

em code tính tổng từ 1 đến n nhưng khi nhập vào ví dụ số 10 thì đáng lý ra là 55 thì nó ra 65 tức là nó đã tính tổng luôn số 11 . em xin hỏi em đã gì sai , em chạy tay rồi nhưng không hiểu sai chỗ nào

Minh Trí viết 19:48 ngày 01/10/2018

em đã tìm ra lỗi rồi :))).

Aragami1408 viết 19:52 ngày 01/10/2018

Tìm ra lỗi không phải là để đấy là xong

@drgnz nên close bài này. Nếu tiện thì unlist luôn.

Nhân tiện bạn nên viết cách giải của bạn vào đây, tick solution để những người sau xem biết lỗi gì thì có ngay bài giải ở đó. Chúng ta nên thể hiện lòng tốt vì đây là cộng đồng văn minh :))

Nguyễn Đình Anh viết 19:46 ngày 01/10/2018

i+=1;
sum+=i;

Đảo ngược 2 dòng này lại, vì nếu i = 10 thì vẫn đúng nên sum sẽ cộng thêm 11. Đảo ngược lại sẽ giải quyết được chuyện này. Với cả để luôn sum = 0

Minh Trí viết 19:45 ngày 01/10/2018

mình đã tìm ra lỗi là ở dòng while (i<n) mới đúng nhé

Nguyễn Đình Anh viết 19:49 ngày 01/10/2018

Cách đấy cũng được Nhưng Logic không được hay cho lắm

Nyan Ko viết 19:43 ngày 01/10/2018

Sai de quy nhe ban

int summax(int n){
    if (n == 0)
        return n;
    else
        return n + summax(n-1);	
}

Anh viết 19:50 ngày 01/10/2018

m thấy nên hạn chế dùng đệ quy, dùng đệ quy dễ gây lỗi stack overflow lắm.
Thêm vào đó là việc đệ quy sẽ làm tốc độ của chương trình chậm hơn nhiều.

rogp10 viết 19:51 ngày 01/10/2018

Để ++i cuối cùng thì giống như for vậy => nên để như vậy.

while này nên dùng khi for không làm được, như trường hợp thân vòng lặp phải chạy ít nhất một lần (for kiểm tra sau khi khởi tạo/tăng - trước khi lặp)

Minh Trí viết 19:36 ngày 01/10/2018

mình newbie bạn ơi :))))

Mai Anh Dũng viết 19:40 ngày 01/10/2018

Anh off-topic một chút nhưng bài này có một cách giải đơn giản là dùng toán, học ở cấp 2, là được rồi