Diskuze: Chyba v programu na odhad pi po druhé

Naučíme tě pracovat na home-office.

Zjistit více...

V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity

Regedin the Immortal

Člen

Regedin the Immortal:14.1.2017 10:57

Už jednou jsem se na fóru ptal ohledně předchozí verze aplikace. Nakonec se ji podařilo zprovoznit a dává celkem reálné hodnoty. Teď jsem se k ní však náhodně vrátil a zkusil ji trochu vylepšit ( základní princip zde ). Zkoušel jsem přidat seznam (list) do kterého by se ukládaly vypočtené hodnoty a z prvků seznamu by se potom určila směrodatná odchylka. Když by směrodatná odchylka poklesla pod pevně stanovenou mez, hodnoty v seznamu by se zprůměrovaly a tento průměr by byl výstupem programu. Někde je ale chyba a sekne se to u řádku ve kterém nevidím chybu. Za každou radu bych byl vděčný

Souradnice souradnice = new Souradnice();
            List<double> hodnoty = new List<double>();
            double s_odchylka = 0;
            int n = 0;
            int h = 0;
            int u = 0;
            double pi;
            double prumer;
            double soucet = 0;
            double soucet_s_odchylka = 0;
            double x;
            double y;
            float pravdepodobnost;
            Console.WriteLine("zadej požadovanou přesnost (přirozené číslo, nesmí překročit rozsah int)");
            int presnost = int.Parse(Console.ReadLine());
            while (Math.Sqrt(s_odchylka) >= 0.01F || (n == 0))
            {
                prumer = 0;
                soucet_s_odchylka = 0;
                u = 0;
                h = 0;
                while (h< presnost)
            {
                x = souradnice.hod();
                y = souradnice.hod();
                if (Math.Sqrt(Math.Pow(x,2)+Math.Pow(y, 2))<=1)
                {
                    u++;
                }
                    h++;
                if (h % (presnost/100) == 0)
                {
                    pravdepodobnost = 100 * (float)u / h;
                    Console.WriteLine("hotovo výpočtu : {0}%     p : " + pravdepodobnost + " %",100 * (float)h/presnost);
                    Console.WriteLine();
                }
                }

            pi = 4 * (double)u / h;
            hodnoty.Add(pi);
                n++;
                Console.WriteLine("předbězně určená hodnota je " + hodnoty[n-1]);
                foreach (int i in hodnoty)
                {
                    soucet = soucet + hodnoty[i];
                }
                prumer = soucet / (double)n;
                foreach(int i in hodnoty)
                {
                    soucet_s_odchylka = soucet_s_odchylka + Math.Pow((prumer - hodnoty[i]),2);
                }
                s_odchylka=soucet_s_odchylka / (double)n;
                Console.WriteLine("určená hodnota pi je " + prumer);
            Console.WriteLine("systémem určená hodnota je " + Math.PI);
            double rozdil = Math.Abs(prumer - Math.PI);
            Console.WriteLine("rozdíl hodnot je " + rozdil);
            }
            Console.WriteLine("aplikace bude ukončena stiskem libovolného tlačítka");
            Console.ReadKey();

A třída souřadnice

class Souradnice
    {
        public double horni_mez;
        private Random random;
        public Souradnice()
        {
            this.horni_mez = 1;
            random = new Random();
        }
        public Souradnice(double horni_mez_a)
        {
            this.horni_mez = horni_mez_a;
            random = new Random();
        }
        public double hod()
        {
            int mezi_cislo = random.Next(1, 1000001);             // dolní mez se dá nastavit odstraněním absolutní hodnoty a přičtením pevné hodnoty
            return (mezi_cislo * horni_mez / 1000000);

        }
    }

Chybu to vyhazuje u toho

soucet = soucet + hodnoty[i];

Ale upřímně nevím co konkrétně je tam špatně.

Odpovědět

14.1.2017 10:57

Roman Havlíček

Člen

Roman Havlíček:14.1.2017 14:16

Ahoj, smotal jsi dvě různé věci dohromady, buď použij

for (int i = 0; i< hodnoty.Count)
{
   soucet = soucet + hodnoty[i];
}

nebo

foreach (var hodnota in hodnoty)
{
   soucet = soucet + hodnota;
}

Akceptované řešení
+20 Zkušeností
+2,50 Kč

Nahoru Odpovědět

14.1.2017 14:16

Roman Havlíček

Člen

Roman Havlíček:14.1.2017 20:25

chybička se mi tam vloudila

for (int i = 0; i< hodnoty.Count; i++)
{
   soucet = soucet + hodnoty[i];
}

Nahoru Odpovědět

14.1.2017 20:25

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 3 zpráv z 3.

Nejčastěji vyhledáváné

Diskuze: Chyba v programu na odhad pi po druhé