Выберите случайно строку из массива в C

Question

void pick() { 
    char* words[2]; 
    words[0] = "blah"; 
    words[1] = "hmm"; 

    char random; 
    srand(time(NULL)); 
    random = words[rand() % 2]; 
    printf(random); 
    return; 
} 

Это мой код, и я хочу, чтобы выбрать случайным образом слова из слов массива, но когда я запускаю его компилятор говорит: [Warning] назначение делает целое число от указателя без приведения [Enabled по умолчанию]

Answer 1

Вы объявили random как char, который представляет собой однобайтовое целочисленное значение. Вы назначаете ему элемент из массива words, и каждый из этих элементов имеет тип char*. Следовательно, вы получаете сообщение об ошибке при попытке присвоить значение char* целочисленному значению.

Вы хотели объявить random как char*.

Другие вещи, которые я укажу о коде:

void pick() { 
    char* words[2]; // 1 
    words[0] = "blah"; 
    words[1] = "hmm"; 

    char random; // 2 
    srand(time(NULL)); 
    random = words[rand() % 2]; // 3 
    printf(random); // 4 
    return; 
} 

1. Это должен быть объявлен как массив const char*, так как вы назначая строковые литералы (которые неизменны) к нему.

2. random также должен быть объявлен как const char*.

3. Использование % для получения случайных чисел в определенном диапазоне традиционно не очень хорошо. Также см. Q13.16 How can I get random integers in a certain range? из часто задаваемых вопросов по comp.lang.c.

4. printf(random) является опасным. Если строка, которую вы печатаете, включает в себя % символов, то printf будет ошибочной (и это потенциально может быть уязвимостью безопасности). Вы всегда должны предпочесть printf("%s", random). И поскольку вам, вероятно, нужна конечная новая линия, она должна быть printf("%s\n", random) или просто puts(random).