• 1
  • 2
2015-04-10, 00:03
  #1
Medlem
Kembalis avatar
Hellu flashbackare!

Jag har skrivit lite kod, som ska försöka föreställa en lättare miniräknare. Jag har lite problem med mitt case 5 då jag inte är nöjd över syntaxen, tycker det är allt för mycket kod, men jag har försökt bryta ut till med funktioner men utan resultat.

Jag har också lite problem med IEnumerator, om jag har förstått rätt så används den för att det ska vara möjligt att foreach loopa ut object från en list?

Övrigt är alla synpunkter uppskattade, jag vet att flashbackare gillar att köra över folk! Men ärligt så är jag noob inom kodning men har lärt mig se det roliga i att försöka skriva snygg kod.

Väl mött!


Kod:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Collections;
using System.IO;

namespace 
ConsoleApplicationEASTER
{
    class 
ClassFunctions {
        public 
int substraktion(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }

        public 
int multiplikation(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }
        public 
int addition(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }
        public 
int division(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }
    }


    class 
userActivity
    
{
        public 
int inData1 getset; }
        public 
int inData2 getset; }
    }


    class 
Program
    
{
        static 
void menu(){
            
Console.WriteLine("1. Multiplikation" Environment.NewLine 
                
"2. Additation" Environment.NewLine "3. Division" 
                    
Environment.NewLine "4. Substraktion" +
                        
Environment.NewLine "5. Multi list" 
                            
Environment.NewLine "6. Exit");
        }

        static 
string tal1() {
            return 
"Tal 1: ";
        }
        static 
string tal2()
        {
            return 
"Tal 2: ";
        }

        static 
void Main(string[] args)
        {
            
Console.Clear();
            
menu();
            
int selector 0;
            
bool good false;
            
good int.TryParse(Console.ReadLine(), out selector);


            if (
good)
            {
                
Console.Clear();
                
Console.WriteLine("**En simpel miniräknare**");

                
//Get instance
                
ClassFunctions callFunction = new ClassFunctions();
                
userActivity userInput = new userActivity(); 
                
                
Console.WriteLine("");
                switch (
selector)
                {
                    case 
1:
                                    
Console.WriteLine("Case1 - Multiplikation" 
                                            
Environment.NewLine "Skriv in två heltal.");

                                    
Console.Write(tal1());
                                    
userInput.inData1 Convert.ToInt32(Console.ReadLine());
                                    
Console.Write(tal2());
                                    
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                                    
Console.Write("Svar: {0} * {1} = "userInput.inData1,userInput.inData2);
                                    
Console.WriteLine(callFunction.multiplikation(userInput.inData1,userInput.inData2));
                                    
Console.ReadLine();
                                    
Main(args);
                        break;

                    case 
2:
                         
Console.WriteLine("Case2 - Additation"+
                             
Environment.NewLine +
                             
"Skriv in 2 heltal");

                         
Console.Write(tal1());
                         
userInput.inData1 Convert.ToInt32(Console.ReadLine());
                         
Console.Write(tal2());
                         
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                        
Console.Write("Svar: {0} + {1} = "userInput.inData1userInput.inData2);
                        
Console.WriteLine(callFunction.addition(userInput.inData1userInput.inData2));
                          
Console.ReadLine();
                          
Main(args);
                        break;

                    case 
3:
                        
Console.WriteLine("Case3 - Division");

                        
Console.Write(tal1());
                        
userInput.inData1 Convert.ToInt32(Console.ReadLine());
                        
Console.Write(tal2());
                        
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                        
Console.Write("Svar: ");
                        
Console.WriteLine(callFunction.division(userInput.inData1userInput.inData2));

                        
Console.ReadLine();
                        
Main(args);


                        break;

                    case 
4:
                        
Console.WriteLine("Case4 - Substraktion");
                        
Console.Write(tal1());
                        
userInput.inData1 Convert.ToInt32(Console.ReadLine());

                        
Console.Write(tal2());
                        
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                        
Console.WriteLine(callFunction.substraktion(userInput.inData1userInput.inData2));

                       
Console.ReadLine();
                       
Main(args);
                        break;

                    case 
5:
                          
Console.WriteLine("MULTI = Addition - Substraktion, Press Q for menu.");
                          
Console.WriteLine();

                          
multipleIncome listIncome = new multipleIncome();
                          
int uInt;
                        
                    
//Do while if readline is not null or "q"
                              
do
                              {
                                  
string uInput Console.ReadLine();

                                  if (
uInput.Contains("q") == true)
                                  {
                                      
Main(args);
                                  }

                                  if (
int.TryParse(uInputout uInt))
                                  {
                                      
listIncome.AddTal(uInt);
                                  }
                                  else {
                                      
Console.WriteLine("Endast siffror!");
                                  }

                                  
int summaObject 0;
                                  
Console.WriteLine("*******");
                                  foreach (
int obj in listIncome)
                                  {
                                      
summaObject += obj;
                                      
Console.Write(" + {0}"obj);
                                  }

                                  
Console.WriteLine("");
                                  
Console.WriteLine("summaObject = {0}"summaObject);
                              } while (
uInt != null);
                        break;

                    case 
6:
                        
Console.WriteLine("Du har tryckt noll!");
                        
                        
Environment.Exit(0);
                        break;
                      
                }
            }
            else
            {
                
Console.WriteLine("Wrong selection" Environment.NewLine "Press any key to exit");
                
Console.ReadKey();
            }
            
Console.ReadKey();
        }
    }



 
//https://msdn.microsoft.com/en-us/library/dscyy5s0.aspx

 // Collection of tal objects. This class 
 // implements IEnumerable so that it can be used 
 // with ForEach syntax.
 
class multipleIncome {

    
// Private members. 
     
private List<userActivitytal5 = new List<userActivity>();

    
// Public methods. //Add to list
    
public void AddTal(int name)
    {
        
tal5.Add(new userActivity inData1 name });
    }

    public 
IEnumerator GetEnumerator()
    {
        foreach (
userActivity theAnimal in tal5)
        {
            
yield return theAnimal.inData1;
        }
    }
  
 }

Citera
2015-04-10, 00:19
  #2
Medlem
Vissens avatar
Forumet e för "C# / VB.NET / .NET Framework" men din kod är skriven i java?
Citera
2015-04-10, 00:21
  #3
Medlem
Kembalis avatar
Citat:
Ursprungligen postat av Vissen
Forumet e för "C# / VB.NET / .NET Framework" men din kod är skriven i java?


Ne, det är det inte. Det är C#. Jag hoppas att detta är C# annars så har jag haft tur, för koden fungerar på de tutorials jag läst från.

Det är en console applikation om det kanske bör tilläggas. Jag har kodat med PHP det senaste året, kanske har blivit lite influerad, om du tycker att min kod är lite knäpp
__________________
Senast redigerad av Kembali 2015-04-10 kl. 00:24.
Citera
2015-04-10, 00:22
  #4
Medlem
Vissens avatar
Citat:
Ursprungligen postat av Kembali
Ne, det är det inte. Det är C#. Jag hoppas att detta är C# annars så har jag haft tur, för koden fungerar på de tutorials jag läst från.

Haha ursäkta, jag vet inte vad jag håller på med. Jag går och lägger mig nu, lycka till.
Citera
2015-04-10, 15:57
  #5
Medlem
poolos avatar
Alla klassnamn ska börja med stor bokstav.
Alla public funktioner ska börja med stor bokstav.
Alla privata funktioner ska ha liten bokstav.

Se till att du har kontinuerlig bracket-syntax. Ex. fixa till tal1 så det ser ut som tal2
Kod:
        static string tal1() { 
            return "Tal 1: "; 
        } 
        static string tal2() 
        { 
            return "Tal 2: "; 
        } 

Byt namn på variabler så de förklar vad de gör. Ett tips är att booleaner har namn som en fråga. Så din "bool good" bör heta "bool isValidInteger"

Bryt ut din kod till funktioner. Du kan börja med att bryta ut din kod i varje case till en egen funktion.

Tänk på när du refaktoriserar koden att du testar att programmet funkar mellan varje förändring så du inte förstör allt och har svårt att ställa tillbaka till ursprungsläge. Gör ett nytt inlägg när du uppdaterat koden så kan du få mer input.
Citera
2015-04-10, 20:12
  #6
Medlem
Kembalis avatar
Citat:
Ursprungligen postat av poolo
Alla klassnamn ska börja med stor bokstav.
Alla public funktioner ska börja med stor bokstav.
Alla privata funktioner ska ha liten bokstav.

Se till att du har kontinuerlig bracket-syntax. Ex. fixa till tal1 så det ser ut som tal2
Kod:
        static string tal1() { 
            return "Tal 1: "; 
        } 
        static string tal2() 
        { 
            return "Tal 2: "; 
        } 

Byt namn på variabler så de förklar vad de gör. Ett tips är att booleaner har namn som en fråga. Så din "bool good" bör heta "bool isValidInteger"

Bryt ut din kod till funktioner. Du kan börja med att bryta ut din kod i varje case till en egen funktion.

Tänk på när du refaktoriserar koden att du testar att programmet funkar mellan varje förändring så du inte förstör allt och har svårt att ställa tillbaka till ursprungsläge. Gör ett nytt inlägg när du uppdaterat koden så kan du få mer input.


Tack! Jag har en tendens att alltid glömma bort och jag har framförallt jäkligt svårt att namnsätta variabler och liknande. Hehe. Jag har försökt med att bryta ut varje case till functioner, ska jag göra det till en egen class? Typ, Display.case1, etc.

Igen, tack för tipsen!


Mvh
Citera
2015-04-11, 10:57
  #7
Medlem
Kembalis avatar
Uppdaterade lite, om någon är intresserad, verkar som att det inte går att loopa ut sum från list i en function, då det är lagrade object??

Tips?


Kod:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Collections;
using System.IO;

namespace 
ConsoleApplicationEASTER
{
    public class 
ClassFunctions ClassMultipleIncome
 
{
        public 
int substraktion(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }

        public 
int multiplikation(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }
        public 
int addition(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }
        public 
int division(int tal1int tal2)
        {
            
int results tal1 tal2;
            return 
results;
        }
    }


 public class 
ClassUserActivity
    
{
        public 
int inData1 getset; }
        public 
int inData2 getset; }
    }


    class 
Program
    
{
        static 
void menu(){
            
Console.WriteLine("1. Multiplikation" Environment.NewLine 
                
"2. Additation" Environment.NewLine "3. Division" 
                    
Environment.NewLine "4. Substraktion" +
                        
Environment.NewLine "5. Multi list" 
                            
Environment.NewLine "6. Exit");
        }

        static 
string tal1() {
            return 
"Tal 1: ";
        }
        static 
string tal2()
        {
            return 
"Tal 2: ";
        }

        static 
void Main(string[] args)
        {
            
Console.Clear();
            
menu();
            
int selector 0;
            
bool ifValidInteger false;
            
ifValidInteger int.TryParse(Console.ReadLine(), out selector);


            if (
ifValidInteger)
            {
                
Console.Clear();
                
Console.WriteLine("**En simpel miniräknare**");

                
//Get instance
                
ClassUserActivity userInput = new ClassUserActivity();
                
ClassDisplayCalculator display = new ClassDisplayCalculator();
                
                
Console.WriteLine("");
                switch (
selector)
                {
                    case 
1:
                                    
Console.WriteLine("Case1 - Multiplikation" 
                                            
Environment.NewLine "Skriv in två heltal.");
                                    
Console.Write(tal1());
                                    
userInput.inData1 Convert.ToInt32(Console.ReadLine());
                                    
Console.Write(tal2());
                                    
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                                    
//DisplayMultiplication
                                    
display.DisplayMultiplication(userInput.inData1userInput.inData2);
                                    
Main(args);
                        break;

                    case 
2:
                         
Console.WriteLine("Case2 - Additation"+
                             
Environment.NewLine +
                             
"Skriv in 2 heltal");
                         
Console.Write(tal1());
                         
userInput.inData1 Convert.ToInt32(Console.ReadLine());
                         
Console.Write(tal2());
                         
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                         
//DisplayMultiplication
                         
display.DisplayAddition(userInput.inData1userInput.inData2);
                         
Main(args);
                        break;

                    case 
3:
                        
Console.WriteLine("Case3 - Division");
                        
Console.Write(tal1());
                        
userInput.inData1 Convert.ToInt32(Console.ReadLine());
                        
Console.Write(tal2());
                        
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                        
//DisplayMultiplication
                        
display.DisplayDiv(userInput.inData1userInput.inData2);
                        
Main(args);
                        break;

                    case 
4:
                        
Console.WriteLine("Case4 - Substraktion");
                        
Console.Write(tal1());
                        
userInput.inData1 Convert.ToInt32(Console.ReadLine());

                        
Console.Write(tal2());
                        
userInput.inData2 Convert.ToInt32(Console.ReadLine());

                        
//DisplayMultiplication
                        
display.DisplaySub(userInput.inData1userInput.inData2);
                       
Main(args);
                        break;

                    case 
5:
                          
Console.WriteLine("MULTI = Addition - Substraktion, Press Q for menu.");
                          
Console.WriteLine();
                          
int uInt;
                    
//Do while if readline is not null or "q"
                              
do
                              {
                                  
string uInput Console.ReadLine();
                                  
uInt 0;
                                  
display.DisplayMultipleIncome(uInput);

                                  
int summaObject 0;
                                  
Console.WriteLine("*******");
                                  foreach (
int obj in display)
                                  {
                                      
summaObject += obj;
                                      
Console.Write(" + {0}"obj);
                                  }
                                  
Console.WriteLine("");
                                  
Console.WriteLine("summaObject = {0}"summaObject);
                              } while (
uInt != null);
                        break;

                    case 
6:
                        
Console.WriteLine("Du har tryckt noll!");
                        
Environment.Exit(0);
                        break;
                }
            }
            else
            {
                
Console.WriteLine("Wrong selection" Environment.NewLine "Press any key to exit");
                
Console.ReadKey();
            }
            
Console.ReadKey();
        }
    }


    
//NOTIS!! -----  class UserActivity->MultipleIncome->Function->Display.. Inheritence

 // Collection of tal objects. This class 
 // implements IEnumerable so that it can be used 
 // with ForEach syntax.
 
public class ClassMultipleIncome {

     
int antal;
    
// Private members. 
     
public List<ClassUserActivitytal5 = new List<ClassUserActivity>();

 
//Add to list
    
public void AddTal(int name)
    {
        
tal5.Add(new ClassUserActivity inData1 name });
    }

    public 
IEnumerator GetEnumerator()
    {
        foreach (
ClassUserActivity theAnimal in tal5)
        {
            
antal = +theAnimal.inData1;

           
// yield return theAnimal.inData1;
            
yield return antal;
        }
    }
}

  

public class 
ClassDisplayCalculator  ClassFunctions
    
{
            public 
void DisplayAddition(int tal1int tal2)
            {
                
Console.Write("Svar: {0} + {1} = "tal1tal2);
                
Console.WriteLine(this.addition(tal1tal2));
                
Console.ReadLine();
            }

            public 
void DisplayDiv(int tal1int tal2)
            {
                
Console.Write("Svar: {0} / {1} = "tal1tal2);
                
Console.WriteLine(this.division(tal1tal2));
                
Console.ReadLine();
            }

            public 
void DisplaySub(int tal1int tal2)
            {
                
Console.Write("Svar: {0} - {1} = "tal1tal2);
                
Console.WriteLine(this.substraktion(tal1tal2));
                
Console.ReadLine();
            }

            public 
void DisplayMultiplication(int tal1int tal2)
            {
                
Console.Write("Svar: {0} * {1} = "tal1tal2);
                
Console.WriteLine(this.multiplikation(tal1,tal2));
                
Console.ReadLine();
               
            }

        public 
void DisplayMultipleIncome(string checkIfq)
        {
            
int addInt;
            
bool contains checkIfq.Contains("q");

            if (
contains)
            {
                
Console.Clear();
                
Console.WriteLine("DU HAR VALT ATT AVSLUTA!");

            }
            else if (
int.TryParse(checkIfqout addInt))
            {
                
this.AddTal(addInt);
                
Console.Clear();
            }
            else
            {
                
Console.Clear();
                
Console.WriteLine("Endast siffror!");
            }

    }
 }

