Petite correction sur mon code

Je suis actuellement le cour de zeste-de-savoir sur le C++, voila je voulais quelqu’un de plus expérimenté dans ce langage revois mon code… Et si possible avoir un point de vue critique afin de m’améliorer encore plus. Merci en avance pour votre attention.

Pour trouver le PGCD d’un nombre j’ai utilisé la méthode de soustraction :

#include <iostream>

int main () { std::cout « "a = "; int a { 0 };

while (!(std::cin >> a))
{
    std::cout << "Erreur de saisie, re-saisir a = ";
    std::cin.clear(); // remet cin dans un etat fonctionnel
    std::cin.ignore(500, '\n'); // vide le buffer
}

std::cout << "b = ";
int b { 0 };

while(!(std::cin >> b))
{
    std::cout << "Erreur de saisie, re-saisir b = ";
    std::cin.clear(); // remet cin dans un etat fonctionnel
    std::cin.ignore(500, '\n'); // vide le buffer
}

int soustraction { a - b };

while (soustraction != 0)
{
    if (soustraction > b)
    {
        a = soustraction;
        soustraction = a - b;
    }

     else
    {
        a = b;
        b = soustraction;
        soustraction = a - b;
    }
 }

std::cout << "Le PGCD vaut : " << a << ".\n";

return 0;

}

Tu as des lignes vides inutiles aux lignes 27 et 34… Bon, à part ça, je vois rien à dire.

Ah d’accord j’ai vu mais je l’avais fait pour rendre mon code plus lisible en cas de relecture de ma part… pour celui de 34 c’était involontaire et j ai rectifié sur mon éditeur. Merci sinon

Oui je partais du principe que a > b, c’était à partir de là que j’avais construit mon raisonnement.

Et pour ce qui est de créer une fonction, je ne possède pas encore le bagage nécessaire, je ne suis qu’au chapitre sur le Tableau. Mais je compte bien le faire.

J’ai fais de mon mieux pour que ce petit programme ne rentre pas dans une boucle infinie (si a < b) J’ai donc réécris mon code comme ci-dessous :

#include <iostream>

int main () { std::cout « "a = "; int a { 0 };

while (!(std::cin >> a))
{
    std::cout << "Erreur de saisie, re-saisir a = ";
    std::cin.clear(); // remet cin dans un etat fonctionnel
    std::cin.ignore(500, '\n'); // vide le buffer
}

std::cout << "b = ";
int b { 0 };

while (!(std::cin >> b))
{
    std::cout << "Erreur de saisie, re-saisir b = ";
    std::cin.clear(); // remet le flux dans un etat fonctionnel 
    std::cin.ignore(500, '\n'); // vide le buffer
}

if (a < b)
{
    int c { a };
    a = b;
    b = c;
}

int soustraction { a - b };

while(soustraction != 0)
{
    if (soustraction > b)
    {
        a = soustraction;
        soustraction = a - b;
    }
    else 
    {
        a = b;
        b = soustraction;
        soustraction = a - b;
    }
}
    
std::cout << "Le PGCD vaut : " << a << std::endl;

return 0;

}

Je vais essayer de repenser autrement afin d’écrire un code qui prend en compte une condition avec les opérateurs >=, > ou <=, <…

Autre question pourquoi préfère-t-on les conditions basées sur les opérateurs >=, >, <=, <

Est-ce seulement à cause du risque d’une boucle infinie ou bien il y a d’autres raisons à ça ?

Ouais je m’en doutais aussi que j’avais mal choisi ce nom (soustraction). Mon premier choix portais sur <résultat> mais je crois que c’est un peu tiré du cheveux et trop général comme nom de variable. C’est mieux et je préfère <différence> par rapport à l’ancien…

En ce qui concerne de créer une fonction afin de séparer les 2 actions, je compte bien le faire plus tard, vu que je n’ai pas encore lu le chapitre sur les fonctions… En fait j’avais constaté que je progressais trop vite. Du coup j’ai préféré prendre un certain recul afin de solidifier mes acquis.

Merci pour votre analyse et exemple, je compte bien l’étudier une fois sur les fonctions.

J’ai vu sur un autre tutoriel que clear(), ignore() sont des méthodes… et sur celui de zeste-de-savoir que ce sont des fonctions. Du coup j’ai considéré que méthode = fonction (je ne me trompe pas j’espère).

Oui tout a fait d’accord et c’est logique, mieux vaut découper son programme en plusieurs actions dont chacune remplie une tache bien précise (fonction)… car c’est bien plus facile de corriger quelques lignes de codes qu’un ensemble de codes qui fait tout à la fois (Si je comprends bien vos propos).

Super ça m’aide bcp l’exemple sur la troisième fonction… Je comprends bcp mieux le rôle des fonctions (découper un problème en sous-tache).

Pour l’instant je préfère garder une définition basique : une méthode désigne une fonction membre… Avec le temps, j’aurai sûrement une définition bien plus détaillé et fourni de ce qu’une méthode…

Merci pour la précision et la petite histoire… ce fut un plaisir de la lire même si je rencontre trop de vocabulaire que je ne comprends point (preuve qu’il me reste bcp de chose à apprendre en C++ et en programmation)… Je relirai sûrement avec le temps.

Le vocabulaire peut varier d’un langage à un autre.

Déjà, ce qui s’appelle communément "fonction" en C et dans les langages qui s’en inspirent, ce n’est absolument pas une fonction dans aucun sens du terme.

Si, c’est juste qu’il y a des paramètres implicites

Ça me parait vraiment bizarre. Ok, les fonctions qui renvoient void sont censées se nommer des procédures, mais toutes les autres fonctions se nomment bien fonction.

On (ceux qui adoptent la terminologie du langage) les appelle fonction, mais dans le dictionnaire, il y a deux sens pour ce mot

• rôle, comme dans "remplir une fonction
• relation entre des quantités (maths)

En programmation, c’est pas une relation, c’est un truc qui effectue un certain traitement.

On le rapproche des fonctions pour diverses raisons

• les fonctions de fortran, qui au début (fin années 50) étaient des relations écrites en début de programmes genre F(X)=A*X + B, et ont été ensuite généralisées en "unités de programme" contenant des instructions quelconques
• les explications par analogie qu’on fait aux débutants à propos des paramètres
• l’existence de bibliothèques de "fonctions predefinies" effectuant les calculs qui correspondent aux fonctions numeriques courantes, racines, trigo, etc
• le fait qu’assez souvent on conseille une approche fonctionnelle, où la fonction des "fonctions" est de calculer et retourner un truc a partir des parametres, sans modifier quoi que ce soit d’autre