c# Zgaduj zgadula przepianie z c++ na c#

Question

Cześć, przepisuję sobie z c++ na c# kod z "zgaduj zgadula" z kursu 4 c++ od Zelenta. Problem w tym że w konsoli, po wpisaniu drugiej liczby program mi się wysypuje.

Jestem początkująca, więc rozwiązanie może wydawać się banalne, ale na nie jeszcze nie wpadłam, ktoś ma pomysł?

using System;
using System.Collections.Generic;
using System.Linq;
using System.Security.Cryptography.X509Certificates;
using System.Text;
using System.Threading.Tasks;
using System.Timers;


namespace zgadywanka
{
    class Program
    {
        static void Main(string[] args)
        {
            int choice;

            Console.WriteLine(" I think about a number between 1 and 100,");

            Random number = new Random();
            int a = number.Next(1, 100);
            Console.WriteLine(a);

            Console.WriteLine("guess what number is it?");
            

            do {
                choice = Console.Read();
                if (choice == a)
                    Console.WriteLine("Correct Number! Congratulations, You win");


                else if (choice < a)
                
                    Console.WriteLine("It`s too little, try again!");
  

                else if (choice > a)
                
                    Console.WriteLine("It`s too much, Try again!");
               

                Console.Read();
                return;

            } while (choice == a);
        }
      
    }
    
}

 

Answer 1

1. Console.Read(); zwraca wartość Int32 z kolejnego znaku na wejściu.

A więc nie otrzymujesz wpisanej liczby a wartość znaku char w postaci int32.

Zalecam wykorzystanie Console.ReadLine() aby oczytać całą wartość wejściową (a nie tylko pierwszy znak) i przekonwertować string do postaci Int32.  Na przykład wykorzystując metodę Int32.Parse(<wartość>);

2. Przenieś Console.Read() który występuje przed return; na koniec kodu po pętli while().

3. Skasuj:

return;

return powoduje zakończenie się funkcji, czyli w tym przypadku całego programu. Gdyż wszystko dzieje się we funkcji Main();

4. Powinno być while(choice != a), gdyż jeśli to zdanie jest prawdziwe (czyli wartości nie są równe) to wykonuje pętlę jeszcze raz.

 

Comments

Dzięki, zmieniłam wszytko o czym pisałeś, jednak w momencie gdy moja odpowiedź jest za pierwszym razem za duża, to mi potem w kółko wypisuje program " too much", niezależnie od następnych odpowiedzi, i na odwrót, natomiast gdy zgadnę to output jest taki, jaki powinien być.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Security.Cryptography.X509Certificates;
using System.Text;
using System.Threading.Tasks;
using System.Timers;


namespace zgadywanka
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine(" I think about a number between 1 and 100,");

            Random number = new Random();
            int a = number.Next(1, 100);
           // Console.WriteLine(a);
     
            Console.WriteLine("guess what number is it?");
         

            var choice = int.Parse(Console.ReadLine());


            do
            {
 

                if (choice == a)
                {
                    Console.WriteLine("Correct Number! Congratulations, You win in! ");
                    Console.ReadLine();
                }


                else if (choice < a)
                {
                    Console.WriteLine("It`s too little, try again!");
                    Console.ReadLine();
                }


                else if (choice > a)
                { 
                    Console.WriteLine("It`s too much, Try again!");
                    Console.ReadLine();
                }



            }while (choice != a);
            

        }


    }

 

---

Ponieważ tylko raz pobierasz liczbę z klawiatury.

Usuń Console.ReadLine() ze wszystkich ifów w pętli.

Przenieś linijkę:

var choice = int.Parse(Console.ReadLine());

Na początek pętli do while, aby pobierać znak w każdej iteracji pętli.

Answer 2

Wykonujesz pętlę raz, a później dopóki liczba podana przez użytkownika jest równa tej wylosowanej.