Konsolenanwendung C# Verbesserungsvorschläge?

Guten Tag! Habe soeben mein erstes richtiges Projekt in C# gemacht. Es ist ein einfaches Schere Stein Papier Spiel als Konsolenanwendung. Habt ihr Ideen, wie ich den "Code" optimieren könnte bzw. weitere Ideen, die Applikation zu erweitern. Ich würde mich sehr freuen, sollte sich jemand die Mühe machen, mir behilflich zu sein.

Bild zum Beitrag

Palladin007

05.09.2021, 21:43

Poste den Code bitte als Text in Code-Tags oder z.B. über Tools wie Pastebin.

MoechtegernCode

Beitragsersteller

06.09.2021, 14:41

https://pastebin.com/DJx8g0AA Funktioniert dieser Link zu Pastebin?

3 Antworten

Vom Beitragsersteller als hilfreich ausgezeichnet

Palladin007

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Computer, Technik, programmieren

06.09.2021, 18:52

Hier:

SchereSteinPapier2 - Pastebin.com

Mit ein paar Kommentaren dabei

Ach und achte auf Code-Guidelines, Microsoft bietet da selber Artikel für an und der Großteil der Community hält sich daran.

Woher ich das weiß:Berufserfahrung – C#.NET Senior Softwareentwickler

FaTech

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Computer, Technik, programmieren

05.09.2021, 16:02

Schau dir mal switch case an. Das würde meiner Meinung nach mehr sinn machen statt if Anfragen. Aber da du if benutzt hast, schau dir auch mal else if und else an. Ich würde dafür switch case benutzen, dennoch schau dir auch else if und else an.

WitchHunter0815

06.09.2021, 08:09

Um mich an Ich würde mich an den Vorschlag anschließen und auch noch
https://docs.microsoft.com/de-de/dotnet/csharp/language-reference/builtin-types/enum
Enumerationen einführen.

enum Gesures
{
  Stein,
  Scheere,
  Papier
}

switch (myGesture)
{
  case Gesures.Stein:
     // mach was
     break;
}

MoechtegernCode

Beitragsersteller

06.09.2021, 14:46

Das mit if else fällt mir gerade auch auf mit switch case habe ich bisher noch keine Erfahrung, werde mich jedoch demnächst einlesen danke für deine Antwort

MoechtegernCode

Beitragsersteller

06.09.2021, 14:44

Danke für deine Antwort! Werde mich auf alle Fälle da mal reinlesen

MagicalGrill

05.09.2021, 16:09

Wenn acceptment == "JA" ist, weißt du ja bereits, dass acceptment weder "NEIN" noch irgendetwas anderes sein kann. Trotzdem werden diese Dinge später abgefragt, was nicht optimal ist. Dies kannst du beheben, indem du aus den anderen beiden ifs jeweils else ifs machst (das letzte else if könntest du sogar gegen ein else ersetzen). Da du allerdings jeweils auf acceptment prüfst, würde sich hier ein switch besser eignen.
Du könntest drüber nachdenken, den User auch Zahlen eingeben zu lassen statt der Strings "Stein", "Schere" und "Papier". Dann könntest du mithilfe von Arithmetik ermitteln, wer gewonnen hat. Dies könnte z.B. in der Konsole so aussehen:

Bitte geben Sie die Nummer Ihrer Waffe ein:
1 -> Schere
2 -> Stein
3 -> Papier

Selbst wenn du bei den strings bleibst, solltest du auch beim Abfragen des decisionCheck dringend auf ein switch bzw wenigstens else if wechseln, da sonst sehr viele string-Vergleiche stattfinden, was relativ langsam ist.
Es ergibt wenig Sinn, "while (gameFinish == false)" auszuschreiben, da gameFinish selbst bereits ein Wahrheitswert ist, den man abfragen kann. Schreib stattdessen "while(!gameFinish)".

MoechtegernCode

Beitragsersteller

06.09.2021, 14:47

Wow! Wirklich vielen Dank für diesen sehr ausführlichen Kommentar werde ihn mir in Ruhe anschauen und darauf meinen Code optimieren

Ähnliche Beiträge

Ideen für C# Projekt?

