IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Langage C++ Discussion :

console corrompue lors d'un "system()"


Sujet :

Langage C++

  1. #1
    Candidat au Club
    Profil pro
    Inscrit en
    Juin 2007
    Messages
    7
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juin 2007
    Messages : 7
    Points : 3
    Points
    3
    Par défaut console corrompue lors d'un "system()"
    Bonjour,

    J'ai un probleme simple, mais qui me cause une grosse incomprehension. Quand je fais un "system()", ma console se trouve a etre corrompue.

    Voici le code qui cause probleme:
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
    13
    14
    15
    16
    17
    18
    19
    20
     
    char sMessage[3];
    sMessage[1] = 0;
    for (int ii=97; ii<100; ii++)
    {
    	sMessage[0] = ii;
    printf_s("%d - %s\n", ii, sMessage);
    }
     
    // extract the metadata from the geotiff
    char sCreateMetadataFile[STRING1024];
    sprintf_s(sCreateMetadataFile, STRING1024, "%slistgeo.exe -d %s > %s", sExePath, pMetadata->sPathRGBFileName, sPathMetadataFileName);
    system(sCreateMetadataFile);
     
    sMessage[1] = 0;
    for (int ii=97; ii<100; ii++)
    {
    	sMessage[0] = ii;
    printf_s("%d - %s\n", ii, sMessage);
    }
    Comme je fais une redirection, l'ecran devrais simplement afficher
    97 - a
    98 - b
    99 - c
    97 - a
    98 - b
    99 - c

    A partir du system(), tous les '\n' de mon code sont corrompus. Je peux mettre n'importe quoi dans ma commande systeme, redirection ou pas, chaque changement de ligne est alors composer de 2 caracteres ASCII au lieu de faire un saut de ligne. De plus, si la ligne est pleine, il recommence a ecrire sur la meme ligne.

    Vous avez une idee de ce qui peut causer ca?

    Voir le fichier attacher pour voir ce que ca donne.
    C'est sur Windows 7, compilé dans VS2010.
    Images attachées Images attachées  

  2. #2
    Expert éminent
    Avatar de transgohan
    Homme Profil pro
    Développeur Temps réel Embarqué
    Inscrit en
    Janvier 2011
    Messages
    3 146
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Maine et Loire (Pays de la Loire)

    Informations professionnelles :
    Activité : Développeur Temps réel Embarqué

    Informations forums :
    Inscription : Janvier 2011
    Messages : 3 146
    Points : 9 386
    Points
    9 386
    Par défaut
    Il renvoie quoi ton programme ? Et surtout où ? (sPathMetadataFileName)

    « Toujours se souvenir que la majorité des ennuis viennent de l'espace occupé entre la chaise et l'écran de l'ordinateur. »
    « Le watchdog aboie, les tests passent »

  3. #3
    Candidat au Club
    Profil pro
    Inscrit en
    Juin 2007
    Messages
    7
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juin 2007
    Messages : 7
    Points : 3
    Points
    3
    Par défaut
    Citation Envoyé par transgohan Voir le message
    Il renvoie quoi ton programme ? Et surtout où ? (sPathMetadataFileName)
    Il renvoie du texte, plus precisement des coordonnees geographiques, mais ce n'est pas le noeud du probleme. Je peux mettre n'importe quoi dans system() et j'aurai le meme probleme. Par example, si je remplace ma commande par
    systeme ("aaa");
    Je devrais voir a l'ecran:
    "'aaa' is not recognized as an internal or external command,
    operable program or batch file."

    Mais deja a ce moment le message est corrompu. Les 2 retours de chariot sont remplaces par les 2 symboles bizarres, et la ligne se reecrit sur elle-meme. Et par la suite, tout ce qui passe par un printf a le meme probleme.

  4. #4
    Expert éminent
    Avatar de transgohan
    Homme Profil pro
    Développeur Temps réel Embarqué
    Inscrit en
    Janvier 2011
    Messages
    3 146
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Maine et Loire (Pays de la Loire)

    Informations professionnelles :
    Activité : Développeur Temps réel Embarqué

    Informations forums :
    Inscription : Janvier 2011
    Messages : 3 146
    Points : 9 386
    Points
    9 386
    Par défaut
    Pour vérifier que ce n'est pas la console qui est impliquée tentes ta chance avec une fonction telle que exec : http://msdn.microsoft.com/en-us/libr...=vs.71%29.aspx

    « Toujours se souvenir que la majorité des ennuis viennent de l'espace occupé entre la chaise et l'écran de l'ordinateur. »
    « Le watchdog aboie, les tests passent »

  5. #5
    Expert éminent sénior

    Femme Profil pro
    Ingénieur développement logiciels
    Inscrit en
    Juin 2007
    Messages
    5 189
    Détails du profil
    Informations personnelles :
    Sexe : Femme
    Localisation : France, Essonne (Île de France)

    Informations professionnelles :
    Activité : Ingénieur développement logiciels

    Informations forums :
    Inscription : Juin 2007
    Messages : 5 189
    Points : 17 141
    Points
    17 141
    Par défaut
    ce sont quoi, tes fonctions printf_s et sprintf_s?

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    char sMessage[3];
    sMessage[1] = 0;
    for (int ii=97; ii<100; ii++)
    {
    	sMessage[0] = ii;
    printf_s("%d - %s\n", ii, sMessage);
    }
    Ceci est étrange, tu écris pars de {?,0,?}, et tu écris quelques fois dans [0]
    ton sMessage est donc {99, 0, ?}
    deplus, il ne sert à rien d'autre que l'affichage.

    printf %c affiche le caractère ascii contenu dans un char.
    donc printf_s("%d - %c\n", ii, ii); afficherait la même chose sans utiliser sMessage du tout. autre variante: for (char ii=97; ii<100;++ii) printf_s("%d - %c\n", ii, ii);Comme tu es en C++, je te recommande d'utiliser plutot <iostream> et cout/cerr/clog

    rajoute std::cerr<<'['<<sCreateMetadataFile<<']'<<std::endl; juste avant l'appel à system.
    Mes principes de bases du codeur qui veut pouvoir dormir:
    • Une variable de moins est une source d'erreur en moins.
    • Un pointeur de moins est une montagne d'erreurs en moins.
    • Un copier-coller, ça doit se justifier... Deux, c'est un de trop.
    • jamais signifie "sauf si j'ai passé trois jours à prouver que je peux".
    • La plus sotte des questions est celle qu'on ne pose pas.
    Pour faire des graphes, essayez yEd.
    le ter nel est le titre porté par un de mes personnages de jeu de rôle

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo