Преобразование программы в функцию

[fs444]

У Дейтлов есть задача:

После того, как вы определили, что делает программа в упражнении 3.50, преобразуй-
те программу в соответствующую функцию, удалив ограничение, требующее, чтобы
второй аргумент был неотрицательным.

Код из упр. 3.50 такой:

#include<iostream>
using std::cout;
using std::cin;
using std::endl;

int mystery (int, int);

int main()
{
   int x, y;
   cout << "Введите два целых числа: ";
   cin >> x >> y;
   cout << "Результат: " << mystery( x, y ) << endl;
   return 0;
}

// Параметр Ь должен быть положительным
// целым для предотвращения бесконечной рекурсии
int mystery (int a, int b )
{
   if ( b = 1 )
      return a;
   else
      return a + mystery( a, b - 1 );
}

Я его переделал в

#include<iostream>
using std::cout;
using std::cin;
using std::endl;

#include<windows>

int mystery(int, int);
int mystery2(int, int);

int main()
{
   int x, y;

   mystery2(x, y);

   system("pause");
   return 0;
}

// Параметр b должен быть положительным
// целым для предотвращения бесконечной рекурсии
int mystery (int a, int b )
{
   if ( b = 1 )
      return a;
   else
      return a + mystery(a, b - 1 ) ;
}

int mystery2 (int c, int d)
{
   cout <<"Vvedite dva celih chisla: ";
   cin >> c >> d;
   cout << "Resultat: " << mystery( c, d ) << endl;
}

Как считаете, я справился с поставленной задачей?

[rrrFer]

fs444
нет.

int x, y;
   mystery2(x, y);

переменные не инициализировал,но уже передаешь в функцию
функция mystery2 возвращает int но в ней нет return
второй аргумент по прежнему не может быть отрицательным или равным 0. Чтобы происходил выход при отрицательном второй агрумента возможно надо заменить b=1 на b<=1

[rrrFer]

#include<windows> // эта строчка тебе зачем?

[fs444]

rrrFer, мда, пойду делать =))

#include<windows> // эта строчка тебе зачем?

чтобы работала команда

system("pause");

[atenon]

Еще ошибка

if ( b = 1 )

Рекурсии не будет.
Что касается задания прога должна сложить "а" "b" раз.
Чтобы "b" могло быть и отрицательным его нужно брать по абсолютному значению, либо проверять, если отрицательное то запускать рекурсию со знаком плюс (делать еще одну функцию)

[fs444]

Сделал так:

#include<iostream>
using std::cout;
using std::cin;
using std::endl;

#include<windows>

int mystery(int, int);
int mystery2(int, int);

int main()
{
   int x, y;

   mystery2(x, y);

   system("pause");
   return 0;
}

// Параметр b должен быть положительным
// целым для предотвращения бесконечной рекурсии
int mystery (int a, int b )
{
   if (b >= 0)
   {
      if ( b = 1 )
         return a;
      else
         return a + mystery(a, b - 1 ) ;
   }
   else
   {
      if ( b = 1 )
         return a;
      else
         return a + mystery(a, b + 1 ) ;
   }

}

int mystery2 (int c, int d)
{
   cout <<"Vvedite dva celih chisla: ";
   cin >> c >> d;
   cout << "Resultat: " << mystery( c, d ) << endl;
}

Правильно?

[atenon]

int mystery2 (int c, int d)
{
cout <<"Vvedite dva celih chisla: ";
cin >> c >> d;
cout << "Resultat: " << mystery( c, d ) << endl;
}

Функцие передаются не инициализированные параметры, также она не не возращает значения (void) (Кстати на фиг она нужна?)

if ( b = 1 )

Переменной b присваевается 1 и сразу функция возращает а.
Нужно исправить на if ( b == 1 )

else
{
if ( b = =1 )
return a;
else
return a + mystery(a, b + 1 ) ;
}

В этой части программы при отрицательном "b", число "а" будет складываться не "b" раз, а "b+2" раза.
Здесь if ( b == 1 ) Нужно заменить +1 на -1 .

if (b >= 0)

По условию рекурсии "b" никак не может быть равным нулю.
Я бы лучше создал ДВЕ рекурсивные функции одна для положительного "b" другая для отрицательного.

int main()
{
 int x, y, k;

	cout <<"Vvedite dva celih chisla: ";
	cin >> x >> y;

	if (y>0)
		k=mystery1 (x, y);
	else
		k=mystery2(x, y);

	cout<<"Itog  "<<k<<"\n";
   return 0;
}


int mystery1 (int a, int b )
{
      if ( b == 1 )
         return a;
      else
         return a + mystery1(a, b - 1 ) ;
}
int mystery2 (int a, int b )
{
      if ( b == -1 )
         return a;
      else
         return a + mystery2(a, b + 1 ) ;
}

[fs444]

Функцие передаются не инициализированные параметры

Почему не инициализированные? Я же написал:

   int x, y;

   mystery2(x, y);

также она не не возращает значения (void)

спасибо, поправил.

Кстати на фиг она нужна?

Что именно ты имеешь в виду?

Нужно исправить на if ( b == 1 )

Поправил.

else
{
if ( b = =1 )
return a;
else
return a + mystery(a, b + 1 ) ;
}

В этой части программы при отрицательном "b", число "а" будет складываться не "b" раз, а "b+2" раза.

Почему "b+2"? b ведь каждый раз на 1 уменьшается, потому что оно отрицательное.

Здесь if ( b == 1 ) Нужно заменить +1 на -1 .

Не совсем понял, о чем ты здесь?

По условию рекурсии "b" никак не может быть равным нулю.
Я бы лучше создал ДВЕ рекурсивные функции одна для положительного "b" другая для отрицательного.

Согласен.

А у тебя после выполнения программы консоль сама не закрывается? А то мне вот в твой код приходится system("pause") добавлять.