В чем ошибка?

[L6go1as]

Подскажите, в чем ошибка. Нужно ввести 2 числа и подсчитать сумма всех промежуточных членов. Собственно сперва сумма их считается как нужно, а вот в конечном ответе почему-то не она ... в чем ошибка допереть что-то не могу ...

#include <iostream>

using namespace std;

int main()
{
	cout << "Please enter the first of your namber: ";
	int a = 0;
	cin >> a;

	cout << "Please enter the second of your namber: ";
	int b = 0;
	cin >> b;

	cout << "\n";

	int i, j;
	for (i = a, j = a; i > a, i <= b; i++, j += i)
		cout << j << "\t";

	cout << "\n\nSum of all terms: " << j;

	int x = 0;
	cin >> x;

	return 0;
}

[Abstraction]

0) Условие (i > a, i <= b) - это оригинально. Оператор , вычисляет одно, другое, а в качестве ответа возвращает один из двух результатов. Для объединения логических условий его использовать не надо, но в данном случае я вообще не понимаю, зачем условие i>a.
В принципе, нежелательно объединять через запятую инструкции, порядок выполнения которых существенен (i++, j += i).

1) for(X; Y; Z) A;
Сначала выполняется X, затем проверяется Y, затем выполняется A, затем Z, и только затем снова проверяется Y.
Соответственно, в данном случае на последней итерации i=9, j=44 - вывели 44. Потом i++, j+=i, проверка условия (i<=b неверно) и мы выходим из цикла, но j уже 54.

[kypck]

Сообщение от L6go1as

в чем ошибка допереть что-то не могу ...

Вот исправленный код. Ошибка в том, что после вывода последнего данного j увеличивается еще на i, в итоге сумма получается не та, что нужна.

Вот работающий код

#include <iostream>

using namespace std;

int main()
{
	cout << "Please enter the first of your namber: ";
	int a = 0;
	cin >> a;

	cout << "Please enter the second of your namber: ";
	int b = 0;
	cin >> b;

	cout << "\n";

	int i, j;
	for (i = a, j = a; i <= b; i++, j += i)
		cout << j << "\t";
        j-=i;

	cout << "\n\nSum of all terms: " << j;

	int x = 0;
	cin >> x;

	return 0;
}

[Syuf]

for (i = a, j = a; i <= b; i++, j += i)

for (i = j = a; i <= b; j += ++i)

[L6go1as]

Abstraction, "i > a" - это условие еще с первой "ревизии", когда просто пытался диапазон нужный выделить, как итог добавил еще одно условия, при этом забыв убрать первое.

По пункту 1.) по моему, ты чутка напутал, выход из цикла происходит при достижения "i" заданного числа "b", в данном случае при достижения числа "9", при этом после проходения "j += i", и "j" равно "44", после чего выводится "j". Но вот почему 54 вместо 44 - не понимаю.

Syuf, "for (i = j = a; i <= b; j += ++i)" - результат тот же, 54 вместо 44 (если без других изменений в коде).

kypck, спасибо, теперь работает как нужно. Только вот почему по завершения цикла "j" увеличивается еще раз на "i" ?

[Abstraction]

Сообщение от L6go1as

Abstraction,

По пункту 1.) по моему, ты чутка напутал, выход из цикла происходит при достижения "i" заданного числа "b", в данном случае при достижения числа "9", при этом после проходения "j += i", и "j" равно "44", после чего выводится "j".

Нет, не напутал.
Выход из цикла происходит при условии, что выражение i<=b возвращает "ложь".
Вопрос: при каком минимальном i условие i<=9 ложно?
Второй вопрос: в описании for выше, выполняется ли A после того, как Y было вычислено и получилась "ложь"?

[L6go1as]

Abstraction, теперь вроде бы разобрался, неправильно указал условие, если бы было просто i < b, + запутался как for испольняется. Поэтому смутился когда cout выдал значение 35 - что эквивалент 8, а не 9. Из-за этого добавил "=" - теперь cout выдает 44 - что эквивалент 9, но вот при финале еще идет последняя операция которая выдает уже 54 - что эквивалент уже 10 ...

Вообщем разобрался, спасибо. Ибо считал, что сперва испольняется все то, что в скобках, а потом уже блок ниже ...

[Syuf]

Syuf, "for (i = j = a; i <= b; j += ++i)" - результат тот же, 54 вместо 44 (если без других изменений в коде).

Я код и не смотрел, вы тут и без меня разберетесь, я просто показал, как лаконичнее записать цикл, и чтобы не возникало претензий по поводу запятой справа.

[L6go1as]

Syuf, За это кстати отдельное спасибо, просто такий записи я еще не видел, но оно работает. Другое дело, что через запятую пока воспринимается как "ок", а ваша запись из будущего ))