Random - uloženo stále stejné číslo – .NET – Fórum – Programujte.com

Ahoj :) Potřeboval bych poradit... Nemůžu najít chybu... Nejspíš bude v porovnávání... Je to obyčejná hra kámen nůžky papír... Např: když dám kámen a počítač papír má bod on... 

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

namespace kamen_nuzky_papirr
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        private Random nahoda;
        public int cislo;
        public int skore_hrac;
        public int skore_pocitac;

        public MainWindow()
        {
            InitializeComponent();
            hractext.Text = "0";
            pocitactext.Text = "0";
            this.nahoda = new Random();
        }

        private int Nahoda //náhodné číslo od 1 do 3
        {
            get
            {
                return this.nahoda.Next(1, 4);
            }
        }

        /*
        public enum knp
        {
            kamen = 1,
            nuzky = 2,
            papir = 3
        }
         * */

        /// <summary>
        /// Po kliknutí na kámen se přičtou body buď hráči nebo PC
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void kamen_Click(object sender, RoutedEventArgs e)
        {
            if (Nahoda == 1)
            {
                pcdal.Text = "Kámen";
            }

            else if (Nahoda == 2)
            {
                pcdal.Text = "Nůžky";
            }

            else if (Nahoda == 3)
            {
                pcdal.Text = "Papír";
            }

            hracdal.Text = "Kámen";

            if (Nahoda == 2)
            {
                skore_hrac++;
                hractext.Text = skore_hrac.ToString();
            }

            else if (Nahoda == 3)
            {
                skore_pocitac++;
                pocitactext.Text = skore_pocitac.ToString();
            }
        }

        /// <summary>
        /// Po kliknutí na nůžky se přičtou body buď hráči nebo PC
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void nuzky_Click(object sender, RoutedEventArgs e)
        {
            if (Nahoda == 1)
            {
                pcdal.Text = "Kámen";
            }

            else if (Nahoda == 2)
            {
                pcdal.Text = "Nůžky";
            }

            else if (Nahoda == 3)
            {
                pcdal.Text = "Papír";
            }

            hracdal.Text = "Nůžky";

            if (Nahoda == 3)
            {
                skore_hrac++;
                hractext.Text = skore_hrac.ToString();
            }

            else if (Nahoda == 1)
            {
                skore_pocitac++;
                pocitactext.Text = skore_pocitac.ToString();
            }
        }

        /// <summary>
        /// Po kliknutí na papír se přičtou body buď hráči nebo PC
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void papir_Click(object sender, RoutedEventArgs e)
        {
            if (Nahoda == 1)
            {
                pcdal.Text = "Kámen";
            }

            else if (Nahoda == 2)
            {
                pcdal.Text = "Nůžky";
            }

            else if (Nahoda == 3)
            {
                pcdal.Text = "Papír";
            }

            hracdal.Text = "Papír";

            if (Nahoda == 1)
            {
                skore_hrac++;
                hractext.Text = skore_hrac.ToString();
            }

            else if (Nahoda == 2)
            {
                skore_pocitac++;
                pocitactext.Text = skore_pocitac.ToString();
            }
        }
    }
}

nějak nechápu, proč máš privátní vlastnost "Nahoda"...

zkus místo té vlastnosti dát obyčejnou funkci..

private int Nahoda()
{
    return nahoda.Next(1, 3);
}

a v podmínkách volej "Nahoda()"

a možná by ještě bylo lepší, kdyby si tohle celý dal do fce...

private knp Nahoda()
{
    switch (nahoda.Next(1,3))
    {
    case 1: return knp.kamen;
    case 2: return knp.nuzky;
    case 3: return knp.papir;
    }
    return knp.kamen; // nema vyznam
}

a potom v podmínkách porovnávat přímo hodnoty 

if (Nahoda() == knp.kamen) { ... }

#2 ingiraxo
To nefunguje... píše to: Error 2 Operator '==' cannot be applied to operands of type 'method group' and 'int' 

#3 Flouwie
jo aha... zkusím ;)

a jak tak koukám, tak všude uvádíš ještě název.. takže bych si tuhle fci přetížil na 

private knp Nahoda(out string str)
{
    str = null;

    switch (nahoda.Next(1,3))
    {
    case 1: str = "Kámen"; return knp.kamen;
    case 2: str = "Nůžky"; return knp.nuzky;
    case 3: str = "Papír"; return knp.papir;
    }
    return knp.kamen;
}