Guten Tag! Ich versuche momentan mein Wissen in C# zu erweitern und auch zu vertiefen. Ich möchte lediglich ein paar Ideen für Konsolenanwendungen in Visual Studio haben und nicht grafisch darstellen. Habt ihr Ideen für ein Projekt z. B. Passworterstellung + Passwortabfrage so was Anfängerfreundliches in die Richtung vielen Dank im Voraus Lg

...zum Beitrag

Zufallsexperiment - Schere, Stein, Papier?

Ich mache irgendwas falsch.. Die Aufgabe ist: untersuche die Verlierchancen für Schere und Stein und Papier und Brunnen beim Spiel Schere Stein Papier(Brunnen)... Der Ergebnisraum ist {Schere;Stein;Papier;Brunnen} , Schere schneidet Papier . Papier bedeckt Brunnen , Stein schleift Schere und Stein bzw Schere fällt in den Brunnen.. E(Schere verliert)={Stein;Brunnen} E(Papier verliert)={Schere} E(Brunnen verliert)={Papier} E(Stein verliert)={Brunnen;Papier}.. Aber das kann nicht sein ? Weil allein die Elemente in den Ereignissen mehr sind als der Ergebnisraum.. ICH mache irgendwas falsch ! Bitte kann mir wer helfen und es erklären

...zum Beitrag

ZUFALLSEXPERIMENT über Schere Stein Papier Brunnen - Mathe Klasse 7?

...zum Beitrag

Schere Stein Papier zu dritt?

Wir haben gerade Schere Stein Papier zu dritt gespielt und wir haben uns gefragt was passiert,wenn einer Schere,einer Papier und einer Stein benutzt,kriegt dann jeder einen Punkt oder keiner? 🤔

...zum Beitrag

Zu dritt Schere, Stein, Papier - wie umsetzen?

Wie würdet ihr zu dritt das Spiel Schere, Stein, Papier umsetzen.

...zum Beitrag

Schere,Stein,Papier mit mehreren?

Wie sind die Spielregeln, wenn mehrere Menschen gleichzeitig Schere,Stein,Papier spielen? Also dann gibt es ja mehrere mit z.B. Papier und Stein. Wie geht das?

...zum Beitrag

Kann mir jemand dabei helfen, einen Algorithmus für Schere-Stein-Papier zu finden?

Ich muss einen Algorithmus für das Spiel Schere-Stein-Papier (auf Papier) schreiben. Kann mir jemand helfen?

Danke im Voraus.

...zum Beitrag

Java schere stein papier?

Hi Leute, ich bin seit c.a 1 Woche dabei Java zu lernen. nun versuch ich Schere Stein Papier in Konsolen Ausgabe zu programmieren. Das Problem dabei ist das Wenn ich etwas eingebe (zum beispiel "Schere")bekomme ich nichts zurück.

brauche Hilfe

hier mein Code

import java.util.Scanner;
   import java.util.Random;

  public class SSP {

private static Scanner spieler_auswahl;

public static void main(String[] args){
    String Wahl_spieler;
    int punkte_spieler = 0;
    int punkte_pc = 0;
    int Runden = 0;

    System.out.println("Schere Stein Papier \n");
    System.out.println("Such dir eine Wahl aus");
    
    
    while(Runden < 12){
        Random zG = new Random(); 
         
        String[] auswahl_pc = new String[3];
        auswahl_pc[0] = "Schere";
        auswahl_pc[1] = "Stein";
        auswahl_pc[2] = "Papier";
        
        
            for(int a = 0; a < 1; a++){
                
                int auswahl =zG.nextInt(auswahl_pc.length); // Zufallauswahl

                }
            
        Scanner spieler_wahl = new Scanner(System.in);
        Wahl_spieler = spieler_wahl.nextLine();
        
        if(auswahl_pc.equals(spieler_wahl)){
            System.out.println("Unetntschieden");
            Runden++;
        }
        // schere vergleich
        if(auswahl_pc.equals("Schere") && Wahl_spieler == "Stein" ){
            System.out.println("Du hast gewonnen");
            Runden++;
            punkte_spieler++;
        }
        
        if(auswahl_pc.equals("Schere") && Wahl_spieler == "Papier"){
            System.out.println("Computer hat gewonnen");
            Runden++;
            punkte_pc++;
        }
        
        //Vergleich Stein
        if(auswahl_pc.equals("Stein") && Wahl_spieler == "Schere"){
            System.out.println("Computer hat gewonnen");
            Runden++;
            punkte_pc++;
        }
        if(auswahl_pc.equals("Stein") && Wahl_spieler == "Papier"){
            System.out.println("Du hast Gewonnen");
            Runden++;
            punkte_spieler++;
        }
        
        // Vergleich Papier
        if(auswahl_pc.equals("Papier") && Wahl_spieler == "Stein"){
            System.out.println("Computer hat gewonnen");
            Runden++;
            punkte_pc++;
        }
        if(auswahl_pc.equals("Papier") && Wahl_spieler == "Schere"){
            System.out.println("Du Gewinnst");
            Runden++;
            punkte_spieler++;
        }
    }
}

}

