Говнокод #16253 — C# — Говнокод.ру

C# / Говнокод #16253

+130

class ProducerConsumer
    {
        private static Semaphore semaphore = new Semaphore(1, 2);
        static object locker = new object();
        static int product = 0;
        private static bool work = true;
        private static bool valueSet = false; // why??

        private static void Producer() // производитель
        {
            while (work)
            {
                Console.WriteLine("Thread Producer start");
                int sqr = 0;
                semaphore.WaitOne(); // декрементируем счётчик семафора
                for (int i = 0; i < 15; i++)
                {
                    sqr = i * i;
                }
                lock (locker) // error
                {

                    while (valueSet)
                    {
                        Thread.Yield();
                    }
                    product += sqr;
                    valueSet = true;
                    Console.WriteLine("Product put: " + sqr);
                    Console.WriteLine("Product now: " + product);
                }
                semaphore.Release(); // выход из семафора
                Thread.Sleep(5000);
            }
        }

        private static void Consumer() // потребитель
        {
            const int MAX = 5;
            int[] arr = new int[MAX];
            int result = 0;
            Random rand = new Random();

            while (work)
            {
                Console.WriteLine("Thread Consumer start");
                semaphore.WaitOne(); 
                for (int i = 0; i < 5; i++)
                {
                    arr[i] = rand.Next(0, 1024);
                }
                for (int i = 0; i < 5; i++)
                {
                    result += arr[i];
                }
                result /= 5;
                while (!valueSet)
                {
                    Thread.Yield();
                }
                lock (locker)
                {
                    if (product - result > 0) // исключаем отриц.кол-ва продуктов
                    {
                        product -= result;
                        Console.WriteLine("Product get: " + result);
                    }
                    else 
                    {
                        Console.WriteLine("Product < 0");
                    }
                    valueSet = false;
                    Console.WriteLine("Product now: " + product);
                }
                semaphore.Release();
                Thread.Sleep(5000);
            }
        }

        public static void Main()
        {
            Thread threadProducer = new Thread(Producer);
            threadProducer.Start();

            Thread threadConsumer = new Thread(Consumer);
            threadConsumer.Start();

            Thread.Sleep(5000);

            Console.WriteLine("Main thread start.");
            String str = System.Console.ReadLine();
            Console.ReadKey();
        } 
}

Корявый пример решения задачи "Producer-Consumer".

Запостил: qstd, 29 Июня 2014

Комментарии (23) RSS

brutushafens 29.06.2014 14:53 # +1

cleaned
Ответить

kegdan 29.06.2014 16:18 # +2

for (int i = 0; i < 15; i++)
                {
                    sqr = i * i;
                }

ну ахуеть теперь

const int MAX = 5;
            int[] arr = new int[MAX];

Это. я так понимаю, на будущее.

if (product - result > 0) // исключаем отриц.кол-ва продуктов
                    {
                        product -= result;
                        Console.WriteLine("Product get: " + result);
                    }

Гениально

Thread.Sleep(5000);

Sweet dreams

Ответить

koodeer 29.06.2014 16:40 # +2

Очередная унылая лаба.
Использовать не volatile булевы поля для взаимодействия разных потоков - эпик фэйл.
Ответить
- bormand 29.06.2014 16:43 # +1
  
  > эпик фэйл
  К сожалению, на x86 будет работать ;(
  Ответить
  - koodeer 29.06.2014 17:08 # +1
    
    Чисто теоретически, компилятор может "соптимизировать" код, выкинув проверки нафик. Во многих статья по GC и управлению памятью в дотнете это обсасывается.
    
    Единого мнения нет. Отчасти из-за различия в спецификации ECMA и реализации Microsoft. Например, в Mono будет одно поведение, в CLR другое. Для переносимого кода лучше сразу писать "по правилам": либо с volatile, либо другие меры предпринимать.
    Ответить
    - bormand 29.06.2014 17:12 # +3
      
      Дык поэтому я и пишу "к сожалению" :) Вроде бы работает, а потом перестанет, при каких-нибудь условиях.
      Ответить
kegdan 29.06.2014 16:55 # +1

+ ко всему - семафоры тут нахуй не нужны
Ответить

kegdan 29.06.2014 19:39 # 0

Хуясе какой кусок говна я пропустил

for (int i = 0; i < 5; i++)
                    {
                        arr[i] = rand.Next(0, 1024);
                    }
                    for (int i = 0; i < 5; i++)
                    {
                        result += arr[i];
                    }
                    result /= 5;

Ответить

kegdan 29.06.2014 19:59 # 0

если не гоню (а я могу) вот код немного очишенный от говна - функциАнал тот же

