>В итоге поучилось нечто такое, если где то есть ошибка сообщите. Всем
>спасибо за участие. Извиняюсь за наглость в этом посте, я просто написал всё что я думаю об этом коде, я не хочу вас обидеть ни в коей мере.
Я уже показал как обычно делаются такие вещи в предыдущем посте. А именно:
if(pid == fork())
{
это родитель, проверяем ошибки
}
else
{
это потомок, делаем работу и обязательно выходим!
}
Вот эта конструкция является идиомой в программировании, её умеют "увидеть" сотни и тысячи программистов. Облегчите поддержку вашего кода :)
В вашем коде:
if(cpid == getpid() && child_proc_count < MAXPROC ) { /* Если этот процесс первый */
вот эту строку я сначала без комментария вообще не понял, и поэтому неправильно сказал что это форк-бомба, за что мне очень стыдно. К тому же, при каждом шаге цикла вызывается getpid(), без которого можно обойтись (в моём коде его нет)
child_proc_count++; /* Увеличиваем счетчик потомков*/
нелогично, в этот момент потомка ещё нет и неизвестно -- может даже не хватит памяти его создать
pid = fork(); /* Рождение потомка */
см. выше насчёт идиомы программирования
switch из 0 и default это эффективно является тем же if() что и в идиоме, только это снова неочевидно
printf("Запущенно потомков - %d\n Ждем окончание работы потомка PID=%d\n", child_proc_count, wait(&status));
вынесите wait() в отдельную строку и сделайте проверку ошибок. логически (с первого взгляда) эта строка кода выводит текст, а реально -- ждёт потомка и только потом выводит текст. (фактически, изменяется способ исполнения программы -- она засыпает, нужно это подчеркнуть)
