[C] Fork () <- Ostatnia linijka wykonuje sie 3 razy (a nie raz)

Question

Zadanie: Stwórz drzewo procesów następujący sposób:

  - 3 - 4
1
  - 2- 5

Każdy proces uśpij . Główny proces ma czekać na zakończenie procesu dzieci.

Kod:

#include <sys/types.h>
#include <unistd.h>
#include <stdio.h>
#include <sys/types.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/wait.h>

int main (void){
pid_t child_a, child_b;

child_a = fork();

if (child_a == 0) {
    printf ("Process 2 \n");
    sleep(10);
	pid_t child = fork();
	if (child == 0){
	printf ("Process 3");
	sleep(10);
	}
	else{
	wait(NULL);
	}
}
 else {
	printf("Process 1 /n");
    child_b = fork();

    if (child_b == 0) {
        /* Child B code */
        printf("Process 4 \n");
        sleep(10);
        pid_t child1 = fork();
        if (child1 == 0 ){
			printf ("Process 5 \n");
			sleep(10);
		}
		else {
		wait (NULL);
		}
        //ParentChild Code
    }
    wait(NULL);
    printf ("Process Ended...\n");
    
}
	
}

I wszystko pięknie się robi, ale nie wiem dlaczego zawsze drukuje process ended gdy jakikolwiek sie zakonczy. A nie wiem jak to naprawic. Ma ktos jakieś pomysły?

Answer 1

Nie wiem czy dobrze rozumiem to drzewo procesów, które podałeś, ale wydaje mi się, że wynika z niego, że proces 1 ma mieć dwa procesy potomne o numerach 2 i 3. Proces 2 ma mieć potomka o numerze 5, a proces 3 potomka o numerze 4. Z tego co Ty zrobiłeś wynika, że proces 1 ma potomków 2 i 4, proces 2 ma potomka 3, a proces 4 ma potomka 5. Tak więc ja bym nadał numery w printf: patrząc od góry 2,5,1,3,4.

printf z process ended masz w bloku else, który zaczyna się w 26 linijce i kończy w 47. Wykonuje się on 3 razy, ponieważ w tym bloku else są 3 procesy. Według Twojego numerowania: 1,4,5. Nie kończysz wcześniej procesów 4 i 5, więc one także dochodzą do tego momentu. Na końcu bloku if, który zaczyna się w linijce 30 napisz return 0, aby w tym miejscu zakończyły się procesy 4 i 5.

W 44 linijce masz wait(NULL), ale przydałoby się napisać to jeszcze raz, bo proces 1 ma dwóch potomków, powinien więc poczekać na obu, a nie tylko na jednego. Brakuje też return 0 na koniec programu i masz zły ukośnik w printf process1. Niezbyt czytelne formatowanie kodu. (np. dlaczego wnętrze if z linijki 18 nie ma wcięcia względem tego if?).

Comments

Bo to geany, a ja się przyzwyczaiłem się do automatycznych wcięc np: w netbeans. Tak czy siak rozwiązało to problem.