a potom v podmínkách použít 

if (Nahoda(out temp) == knp.kamen)
{
    pcdal.Text = temp;
}

samozřejmě proměnná temp je instanční

a takto se zbavíš duplicity toho textu

#5 ingiraxo
u toho str mi to háže chybu :/ Error 1 The out parameter 'str' must be assigned to before control leaves the current method 

co s tím?

#6 Flouwie
napadlo mě tam dát ref... to by fungovalo ne? :)

nad switch to ještě initni na null

#8 ingiraxo
achjo :( stejně mi to pořád špatně generuje ty náhodné čísla... u PC se neobjevuje papír :/

#9 Flouwie
ingiraxo tam ma malou botu..  ma to byt samozrejme  .next(1,4)  pocatek je vcetne, konec tam nepatri

#10 KIIV
jojo už jsem si všiml :) ale někdy když začínám tu hru, tak se u pc neobjeví vůbec nic... to by mohlo být čím? už mě to **** :D

jo pardon, má tam bejt (1,4) =)

tak po spustění to hned initni na nějakou hodnotu, pust tu fci Nahoda, ať PC něco má

kdyžtak ukaž jak to momentálně máš a dej to do code a né jako text

#12 ingiraxo
 

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

namespace kamen_nuzky_papirr
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        private Random nahoda;
        private string temp;
        public int cislo;
        public int skore_hrac;
        public int skore_pocitac;

        public MainWindow()
        {
            InitializeComponent();
            hractext.Text = "0";
            pocitactext.Text = "0";
            this.nahoda = new Random();
        }

        public enum knp
        {
            kamen = 1,
            nuzky = 2,
            papir = 3
        }

        private knp Nahoda(out string str) //náhodné číslo od 1 do 3
        {
            str = null;

            switch (nahoda.Next(1, 4))
            {
                case 1:
                    str = "Kámen";
                    return knp.kamen;
                case 2:
                    str = "Nůžky";
                    return knp.nuzky;
                case 3:
                    str = "Papír";
                    return knp.papir;
            }
            return knp.kamen;
        }

        /// <summary>
        /// Po kliknutí na kámen se přičtou body buď hráči nebo PC
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void kamen_Click(object sender, RoutedEventArgs e)
        {
            if (Nahoda(out temp) == knp.kamen || Nahoda(out temp) == knp.nuzky || Nahoda(out temp) == knp.papir)
            {
                pcdal.Text = temp;
            }

            hracdal.Text = "Kámen";

            if (Nahoda(out temp) == knp.nuzky)
            {
                skore_hrac++;
                hractext.Text = skore_hrac.ToString();
            }

            else if (Nahoda(out temp) == knp.papir)
            {
                skore_pocitac++;
                pocitactext.Text = skore_pocitac.ToString();
            }
        }

        /// <summary>
        /// Po kliknutí na nůžky se přičtou body buď hráči nebo PC
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void nuzky_Click(object sender, RoutedEventArgs e)
        {
            if (Nahoda(out temp) == knp.kamen || Nahoda(out temp) == knp.nuzky || Nahoda(out temp) == knp.papir)
            {
                pcdal.Text = temp;
            }

            hracdal.Text = "Nůžky";

            if (Nahoda(out temp) == knp.papir)
            {
                skore_hrac++;
                hractext.Text = skore_hrac.ToString();
            }

            else if (Nahoda(out temp) == knp.kamen)
            {
                skore_pocitac++;
                pocitactext.Text = skore_pocitac.ToString();
            }
        }

        /// <summary>
        /// Po kliknutí na papír se přičtou body buď hráči nebo PC
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void papir_Click(object sender, RoutedEventArgs e)
        {
            if (Nahoda(out temp) == knp.kamen || Nahoda(out temp) == knp.nuzky || Nahoda(out temp) == knp.papir)
            {
                pcdal.Text = temp;
            }

            hracdal.Text = "Papír";

            if (Nahoda(out temp) == knp.kamen)
            {
                skore_hrac++;
                hractext.Text = skore_hrac.ToString();
            }

            else if (Nahoda(out temp) == knp.nuzky)
            {
                skore_pocitac++;
                pocitactext.Text = skore_pocitac.ToString();
            }
        }
    }
}