it-swarm.dev

Рекурсивный Фибоначчи

Мне трудно понять почему

#include <iostream>

using namespace std;

int fib(int x) {
    if (x == 1) {
        return 1;
    } else {
        return fib(x-1)+fib(x-2);
    }
}

int main() {
    cout << fib(5) << endl;
}

приводит к ошибке сегментации. Как только х падает до 1, разве он не должен вернуться?

34
blcArmadillo

Когда x==2 вы вызываете fib(1) и fib(0):

return fib(2-1)+fib(2-2);

Посмотрите, что произойдет, когда fib(0) оценивается ...

149
Georg Fritzsche

Причина в том, что последовательность Фибоначчи начинается с двух известных сущностей, 0 и 1. Ваш код проверяет только одну из них (будучи одной).

Измените свой код на 

int fib(int x) {
    if (x == 0)
        return 0;

    if (x == 1)
        return 1;

    return fib(x-1)+fib(x-2);
}

Чтобы включить и 0 и 1.

39
LiraNuna

Почему бы не использовать итерационный алгоритм?

int fib(int n)
{
    int a = 1, b = 1;
    for (int i = 3; i <= n; i++) {
        int c = a + b;
        a = b;
        b = c;
    }           
    return b;
}
11
Dzmitry Huba

По определению, первые два числа в последовательности Фибоначчи - это 1 и 1 или 0 и 1. Поэтому вы должны с этим справиться.

#include <iostream>
using namespace std;

int Fibonacci(int);

int main(void) {
    int number;

    cout << "Please enter a positive integer: ";
    cin >> number;
    if (number < 0)
        cout << "That is not a positive integer.\n";
    else
        cout << number << " Fibonacci is: " << Fibonacci(number) << endl;
}

int Fibonacci(int x) 
{
    if (x < 2){
     return x;
    }     
    return (Fibonacci (x - 1) + Fibonacci (x - 2));
}
7
Vanji

Я думаю, что это решение короткое и выглядит красиво: 

long long fib(int n){
  return n<=2?1:fib(n-1)+fib(n-2);
}

Правка: как упоминал jweyrich, истинная рекурсивная функция должна быть: 

long long fib(int n){
      return n<2?n:fib(n-1)+fib(n-2);
    }

(потому что fib (0) = 0. но на основе приведенной выше рекурсивной формулы, fib (0) будет 1)

Чтобы понять алгоритм рекурсии, вы должны нарисовать на бумаге, и самое главное: «Думай нормально, как часто».

3
hqt

Это мое решение проблемы Фибоначчи с рекурсией.  

#include <iostream>
using namespace std;

int fibonacci(int n){
    if(n<=0)
        return 0;
    else if(n==1 || n==2)
        return 1;
    else
        return (fibonacci(n-1)+fibonacci(n-2));
}

int main() {
    cout << fibonacci(8);
    return 0;
}
3
Pedro Eugénio
int fib(int n) {
    if (n == 1 || n == 2) {
        return 1;
    } else {
        return fib(n - 1) + fib(n - 2);
    }
}

в последовательности Фибоначчи первые 2 числа всегда заканчиваются на 1, затем каждый раз, когда значение становится равным 1 или 2, оно должно возвращать 1

2
noelyahan
int fib(int x) 
{
    if (x < 2)
      return x;
    else 
      return (fib(x - 1) + fib(x - 2));
}
1
zod
int fib(int x) 
{
    if (x == 0)
      return 0;
    else if (x == 1 || x == 2) 
      return 1;
    else 
      return (fib(x - 1) + fib(x - 2));
}
1
user2331083
if(n==1 || n==0){
    return n;
}else{     
    return fib(n-1) + fib(n-2);
}

Однако использование рекурсии для получения числа Фибоначчи является плохой практикой, потому что функция вызывается примерно в 8,5 раз больше, чем полученное число. чтобы получить число Фибоначчи от 30 (1346269) - функция вызывается 7049122 раза!

1
Jokerius

Я думаю, что это лучшее решение Фибоначчи с использованием рекурсии.

#include<bits/stdc++.h>
typedef unsigned long long ull;
typedef long long ll;
ull FIBO[100005];
using namespace std;
ull fibo(ull n)
{
    if(n==1||n==0)
        return n;
    if(FIBO[n]!=0)
        return FIBO[n];
    FIBO[n] = (fibo(n-1)+fibo(n-2));
    return FIBO[n];
}
int main()
{
    for(long long  i =34;i<=60;i++)
        cout<<fibo(i)<<" " ;
    return 0;
}
0
Md. Ashraful Haque

Мое решение:

#include <iostream>


    int fib(int number);

    void call_fib(void);

    int main()
    {
    call_fib();
    return 0;
    }

    void call_fib(void)
    {
      int input;
      std::cout<<"enter a number\t";
      std::cin>> input;
      if (input <0)
      {
        input=0;
        std::cout<<"that is not a valid input\n"   ;
        call_fib();
     }
     else 
     {
         std::cout<<"the "<<input <<"th fibonacci number is "<<fib(input);
     }

    }





    int fib(int x)
    {
     if (x==0){return 0;}
     else if (x==2 || x==1)
    {
         return 1;   
    }

    else if (x>0)
   {
        return fib(x-1)+fib(x-2);
    }
    else 
     return -1;
    }

возвращает fib (0) = 0 и ошибку, если отрицательная

0
Cadence Weddle

Я думаю, что все эти решения неэффективны. Они требуют много рекурсивных вызовов, чтобы получить результат.

unsigned fib(unsigned n) {
    if(n == 0) return 0;
    if(n == 1) return 1;
    return fib(n-1) + fib(n-2);
}

Этот код требует 14 вызовов для получения результата для fib (5), 177 для fin (10) и 2.7kk для fib (30).

Вам лучше использовать этот подход или, если вы хотите использовать рекурсию, попробуйте это:

unsigned fib(unsigned n, unsigned prev1 = 0, unsigned prev2 = 1, int depth = 2)     
{
    if(n == 0) return 0;
    if(n == 1) return 1;
    if(depth < n) return fib(n, prev2, prev1+prev2, depth+1);
    return prev1+prev2;
}

Эта функция требует n рекурсивных вызовов для вычисления числа Фибоначчи для n. Вы все еще можете использовать его, вызывая fib (10), потому что все остальные параметры имеют значения по умолчанию.

0
Артем Васильев