함수가 아무 값도 반환하지 않습니다.

Question

이것은 내 코드이며 작동하지 않습니다.

상기 PREV 위치에 도착이 수행 하나에서 크거나 작은 경우 경우 해당 I 배열의 어느 시점에 도착의 이전 위치를 값 1

을 경우 반환해야하므로 모든 어레이 큰 다음 중 하나 그렇게 그 어떤 도움

#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

당신은 else 블록에 반환하지 않아야의 값 0 감사를 반환해야 따릅니다 크다. 이것은 증가하는 두 개의 요소를 찾자 마자 즉시 1을 반환 할 것이고, 나머지 배열을 검사하지 않을 것입니다. 결과적으로 함수는 배열의 처음 두 요소를 확인합니다.

대신 전체 배열을 찾을 때까지 기다리십시오. 루프 내부에 0을 반환하지 않았다면 모든 요소가 오름차순임을 의미하므로 1을 반환 할 수 있습니다.

i이 Size - 2 일 때 배열을 중지해야하므로 arr[i+1]이 여전히 배열 안에 있도록 배열 밖에서 액세스하지 않아야합니다. else if의 조건이 if 조건의 정반대 때

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하지 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; 
}