1

再帰を使用して配列の最大要素と最小要素を見つけたいです。これが私が書いたプログラムです。ロジックを調べたところ、完璧に見えます。しかし、プログラムをコンパイルすると、入力後にプログラムが動かなくなります。

これが私のプログラムです：

#include<stdio.h>
#include<conio.h>

int max(int a[], int n);
int min(int a[], int n);

void main(){
    int a[100],n,i,maxi,mini;
    clrscr();
    printf("Enter the number of elements ");
    scanf("%d",&n);
    printf("Enter the elements of array ");
    for(i=0;i<n;i++)
    {
        scanf("%d \n",&a[i]);
    }
    maxi = max(a,n);
    mini = min(a,n);
    printf("\nMaximum element : %d",maxi);
    printf("\nMinimum element : %d",mini);
    getch();
}

int max(int a[],int n){
    int maxo=0,i=0;
    if(i<n){
        if(maxo < a[i]){
           maxo=a[i];
        }
        i++;
        max(a,n);
    }
    return maxo;
}

int min(int a[],int n){
    int mino=999,i=0;
    if(i<n){
        if(mino > a[i]){
            mino=a[i];
        }
        i++;
        min(a,n);
    }
    return mino;
}
4

2 に答える 2

2

コードは無期限に再帰します

OPのコードに加えられた2つの変更

#include <limits.h>

int max(int a[],int n){
    //int maxo=0,i=0;
    int maxo=INT_MIN,i=0;  // Initialize maxo to the minimum int
    if(i<n){
        if(maxo < a[i]){
           maxo=a[i];
        }
        i++;
        // max(a,n);  
        // go to next element in the array, decrement array size. 
        // Without eventually reducing the array size, code recurses indefinitely.
        // So here max() works on a 1 smaller array.
        int m = max(a+1,n-1);  
        // save the greater one
        if (m > maxo) maxo = m;
    }
    return maxo;
}

ループの方が優れている場合に再帰を使用すると、再帰に悪い名前が付けられます。各呼び出しで配列サイズを半分にする以下を検討してください。最終的にはまだ約のn呼び出しmax()がありますが、再帰の深さはlog2(n)ではなくn.

int find_max(const int *a, size_t n) {
  if (n <= 1) {
    return (n == 0) ? INT_MIN : a[0];
  }
  int left = find_max(a, n/2);
  int right = find_max(&a[n/2], n - n/2);
  return left > right ? left : right;
}
于 2015-08-25T18:35:10.123 に答える