C # funzione di Fibonacci errori ritorno
-
21-08-2019 - |
Domanda
sto praticando un C # applicazione console, e sto cercando di ottenere la funzione di verificare se il numero appare in una serie di Fibonacci o no, ma sto ottenendo gli errori.
Quello che ho fatto è stato:
class Program
{
static void Main(string[] args)
{
System.Console.WriteLine(isFibonacci(20));
}
static int isFibonacci(int n)
{
int[] fib = new int[100];
fib[0] = 1;
fib[1] = 1;
for (int i = 2; i <= 100; i++)
{
fib[i] = fib[i - 1] + fib[i - 2];
if (n == fib[i])
{
return 1;
}
}
return 0;
}
}
Qualcuno può dirmi che cosa sto facendo male qui?
Soluzione
E qui è una soluzione che batte tutti del tuo!
A causa, il motivo per cui iterazione quando si ha intelligenti matematici facendo soluzioni in forma chiusa per voi? :)
static bool IsFibonacci(int number)
{
//Uses a closed form solution for the fibonacci number calculation.
//http://en.wikipedia.org/wiki/Fibonacci_number#Closed-form_expression
double fi = (1 + Math.Sqrt(5)) / 2.0; //Golden ratio
int n = (int) Math.Floor(Math.Log(number * Math.Sqrt(5) + 0.5, fi)); //Find's the index (n) of the given number in the fibonacci sequence
int actualFibonacciNumber = (int)Math.Floor(Math.Pow(fi, n) / Math.Sqrt(5) + 0.5); //Finds the actual number corresponding to given index (n)
return actualFibonacciNumber == number;
}
Altri suggerimenti
Ecco una soluzione divertente utilizzando un blocco iteratore infinita:
IEnumerable<int> Fibonacci()
{
int n1 = 0;
int n2 = 1;
yield return 1;
while (true)
{
int n = n1 + n2;
n1 = n2;
n2 = n;
yield return n;
}
}
bool isFibonacci(int n)
{
foreach (int f in Fibonacci())
{
if (f > n) return false;
if (f == n) return true;
}
}
Io in realtà piace molto questo tipo di implementazione Fibonacci vs la soluzione ricorsiva tradizione, perché mantiene il lavoro utilizzato per completare un termine a disposizione per completare il prossimo. La soluzione tradizionale ricorsiva duplica po 'di lavoro, perché ha bisogno di due chiamate ricorsive ogni termine.
Il problema sta nel <= la seguente dichiarazione:
for (int i = 2; i <= 100; i++)
più al punto i =. Non v'è alcun fib [100] (C # a zero conteggi) in modo che quando si controlla il i = 100 si ottiene un'eccezione.
la dichiarazione corretta dovrebbe essere
for (int i = 2; i < 100; i++)
o meglio ancora
for (int i = 2; i < fib.Length; i++)
Beh, per cominciare l'array è lungo solo 10 e si sta riempiendo con ~ 100 elementi (out-of-range-eccezione) -, ma ci sono modi migliori per fare questo ...
Ad esempio, utilizzando questo post :
long val = ...
bool isFib = Fibonacci().TakeWhile(x => x <= val).Last() == val;
int[] fib = new int[10];
for (int i = 2; i <= *100*; i++)
Si sta andando fuori dai limiti della propria matrice perché il ciclo condizionale è troppo grande. Un approccio più tradizionale sarebbe vincolato all'anello dalla dimensione della matrice:
for (int i = 2; i < fib.Length; i++)
E rendere l'array più grande, ma come ha detto Marc, ci sono modi migliori per fare questo, e vi consiglio di passare un po 'tempo a leggere l'articolo di Wikipedia su numeri di Fibonacci .
Una cosa che puoi fare è verificare la presenza di una rapida uscita. Dal momento che si sta cercando di determinare se un dato numero è nella sequenza di Fibonacci, si può fare il controllo dei limiti di uscire presto.
Esempio:
static bool isFibonacci(int n)
{
int[] fib = new int[100];
fib[0] = 1;
fib[1] = 1;
for (int i = 2; i <= fib.Length; i++)
{
fib[i] = fib[i - 1] + fib[i - 2];
if (n == fib[i])
{
return true;
}
else if (n < fib[i])
{
return false; //your number has been surpassed in the fib seq
}
}
return false;
}
public static int FibNo(int n) {
int result = 0; int No = 0; int N1 = 1;
if (n< 0)
{ throw new ArguementException("number must be a positive value"); }
if (n <= 1)
{ result = n; return result; }
for(int x=1; x < n; x++)
{ result = No + N1; No = N1; N1=result; }
return result;
}