...zum Beitrag

Schere Stein Papier in Javascript. Wo ist mein Fehler?

Ich habe versucht in Javascript Schere Stein Papier zu programmieren. Das Problem ist, dass es jedes Mal erst 3 zu 3 stehen muss, damit man gewonnen hat. Irgendwie kommt man sonst nicht aus der Schleife raus. Weiß jemand wo das Problem liegt?

Schon mal Danke im Voraus.

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title> Schere Stein Papier </title>
</head>
<body>
    
<script>


var symbolSpieler;
var symbolComputer;
var anzahlRunden = 0;
var gewinneComputer= 0;
var gewinneSpieler = 0;
var gewinnBedinung = 3;
var wahlSpieler;
var ausgedachteZahl;


do{
ausgedachteZahl= Math.random() *3;
ausgedachteZahl = Math.round(ausgedachteZahl + 0.5);


if ( ausgedachteZahl==1) {symbolComputer= "Schere"};
if (ausgedachteZahl==2){symbolComputer ="Stein"};
if (ausgedachteZahl==3){symbolComputer= "Papier"};


wahlSpieler=prompt("Gib deine Wahl ein", "Schere ist 1, Stein ist 2, Papier ist 3");


if(wahlSpieler==1){symbolSpieler= "Schere"};
if(wahlSpieler==2){symbolSpieler ="Stein"};
if(wahlSpieler==3){symbolSpieler= "Papier"};


if(symbolSpieler==symbolComputer){ alert("Unentschieden");}


if (symbolComputer== "Schere" && symbolSpieler== "Stein"){gewinneSpieler++;
alert("Du gewinnst gegen Schere");}


if(symbolComputer=="Schere" && symbolSpieler=="Papier") {gewinneComputer++;
alert("Computer gewinnt mit Schere");}


if(symbolComputer=="Stein" && symbolSpieler=="Schere"){
    gewinneComputer++;
    alert("Computer gewinnt mit Stein");
}


if(symbolComputer=="Stein" && symbolSpieler=="Papier"){
    gewinneSpieler++;
    alert("Du gewinnst gegen Stein");
}


if (symbolComputer=="Papier" && symbolSpieler=="Schere") {
    symbolSpieler++;
    alert("Du gewinnst gegen Papier"); 
}


if (symbolComputer=="Papier" && symbolSpieler=="Stein") { 
    gewinneComputer++;
    alert("Computer gewinnt mit Papier");
    
}
anzahlRunden++;
} while (gewinneSpieler<3 && gewinneComputer<3) 


if (gewinneSpieler=3) {alert("Du hast das Spiel gewonnen");}
if(gewinneComputer=3){alert("Schade, du hast das Spiel verloren :(");}


alert(gewinneSpieler + " zu "+ gewinneComputer);
</script>





</body>
</html>

...zum Beitrag

Code:

#Imports
import time
import random
random.seed()

	#Headline
print("*************************"); time.sleep(0.5)
print("*Schere - Stein - Papier*"); time.sleep(0.5)
print("*************************"); time.sleep(0.5)

true = True

