Additionner,Soustraire,Multiplier et diviserRésolu

Question

Salut à tous, voici ma question: Dans le code suivant, il faut choisir entre additionner, soustraire, multiplier et diviser. Quelqu'un peut-il m'aider à réorganiser le code pour que dans chaque Case de Switch je ne sois pas obligé de retaper la portion concernant la saisie du premier et du deuxième nombre (en italique souligné dans le code), puisqu'à chaque fois c'est la même chose. Je suis vraiment novice, alors soyez indulgent et aidez-moi ! using System ; namespace Operators { ///

/// Effectue des multiplications additions divisions soustractions ///

public class Class1 { public static void Main(string[] args ) { // Demande le type d'opération à effectuer : // 1 pour additionner // 2 pour multiplier // 3 pour diviser // 4 pour soustraire Console .WriteLine("Type d'opération à effectuer :"); Console .WriteLine(); // Saut de ligne Console .WriteLine("1-Addition"); Console .WriteLine("2-Multiplication"); Console .WriteLine("3-Division"); Console .WriteLine("4-Soustraction"); Console .WriteLine(); // Saut de ligne Console .WriteLine("Choisissez le numéro correspondant à l'opération que vous souhaitez faire : "); string sMyChoice = Console . ReadLine (); // Convertit ce qui a été saisi en un entier int nMyChoice = Convert .ToInt32(sMyChoice ); string sFirstNbr ; string sSecondNbr ; decimal mFirstNbr ; decimal mSecondNbr ; decimal mResult ; switch(nMyChoice ) { case 1: //Demande le premier nombre Console.Write("Entrez le premier nombre : " ); sFirstNbr = Console .ReadLine (); // Convertit ce qui a été saisi en un decimal mFirstNbr = Convert .ToDecimal(sFirstNbr ); //Demande le second nombre Console.Write("Entrez le deuxième nombre : " ); sSecondNbr = Console .ReadLine (); //Convertit ce qui a été saisi en un decimal mSecondNbr = Convert .ToDecimal(sSecondNbr ); // Additionne le premier et le deuxième nombre mResult = mFirstNbr + mSecondNbr ; // Affiche le résultat Console . WriteLine (); // Saute une ligne Console .WriteLine(mFirstNbr + " plus " + mSecondNbr + " est égal à " + mResult ); break; case 2: //Demande le premier nombre Console .Write("Entrez le premier nombre : "); sFirstNbr = Console .ReadLine(); // Convertit ce qui a été saisi en un decimal mFirstNbr = Convert . ToDecimal (sFirstNbr ); //Demande le second nombre Console .Write("Entrez le deuxième nombre : "); sSecondNbr = Console .ReadLine(); //Convertit ce qui a été saisi en un decimal mSecondNbr = Convert . ToDecimal (sSecondNbr ); // Multiplie le premier et le deuxième nombre mResult = mFirstNbr * mSecondNbr ; // Affiche le résultat Console .WriteLine(); // Saute une ligne Console .WriteLine(mFirstNbr + " multiplié par " + mSecondNbr + " est égal à " + mResult ); break; case 3: //Demande le premier nombre Console .Write("Entrez le premier nombre : "); sFirstNbr = Console . ReadLine (); // Convertit ce qui a été saisi en un decimal mFirstNbr = Convert .ToDecimal(sFirstNbr ); //Demande le second nombre Console .Write("Entrez le deuxième nombre : "); sSecondNbr = Console . ReadLine (); //Convertit ce qui a été saisi en un decimal mSecondNbr = Convert .ToDecimal(sSecondNbr ); // Divise le premier et le deuxième nombre mResult = mFirstNbr / mSecondNbr ; // Affiche le résultat Console . WriteLine (); // Saute une ligne Console .WriteLine(mFirstNbr + " divisé par " + mSecondNbr + " est égal à " + mResult ); break; case 4: //Demande le premier nombre Console .Write("Entrez le premier nombre : "); sFirstNbr = Console .ReadLine(); // Convertit ce qui a été saisi en un decimal mFirstNbr = Convert . ToDecimal (sFirstNbr ); //Demande le second nombre Console .Write("Entrez le deuxième nombre : "); sSecondNbr = Console .ReadLine(); //Convertit ce qui a été saisi en un decimal mSecondNbr = Convert . ToDecimal (sSecondNbr ); // Divise le premier et le deuxième nombre mResult = mFirstNbr - mSecondNbr ; // Affiche le résultat Console .WriteLine(); // Saute une ligne Console .WriteLine(mFirstNbr + " moins " + mSecondNbr + " est égal à " + mResult ); break; default: Console .WriteLine("Vous devez saisir un nombre compris entre 1 et 4, Veuillez recommencer."); Console .WriteLine(); // Saute une ligne break; } } } }

cs_coq · Accepted Answer

Salut,

Une m&#233;thode ?
private decimal GetNumber(string question)
{
Console.Write(question);
string sNb = Console.ReadLine();
// Convertit ce qui a &#233;t&#233; saisi en un decimal
return Convert.ToDecimal(sNb);
}

Appel : 
//Demande le premier nombre 
mFirstNbr = this.GetNumber("Entrez le premier nombre : ");
//Demande le second nombre 
mSecondNbr = this.GetNumber("Entrez le deuxi&#232;me nombre : ");

/*
coq
MVP Visual C#
*/

cs_v1 · Answer

J'ai validé la réponse car je suis sûr qu'il faut faire comme ça, mais le compiler me dit "keyword this is not valid in a static property, static method, or static field initializer". Voici ce que j'ai mis (j'ai retiré le reste des Case, je n'ai laissé que le 1 et le default, pour simplifier), mais je pense que je me suis trompé en remettant la méthode telle quelle dans le code. Si tu m'aides encore une fois ce serait bien, (-coq-) après je te fous le paix pour aujourd'hui, et je vais m'informer sur les méthodes et les fonctions ! ps : ben dis-donc c'est pas évident quand on débute, quelle galère ! using System ; namespace Operators { ///

/// Effectue des multiplications ///

public class Class1 { public static void Main(string[] args ) { // Demande le type d'opération à effectuer : // 1 pour additionner // 2 pour multiplier // 3 pour diviser // 4 pour soustraire Console .WriteLine("Type d'opération à effectuer :"); Console .WriteLine(); // Saut de ligne Console .WriteLine("1-Addition"); Console .WriteLine("2-Multiplication"); Console .WriteLine("3-Division"); Console .WriteLine("4-Soustraction"); Console .WriteLine(); // Saut de ligne Console .WriteLine("Choisissez le numéro correspondant à l'opération que vous souhaitez faire : "); string sMyChoice = Console . ReadLine (); // Convertit ce qui a été saisi en un entier int nMyChoice = Convert .ToInt32(sMyChoice ); string sFirstNbr ; string sSecondNbr ; decimal mFirstNbr ; decimal mSecondNbr ; decimal mResult ; switch(nMyChoice ) { case 1: //Demande le premier nombre mFirstNbr = this .GetNumber("Entrez le premier nombre : " ); //Demande le second nombre mSecondNbr = this .GetNumber("Entrez le deuxième nombre : "); // Affiche le résultat Console .WriteLine(); // Saute une ligne Console .WriteLine(mFirstNbr + " plus " + mSecondNbr + " est égal à " + mResult ); break; default: Console .WriteLine("Vous devez saisir un nombre compris entre 1 et 4, Veuillez recommencer."); Console .WriteLine(); // Saute une ligne break; } } private decimal GetNumber(string question ) { Console .Write(question ); string sNb = Console .ReadLine(); // Convertit ce qui a été saisi en un decimal return Convert .ToDecimal(sNb ); } } }

cs_badrbadr · Answer

Code source modifi&#233; pour &#233;conomiser d'avantage d'espace


using System;


namespace Operators


{


public 
class Class1


{


public 
static 
void Main(
string[] args)


{ Console.WriteLine("Type d'op&#233;ration &#224; effectuer :");


Console.WriteLine(); 
// Saut de ligne


Console.WriteLine("1-Addition
2-Multiplication
3-Division
4-Soustraction
");


Console.WriteLine("Choisissez le num&#233;ro correspondant &#224; l'op&#233;ration que vous souhaitez faire : "); 



string sMyChoice = Console.ReadLine();



// Convertit ce qui a &#233;t&#233; saisi en un entier



int nMyChoice = Convert.ToInt32(sMyChoice);



string sFirstNbr; 
string sSecondNbr;



decimal mFirstNbr; 
decimal mSecondNbr;



decimal mResult;


Console.Write("Entrez le premier nombre : ");


sFirstNbr = Console.ReadLine();


mFirstNbr = Convert.ToDecimal(sFirstNbr);


Console.Write("Entrez le deuxi&#232;me nombre : ");


sSecondNbr = Console.ReadLine();


mSecondNbr = Convert.ToDecimal(sSecondNbr);



switch(nMyChoice)


{



case 1:


mResult = mFirstNbr + mSecondNbr;


Console.WriteLine(); 
// Saute une ligne


Console.WriteLine(mFirstNbr + " plus " + mSecondNbr + " est &#233;gal &#224; " + mResult);



break; 



case 2: 


mResult = mFirstNbr * mSecondNbr;


Console.WriteLine(); 
// Saute une ligne


Console.WriteLine(mFirstNbr + " multipli&#233; par " + mSecondNbr + " est &#233;gal &#224; " + mResult);



break; 



case 3:



if(mSecondNbr!=0)


{


mResult = mFirstNbr / mSecondNbr;


Console.WriteLine(); 
// Saute une ligne


Console.WriteLine(mFirstNbr + " divis&#233; par " + mSecondNbr + " est &#233;gal &#224; " + mResult);


} 



else


{


Console.WriteLine(); 
// Saute une ligne


Console.WriteLine("division par z&#233;ro impossible");


}



break;



case 4:


mResult = mFirstNbr - mSecondNbr;


Console.WriteLine(); 
// Saute une ligne


Console.WriteLine(mFirstNbr + " moins " + mSecondNbr + " est &#233;gal &#224; " + mResult);



break;



default:


Console.WriteLine("Vous devez saisir un nombre compris entre 1 et 4, Veuillez recommencer.");


Console.WriteLine(); 
// Saute une ligne



break;


}


}


}


}

@++

cs_v1 · Answer

Oui c'est pas mal badrbadr, mais ce n'est plus tr&#232;s logique dans de cas, parce que si je recopie le code, je me rend compte que je peux sasir des nombres sup&#233;rieur &#224; 4 au d&#233;but, et c'est seulement apr&#232;s avoir saisi les deux nombres que la phrase du "Default" appara&#238;t !

LUDINSKI · Answer

Salut V1,

static void Main(string
[] args)
{
   Console.WriteLine( "Type d'op&#233;ration &#224; effectuer :
" );
   Console.WriteLine( "1-Addition
2-Multiplication
3-Division
4-Soustraction
" );


   bool operatorOK = false
;
   int
 nMyChoice = 0;
   do
   {
      Console.WriteLine( "Choisissez le num&#233;ro correspondant &#224; l'op&#233;ration que vous souhaitez faire : " ); 
      string
 sMyChoice = Console.ReadLine();
      nMyChoice = Convert.ToInt32( sMyChoice );

      if( nMyChoice < 1 || nMyChoice > 4 )
      {
         Console.WriteLine( "Vous devez saisir un nombre compris entre 1 et 4, Veuillez recommencer.
" );
      }
      else
         operatorOK = true
;
   }
   while
( ! operatorOK );

   string sFirstNbr, sSecondNbr;
   decimal
 mFirstNbr, mSecondNbr;

   Console.Write( "Entrez le premier nombre : " );
   sFirstNbr = Console.ReadLine();
   mFirstNbr = Convert.ToDecimal( sFirstNbr );

   Console.Write( "Entrez le deuxi&#232;me nombre : " );
   sSecondNbr = Console.ReadLine();
   mSecondNbr = Convert.ToDecimal( sSecondNbr );

   Console.WriteLine(); // Saute une ligne

   decimal mResult;
   string
 operation = "";
   switch
( nMyChoice )
   {
      case
 1:

         mResult = mFirstNbr + mSecondNbr;
         operation = " plus ";
         break
;

      case
 2: 

         mResult = mFirstNbr * mSecondNbr;
         operation = " multipli&#233; par ";
         break
; 

      case 3:

         if( mSecondNbr == 0 )
         {
            Console.WriteLine( "division par z&#233;ro impossible" );
            return
;
         }

         mResult = mFirstNbr / mSecondNbr;
         operation = " divis&#233; par ";
         break
;

      default:

         mResult = mFirstNbr - mSecondNbr;
         operation = " moins ";
         break
;
      }

      Console.WriteLine( mFirstNbr + operation + mSecondNbr + " est &#233;gal &#224; " + mResult );
   }

Ca devrait faire ce qui faut qu'&#231;a fasse... ;)

Additionner,Soustraire,Multiplier et diviser

5 réponses

Votre réponse

Discussions similaires