[C++]Shell Linux

[trallallero]

Originariamente inviato da giovessj

ok ti kiedo solo un favore...qndo hai tempo potresti modificarmi il codice e aggiustarmelo in maniera corretta?...anke se è in c nn fa nulla al max poi lo modifico in c++...cmq ora ti lascio al tuo lavoro...se e qndo puoi riesci a farmi sto piacere te ne sarei grato...cmq grazie mille di tt

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


char **crea_vettore(char  *stringa);
void   dealloca    (char **array_di_stringhe);

int main(int argc, char *argv[])
{
	int pid, i;
	int status;
	char *stringa;
	//stringa dove memorizzo la stringa dei comandi
	char **stringhe;
	//array di stringhe

	stringa = (char *) calloc(20, sizeof(char));

	if ((stringa = fgets(stringa, 20, stdin)) == NULL)
		exit(1);
	while (strncmp(stringa, "stop", 4) != 0)
	{
		stringhe = crea_vettore(stringa);

		if ((pid = fork()) < 0)
		{
			perror("fork");
			exit(1);
		}
		else
		{
			if (pid == 0)
			{
				//figlio
					if (execvp(stringhe[0], stringhe) < 0)
					   perror("execvp"), exit(1);
			}
			else
			{
				//padre
					// aspetto la terminazione di mio figlio
					wait(&status);
				//dealloca(stringhe);
				//libero memoria
			} //fine
		} //fine
	} //fine while

	return 0; // CI VUOLE UN RITORNO 
}

char **crea_vettore(char *stringa)
{

	char **stringhe;
	char  *parola = NULL;
	int i = 0, size = 1, j = 0;

	if ((stringhe = (char **) malloc(size * sizeof(char *))) == NULL)
		perror("malloc"), exit(1);
	//errore allocazione

	while (stringa[i] != '\0')
	{
		//scandisco tutta la stringa e la divido in tante parole
		while ((stringa[i] != ' ') && (stringa[i] != '\0') && (stringa[i] != '\n'))
		{
			//costruisco la parola
			parola = (char *) realloc(parola, (j + 1) * sizeof(char *));
			parola[j] = stringa[i];
			j = j + 1;
			i = i + 1;
		}
		if ((stringa[i] == ' ') || (stringa[i] == '\n'))
			i = i + 1;

		printf("parola= %s\n", parola);

		if (parola != NULL)
		{
			//parola trovata
			stringhe = (char **) realloc(stringhe, (size + 1) * sizeof(char *));

			if ((stringhe[size - 1] = (char *) malloc(strlen(parola) + 1)) == NULL)
				perror("malloc"), exit(1);	//errore allocazione

			strcpy(stringhe[size - 1], parola);
			size = size + 1;
		}
		parola = NULL;
		j = 0;
	}
	stringhe[size - 1] = NULL;
	//fine dell array di stringhe

	return (stringhe);
}


void dealloca(char **array_di_stringhe)
{
	int i = 0;

	while (array_di_stringhe[i] != NULL)
	{
		free(array_di_stringhe[i]);
		i = i + 1;
	}
	free(array_di_stringhe[i]);
	free(array_di_stringhe);
}

ti ho corretto gli errori (non quelli logici) e adesso compila
ma non mi sembra corretta la "crea_vettore".
Solo che é un pó lungo ... devo solo fare una cosa poi
se entro le 17 la finisco lo leggo meglio

ps: scusa ma ho reindentato piú o meno a modo mio per capire meglio

[Andlea]

Originariamente inviato da giovessj

No io devo scrivere il codice della shell...devo farla io la gestione della stringa in ingresso e la gestione dei processi padre e figlio...il codice ke mi hai scritto a ke serve????

Se ho capito bene ti basta fare una pipe, fare la fork, chiudere scrittura al padre, chiudere stdin, dupare la pipe in lettura, nel figlio chiudere estremo di lettura della pipe, chiudere stdout, dupare la pipe di scrittura e lanciare l'exec con l'argomento passato a parametro

[giovessj]

Andlea è da dire ke io sn alle prime armi e quindi sinceramente nn ho capito quello ke vuoi dire...se potresti essere + gentile da spiegarti cn qlke esempio....

[Andlea]

Ogni volta che ricevi un comando, crei una nuova pipe

int piped[2];
...
pipe(piped);

dopodichè fai la fork()
Nel padre farai così

close(piped[1]);  // chiudi estremo di scrittura della pipe
while((nread=read(piped[0], buff, MAX_BUFF))>0)
printf("%s", buff);

buff è un char*, MAX_BUFF una costante che puoi impostare ad esempio a 20
Questa parte di codice non fa altro che leggere quello che esce dalla pipe e visualizzarlo a schermo

Il figlio dovrà scrivere sulla pipe, per farlo fa così

close(piped[0]);
close(1);
dup(piped[1]);
close(piped[1]);
exec(.....);