Citera
2015-04-11, 14:33
  #8
Medlem
tj.s avatar
Properties bör följa samma namnkonventioner som funktioner och klasser, dvs UpperCamelCase. Undvik onödig nestling av if-satser. Det blir både krångligare i längden och gör koden mindre läsbar.
Kod:
if (!validInput)
{
    
//Gör något här
}

switch (...)
{
    ...
    ...
    ...

Använd enums istället för hårdkodade strängar eller numeriska värden.
Kod:
public enum Operator
{
    
Addition,
    
Subtraction,
    
Multiplication,
    
Division
}

switch (
operator)
{
    case 
Addition:
        ...
        ...
        break;
    ...

Sluta koda på svenska. Dina funktioner och klassnamn verkar till synes vara på engelska. Ändå har du flera variabler på svenska. Usch. Sluta upp med det, bums. Dela upp din main-funktion i flera metoder. En funktion bör ha ett syfte, inte femtioelva stycken.
Citera
2015-04-11, 16:36
  #9
Medlem
tj.s avatar
Ytterligare synpunkter. Varför använder du Environment.Newline i dina utskrifter? Console.WriteLine(...); gör just vad den heter, den skriver ut en rad.
Istället för att skriva
Kod:
Console.WriteLine("Case1 - Multiplikation" +  
                                            
Environment.NewLine "Skriv in två heltal."); 

bör du alltså skriva
Kod:
Console.WriteLine("Case1 - Multiplikation");
Console.WriteLine("Skriv in två heltal."); 
Kod:
public int addition(int tal1int tal2

    
int results tal1 tal2
    return 
results

Dels är funktionen på svenska (tvi!), dels följer den inte C#:s kodkonventioner om UpperCamelCase, dels ska funktioner vara verb.
Kod:
public int Add(int xint y)

    return 
y;

Dessutom behöver du kolla efter DivisionByZero i din divisionfunktion.
Citera
2015-04-12, 14:54
  #10
Medlem
Kembalis avatar
Citat:
Ursprungligen postat av tj.
Sluta koda på svenska. Dina funktioner och klassnamn verkar till synes vara på engelska. Ändå har du flera variabler på svenska. Usch. Sluta upp med det, bums. Dela upp din main-funktion i flera metoder. En funktion bör ha ett syfte, inte femtioelva stycken.

Haha! Det är dessa tips jag har letat efter, rakt på sak! Stort tack tj! När jag får tid ska jag fixa till min kod! Hehe

Igen, tack!
Citera
2015-04-14, 21:12
  #11
Medlem
Kembalis avatar
*Updated*

tja, jag har uppdaterat lite om någon är intresserad. Kritik och synpunkter uppskattas gärna!

Kod:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Collections;
using System.IO;

namespace 
ConsoleApplicationEASTER
{
    public class 
ClassFunctions ClassMultipleIncome
 
{
        
internal int Substract(int xint y){
            return 
y;
        }

        
internal int Multiply(int xint y){
            return 
y;
        }

        
internal int Add(int xint y){
            return 
y;
        }

        
internal int Divide(int xint y){
            return 
y;
        }
    }


 public class 
ClassUserActivity{
        public static 
int inData1 getset; }
        public static 
int inData2 getset; }
        public static 
string inString getset; }
    }


 public class 
Program
    
{
        static 
void Menu(){
            
Console.WriteLine("1. Multiplikation" +
               
Environment.NewLine "2. Additation" +
                 
Environment.NewLine "3. Division" +
                     
Environment.NewLine "4. Substraktion" +
                         
Environment.NewLine "5. Multi list" +
                             
Environment.NewLine "6. Exit");
        }

        static 
string tal1() {
            return 
"Tal 1: ";
        }
        static 
string tal2(){
            return 
"Tal 2: ";
        }

public 
enum Operator{
            
Multiplication 1,
            
Addition,
            
Division,
            
Substraction,
            
MultipleAddition,
            Exit
    }


public static 
void Main(string[] args){
                
Menu();
                
// get value user provided
                
int OperatorInt Int32.Parse(Console.ReadLine());
                
Operator IncomeInteger = (Operator)OperatorInt;
                
Console.Clear();
                
Console.WriteLine("**En simpel miniräknare**");

                
//Get instance
                
ClassDisplayCalculator display = new ClassDisplayCalculator();
                
Console.WriteLine();
                switch (
IncomeInteger)
                {
                    case 
Operator.Multiplication:
                                    
Console.WriteLine("Case1 - Multiplikation");
                                    
Console.WriteLine("Skriv in två heltal.");

                                    
Console.Write(tal1());
                                    
ClassUserActivity.inData1 Convert.ToInt32(Console.ReadLine());
                                    
Console.Write(tal2());
                                    
ClassUserActivity.inData2 Convert.ToInt32(Console.ReadLine());

                                    
//DisplayMultiplication
                                    
display.DisplayMultiplication(ClassUserActivity.inData1ClassUserActivity.inData2);
                                    
Main(args);
                        break;

                    case 
Operator.Addition:
                                    
Console.WriteLine("Case2 - Addition");
                                    
Console.WriteLine("Skriv in två heltal.");
            
                                     
Console.Write(tal1());
                                     
ClassUserActivity.inData1 Convert.ToInt32(Console.ReadLine());
                                     
Console.Write(tal2());
                                     
ClassUserActivity.inData2 Convert.ToInt32(Console.ReadLine());
            
                                     
//DisplayAddition
                                     
display.DisplayAddition(ClassUserActivity.inData1ClassUserActivity.inData2);
                                     
Main(args);
                        break;

                    case 
Operator.Division:
                                    
Console.WriteLine("Case3 - Division");
                                    
Console.WriteLine("Skriv in två heltal.");
            
                                    
Console.Write(tal1());
                                    
ClassUserActivity.inData1 Convert.ToInt32(Console.ReadLine());
                                    
Console.Write(tal2());
                                    
ClassUserActivity.inData2 Convert.ToInt32(Console.ReadLine());
            
                                    
//DisplayDivision
                                    
display.DisplayDiv(ClassUserActivity.inData1ClassUserActivity.inData2);
                                    
Main(args);
                        break;

                    case 
Operator.Substraction:
                                    
Console.WriteLine("Case4 - Substraktion");
                                    
Console.WriteLine("Skriv in två heltal.");
            
                                    
Console.Write("Skriv in tal 1: ");
                                    
ClassUserActivity.inData1 Convert.ToInt32(Console.ReadLine());
                                    
Console.Write("Skriv in tal 2: ");
                                    
ClassUserActivity.inData2 Convert.ToInt32(Console.ReadLine());
            
                                    
//DisplaySubstraction
                                    
display.DisplaySub(ClassUserActivity.inData1ClassUserActivity.inData2);
                                    
Main(args);
                        break;

                    case 
Operator.MultipleAddition:
                                  
Console.WriteLine("MULTI = Addition - Substraktion, Press Q for menu.");
                                  
Console.WriteLine();
                                  
int uInt;
                                  
                                
//Do while if readline is not null or "q"
                                      
do
                                      {
                                          
uInt 0;
                                          
ClassUserActivity.inString Console.ReadLine();
                                          
display.DisplayMultipleIncome(ClassUserActivity.inString);
                                      } while (
uInt != null);
                        break;

                    case 
Operator.Exit:
                                
Console.WriteLine("Du har tryckt noll!");
                                
Environment.Exit(0);
                        break;
                }
            
Console.ReadKey();
        }
    }


//NOTIS!! -----  class UserActivity->MultipleIncome->Function->Display.... Inheritence

public class ClassMultipleIncome {

   protected List<
intMultipleIncomeList = new List<int>();
   protected 
int listSumma;
   
    
//Add to list
   
protected void AddtoList(int x){
        
MultipleIncomeList.Add(x);
    }
     
   protected 
int ShowSum(){
          
listSumma MultipleIncomeList.Sum();
          return 
listSumma;
     }

      
// Loop out the items.
  
protected void ShowAllItemsInList(){
          foreach (
int value in this.MultipleIncomeList ){
                
Console.Write(" + {0}",value);
          }
      }
 }

  

public class 
ClassDisplayCalculator  ClassFunctions{
  
internal void DisplayAddition(int xint y){
                
Console.Write("Svar: {0} + {1} = "xy);
                
Console.WriteLine(this.Add(xy));
                
Console.ReadLine();
            }

 
internal void DisplayDiv(int xint y){
                
Console.Write("Svar: {0} / {1} = "xy);
                
Console.WriteLine(this.Divide(xy));
                
Console.ReadLine();
            }

 
internal void DisplaySub(int xint y){
                
Console.Write("Svar: {0} - {1} = "xy);
                
Console.WriteLine(this.Substract(xy));
                
Console.ReadLine();
            }

  
internal void DisplayMultiplication(int xint y){
                
Console.Write("Svar: {0} * {1} = "xy);
                
Console.WriteLine(this.Multiply(x,y));
                
Console.ReadLine();
            }

  
internal void DisplayMultipleIncome(string checkString){
            
int validInteger;
            
bool ifValidInteger int.TryParse(checkStringout validInteger);

            if (
ifValidInteger){
                
Console.Clear();
                
this.AddtoList(validInteger);
                
Console.WriteLine("*******");

                
//Show all items in list
                
this.ShowAllItemsInList();
                
Console.WriteLine();
                
Console.WriteLine("Total Summa = {0}"this.ShowSum());
                
Console.Write("Skriv in tal: ");
            }
            else if (
checkString.Contains("q")){
                
Console.Clear();
                
Console.WriteLine("DU HAR VALT ATT AVSLUTA!");
                
//Main(args);
                //Program.Main();
            
}
            else{
                
Console.Clear();
                
Console.WriteLine("Endast siffror!");
            }
        }
        
        
 }

Citera
2015-04-15, 17:51
  #12
Medlem
Kod:
static void Menu(){ 
    Console.WriteLine("1. Multiplikation" + 
    Environment.NewLine + "2. Additation" + 
    Environment.NewLine + "3. Division" + 
    Environment.NewLine + "4. Substraktion" + 
    Environment.NewLine + "5. Multi list" + 
    Environment.NewLine + "6. Exit"); 
} 

nu ser väl det mesta vettigt ut förrutom det ovan.

Kod:
static void Menu(){ 
    Console.WriteLine("1. Multiplikation");
    Console.WriteLine("2. Additation"); // <--- sen stava rätt här också ;)
    Console.WriteLine("3. Division");
    Console.WriteLine("4. Substraktion";  // <--- och här med...
    Console.WriteLine("5. Multi list";
    Console.WriteLine("6. Exit"); 
} 

Brukar vara vanligare att man skriver detta på det här sättet. Environment.NewLine brukar vara vanligare om du ska skriva något till en stream till en fil eller liknande och det ska kunna köras på både win och linux eller så. Det Environment.NewLine gör är att lägga in en "\n". Men ska man skriva till konsollen så kan man lika gärna använda de verktyg man får gratis.
Citera
  • 1
  • 2

Skapa ett konto eller logga in för att kommentera

Du måste vara medlem för att kunna kommentera

Skapa ett konto

Det är enkelt att registrera ett nytt konto

Bli medlem

Logga in

Har du redan ett konto? Logga in här

Logga in