C# Можете объяснить почему программа выдает неправильный ответ?

[shkap]

Условие задачи: Даны действительные числа a, b ( a << b ). Найти третий член из генерации чисел ((-1)^i) * (1 + 1/2 +.....1/i); (i = 1, 2...), которая не пpинадлежит отрезку [a,b].

namespace ConsoleApplication15
{
class Program
{
static void Main(string[] args)
{
double a = 0, b = 0, r = 0, sum = 0, s = 0, l = 0, k = 32000;
Console.Write("Введите значение a = ");
a = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите значение b = ");
b = Convert.ToDouble(Console.ReadLine());
for (double i = 1; i <= k; i++)
{
do
{
l = l + 1;
sum += (1 / l);
r = Math.Pow(-1, i) * sum;
if ((r < a) || (r > b))
s++;
}
while (s == 3);
} 
Console.WriteLine("Третье число: " + r);
Console.ReadKey();
}
} 
}

[Sciv]

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

for (double i = 1; i <= k; i++)

В чём сакральный смысл использовать тут double?! По большому счёту это вообще лишний цикл, как и переменная k.

И зачем вы разделяете i и I? В целом формула выглядит как сумма членов (-1)^i * 1/i

И ещё: -1 в степени i даст либо 1 либо -1 - имеет ли смысл вызывать Pow, если проще и быстрее определить знак по чётности i?

[shkap]

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

В чём сакральный смысл использовать тут double?! По большому счёту это вообще лишний цикл, как и переменная k.

И зачем вы разделяете i и I? В целом формула выглядит как сумма членов (-1)^i * 1/i

И ещё: -1 в степени i даст либо 1 либо -1 - имеет ли смысл вызывать Pow, если проще и быстрее определить знак по чётности i?

А как тогда нужно сделать?

[Sciv]

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

А как тогда нужно сделать?

Предполагаю, что как-то так (писано на коленке в Борман Си):

#include <stdio.h>
#include <iostream>


int getSign(int num){
  if (num % 2 == 0) {
	   return 1;
  }
  else{
	  return -1;
  }
}

double getCurrentNumber(int num){
	return  1 / (float)num;
}

int main(int argc, _TCHAR* argv[])
{
	using namespace std;

	double a,b, result;
	int count = 0;
	double sum = 0;
	int i = 1;

	cout << "Enter a: ";
	cin >> a;
	cout << "Enter b: ";
	cin >> b;
	while (count < 3){
		sum = sum + getCurrentNumber(i);
		result = getSign(i) * sum;
		if ((result < a)||(result > b)) {
			count++;
			cout << "Number #" << count << " = " << result << "\n";
		}
		i++;
	}

	cout << "Result Number = " << result << "\n";
	system("pause");
}

[shkap]

Спасибо!)