In questo modo chiude l'estremo di lettura della pipe, chiude lo standard output, e rimpiazza il descrittore dello stdout con l'estremo di scrittura della pipe
nell'exec farai la chiamata al comando impartito dall'utente, e il risultato verrà mandato nella pipe, ricvuta e visualizzata dal padre.

Una volta finito fai la wait(&status); e ricomincerai chiedendo il comando, ecc..

[trallallero]

problema oracle rimandato a mercoledi

per adesso vedo un errore

sbagliato:
parola = (char *) realloc(parola, (j + 1) * sizeof(char *));
corretto
parola = (char *) realloc(parola, (j + 1) * sizeof(char));
stai allocando una stringa non una matrice

poi non ho capito il problema ...
il programma chiede una serie di comandi e tu per ogni comando lanci un figlio che lo esegue ?

[trallallero]

while (strncmp(stringa, "stop", 4) != 0)

ciclo infinito

se stringa é "ciao pippo pluto stop" la strncmp
punterá sempre a ciao quindi condizione sempre falsa

[giovessj]

si esattamente...cioè nn fa altro ke ad ogni fork lanciare un figlio....

[giovessj]

quindi il ciclo while così fatto nn va bene??

[trallallero]

no, penso di aver capito

tu devi creare 1 figlio con la fork.
La execvp ha una lista di argomenti e serve per trasformare il figlio
nel programma che vuoi tu.
Il primo argomento é infatti il nome del programma (per ex. ls)
poi gli argomenti per i programma (per ex. -ltr pippo ...)
l'ultimo argomento é NULL per far capire che é finita la lista.
Il figlio quindi si trasforma nel programma specificato e gli passa gli argomenti restanti.

[giovessj]

si xkè tipo puo' accadere ke cm riga di comando ci possa essere:
g++ -o shell.cpp e la mia shell cmq deve rilevare tt le parole presenti nela riga...

[trallallero]

Originariamente inviato da giovessj

si xkè tipo puo' accadere ke cm riga di comando ci possa essere:
g++ -o shell.cpp e la mia shell cmq deve rilevare tt le parole presenti nela riga...

Appunto, non é un figlio per parola ma un solo figlio che fa tutto.

Devi fare cosí:

- ottieni stringa (senza stop, non serve)
- crei vettore
- chiami la fork
- chiami la execvp passando il vettore terminato da NULL

io fra un pó devo andare. Ti posto il codice di un client
di un sistema client/server che ho fatto qualche anno fa.
Il sistema é lo stesso.
Il server si clona, chiama la execvp e si trasforma in child
passando gli argomenti.
Ovviamente io ho fatto anche child.c che gestisce il client
ma a te non serve perché chiami i comandi di sistema.
Spero ti torni utile, é molto commentato.

/* -----------------------------------------*\
      AVVIA UN PROCESSO FIGLIO.
\* -----------------------------------------*/
static void ChildBorning( int ThisSocket )
{
   int      pid;

   if ( ThisSocket <= (stderr - stdin) ) return;

   pid = fork();   // SDOPPIA IL PROCESSO ATTUALE.

   if ( pid < 0 )       //  ERRORE
      DiagMsg(__LINE__ ,"'fork': %s" ,strerror(errno));
   else if ( pid > 0 )  //  PARENT
      DiagMsg(__LINE__ ,"Connection signal received from PPSERVER; child starting...");
   else                 //  CHILD
   {  // RILASCIA IL FILE_DESCRIPTOR DEL SOCKET DEL GENITORE.
      // --- SOSTITUISCE L'IMMAGINE DEL PROGRAMMA ATTUALE ---
      int   parm = 0;
      char  buff[16];
      char *arglist[7];

      // ID DELLA CODA_MESSAGGI PASSATO COME PARAMETRO AL PROGRAMMA CHE SUBENTRA AL CLONE DEL GENITORE.
      sprintf( buff, "%05.5d", ThisSocket );

      close(sockfd);
      // ACCATASTA I PARAMETRI PER IL PROGRAMMA 'CHILD'.
      arglist[parm++] = "child";                      // NOME PROCESSO
      arglist[parm++] = buff;                         // NUMERO CODA MESSAGGI
      if (Diagnostic) arglist[parm++]  = Diagnostic;  // EVENTUALE FLAG DI DIAGNOSTICA ( -d, -D )
      arglist[parm++] = NULL;                         // FINE ARGOMENTI


      // STAMPA IL COMANDO COMPLETO PER L'ESECUZIONE DI 'CHILD'.
      ArgListing( arglist, "Executed", "execvp" );
      execvp( arglist[0], arglist );

      // SE SIAMO QUI NON RIUSCITA 'exec'
      // STAMPA IL COMANDO COMPLETO PER L'ESECUZIONE  DI 'CHILD'.
      ArgListing(arglist ,"Failed"  ,"execvp");
   }
}
/* -----------------------------------------------------------------------------------*/

[giovessj]

Ok grazie mille x la tua disponibilità sei stato gentilissimo....