Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
using System;
using System.Net;
using System.Net.Sockets;
using System.IO;
using System.Linq;
using System.Collections.Generic;
namespace CSharp_Shell
{
public class Program
{
static int flipACoin() {
Random rand = new Random();
int coin = rand.Next(0,2);
return coin;
}
public static void Main(string[] args)
{
int headsCount = 0;
int tailsCount = 0;
int tmp;
Console.Write("Enter a number of flips\n");
for(int flips = int.Parse(Console.ReadLine()); flips > 0; flips--)
{
Console.Write("You flipped: ");
tmp = flipACoin();
if(tmp == 1)
{
Console.Write("Heads\n");
headsCount++;
}
else
{
Console.Write("Tails\n");
tailsCount++;
}
}
Console.Write("Heads: " + headsCount + "\nTails: " + tailsCount + "\n");
}
}
}
Флипает коинсу по аглицки. Есть ли это говнокодом?
Неработающее говно, курите мануал: https://msdn.microsoft.com/en-us/library/h343ddh9(v=vs.110).aspx.
>>> Initializes a new instance of the Random class, using a time-dependent default seed value.
А поскольку в цикле новые объекты Random создаются очень быстро, семя не успевает измениться, как следствие - rand.Next(0,2); всегда выдаёт одно и то же значение:
Heads: 0
Tails: 100
Ну и даже если бы успевало, куски подпоследовательностей случайных чисел от ГПСЧ, инициализированных разными seed'ами, соединённые в одну последовательность, будут (скорее всего) иметь очень хреновое распределение.
Зачем создавать рандомизатор при каждом вызове функции?
Где обработка ошибок?
Почему в одном месте ты объявляешь переменную сразу в конструкции языка, в другом - отдельно? Ты шизофреник?
Если что, есть
>>> Initializes a new instance of the Random class, using a time-dependent default seed value.
А поскольку в цикле новые объекты Random создаются очень быстро, семя не успевает измениться, как следствие - rand.Next(0,2); всегда выдаёт одно и то же значение:
Ну и даже если бы успевало, куски подпоследовательностей случайных чисел от ГПСЧ, инициализированных разными seed'ами, соединённые в одну последовательность, будут (скорее всего) иметь очень хреновое распределение.
Не надо Random rand = new Random(); каждый раз
зы: gost вон выше уже объяснил почему
Я вас поздравляю, вы получаете бронзовую мендаль.
у вас же var, вывод типов, вот это всё