FINÁLNÍ VÝPRODEJ KREDITŮ na ITnetwork je tady. Využij poslední šanci a získej až 80 % extra kreditů ZDARMA na náš interaktivní e-learning. Zjisti více:
NOVINKA: Kurz kybernetické bezpečnosti s akreditací MŠMT, nyní již od 0 Kč. Staň se žádaným profesionálem. Zjisti více:
Avatar
David Liška
Člen
Avatar
David Liška:12.5.2016 2:10

Ahoj, mám takový menší problém a potřeboval bych s ním poradit. Mám jednu třídu a v ní dvě metody. První metoda je static void Main(), do které chci zavolat cyklus z druhé metody static float cyklus(float x), který by uživatele nepustil dál, dokud se nesplní podmínka správného zadání datového typu. Zkoušel jsem to už mnoha různými způsoby, ale vždy se objevila nějaká ta chyba. V této verzi jsem se dostal nejdál, hlásí mi to už pouze jednu chybu (Error 1: Use of unassigned local variable 'i'), ale dál už si s tím nevím rady. Chtěl bych proto poprosit o pomoc každého, kdo se v tom vyzná. V programování jsem teprve začátečník a tak budu vděčný za každou radu. Předem děkuji za ochotu.

namespace ConsoleApplication1
{
    class volani
    {

        static float cyklus(float x)
        {

           while (!float.TryParse(Console.ReadLine(), out x))
           Console.WriteLine("Neplatné číslo, zadejte prosím znovu:");

           return x;

        }


        static void Main()
        {
            Console.WriteLine("Zadej první číslo");
            float i;
            cyklus(i);


            Console.WriteLine("Zadej druhé číslo");
            float o;
            cyklus(o);


            Console.ReadKey();
        }
    }
}
 
Odpovědět
12.5.2016 2:10
Avatar
Ladislav Niderle
Člen
Avatar
Odpovídá na David Liška
Ladislav Niderle:12.5.2016 6:50

V tomto případě zbytečně deklaruješ proměnné "i" a "o". Stačí ti jedna proměnná přímo v samotné funkci a už ti to začne fungovat. V tomto případě se snažím předávat proměnnou, kterou jsi jenom nadeklaroval, ale není nastavená aspoň na nějakou hodnotu a proto Ti to háže chybu. Jako další by se dalo i říci, že sis vytvořil metodu cyklus, která ti má vrátit to číslo, které je zadané, ale když se do hlavní části kódu podíváš, tak tu metodu voláš, jako by byla typu void, tedy bez návratové hodnoty, se kterou by jsi dále v programu pracoval. Takže pokud s tou hodnotou nehodláš dále nic dělat, tak bych i doporučil tu metodu cyklus opravit z typu float na typ void a tím pádem i zrušíš řádek s return. Snad to je aspoň trošku pochopitelné jak jsem to napsal.

 
Nahoru Odpovědět
12.5.2016 6:50
Avatar
David Liška
Člen
Avatar
David Liška:12.5.2016 11:27

Rozumim tý části s void, teď jsem to poupravil a už mi to jde, ale když budu chtít třeba na těch dvou číslech, které uživatel zadá, provést např. součet, tak tam void nemuže být ne? Potřebuju aby mi to vrátilo tu proměnnou, abych s ní pak mohl pracovat dál.

 
Nahoru Odpovědět
12.5.2016 11:27
Avatar
Ladislav Niderle
Člen
Avatar
Odpovídá na David Liška
Ladislav Niderle:12.5.2016 12:59

Ano pokud by jsi potom chtěl s tou hodnotou pracovat, tak tam necháš ten float, ale taky by jsi měl správně udělat následující:

float cislo1 = cyklus();
float cislo2 = cyklus();
float součet = cislo1 + cislo2;

Což znamená, že v metodě cyklus musí být spoluse změnou void na float i zpět return x;.

Stačí takto? Klidně se ptej pokud Ti něco není z toho jasné nebo máš otázky k něčemu jinému.

 
Nahoru Odpovědět
12.5.2016 12:59
Avatar
David Liška
Člen
Avatar
David Liška:12.5.2016 23:47

Teď to mám takhle a stejně to nejde. Furt to píše tu stejnou chybu. Error 1: Use of unassigned local variable 'i' a Error 2: Use of unassigned local variable 'o'. Pořád někde musím dělat nějakou blbost.

namespace ConsoleApplication1
{
    class volani
    {

        static float cyklus(float x)
        {

            while (!float.TryParse(Console.ReadLine(), out x))
                Console.WriteLine("Neplatné číslo, zadejte prosím znovu:");

            return x;

        }


        static void Main()
        {
            Console.WriteLine("Zadej první číslo");

            float i = cyklus(i);



            Console.WriteLine("Zadej druhé číslo");

            float o = cyklus(o);





            float vysledek;
            vysledek = i + o;
            Console.WriteLine("Výsledek: " + vysledek);



            Console.ReadKey();
        }
    }
}
 
Nahoru Odpovědět
12.5.2016 23:47
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na David Liška
Neaktivní uživatel:13.5.2016 0:01

Samozřejmě, když napíšeš float i = něco, tak zakládáš proměnnou. ta proměnná nemá žádnou přiřazenou hodnotu, a ty ji hned za "=" nacpeš do metody jako parametr. Proto to nefunguje. Řešení je jednoduché: První proměnnou inicializuješ: float i = 0; A až poté ji předáš do t metody cyklus: i = cyklus(i);

Nahoru Odpovědět
13.5.2016 0:01
Neaktivní uživatelský účet
Avatar
David Liška
Člen
Avatar
David Liška:13.5.2016 1:42

Jasný chápu, taková hloupá chyba :) Teď už to konečně funguje. Všem moc děkuji za pomoc.

 
Nahoru Odpovědět
13.5.2016 1:42
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 7 zpráv z 7.