[C] Probleme commande système

Recherche :

Electronique : pat_man_34, 1 utilisateur anonyme et 10 utilisateurs inconnus

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C] Probleme commande système

k4rt3x

Davantage d'avantages avantage

Salut,

J''ai un petit probleme, je suis en train de creer un programme
en console qui a pour but de formater le Disque dur, la clé usb, ou la disquette
qu'on lui de demande de formater avec un menu.

j'ai creer une fonction "hdd" qui formate le disque dur qu'on lui demande de formater :

Seulement la commande system() ne marche pas car il prend \\\\\"hdd\\\\\" pour un deuxieme argument

Alors je voudrait savoir comment faire pour que la commande system

formate la lettre rentrée dans hdd

Merci pour vos reponses.

EDIT: Je n'arrive pas a mettre le code de la fonction dans le sujet il rajoute plein de \\\\ partout !

le code est ici : www.kokoom.com/rektetex/code.txt

EDIT2 : Pour le formatage de disquette, il n'y a pas besoin de stocker une lettre dans une variable donc
j'ai fait

system(\\\"FORMAT A:\\\" )(les deux slash n'y sont pas, je ne sais pas pourquoi ça les rajoutes quand je post.)

mais le fomatage est très long :sleep:
Je me demandais comment faire un formatage rapide (comme quand on le fait avec windows où il faut juste cocher la case formatage rapide)

merci.

Message édité par k4rt3x le 30-08-2006 à 22:35:06

Google

Deadog

Dain Bramaged

Code :

char hdd ;
...
scanf ("%s", &hdd);
...
system("FORMAT %s: ", hdd);

ohlala [:deadog:2]

Code :

char hdd ;
...
scanf ("%c", &hdd);
...
system("FORMAT %c: ", hdd);

k4rt3x

Davantage d'avantages avantage

Merci de ta reponse, mais le probelme est toujours le meme:
à la compilation : "too many argument in fonction system"

Deadog

Dain Bramaged

ah oui, javais même pas capté :sarcastic:
enfin, tu l'aurais dit plus tôt l'erreur :

Code :

char buf[11];
sprintf(buf, "FORMAT %c: ", hdd);
system(buf);

Message édité par Deadog le 31-08-2006 à 01:21:29

k4rt3x

Davantage d'avantages avantage

Code :

int hdd()
{
int choix;
char hdd ;
system("cls" );
char buf[11];
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
scanf ("%c", &hdd);
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
scanf ("%d", &choix);
switch (choix)
{
case 1:
printf ("Debut du formatage: " );
break;
default:
exit(0);
}
system(buf);
system("exit" );
}

Maintenant, le

Code :

scanf ("%c", &hdd);

ne se fait plus ... Il passe directement à 'Entrez 1 pour commencer le formatage ou 0 pour quitter.'
:??:

Message édité par k4rt3x le 31-08-2006 à 00:49:39

k4rt3x

Davantage d'avantages avantage

Comme ça, ça marche:

Code :

int choix;
char hdd;
system("cls" );
char buf[11];
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
scanf ("%s", &hdd);
printf("%c",hdd);

( en faisant scanf avec %s, et printf avec %c ça marche ... :pt1cable:

Message cité 1 fois

Deadog

Dain Bramaged

les joies du scanf ... surtout quand celui la n'a pas les même comportements suivant l'OS, car chez moi (linux) il se fait ...

à dire vrai j'ai toujours banni le scanf pour ce genre de truc, je suis toujours passé par des solutions un poil plus compliqué mais au moins j'suis sur que ça marche.

Message édité par Deadog le 31-08-2006 à 01:22:05

Deadog

Dain Bramaged

k4rt3x a écrit :

Comme ça, ça marche:

Code :

int choix;
char hdd;
system("cls" );
char buf[11];
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
scanf ("%s", &hdd);
printf("%c",hdd);

( en faisant scanf avec %s, et printf avec %c ça marche ... :pt1cable:

oui mais non, ça ça n'a pas de sens
tu lui dit de lire une chaîne pour écrire dans un octet, écrit plein plein de truc après et tu veras qu'il va planter aléatoirement car il aura écrit plus loin que l'octet alloué (c'est un dépassement de mémoire, buffer overflow, tu sais, le truc qui donne 70% des failles de sécurité dans les programmes [:dawa] )

edit : même en écrivant qu'un seul caractère il dépasse quand même car il cherche aussi à écrire le 0 de fin de chaîne, voir le \n

Message édité par Deadog le 31-08-2006 à 01:09:37

k4rt3x

Davantage d'avantages avantage

Ah ok, Quelles "solutions un poil plus compliqué" je pourrai utiliser ici ?

k4rt3x

Davantage d'avantages avantage

Arf, on a post en meme temps, j'ai essayer d'ecrire une phrase au lieu d'une lettre, il n'a pas planter pourtant ???

EDIT: Le programme est ici : www.kokoom.com/rektetex/format.exe
EDIT2: Quand je dis il n'a pas planter, c'est qu'il ne s'est pas fermer de lui-meme en affichant un message d'erreur quoi,
effectivement ça ma renvoyer direct au menu on peut appeler ça un plantage ?

Message édité par k4rt3x le 31-08-2006 à 01:13:34

k4rt3x

Davantage d'avantages avantage

Ah, ca ne marche pas !
Enfet

Code :

- int choix;
- [*] char hdd;
- [*]
- [*] system("cls" );
- [*] char buf[11];
- [*]
- [*] printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
- [*]
- [*]printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
- [*]scanf ("%s", &hdd);
- [*]
- [*]printf("%c",hdd);

la ça marche mais

Code :

int choix;
char hdd;
system("cls" );
char buf[11];
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
scanf ("%s", &hdd);
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
scanf ("%d", &choix);

la ca marche plus

Message édité par k4rt3x le 31-08-2006 à 01:16:13

Deadog

Dain Bramaged

je t'ai dit, c'est aléatoire.
si tu prend l'habitude d'utiliser scanf comme ça tu va au devant de gros bugs chelou plus tard
je sais que y'a une "bonne" méthode d'utiliser scanf mais j'me suis jamais atardé dessus car la plus part du temps on préfère utiliser ce genre de truc :

Code :

char buffer[BUFSIZ];
char hdd;
fgets (buffer, BUFSIZ, stdin);
hdd = buffer[0];

BUFSIZ est normallement déjà défini dans stdio.h (il vaut 4096 la plus part du temps)
tu peux le réduire si tu veux, mais pas moins de 3 pour que tout aille bien

Message édité par Deadog le 31-08-2006 à 01:23:31

k4rt3x

Davantage d'avantages avantage

Meme avec

Code :

- char buffer[BUFSIZ];
- [*] char hdd;
- [*]
- [*] fgets (buffer, BUFSIZ, stdin);
- [*] hdd = buffer[0];

Ca saute le scanf,

Code :

#include <stdio.h>
#include <stdlib.h>
int main ()
{
int hdd();
int disk();
int usb();
int menu();
int choixMenu = 0;
printf ("\n\n\t\t\tFORMATAGE" );
printf ("\n\n\n\n\t Menu:\n\n" );
choixMenu = menu(&choixMenu);
switch (choixMenu)
{
case 1:
hdd();
/* case 2:
disk();
case 3:
usb();
case 4:
exit(0); */
default:
system("cls" );
printf ("\nChoix incorrect, tappez un nombre du menu:\n\n" );
choixMenu = menu();
}
return 0;
}
int menu(int *ptrChoixMenu)
{
int choixMenu;
printf ("1. Formater un disque dur\n" );
/*
printf ("2. Formater une disquette\n" );
printf ("3. Formater une cle usb ou lecteur MP3\n" );
printf ("4. Quitter le programme\n\n" );
printf ("Quel est votre choix ?\n" );
*/
scanf ("%ld", &choixMenu);
*ptrChoixMenu = choixMenu;
}
int hdd()
{
int choix;
char buffer[BUFSIZ];
char hdd;
char buf[11];
system("cls" );
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
fgets (buffer, BUFSIZ, stdin);
hdd = buffer[0];
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
scanf ("%d", &choix);
switch (choix)
{
case 1:
printf ("Debut du formatage: " );
break;
default:
exit(0);
}
system(buf);
}
int disk()
{
int choix;
system("cls" );
printf ("\n\n\t\t\tFORMATER UNE DISQUETTE" );
printf ("\n\n\n\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
scanf ("%d", &choix);
system("cls" );
switch (choix)
{
case 1:
printf ("Debut du formatage: " );
break;
default:
exit(0);
}
system("FORMAT A:" );
return 0;
}
int usb()
{
int choix;
char hdd ;
system("cls" );
char buf[11];
char buffer[BUFSIZ];
printf ("\n\n\t\t\tFORMATER UN MP3 OU CLE USB" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du volume \n(Allez dans poste de travail si vous ne connaissez pas la lettre: " );
fgets (buffer, BUFSIZ, stdin);
hdd = buffer[0];
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
scanf ("%d", &choix);
switch (choix)
{
case 1:
printf ("Debut du formatage: " );
break;
default:
exit(0);
}
system(buf);
}

voila le code source entier du programme peut etre que je doit avoir fait des erreurs quelque part :pfff:

En tous cas, merci pour ton aide et désoler pour la longueur du post

Message édité par k4rt3x le 31-08-2006 à 01:39:39

Deadog

Dain Bramaged

il doit avoir du reste dans le stdin à cause du précédent scanf (c'est entre autre pour cela que j'aime pas scanf )

attend voir ...

k4rt3x

Davantage d'avantages avantage

Euh, une question , C'est quoi le stdin en fait ?

Deadog

Dain Bramaged

le fichier d'entrée standart
tout ce que tu rentre au clavier est mit dans un faux fichier qu'on appel stdin

Code :

while(getchar () != 10);
fgets (buffer, BUFSIZ, stdin);

pas sur que ça marche sous windows à cause des fins de ligne qui se font en \10\13 au lieu de \10 seulement sous linux.
sinon ça fait :

Code :

while(getchar () != 10);
while(getchar () != 13);
fgets (buffer, BUFSIZ, stdin);

Message édité par Deadog le 31-08-2006 à 02:05:30

k4rt3x

Davantage d'avantages avantage

Code :

printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
while(getchar () != 10);
fgets (buffer, BUFSIZ, stdin);
hdd = buffer[0];
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
scanf ("%d", &choix);

La ca marche jusqu'a la ligne 11 où je retourne au menu

Code :

- while(getchar () != 10);
- [*] while(getchar () != 13);
- [*] fgets (buffer, BUFSIZ, stdin);

Et là bah quand il me demande la lettre du disque je la tape et j'appui sur ENTRé mais ca me refait taper quelquechose :heink:
(www.kokoom.com/reketetex/format3.exe)

Deadog

Dain Bramaged

c'est quoi que t'appel menu ? lequel ?
(je suis à l'aveugle la, car ça marche chez moi, sous nux, et ton exe je peut pas l'exécuter )

Message édité par Deadog le 31-08-2006 à 02:23:40

k4rt3x

Davantage d'avantages avantage

Le menu du depart ou il demande si tu veux formater un disque dur
( normalement une disquette et une cle usb mais je les ai enlever le temps de trouver comment faire marcher pour disque dur)

Deadog

Dain Bramaged

rectification, en fait, je viens de réellement compilé tout ton programme et c'est la merde de tout les côté je comprend même pas comment ça peut compiler chez toi ...

attend voir que je te commente ton code

k4rt3x

Davantage d'avantages avantage

:pt1cable:

Tu le compile avec quoi toi ? :bounce:

Deadog

Dain Bramaged

avec gcc, et avec les options les plus gore, c'est à dire quasiment aucune vérification, et pourtant, il m'a jeté comme un mal propre en essayant de compiler

Code :

#include <stdio.h>
#include <stdlib.h>
int main ()
{
// La définition de fonction n'a rien à faire ici
int hdd();
int disk();
int usb();
int menu();
int choixMenu = 0;
printf ("\n\n\t\t\tFORMATAGE" );
printf ("\n\n\n\n\t Menu:\n\n" );
choixMenu = menu(&choixMenu); // faut choisir, soit tu passe un pointeur en paramètre, soit tu renvoie qqlch, mais pas les deux, surtout quand c'est la même variable !
switch (choixMenu)
{
case 1:
hdd();
// Si y'a pas de break ici, sortie de hdd() il va direct exécuter default, ce qui explique pourquoi tu retournais dans le menu
/* case 2:
disk();
case 3:
usb();
case 4:
exit(0); */
default:
system("cls" );
printf ("\nChoix incorrect, tappez un nombre du menu:\n\n" );
choixMenu = menu(); // int menu () n'existe pas, c'est int menu (int *) qui existe seulement
}
return 0;
}
int menu(int *ptrChoixMenu)
{
int choixMenu;
printf ("1. Formater un disque dur\n" );
/*
printf ("2. Formater une disquette\n" );
printf ("3. Formater une cle usb ou lecteur MP3\n" );
printf ("4. Quitter le programme\n\n" );
printf ("Quel est votre choix ?\n" );
*/
scanf ("%ld", &choixMenu); // Pourquoi ld ? d suffirait
*ptrChoixMenu = choixMenu; // tant qu'à faire, autant écrire directement dans ptrChoixMenu
// la fonction est déclaré renvoyant u int mais y'a pas de return ?!
}

déjà pour le début

Message édité par Deadog le 31-08-2006 à 02:43:38

Deadog

Dain Bramaged

correction à l'aveugle
je t'ai fait 2 fonction menu : une qui marche en pointeur, l'autre avec une valeur de retour

Code :

#include <stdio.h>
#include <stdlib.h>
int hdd();
int disk();
int usb();
int menu();
void menu_ptr (int *);
int main ()
{
int choixMenu = 0;
printf ("\n\n\t\t\tFORMATAGE" );
printf ("\n\n\n\n\t Menu:\n\n" );
menu_ptr(&choixMenu);
switch (choixMenu)
{
case 1:
hdd();
break;
/* case 2:
disk();
case 3:
usb();
case 4:
exit(0); */
default:
system("cls" );
printf ("\nChoix incorrect, tappez un nombre du menu:\n\n" );
choixMenu = menu();
}
return 0;
}
void menu_ptr(int *ptrChoixMenu)
{
int choixMenu;
printf ("1. Formater un disque dur\n" );
/*
printf ("2. Formater une disquette\n" );
printf ("3. Formater une cle usb ou lecteur MP3\n" );
printf ("4. Quitter le programme\n\n" );
printf ("Quel est votre choix ?\n" );
*/
scanf ("%d", ptrChoixMenu);
}
int menu()
{
int choixMenu;
printf ("1. Formater un disque dur\n" );
/*
printf ("2. Formater une disquette\n" );
printf ("3. Formater une cle usb ou lecteur MP3\n" );
printf ("4. Quitter le programme\n\n" );
printf ("Quel est votre choix ?\n" );
*/
scanf ("%d", &choixMenu);
return choixMenu;
}

Message édité par Deadog le 31-08-2006 à 02:43:52

k4rt3x

Davantage d'avantages avantage

Merci beaucoup, par contre je vais aller me coucher je verrai tous ça demain, merci @+

k4rt3x

Davantage d'avantages avantage

Voila, j'ai refait le programme en utilisant uniquement le menu qui marche avec pointeur
mais :

Code :

int hdd()
{
int choix;
char buffer[BUFSIZ];
char hdd;
char buf[11];
system("cls" );
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
fgets (buffer, BUFSIZ, stdin);
hdd = buffer[0];
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
switch (choix)
{
case 1:
printf ("Debut du formatage: " );
break;
default:
exit(0);
}
system(buf)

Là ca ne marche toujours pas, ça passe directement à 'Entrez 1 pour commencer le formatage ou 0 pour quitter.'

sans attendre que l'on ait taper une lettre. :pfff:

J'ai essayer ça:

Code :

int hdd()
{
int choix;
char buffer[BUFSIZ];
char hdd;
char buf[11];//11 normalement
system("cls" );
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater:" );
scanf ("%s", &hdd);
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nLe formatage va commencer ...\n\n" );
system(buf);
}

Et ca marche donc je pense que je vais tout laisser comme ça.

Message cité 1 fois

Mossieur Propre

le scanf, c'est vraiment de la merde .... banni de mon lexique C

---------------
Forever Yoko in my heart. | http://www.phoenixfr.org - réseau IRC francophone | ~~knexos~~ xstephx a dit : "blabla"

Deadog

Dain Bramaged

k4rt3x a écrit :

Voila, j'ai refait le programme en utilisant uniquement le menu qui marche avec pointeur
mais :

Code :

int hdd()
{
int choix;
char buffer[BUFSIZ];
char hdd;
char buf[11];
system("cls" );
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater: " );
fgets (buffer, BUFSIZ, stdin);
hdd = buffer[0];
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nEntrez 1 pour commencer le formatage ou 0 pour quitter. " );
switch (choix)
{
case 1:
printf ("Debut du formatage: " );
break;
default:
exit(0);
}
system(buf)

Là ca ne marche toujours pas, ça passe directement à 'Entrez 1 pour commencer le formatage ou 0 pour quitter.'

sans attendre que l'on ait taper une lettre. :pfff:

J'ai essayer ça:

Code :

int hdd()
{
int choix;
char buffer[BUFSIZ];
char hdd;
char buf[11];//11 normalement
system("cls" );
printf ("\n\n\t\t\tFORMATER UN DISQUE DUR" );
printf ("\n\n\n\nVeuillez entrer entrer la lettre du disque a formater:" );
scanf ("%s", &hdd);
sprintf(buf, "FORMAT %c: ", hdd);
printf ("\nLe formatage va commencer ...\n\n" );
system(buf);
}

Et ca marche donc je pense que je vais tout laisser comme ça.

bah oui, t'as pas remit le while

si tu laisse un scanf("%s", ...) sur un char c'est vraimant imonde

CryoGen

on peut flush() le stdin aussi non ? ca fait longtemps que j'ai plus fait de C qui gere une entrée clavier ^^

Message cité 1 fois

---------------
I am the bone of my sword...

Deadog

Dain Bramaged

CryoGen a écrit :

on peut flush() le stdin aussi non ? ca fait longtemps que j'ai plus fait de C qui gere une entrée clavier ^^

non non et non pas de fflush du stdin :fou: c'est pire que tout

CryoGen

ok j'ai rien dit

--> [EXIT]

---------------
I am the bone of my sword...

cybersplash

Laissez moi chanter !!!

au risque de dire une connerie pourquoi ne pas utiliser le getchar() ?
en plus ça evite le buffer overflow car on ne récupère qu'un caractère

hdd=getchar();
if(tmp=getchar()!=\n) {/*Gestion de l'erreur*/}
else {/* controle de la validité du caractère */}

voila le petit bout de code pour faire comme un scanf

(si je me trompe soyez indulgent mon c se fait vieux )

Google

FORUM Syndrome-OC - Jacky-PC

Programmation

Langage haut niveau(C, Pascal...)

[C] Probleme commande système

Plan du forum
Forum MesDiscussions.Net, Version 2007.3
(c) 2000-2008 Doctissimo Software

Page générée en 0.321 secondes