Risolto Sequenza di fibonacci

[Davide27]

Ciao a tutti per caso riuscite a dirmi se questo codice per realizzare la sequenza di fibonacci è giusto? Ho dei dubbi sulla parte di void sequnzaFibonacci().

#include <iostream>
using namespace std;

int a=0,b=1,N,F,i;

void sequenzaFibonacci(int){
    if(N==0){
        cout<<"0"<<endl;
        return;
    }
    cout<<a<<endl<<b<<endl;
    for(i=1;i<N;i++){
        F=a+b;
        a=b;
        b=F;
        cout<<F<<endl;
    }
}

int main(){
    cout<<"Inserire un numero: ";
    cin>>N;
    sequenzaFibonacci(N);
}

Grazie mille!!!

 

[--- Ra ---]

Ciao @Davide27 , sostanzialmente il codice va bene, te l'ho aggiustato solo un pochino, per esempio nelle procedure e funzioni conviene passarlo in modo completo il parametro, oltre a specificare il tipo. Poi è buon uso utilizzare prototipi prima del Main e implementazione dei prototipi dopo il Main:

#include <iostream>

using namespace std;

void sequenzaFibonacci(int N);

int main(){
    int N;
    
    cout << "Inserire un numero: " << endl;
    cin >> N;
    cout << "Successione di Fibonacci:" << endl;
    sequenzaFibonacci(N);
    
    return 0;
}

void sequenzaFibonacci(int N){
    int a = 0, b = 1, F;
    
    if(N == 0){
        cout << "0" << endl;
        return;
    }
    
    cout << a << endl << b << endl;
    for(int i=1;i<N;i++){
        F = a+b;
        a = b;
        b = F;
        cout << F << endl;
    }
    return;
}

 

[Davide27]

Ciao @Davide27 , sostanzialmente il codice va bene, te l'ho aggiustato solo un pochino, per esempio nelle procedure e funzioni conviene passarlo in modo completo il parametro, oltre a specificare il tipo. Poi è buon uso utilizzare prototipi prima del Main e implementazione dei prototipi dopo il Main:

#include <iostream>

using namespace std;

void sequenzaFibonacci(int N);

int main(){
    int N;
   
    cout << "Inserire un numero: " << endl;
    cin >> N;
    cout << "Successione di Fibonacci:" << endl;
    sequenzaFibonacci(N);
   
    return 0;
}

void sequenzaFibonacci(int N){
    int a = 0, b = 1, F;
   
    if(N == 0){
        cout << "0" << endl;
        return;
    }
   
    cout << a << endl << b << endl;
    for(int i=1;i<N;i++){
        F = a+b;
        a = b;
        b = F;
        cout << F << endl;
    }
    return;
}

grazie mille!!!