Счетчик циклов, не увеличивающий

Question

Функция aquire_marks предназначена для того, чтобы принимать 5 номеров, введенных пользователем, и хранить их в marks. При запуске цикл for работает как ожидалось с j=0, а затем, как и ожидалось, для j=1, но тогда j никогда не увеличивается до 2 и остается в 1, оставив функцию в бесконечном цикле. Если цикл помещен внутри main, эта проблема не возникает, но я предпочел бы иметь ее внутри отдельной функции, если это возможно. Вы знаете, что здесь происходит?

#include<stdio.h> 

void aquire_marks(char marks[], char names[][11]){ // 11 - max name length + 1 
    char j, mark; 

    for(j=0; j<5; j++){ 
     printf("Enter mark for %s: ", names[j]); 
     scanf("%d", &mark); 
    } 
} 

int main() { 
    char names[5][11] = {"a", "b", "c", "d", "e"}; 
    char marks[5]; 

    aquire_marks(marks, names); 
} 

Answer 1

Вы объявили mark как char, но в scanf вызова вы используете спецификатор d преобразования, которая ожидает его соответствующий аргумент имеет тип int *, не char *. Использование неправильного спецификатора преобразования приводит к неопределенному поведению.

Я собираюсь предположить, что mark и j являются смежными в памяти, и что в процессе чтения целочисленное значение для mark (который может где-нибудь от 2-х до 4 байт в ширину) значение j становится перезаписаны. (90, 75 и т. Д.)? Если это так, измените тип mark от char до int.

Если mark должна представлять собой буквенную (A, B и т.д.), а затем изменить scanf вызов

scanf(" %c", &mark); // note leading whitespace 

Answer 2

Я хотел бы изменить декларацию J к int.

char mark; 

for(int j=0; j<5; j++){ 
    printf("Enter mark for %s: ", names[j]); 
    scanf("%d", &mark); 
} 

Answer 3

Вы передаете адрес 1 byte переменной (char) на функцию, которая будет угроза его как int указатель.

Если мы предположим, что int является 4 bytes, scanf перезаписать 3 byte стека рядом с mark переменной.

Первый, предположительно, j;