c «каждый» макрос не перебирает все элементы

Question

вот мой код, я нахожу, что «каждый» макрос только перебирает первый элемент и выводит «s1», как заставить его перебирать все элементы?

#include "stdio.h" 

#define each(item, array) \ 
    for (int keep = 1, count = 0, size = sizeof(array)/sizeof *(array); keep && count != size; keep = !keep, count++) \ 
     for (item = array[count]; keep; keep = !keep) 

char *join(char const *ss[]) { 
    char *r = ""; 
    each(char *s, ss) { 
      puts(s); 
     } 
    return (r); 
} 

int main(int argc, char **argv) { 
    char const *ss[] = {"s1", "s2"}; 
    join(ss); 
}; 

Answer 1

Если вы звоните each из main, то есть место, где ss определяется, ваша программа работает, как ожидалось:

int main(int argc, char **argv) { 
    char const *ss[] = {"s1", "s2"}; 

    each(char *s, ss) { 
     puts(s); 
    } 
}; 

Это работает, потому что each имеет доступ к ss декларации, так что sizeof a/sizeof *a равно числу элементов в массиве.

Проблема в том, что если вы передадите массив ss в другую функцию (например, join), это decays to pointer. Поскольку ss в join является указателем, sizeof array больше не относится к размеру массива, а выражение sizeof a/sizeof *a дает sizeof(char**)/sizeof(char*), что равно 1 на вашей платформе.

Смотрите также

What is array decaying?

Answer 2

Учитывая ниже терминологию

for(initialization statement; condition; update statement) { 
     code/s to be executed; 
} 

Ваш condition имеет заявление keep && count != size;, который проверяет наличие значения keep быть отличны от нуля.

И в update statement вы делаете keep = !keep, что составляет keep значение 0.

Следовательно, ваша петля выходит из одной итерации. Модифицируйте свое состояние в соответствии с циклом несколько раз.

Answer 3

Здесь макрос не создает какие-либо проблемы макросъемки просто место код всякий раз, когда вы использовали,

вам нужно проверить for петли,

size = sizeof(array)/sizeof *(array); 

В заявлении выше sizeof(array) и sizeof *(array) оба дают sizeof указателей. Таким образом, size всегда 1.

Next находится внутри или второй for цикл повторяется только один раз, потому что вы сделали keep = !keep, которые меняют keep = 0;

Так что теперь она завершена первая итерация, она собирается обновить keep = !keep, count++), После этого проверьте состояние keep && count != size в этом

keep = 1; 

count= 1; и

size = 1; 

так завершает цикл, так проверьте свою часть состояния цикла.

если у вас есть намерение просто итерацию для всех элементов в массив символов, то там количество раствора сделать это, я просто листинг один из них изменить в коде

В макро

for (int keep = 1, count = 0, size = sizeof(ss)/sizeof *(ss); ss[count] != "\0"; keep = !keep, count++) \ 

В основная функция

char const *ss[] = { "s1", "s2","\0" }; 

Теперь вы получите как ожидается выход.