関数は値を返さない

Question

これは私のコードであり、動作しません。

前の位置にARRは、以下のものから大きい又は小さい場合であれば、そのiは配列の中でいくつかの点でARRの前の位置を値1

の場合に返すべきであるように、すべてのアレイのためのより大きいあなたはelseブロックに返すべきではない任意の助けのため

#include <stdio.h> 
#include <stdlib.h> 
int Up_array(int *arr,int Size) 
{ 
    int i; 
    for(i=0;i<Size;i++) 

     if (arr[i] > arr[i+1]) 
     { 
      return 0; 
     } 
     else if(arr[i] <= arr[i+1]) 
     { 
      return 1; 
     } 
}  

void main() 
{ 
int *arr,Size,i; 

    printf("please enter the size of the array\n"); 
    scanf("%d",&Size); 
    arr=(int*)malloc(Size*sizeof(int)); 
    printf("please enter the array\n"); 
    for (i=0; i<Size ;i++) 
     scanf("%d",&arr[i]); 
    Up_array(arr,Size); 
    free(arr); 
    system("pause"); 
} 

Answer 1

を1つのので、その値を返す必要があります0感謝を従って、その後大きいです。これは、増加する2つの要素が見つかるとただちに1を返します。それは配列の残りの部分をチェックしません。その結果、関数は配列の最初の2つの要素をチェックします。

代わりに、アレイ全体を通過するまで待ちます。ループ内に0を返さなかった場合は、すべての要素が昇順であることを意味するので、1を返すことができます。

また、iがSize - 2のときにアレイを停止する必要があるため、arr[i+1]がアレイ内に残るようにする必要があります。

int Up_array(int *arr,int Size) 
{ 
    int i; 
    for(i=0;i<Size-1;i++) 
    { 
     if (arr[i] > arr[i+1]) 
     { 
      return 0; 
     } 
    } 
    return 1; 
}  

そして一般的には、else ifの条件があなただけのelse if、elseを使用する必要がありません、if条件とは正反対です。

Answer 2

ほぼ正しいです。ここでは、コードは次のようになります。

#include <stdio.h> 
#include <stdlib.h> 
int up_array(int *arr,int Size) 
{  
    // Checking corner case where Size = 1 (no index out of bound) 
    if(Size > 1) { 
     for(int i=0; i<Size-1; ++i) { 
      if (arr[i] > arr[i+1]) { 
       return 0; 
      } 
     } 
    } 
    return 1; 
}  

int main() 
{ 
    int *arr,Size; 

    printf("Please enter the size of the array:\n"); 
    scanf("%d",&Size); 
    arr=(int*)malloc(Size*sizeof(int)); 
    printf("Please enter the array (press enter after any number)\n"); 
    for (int i=0; i<Size ;i++) { 
     scanf("%d",&arr[i]); 
    } 

    printf("Result %d", Up_array(arr,Size)); 

    free(arr); 
    return 1; 
}