구조체 문제는 C

Question

에서 나는 다음과 같은 구조체

typedef struct rotor { 
    char characters[26]; 
    char* rotationPos; 
    char* aPosition; 
} rotor; 

을하고 난 이런 식으로 3 개 인스턴스를 만들려면 :

rotor* createRotor(char* cadeia, char* c) { 
    int aux; 
    rotor* r; 
    for (aux = 0; aux < 26; aux++) { 
     if (cadeia[aux] == NULL || strlen(cadeia) > 26) { 
      return -1; 
     } else 
      r->characters[aux] = cadeia[aux]; 
    } 
    r->characters[26] = '\0'; 
    r->rotationPos = c; 
    r->aPosition = r->characters[0]; 
    return r; 
} 

:

char cadeia3[26] = "BDFHJLCPRTXVZNYEIWGAKMUSQO"; 
char cadeia2[26] = "AJDKSIRUXBLHWTMCQGZNPYFVOE"; 
char cadeia1[26] = "EKMFLGDQVZNTOWYHXUSPAIBRCJ"; 

rotor* r1; 
rotor* r2; 
rotor* r3; 
r1 = createRotor(cadeia1, 20); 
r2 = createRotor(cadeia2, 20); 
r3 = createRotor(cadeia3, 20); 

가 createRotor 기능은 자동으로 회 전자를 생성

첫 번째 로터에서만 작동합니다 ... 콘텐츠를 확인하거나 사용하고 싶을 때마다 함수에서 다른 것들은 프로그램이 "종료"됩니다. 나에게 이상한 일은 완벽하게 일하고 다른 일을 위해 일하는 것이 아닙니다.

내가 뭘 잘못하고 있니?

Answer 1

나에게 정의되지 않은 동작처럼 보인다.

rotor에 필요한 메모리를 (동적으로) 할당하지 않습니다. createRotor() 함수는 임의의 정크 주소 (즉, r 값)에 데이터를 저장하고 액세스하려고합니다. 프로그램에 할당 된 주소로 r을 초기화하지 마십시오. 당신이, 상수 ("20")처럼, "문자의 C"로 매개 변수를 정의하지 "의 char *에 c"바이트를 사용하는 경우

1) :

Answer 2

char cadeia3[26] = "BDFHJLCPRTXVZNYEIWGAKMUSQO"; 

문자열 리터럴은 이미 26 자 길이입니다. 즉, 배열에 널 종결 자의 여유 공간이 없습니다. 따라서이 전화는 다음과 같습니다.

strlen(cadeia); 

은 정의되지 않은 동작을 호출합니다.

사용 :

char cadeia3[27] = "BDFHJLCPRTXVZNYEIWGAKMUSQO"; 

또는

char cadeia3[] = "BDFHJLCPRTXVZNYEIWGAKMUSQO"; 

Answer 3

세 가지 변경 사항을 제안

rotor* createRotor(char* cadeia, char* c) 

당신의 문자열) 26 자 사이 ... 다음 문자 배열이 (적어도 있어야합니다 27 개 문자 인 경우

2) 더 중요한 :

이

typedef struct rotor { 
    char characters[27]; 
    char* rotationPos; 
    char* aPosition; 
} rotor; 
... 
char cadeia3[27] = "BDFHJLCPRTXVZNYEIWGAKMUSQO"; 

3) 가장 중요한 있는지 확인합니다을 ALLOCATE "회 전자"가 어딘가에 구조화되어 있습니다.