internal class ProducerConsumer
    {
        private static readonly object Locker = new object();
        private static int _product;
        private static bool _valueSet; // why??
        private static readonly Random Rand = new Random();

        private static void Producer() // производитель
        {
            while (true)
            {
                lock (Locker) // error
                {
                    Console.WriteLine("Thread Producer start");
                    int sqr = 14*14;
                    if (_valueSet) Thread.Yield();
                    _valueSet = true;
                    Console.WriteLine("Product put: " + sqr);
                    Console.WriteLine("Product now: " + (_product += sqr));
                }
                Thread.Sleep(5000);
            }
        }

        private static void Consumer() // потребитель
        {
            while (true)
            {
                if (!_valueSet) Thread.Yield();
                lock (Locker)
                {
                    Console.WriteLine("Thread Consumer start");
                    var result = Rand.Next(0, 1024);
                    if (_product > result) // исключаем отриц.кол-ва продуктов
                    {
                        _product -= result;
                        Console.WriteLine("Product get: " + result);
                    }
                    else Console.WriteLine("Product < 0");
                    _valueSet = false;
                    Console.WriteLine("Product now: " + _product);
                }
                Thread.Sleep(5000);
            }
        }

        public static void Main()
        {
            new Thread(Producer).Start();
            new Thread(Consumer).Start();
            Thread.Sleep(5000);
            Console.WriteLine("Main thread start.");
            Console.ReadKey();
        }
    }

Ответить

bormand 29.06.2014 20:26 # 0

> if (_valueSet) Thread.Yield();
Это еще что такое. Цикл в коде ОП'а в этом плане лучше был, он хотя бы работал... А твой if один разок попробует подождать и попрет дальше. Совсем не айс.

Тут надо юзать что-то в духе wait/notify, или как оно у вас там в .net называется, M$ же любит переименовывать. Чтобы отправитель, добавляя элемент попутно будил получателя. А получатель, забирая элемент, будил отправителя.
Ответить
- bormand 29.06.2014 20:32 # 0
  
  > или как оно у вас там в .net называется
  Monitor.Pulse и Monitor.Wait судя по всему.
  Ответить
  - kegdan 29.06.2014 21:03 # 0
    
    Я не правил все, я так поковырялся.
    Да, оно
    Ответить

bormand 29.06.2014 20:36 # 0

P.S. А вообще, оригинал тоже не работал:

// producer
lock (locker) { // 1. захватили лок в момент когда valueSet был true
   ...
   while (valueSet) { ... } // 2. и ждем пока valueSet станет false
   ...
}
// consumer
lock (locker) { // 3. но consumer заблочится тут
   ...
   valueSet = false; // 4. и никогда не выполнит эту строчку
   ... 
}
// так что ждать придется вечно. deadlock.

Ответить

bormand 29.06.2014 20:49 # +2
Примерно так это должно выглядеть, если я нигде не затупил:
```
// производитель
private static void Producer() {
    Console.WriteLine("Thread Producer start");
    while (true) {
        Thread.Sleep(500); // видимость полезной работы
        int newProduct = ...; // которую мы делаем вне lock'а
        lock (locker) {
            // ждем, пока заберут значение, если его еще не забрали
            while (valueSet)
                Monitor.Wait(locker);
            // помещаем новое значение
            product = newProduct;
            valueSet = true;
            // и уведомляем получателя
            Monitor.Pulse(locker);
        }
        Console.WriteLine("Product put: " + newProduct);
    }
}
// получатель
private static void Consumer() {
    Console.WriteLine("Thread Consumer start");
    while (true) {
        int newProduct;
        lock (locker) {
            // ждем, пока появится значение, если его еще нет
            while (!valueSet)
                Monitor.Wait(locker);
            // получаем новое значение
            newProduct = product;
            valueSet = false;
            // и уведомляем получателя
            Monitor.Pulse(locker);
        }
        Console.WriteLine("Got product: " + newProduct);
    }
}
```
Ответить
- kegdan 29.06.2014 21:04 # 0
  
  похоже на правду. Мне после футбола в лом смотреть код-с
  Ответить
  - bormand 29.06.2014 21:07 # +1
    
    > в лом смотреть
    Лом с отверстием?
    Ответить
    - kegdan 29.06.2014 21:10 # +1
      
      с двумя - в одно смотрю из другого видно)
      Ответить
      - bormand 29.06.2014 21:13 # 0
        
        Эм, там внутри лома система зеркал?
        Ответить
        
        kegdan 29.06.2014 21:16 # 0
        
        Ду рак ты
        Одно С моей стороны, другое - со стороны монитора. А через одно отвестие видно только лом.
        Ашькальмэ!
        Ответить
        
        bormand 29.06.2014 21:22 # 0
        
        Дык если отверстие в ломе сквозное - то оно всё-таки одно. Разве нет?
        
        > А через одно отвестие видно только лом.
        Это если оно не насквозь просверлено.
        Ответить
        
        kegdan 29.06.2014 21:27 # +3
        
        Понимаешь, тут нужна степень по философии науки. Вот смотри, Когда отверстие сквозное в ломе 2 отверстия если смотреть чисто на поверхность лома. А если смотреть на топологию лома, то она тоже изменилась на 1 отверстие. (несквозное отверстие с точки зрения топологии нихуя не значит, что лом, что кружка - один хуй(у меня кружка с одним креплением на ручке)) Вот тебе и дуализм сквозной дыры в природе
        Ответить
        
        brutushafens 29.06.2014 22:26 # 0
        
        cleaned
        Ответить
        
        kegdan 29.06.2014 22:34 # 0
        
        Это не троллинг. Я пьян.
        Ответить

Говнокод: по колено в коде.

C# / Говнокод #16253

Комментарии (23) RSS

Добавить комментарий