如何從函數返回一個數組並循環遍歷它？

Question

#include <iostream> 

int* fib(int); 

int main() 
{ 
    int count; 
    std::cout<<"enter number up to which fibonacci series is to be printed"<<std::endl; 
    std::cin>>count; 
    int *p=new int[count]; 
    p=fib(count); 
    int i; 
    for(i<0;i<=count;i++) 
     std::cout<<p[i]<<std::endl; 
    return 0; 
} 

int* fib(int d) 
{ 
    int *ar=new int[d]; 
    int p=-1,q=1,r; 
    int j; 
    for(j=0;j<=d;j++) 
    { 
     r=p+q; 
     ar[j]=r; 
     p=q; 
     q=r; 
    } 
    return ar; 
    delete ar; 
} 

爲什麼我不能用這種方式打印整個Fibonacci系列數組？

Answer 1

與您的代碼的幾個問題

for(i<0;i<=count;i++) 

實際上應該是

for(i=0;i<count;i++) 

和

for(j=0;j<=d;j++) 

必須閱讀

for(j=0;j<d;j++) 

並刪除行

delete ar; 

，因爲它不具備return語句後有任何影響。此外，您可以在main()，因爲這是你fib功能也做擺脫實例

int *p=new int[count]; 

的。就目前而言，你泄露了你剛剛分配的內存。

Answer 2

一個問題是，您分配數組的一個元素太短。 new[]中的參數是元素的個數，不是的最高索引。

另一個問題是，你在兩個地方分配數組，但只會產生內存泄漏，不會影響結果。

Answer 3

您正在分配的元素太少。您刪除ar的代碼永遠不會運行，因爲它會在返回之後運行。您還會泄漏p，因爲您會用fib()返回的指針覆蓋指針。

如果我是你，我可能會通過p到fib()並獲得fib()來填寫數組。

void fib(int n, int p[]) 
{ 
    p[0] = 1; 
    p[1] = 1; 
    for (int i=2; i<=n; i++) 
     p[i] = p[i-2]+p[i-1]; 
} 

顯然，這種代碼需要n>=2但我會離開錯誤檢查作爲練習讀者！

要調用它使用這樣的代碼：

int p[] = new int[count]; 
fib(count, p); 

如果你想I1和I2之間打印出的價值觀，比如說，像這樣做：

for (int i=i1, i<=i2, i++) 
    std::cout<<p[i]<<std::endl; 

由於您使用C++ ，所有這些代碼在C++矢量類中都會更簡單。

Answer 4

你的我沒有初始化。而不是讓它i = 0，你做我< 0.而在j循環中，最大數量應該是d。所以j < d。不是j < = d。

Answer 5

首先，您不必在主內部分配int *p=new int[count];，因爲您將從fib函數接收指向已分配內存的指針。

其次，return聲明後面的所有內容都是無法訪問的代碼，因此您可以將其刪除或在return之前將其移動。

此外，如果你刪除裏面fib功能陣列，將返回一個空指針。

和主要問題是：

for(i<0;i<=count;i++) 

其中正確的是：

for(i = 0; i <= count; i++) 

Answer 6

的問題正是在這裏：

int i; 
for(i<0;i<=count;i++) 
    std::cout<<p[i]<<std::endl; 

您還沒有分配我任何啓動值。它更改爲：

for (int i = 0; i < count; i++) 
    std::cout << p[i] << std::endl; 

Answer 7

這裏是斐波納契數列的一個例子，並且我開始礦了與一個[0] = 1和a [1] = 1斐波納契數列normaly開始與0和1，但將礦啓動1和1

的#include 「stdafx.h中」 #include <iostream>

使用命名空間std;

int main() 
{ 
    int a[25]; 
    int i,j; 
    int fib[25]; 
    int fibs; 
    char z; 

    a[0] = 1; 
    a[1] = 1; 


     fib[1] = a[0]; 
     fib[2] = a[1]; 
     fibs = 0; 

    for (i=2; i<25; i++) 
    { 
     fibs = fib[1] + fib[2]; 
     fib[1] = fib[2]; 
     fib[2] = fibs; 
     a[i] = fibs; 
    } 

    for(i=0; i<25; i++) 
    { 
     cout << "a[" << i << "]=" << a[i] << endl; 
    }