while true == True:

	print("-------------------------------") #Abtrennung
	#Input
	print("Wählen Sie eine Figur: ")
	spielerwahl = input().lower()
	print("")
	
	computerwahl = ["schere", "stein", "papier"]

	#Output
	print("Sie wählten: ", spielerwahl)
	pc = random.choice(computerwahl)
	print("Der Computer wählte: ", pc)
	print("")

	#Verzweigungen
	if spielerwahl == pc:
		print("*********Unentschieden!*********")
		
	#Spielerwahl = Schere
	elif spielerwahl == "schere" or spielerwahl == "Schere" and pc == "stein":
		print("***********Verloren!***********")
		
	elif spielerwahl == "schere" or spielerwahl == "Schere" and pc == "papier":
		print("***********Gewonnen!***********")
	
	#Spielerwahl = Stein
	elif spielerwahl == "stein" or spielerwahl == "Stein" and pc == "papier":
		print("***********Verloren!***********")
		
	elif spielerwahl == "stein" or spielerwahl == "Stein" and pc == "schere":
		print("***********Gewonnen!***********")
	
	#Spielerwahl = Papier
	elif spielerwahl == "papier" or spielerwahl == "Papier" and pc == "schere":
		print("***********Verloren!***********")
		
	elif spielerwahl == "papier" or spielerwahl == "Papier" and pc == "stein":
		print("***********Gewonnen!***********")
		
	#Fehlermeldung
	else:
		print("*********Nicht möglich!*********")
	
	#Abtrennung
	print("-------------------------------")
	print(""); time.sleep(1.5)

...zum Beitrag

Ich habe angefangen Schere Stein Papier in JavaScript zu programmieren leider funktioniert die Ergebnis Auswertung am Ende nicht. Wo ist der Fehler?

Das ist der JS Code:

FuEingabe();
    function FuEingabe() {
    var eingabe = prompt("Bitte gib: Schere, Stein oder Papier ein!", "Schere, Stein, Papier");
}
    switch(FuEingabe()) {

        case "Schere":
        
        zufallenemyeingabe();
        break;

        case "Stein":
        
        zufallenemyeingabe();
        break;

        case "Papier":
        
        zufallenemyeingabe();
        break;

        default:

        alert("Ungültig!");

}
function zufallenemyeingabe() {
    var max = 3;
    var min = 1;
    var zufallzahl = (Math.random() * (max - min)) + min;
    console.log("Die Zufallzahl lautet: " + zufallzahl);
    var zufallzahlround = Math.round(zufallzahl);
    console.log("Die gerundete Zufallzahl heißt: " + zufallzahlround);
    var StEingabe = "non";

    switch(zufallzahlround){

        case 1:
        StEingabe = "Stein";
        break;

        case 2:
        StEingabe = "Papier";
        break;

        case 3:
        StEingabe = "Schere";
        break;

        default: 
        console.log("Error 01: Computer hat kein Ergebnis generiert!");
    }
    console.log(StEingabe);
    alert("Der Computer hat " + StEingabe) + " gewählt.";
    return StEingabe

}

function ergebnisauswertung() {
    var gewinner = "nil";
    var PlayEingabe = FuEingabe();

    if(PlayEingabe == "Schere" && zufallenemyeingabe == "Schere") { 

        gewinner = "Unentschieden!";

    }
                                                             
    if(PlayEingabe == "Schere" && zufallenemyeingabe == "Stein") {
        gewinner = "PC";

    }
                                                           
    if(PlayEingabe == "Schere" && zufallenemyeingabe == "Papier") {
        gewinner = "Player";

    }
                                                             
                                                              
     //Eigentliches Vergleichen der Wahl - Stein 
    if(PlayEingabe == "Stein" && zufallenemyeingabe == "Stein") {
        gewinner = "Unentschieden!";
        

    }
                                                           
    if(PlayEingabe == "Stein" && zufallenemyeingabe == "Schere") {
        gewinner = "Player";

    }
                                                            
    if(PlayEingabe == "Stein" && zufallenemyeingabe == "Papier") {
        gewinner = "PC";

    }
                                                            
                                                            
    //Eigentliches Vergleichen der Wahl - Papier 
    if(PlayEingabe == "Papier" && zufallenemyeingabe == "Papier") {
    gewinner = "Unentschieden!";

    }
                                                             
    if(PlayEingabe == "Papier" && zufallenemyeingabe == "Schere") { 

        gewinner = "PC";


    }
                                                             
    if(PlayEingabe == "Papier" && zufallenemyeingabe == "Stein") { 

        gewinner = "Player";

    }

    return gewinner;
                                                            
}
var gewinner = ergebnisauswertung();
console.log(gewinner);
if (ergebnisauswertung() == "Player") {
    alert("Du gewinnst!");
}

...zum Beitrag