Würfel automat für Tabletop

  • C#
  • .NET 5–6

Es gibt 2 Antworten in diesem Thema. Der letzte Beitrag () ist von VaporiZed.

    Würfel automat für Tabletop

    Hallo Leute,

    ich habe mich extra für mein Problem Registriert und bin sonst in Foren eher weniger Aktiv, daher entschuldigt bitte, falls ich gegen kleinere gepflogenheiten verstoße

    Ich heiße Gabriel bin 34 Jahre Jung und mache zurzeit eine Ausbildung zum Fachinformatiker.


    Unser Dozent hat uns die Aufgabe gegeben ein eigenes Programm zu erstellen, welches wir dann zweckgebunden auch präsentieren. Also in einer alltäglichen anwendung verwendung findet.
    Was eignet sich hierfür nicht besser als ein Würfelspiel?!

    Leider habe ich hierzu bzw. zu meinem Fehler keine wirkliche Hilfe gefunden. Gegebenenfalls weiß ich auch einfach nicht wonach ich suchen muss.
    Im Grunde geht es bei meinem Würfelspiel darum, aus einer Zufälligen Zahl heraus (12 Seiten) eine Seite zu "erwürfeln" durch zusatzangaben wie Maximal würfeln etz. soll dies Begrenzt werden. Das alles funktioniert schon.
    Jedoch bekomme bei der Ausgabe nur 0 Statt der "gewürfelten" Zahl heraus.

    Anbei mal der Code. GGf. wisst ihr ja woran es liegt?!


    Freundliche Grüße euer Gabriel

    C#-Quellcode

    1. {
    2. Random Rrandom = new Random(); //Random number generation
    3. string dreport;
    4. int dDice = 0; //determs the Eyes of each dice
    5. int dSum = 0; //Sum of the eyes
    6. int dTry = 0; //Shows how often you tried
    7. int dMax = 0; //You can choose a number how often you can roll the dice
    9. if (txtMax.Text == "")
    10. {
    11. MessageBox.Show("How often would you like to roll max ?");
    12. }
    13. else
    14. {
    15. dMax = Convert.ToInt32(txtMax.Text);
    16. txtDice.ResetText();
    17. do
    18. {
    19. dMax = Rrandom.Next(1, 13); //random Eyes between 1 and 12)
    20. txtDice.Text = txtDice.Text +""+ dDice.ToString();
    21. dSum = dSum + dDice;
    22. dTry += 1;
    23. } while (dDice != 12 && dTry < dMax);
    24. txtSum.Text = dSum.ToString();
    25. txtTry.Text = dTry.ToString();
    26. switch (dTry)
    27. {
    28. case 1:
    29. dreport = "Not worthy 1";
    30. break;
    31. case 2:
    32. dreport = "Dragon Slayer 2";
    33. break;
    34. case 3:
    35. dreport = "Impressive 3";
    36. break;
    37. case 4:
    38. dreport = "4";
    39. break;
    40. case 5:
    41. dreport = "5";
    42. break;
    43. case 6:
    44. dreport = "6";
    45. break;
    46. case 7:
    47. dreport = "7";
    48. break;
    49. case 8:
    50. dreport = "8";
    51. break;
    52. case 9:
    53. dreport = "9";
    54. break;
    55. case 10:
    56. dreport = "10";
    57. break;
    58. case 11:
    59. dreport = "11";
    60. break;
    61. case 12:
    62. dreport = "YES! 12";
    63. break;
    65. default:
    66. dreport = "well, may god be with you";
    67. break;
    68. }
    69. MessageBox.Show(dreport, "dice !", MessageBoxButtons.OK, MessageBoxIcon.Information);
    70. }
    72. }
    73. }
    74. }


    CodeTags korrigiert; bitte zukünftig darauf achten, das richtige CodeHighlighting zu verwenden ~VaporiZed

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „VaporiZed“ ()

    @Triton Willkommen im Forum. :thumbup:
    Zunächst packst Du die Zeile

    C#-Quellcode

    1. Random Rrandom = new Random(); //Random number generation
    aus der Prozedur in die Klasse, denn es kann nur einen geben.
    Dann befasst Du Dich mit Debugging:
    Debuggen, Fehler finden und beseitigen
    und überzeugst Dich, dass Dein Programm genau das tut, was es tuen soll.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Willkommen im Forum.

    Wenn man ne 12 würfeln kann, hat man entweder 2 normale 6er-Würfel oder einen speziellen 12-seitigen. Wenn man aber 2 6er Würfel hat, kann man keine 1 würfeln. Wie kommt dann aber auch der switch-default-zusammen. Der kann nie erreicht werden. Da gehört dann eigentlich ein Exception-Throwing rein, aber das ist ein Detail.
    Was kommt vor Zeile#1? Ist das ne Methode? Bitte immer vollständig den eigenen Methodencode hier zeigen.
    Was passiert, wenn jemand in die txtMax das Wort unendlich schreibt und Deine gezeigte Methode ausführt?
    Deine do-Schleife macht und macht und macht. Aber die Ergebnisse werden nicht für jeden Durchgang, sondern erst dann ausgewertet, wenn die Schleife durch ist. ?(
    Schau mal bitte, was Du in Zeile#26 bei dem switch auswertest, dann dürfte klar werden, wo der Wolf verbuddelt liegt.
    Ich frag mich, wie diese Methode entstanden ist. Da passiert sooo viel auf einmal. Wie kann man sich da erst nach soviel Code wundern, dass es nicht klappt? Ah, ich seh grad, dass Du gar nicht nur hier mit dem Thema unterwegs bist, sondern auch auf vbforums …
    Dieser Beitrag wurde bereits 5 mal editiert, zuletzt von „VaporiZed“, mal wieder aus Grammatikgründen.

    Aufgrund spontaner Selbsteintrübung sind all meine Glaskugeln beim Hersteller. Lasst mich daher bitte nicht den Spekulatiusbackmodus wechseln.

Facebook
Telefonspende