strcmp()에 대한 프로그램의 C 코드에서 잘못된 점은 무엇입니까?

Question

strcmp()를 사용하지 않고 두 문자열을 비교하는 프로그램을 작성했습니다. 그러나 원하는 결과를 얻을 수 없습니다. 내 프로그램 코드는 다음과 같습니다.

#include<stdio.h> 
int main(int argc, char const *argv[]) { 
    int i,j; 
    char a[90],b[90]; 
    printf("Enter the first string:"); 
    scanf("%s", &a[90]); 
    printf("Enter the second string:"); 
    scanf("%s", &b[90]); 
    for (i = 0; a[i] != '\0' && b[i] != '\0'; i++) { 
     if (a[i] == b[i]) { 
     /* code */ 
     printf("Equal %d \n", a[i]-b[i]); 
continue; 
     } if (a[i] > b[i]) { 
     /* code */ 
     printf("ai is big %d \n", a[i]-b[i]); 
     break; 
     } 
     if (a[i] < b[i]) { 
     /* code */ 
     printf("%d bi is the biggest \n", a[i]-b[i]); 
     break; 
     } 

    } 


    return 0; 
} 

터미널에서 프로그램을 실행하면 컴파일러에서 입력 문자열을 가져온 다음 중지합니다. 나는 그것을 많이 시도했지만 그것을 이해할 수 없다. 누구든지 나를 도울 수 있습니까 ...!

Answer 1

첫 번째 배열의 이름은 a이고, a[90]이 아닙니다.

마찬가지로 두 번째 배열의 이름은 b[90]이 아니라 b입니다.

a 및 b 끝 부분의 뒤에 하나의 요소 인 a[90] 및 b[90]이 있습니다.

그래서, &a[90]을 쓰기로 &b[90] 당신은 를 작성 scanf를 지시하는 단지 매우 나쁜 잘못 각 배열, 후.

아마 &a[0]과 &b[0]을 의미할까요?

그러나 scanf은 매우 위험하므로 전혀 사용하지 않아야합니다.

Answer 2

표현식 '& a [90]'과 '& b [90]'은 각각 배열 a와 b에서 마지막 항목의 주소를 생성합니다. 즉, 문자열을 배열이 아닌 할당되지 않은 스택 메모리로 읽는 중입니다. scanf() 호출에서 보통 'a'와 'b'만 사용하면됩니다.

"이제 scanf가 문자열을 90 자로 제한하는 법을 알고 있습니까?"라는 궁금증이 생기면 그 해답은 IT가 할 일이 아닙니다. Scanf는 호출자가 입력 된 문자열 (null-char 종료 포함)에 충분한 버퍼 공간을 제공했다고 가정합니다. 임의로 긴 문자열을 입력 할 수 있어야하는 경우 더욱 정교한 버퍼 관리를 수행해야합니다. 90 자 미만의 문자 만 입력하려면 한 번에 한 문자 씩 내려야합니다 (그리고 줄 바꾸기를 명시 적으로 확인해야한다고 생각합니다). 버퍼 오버 플로우.

Answer 3

해당 코드에 문제가 있습니다.

문제

1 사용하지 않은 변수 j.

2 사용자로부터 문자열을받는 동안 scanfa[90] 및 b[90]에 문자열이 90 번째 배열 색인에서 저장된다는 의미입니다. 이로 인해 메모리 문제가 발생할 수 있으며 결과물에 쓰레기 값이 생깁니다.

a[0] 및 b[0]을 scanf에 사용하는 것이 좋습니다. 배열 이름이 배열의 시작 주소를 가리키는 것처럼 scanf에 배열 이름을 사용할 수 있습니다.

scanf ("%s", a); 

scanf ("%s", b); 

올바른 코드 :

#include "stdio.h" 

int main(int argc, char const *argv[]) 
{ 

     int i; 
     char a[90],b[90]; 
     printf("Enter the first string:"); 
     scanf("%s", a); 
     printf("Enter the second string:"); 
     scanf("%s", b); 
     for (i = 0; a[i] != '\0' || b[i] != '\0'; i++) { 
       if (a[i] == b[i]) { 
         /* code */ 
         printf("Equal %d \n", a[i]-b[i]); 
         continue; 
       } if (a[i] > b[i]) { 
         /* code */ 
         printf("a is big %d \n", a[i]-b[i]); 
         break; 
       } 
       if (a[i] < b[i]) { 
         /* code */ 
         printf("%d b is the biggest \n", a[i]-b[i]); 
         break; 
       } 

     } 
     return 0; 
}