Возвращающаяся строка из функции производит мусор

Question

Имея небольшую проблему с этой простой программой. Я могу решить это, сделав ответ [10] глобальной переменной, но я не хочу этого делать. тесты программ для правильного ответа и работает, но возврат строки мусор:

#include <stdio.h> 
#include <string.h> 

char *user_string(char *Setting_Type[]); 

int main() 
{ 
char *response; 
char *test_names[2] = {"Test", "test"}; 

printf("Enter \"Test\" or \"test\": "); 
response = user_string(test_names); 
printf("\nCorrect! Your input is: %s\n", response); 

return 0; 
} 
char *user_string(char *Setting_Type[]) 
{ 
int loop = 1; 
char response[10]; 
char *response_string; 

while(loop = 1) 
    { 
    scanf("%s", &response); 
    response_string = response; 

    if(strcmp(response_string, Setting_Type[0]) != 0 && strcmp(response_string, Setting_Type[1]) != 0) 
     printf("\nWrong! Please try again: "); 
    else 
     break; 
    } 

return response_string; 
} 

Answer 1

Ваша scanf() функция необходима для редактирования
от scanf("%s", &response);
к scanf("%s", response);.
Это поможет устранить проблему.

Поскольку вы не хотите использовать глобалам, почему вы не можете поставить еще один аргумент в

char *user_string(char *Setting_Type[], char *response_string)?

Вам придется выделять для этого память и освобождать ее в вызывающей функции (main()), но она будет работать в этом сценарии. (На самом деле должно быть обеспечение немного памяти для него в любом случае, в его текущем использовании)

Пример: [Проверено, работает]

#include <ansi_c.h> 
#include <stdio.h> 
#include <string.h> 

char *user_string(char *Setting_Type[], char *s); 

int main() 
{ 
    char *response; 
    char *test_names[2] = {"Test", "test"}; 
    char *resp; 

    resp = malloc(80);//I picked 80, you can pick something more appropriate 
    response = malloc(80);//I picked 80, you can pick something more appropriate 
    printf("Enter \"Test\" or \"test\": "); 
    //user_string() returns a char *, so just call it in printf() 
    printf("\nCorrect! Your input is: %s\n", user_string(test_names, response)); 
    free(resp); 
    free(response); 

    return 0; 
} 

char *user_string(char *Setting_Type[], char *response_string) 
{ 
    int loop = 1; 
    char response[10]; 

    while(loop == 1) 
     { 
     scanf("%s", response); //removed & 
     strcpy(response_string,response);//changed from `=` to `strcpy()` 

     if(strcmp(response_string, Setting_Type[0]) != 0 && strcmp(response_string, Setting_Type[1]) != 0) 
      printf("\nWrong! Please try again: "); 
     else 
      break; 
     } 

    return response_string; 
} 

Answer 2

response массив локальный для user_string(), он будет выходить за рамки тот момент, когда функция возвращает, вы не можете использовать его из main(), здесь. Вам нужно либо malloc() памяти для него в user_string(), либо передать в буфер от main(). Много, много дубликатов этого вопроса на SO.

Answer 3

Вы возвращаете адрес локального массива, который останавливается после оператора return.

Также эта строка scanf("%s", &response); вводит возможность переполнения буфера.