Diskuze: Metody a cyklus

C# .NET .NET (C# a Visual Basic) Metody a cyklus American English version English version

Avatar
David Liška
Člen
Avatar
David Liška:

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. května 2:10
Avatar
niderle.l
Člen
Avatar
Odpovídá na David Liška
niderle.l:

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. května 6:50
Avatar
David Liška
Člen
Avatar
David Liška:

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. května 11:27
Avatar
niderle.l
Člen
Avatar
Odpovídá na David Liška
niderle.l:

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. května 12:59
Avatar
David Liška
Člen
Avatar
David Liška:

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. května 23:47
Avatar
Lukas C#
Redaktor
Avatar
Odpovídá na David Liška
Lukas C#:

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  +1 13. května 0:01
Avatar
David Liška
Člen
Avatar
David Liška:

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

 
Nahoru Odpovědět 13. května 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.