programme qui lit une chaine représentant une opération sur des entiers et qui affiche le résultat.

[slashergodzila]

Bonjour , je rencontre un problème dans mon code j'aimerais avoir votre aide car m'aide pas merci.
voici l'énoncé:

Écrire un programme qui lit une chaine représentant une opération sur des entiers et qui
affiche le résultat. Par exemple, si on lui donne la chaine 12+25, elle doit afficher 12 + 25 = 37. Seuls les
opérateurs (+,-, x et /) seront pris en compte. Vous pouvez vous renseigner sur la fonction atoi de la
bibliothèque stdlib.h et ainsi l'utiliser pour convertir les nombres sous formes de chaines en leur version
entière.

et voilà ma tentative de résolution :

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int main (){
    char operation[100],nb1[100], nb2[100] ;
    char signe[] = "+-x/";
    int nombre1, nombre2;
    char signes;
    int resultat;
    printf("veuillez entrer une opération \n");
    fgets(operation, sizeof(operation), stdin);
    char *nba1 = strtok(operation,signe);
    char *nba2 = strtok(NULL,"\0");
     
    
    

    nombre1 = atoi(nba1);
    nombre2 = atoi(nba2);
    signes = operation[strlen(nba1) + 1];  //selection du signe

    if (signes =='+') {
        resultat == nombre1 + nombre2;
        printf("Le résultat de l'addition est : %d + %d = %d\n",nombre1, nombre2, resultat);
    } else if (signes == '-'){resultat = nombre1 - nombre2;
        printf("Le résultat de l'addition est : %d - %d = %d\n",nombre1, nombre2, resultat);}
    else if (signes == 'x'){resultat = nombre1 * nombre2;
        printf("Le résultat de l'addition est : %d x %d = %d\n",nombre1, nombre2, resultat);}
    else if (signes == '/'){resultat = nombre1 / nombre2;
        printf("Le résultat de l'addition est : %d / %d = %d\n",nombre1, nombre2, resultat);}
   

    
    return 0;

    
}

lorsque j'exécute le code , il s'affiche "veuillez entrer une opération" mais lorsque je rentre une opération ,cela ne me retourne rien.
-----

[umfred]

On devrait faire apprendre l'art du débogage.
Si rien ne s'affiche, c'est qu'on ne va dans aucun printf, il faut en trouver la raison.
Dans un 1er temps, pourquoi ne pas afficher la chaine lue par fgets, puis les valeurs de nba1 et nba2 puis nombre1, nombre2 et signes pour vérifier qu'ils ont bien les valeurs attendues.
Si tu as un outils de débogage, utilise le de la même façon en pas à pas pour vérifier les valeurs des variables.

Et on peux aussi relire son code pour voir que l'on a déclaré des variables nb1 et nb2, et que l'on a mis nba1 et nba2 dans le code et voir ensuite que cela génère des erreurs de compilation à résoudre

[umfred]

zut, j'ai été trop vite pour mon dernier point, erreur de lecture de ma part.

[pm42]

On devrait faire apprendre l'art du débogage.

Oui, je ne comprends pas pourquoi on ne commence pas par ça dès qu'on donne des exercices de programmation.
Dans le cas présent, le compilateur donne un warning logique sur : resultat == nombre1 + nombre2 au lieu de resultat = nombre1 + nombre2

Et pour le reste, c'est simplement du au fait que signes = operation[strlen(nba1) + 1] devrait être signes = operation[strlen(nba1)] sur le papier mais que ça ne marche pas parce que strtok modifie la chaine qu'on lui passe.
Il faudrait donc faire une copie de operation, la passer à strtok pour extraire les 2 nombres puis extraire l'opérateur de operation.

[A voir en vidéo sur Futura]

[polo974]

Le premier strtok a remplacé l'operateur par un '\0'...

Pas d'opérateur, pas d'opération.

(Edit, un peu doublon de pm42...)

[Biname]

Salut,
ChatGPT résout le pb en 1 minute
Normalement vous pouvez compiler et tester ce code ici GDB_C_ONLINEhttps://onlinegdb.com/Hcye1h9rxr

 Cliquez pour afficher

#include <iostream>
#include <cstdlib> // Pour la fonction atoi()

using namespace std;

int main() {
    string operation;
    cout << "Entrez une opération sur des entiers : ";
    cin >> operation;

    int resultat = 0;
    char operateur = '+';

    // Recherche de l'opérateur et des opérandes
    int i = 0;
    while (i < operation.length()) {
        if (operation[i] == '+' || operation[i] == '-' || operation[i] == 'x' || operation[i] == '/') {
            operateur = operation[i];
            break;
        }
        i++;
    }

    // Extraction des opérandes
    string operande1 = operation.substr(0, i);
    string operande2 = operation.substr(i + 1);

    // Conversion des opérandes en entiers
    int nombre1 = atoi(operande1.c_str());
    int nombre2 = atoi(operande2.c_str());

    // Calcul du résultat en fonction de l'opérateur
    switch (operateur) {
        case '+':
            resultat = nombre1 + nombre2;
            break;
        case '-':
            resultat = nombre1 - nombre2;
            break;
        case 'x':
            resultat = nombre1 * nombre2;
            break;
        case '/':
            resultat = nombre1 / nombre2;
            break;
        default:
            cout << "Opérateur invalide !" << endl;
            return 0;
    }

    // Affichage du résultat
    cout << operande1 << " " << operateur << " " << operande2 << " = " << resultat << endl;

    return 0;
}

12x2=24 yesse
Biname

[pm42]

L'obsession qu'on certains de donner les solutions complètes contrairement à ce que dit la charte est impressionnante.
Et donner une solution qui n'utilise pas les mêmes fonctions a en plus comme inconvénient d'apprendre encore moins à celui qui poste la question.

[umfred]

@biname ton code c'est du C++, a priori le PO développe en C
Mais le site https://www.onlinegdb.com/ est pratique pour faire des petits programmes à tester sans avoir besoin de tous les IDE installés sur sa machine (il a effectivement en plus un débogueur pour le C et autres langages)

[pm42]

Tiens je n'avais pas fait attention au C++.
La totale.

[slashergodzila]

bonjour, je vous remercie de votre aide, effectivement je n'avais pas tenu compte du fait que strtok modifie la chaîne d'origine. De plus je n'avais pas bien cerné le fonctionnement de strlen . après modification, le code fonctionne parfaitement merci!!