Procentuální podíl v C# – .NET – Fórum – Programujte.com

Píšu si aplikaci, kterou zkouším testovat pravděpodobnost. Její základní podoba 15000 opakuje náhodné generování 1 nebo 2 , zaznamenává hodnoty, a vypíše počet jedniček, počet nul, nejdelší řadu stejných čísel, ale nejde mi počítat procentuální zastoupení jednotlivých čísel. kod je zde.

using System;
namespace console
{
	class Program
	{
		static void Main()
		{
			String a = Console.ReadLine();
			int edna = 0;
			int dva = 0;
			int predchozi = 0;
			int pocet = 0;
			int nejvice =0;			
			Random localhost = new Random();
			for(int i = 1; i<15000;i++ )
			{				
				int generator;
				generator = localhost.Next(1,3);
				if(generator == 1)
				{
					edna++;
				}
				else
				{
					dva++;
				}
				if(predchozi == generator)
				{
					pocet++;		
				}
				else
				{
					pocet = 0;
				}
				if(pocet > nejvice)
				{
					nejvice = pocet;
				}
				predchozi = generator;			
				Console.WriteLine(generator);
								
			}
			double procentjedna = (edna/(dva+edna))*100;
			double procentdva = (dva/(dva+edna))*100;
			Console.WriteLine("pocet jedniček");
		    Console.WriteLine(edna+" což je "+procentjedna+"%");
			Console.WriteLine("pocet dvojek");
			Console.WriteLine(dva+" což je "+procentdva+"%");
			Console.WriteLine("nejdelší řada");
			Console.WriteLine(nejvice);
			Console.ReadLine();
		}
	}
}

Procenta by to mělo počítat ve spodní části.

Console.WriteLine(edna+" což je "+procentjedna.ToString()+"%");

#2 Wikan
Já bych řekl, že problém je ve výpočtu. když místo výpočtu počtu procent dám třeba jen nějaké zkušební čísloí, tak se normálně zobrazí číslo, takhle je ale výstupem vždy 0%

Nazdar,

moje řešení.

1. Změň datový typ edna a dva na double.

double edna = 0;
double dva = 0;

2. A počítej tímto způsobem:

double procentjedna = edna / ((edna+dva) / 100);
double procentdva = dva / ((edna + dva) / 100);

Takhle mi to funguje.

Jen dodám vysvětlení proč ten postup v prvním příspěvku nefunguje, i když je na první pohled správný. C# má pravidlo, které vychází z jazyka C, a které zní, že pokud se vykonává operace mezi dvěma celými čísly, výsledek je opět celé číslo. Takže jelikož to dělení vždycky vyjde jako desetinné číslo, tak se ta desetinná část zahodí a zbude 0. Nejsnadnější řešení je jedno z těch čísel přetypovat na desetinný typ, takže stačí ten původní kód upravit například takto:

double procentjedna = ((float)edna/(dva+edna))*100;
double procentdva = ((float)dva/(dva+edna))*100;

Random localhost = new Random();

Toto ti generuje vždy jiné výsledky? Není lepší tam hodit aspoň nějaký seed, který je vždy unikátní? Např. DateTime.Now.Ticks ;)

#6 amertak
Co takhle se občas kouknout do dokumentace? 

...the parameterless constructor of the Random class uses the system clock to generate its seed value...

http://msdn.microsoft.com/….random.aspx

Tak to je zajímavé, jelikož když sem Random používal dřív, tak sem dostával pořád stejné výsledky. Divné.

#8 amertak
Tak to je hodně divné