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

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

+127
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
11. 11
12. 12
13. 13
14. 14
15. 15
16. 16
17. 17
18. 18
19. 19
20. 20
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
35. 35
36. 36
37. 37
38. 38
39. 39
40. 40
41. 41
42. 42
43. 43
```
if ((textBox2.Text != "") && (textBox3.Text != "") && (Convert.ToInt32(textBox2.Text) < Convert.ToInt32(textBox3.Text)))
{
    int наименование = 0;
    string несозданы = null;
    int количество = 0;
    bool накладнаясоздана = false;
    int накладная = 0;
    наименование = Convert.ToInt32(comboBox1.SelectedValue);
    string наименованиест = наименование.ToString();
    for (int i = Convert.ToInt32(textBox2.Text); i < Convert.ToInt32(textBox3.Text) + 1; i++)
    {
        bool одинаково = false;
        foreach (СкладDataSet.ПоступлениеRow row in складDataSet.Поступление.Rows)
        {
            if ((i == row.__) && (наименованиест == row.Наименование))
            {
                одинаково = true;
                несозданы += " " + i.ToString();
                yes = 2;
            }
        }
        if (одинаково == false)
        {
            складDataSet.Состояние.AddСостояниеRow(наименование.ToString(), i.ToString(), "2", "1", "");
            if (накладнаясоздана == false)
            {
                поступлениесписокTableAdapter.Insert(Convert.ToDateTime(dateTimePicker1.Text).ToString(), наименование, "000", richTextBox1.Text);
                поступлениесписокTableAdapter.Fill(складDataSet.Поступлениесписок);
                накладнаясоздана = true;
                foreach (СкладDataSet.ПоступлениесписокRow row in складDataSet.Поступлениесписок.Rows)
                { if (накладная <= row.Код)  накладная = row.Код; }
            }
            поступлениеTableAdapter.Insert(наименование, i, накладная.ToString());
            количество = количество + 1;
        }
    }
    поступлениесписокTableAdapter.Update(Convert.ToDateTime(dateTimePicker1.Text).ToString(), наименование, количество.ToString(), richTextBox1.Text, накладная, Convert.ToDateTime(dateTimePicker1.Text).ToString(), наименование, "000", richTextBox1.Text);
    if (yes == 2)
        MessageBox.Show(comboBox1.Text + " - " + несозданы + " не созданы т.к. существуют в базе", "Не созданы...", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
    Close();
}
else
    MessageBox.Show("Поле пустое, введите номер прибора!!!", "Внимательнее!", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
```
на готдотнете видел один чел вопрос задавал: http://www.gotdotnet.ru/forums/3/133079/
автору там намекали, что хорошо было бы, чтоб он переделал код, но автор глубоко забил

Запостил: Pauchok-Anaynckiy, 22 Ноября 2010

Tweet
Комментарии (38) RSS
- Nerd 22.11.2010 14:44 # +4
  
  хочется взять и уебать.(с)
  Ответить
  - absolut 22.11.2010 14:49 # +2
    
    не хочется брать ... просто уебать :)
    Ответить
- burdakovd 22.11.2010 14:59 # 0
  
  В рот мне ноги! СкладDataSet.ПоступлениеRow
  Ответить
  - burdakovd 22.11.2010 15:03 # +1
    
    Да тут пиздец в каждой строчке, человек не мог такого написать...
    Ответить
    - dreaktor 25.11.2010 20:56 # 0
      
      ессно, это генеренный код :) суть в том, что классы после генерации никто не переименовал :)
      Ответить
      
      Pauchok-Anaynckiy 25.11.2010 21:13 # +1
      
      а вот это? ))))
      int наименование = 0;
      string несозданы = null;
      int количество = 0;
      bool накладнаясоздана = false;
      int накладная = 0;
      наименование = Convert.ToInt32(comboBox1.SelectedValue) ;
      string наименованиест = наименование.ToString();
      Ответить
      
      Pauchok-Anaynckiy 25.11.2010 21:17 # +1
      
      и вот это: )))
      одинаково = true;
      несозданы += " " + i.ToString();
      yes = 2;
      
      как смотрю заново ржачка накатывает )))))) вытащите меня из-под стола
      Ответить
- 3.14159265 22.11.2010 15:08 # +3
  
  а еще говорят от 1Са не деградируют
  Ответить
- bugmenot 22.11.2010 15:09 # +2
  
  очень мило смотрится comboBox1
  Ответить
- Pauchok-Anaynckiy 22.11.2010 15:13 # +1
  
  там в этой куче добрые люди еще ковырялись, искали причины почему глюки возникают :)))
  Ответить
- Анонимус 22.11.2010 15:22 # +15
  
  1C#.
  
  автору желаю всю жизнь работать с переменной по имени comboBox42
  Ответить
  - absolut 22.11.2010 15:37 # 0
    
    адын-эс-шарп это сильно :)))
    Ответить
- Lure Of Chaos 22.11.2010 15:48 # 0
  
  "гоw.Код" detected
  Ответить
- Lure Of Chaos 22.11.2010 15:49 # 0
  
  > gotdotnet.ru
  ужас. dotnet.gothic.ru
  Ответить
  - Lure Of Chaos 22.11.2010 15:49 # 0
    
    gaddotnet.ru
    Ответить
    - istem 23.11.2010 01:12 # +1
      
      ohmygod.dotnet.ru
      Ответить
      
      inkanus-gray 27.11.2010 04:10 # +1
      
      http://wiki.it-kharkov.com/wiki/Гондурнет
      Ответить
      
      bugmenot 27.11.2010 14:42 # 0
      
      ОМГ
      Ответить
- banderror 26.11.2010 09:28 # +5
  
  Все школьные учителя информатики и некоторые институтские преподы сейчас торжественно разливают шампанское по бокалам: они преодолели профессиональный этап в своей жизни, когда ты пишешь транслитом. Теперь можно писать на великом и могучем, а самые гуру, конечно, сочетают несколько языков в одном имени! Низкий поклон мелкософту за предоставленный простор для творчества.
  Ответить
  - Lure Of Chaos 27.11.2010 00:54 # 0
    
    хороший учитель тот, чей ученик превзошел учителя... И что же - благодаря говнокод.ру видим, что действительно некоторые превзошли и своего учителя, и себя... = )
    Ответить
    - Pauchok-Anaynckiy 27.11.2010 01:09 # 0
      
      да таким хоть кол на голове теши, все похеру. есть учителя, нет учителей, без чужой помощи могут все засрать. сцуко от рождения талант, что-ли
      Ответить
      
      Lure Of Chaos 27.11.2010 01:14 # 0
      
      ну, а если серьезно, то, видимо лень и нежелание все делать основательно.
      Ответить
      
      Pauchok-Anaynckiy 27.11.2010 01:20 # 0
      
      Не, ну это разумеется. Просто я удивляюсь, неужели у него самого нет хоть малейшего чувства гармонии. Ведь полный раздрай, все вразнобой, бессистемно, кричаще, несогласованно. Даже в одном этом куске отдельные места противоречат друг другу.
      Ответить
      
      Lure Of Chaos 27.11.2010 01:22 # 0
      
      вы, наверное, будете удивлены, скольким людям (и программистам, и непрограммистам) чуждо чувство прекрасного и упорядоченного. А так же элементарный вкус и такт.
      Ответить
      
      Pauchok-Anaynckiy 27.11.2010 01:59 # 0
      
      Просто масштабы очень впечатляют ))) Первый раз за все время вижу такой гипертрофированно выраженный случай.
      Ответить
    - Pauchok-Anaynckiy 27.11.2010 01:13 # 0
      
      Кстати, на фоне общей загаженности теряются наиболее яркие места. Все по большей части обращают внимание на смешанный русско-аглицкий диалект, а самые смачные моменты остаются как-то незамеченными.
      Обращаю внимание (выношу в отдельные комменты):
      Ответить
    - Pauchok-Anaynckiy 27.11.2010 01:15 # 0
      
      Здесь КАЖДАЯ инструкция - отборный говнокод. Мля, у чела точняк талант от природы ни одного места не оставить незасранным.
      
      if ((i == row.__) && (наименованиест == row.Наименование))
      {
      одинаково = true;
      несозданы += " " + i.ToString();
      yes = 2;
      }
      Ответить
    - Pauchok-Anaynckiy 27.11.2010 01:17 # 0
      
      Несмотря на то, что все на английском языке, код качественнее не стал:
      
      if ((textBox2.Text != "") && (textBox3.Text != "") && (Convert.ToInt32(textBox2.Text) < Convert.ToInt32(textBox3.Text)))
      Ответить
      
      Lure Of Chaos 27.11.2010 01:20 # 0
      
      как говорят вот, "говнокод, но работает же!" - так в данном случае, эта строчка будет виновата в наиболее частом обсыпании приложения
      Ответить
    - Pauchok-Anaynckiy 27.11.2010 02:13 # +1
      
      Способ именования переменных. Вместо того, чтобы четче описывать смысл, названия наоборот вводят в заблуждение и дезориентируют.
      
      int наименование = 0;
      string несозданы = null;
      bool накладнаясоздана = false;
      int накладная = 0;
      string наименованиест = наименование.ToString();
      bool одинаково = false;
      yes;
      Ответить
- Pauchok-Anaynckiy 27.11.2010 02:16 # 0
  
  Продолжаю выковыривать фрагменты.
  Границы поименовать было конечно же западло. Типа и так понятно, что какой-то индекс (или порядковый номер) инкрементится.
  
  for (int i = Convert.ToInt32(textBox2.Text); i < Convert.ToInt32(textBox3.Text) + 1; i++)
  Ответить
- Pauchok-Anaynckiy 27.11.2010 02:18 # 0
  
  Йопппта!... Оказывается-то это совсем и не порядковый номер. А какое-то хз какое поле. В прямом смысле ХЗ.
  
  if ((i == row.__)
  Ответить
- Pauchok-Anaynckiy 27.11.2010 02:24 # 0
  Что-то неодинаково. Срочно надо что-то сделать. Хз что, но что-то надо.
  
  if (одинаково == false)
  Ответить
- Pauchok-Anaynckiy 27.11.2010 02:34 # 0
  Ах вот, что это пля оказывается за накладная! Это оказывается код накладной!
  АВТОР: Да какая тебе вообще нах разница, что это - код , или номер, или название, непонятно что-ли ваще??? Сказали же тебе выше: "говнокод, но работает же!".
  
  if (накладная <= row.Код) накладная = row.Код;
  Ответить
- Pauchok-Anaynckiy 27.11.2010 02:36 # 0
  О, йессс! Аж 2 раза. Ни куя не создано. О, йессс, еще раз!
  
  if (yes == 2) MessageBox.Show(comboBox1.Text + " - " + несозданы + " не созданы т.к. существуют в базе", "Не созданы...", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
  Ответить
- Pauchok-Anaynckiy 27.11.2010 02:41 # 0
  Чешем репу, пля, к чему же был этот else?.....
  
  else MessageBox.Show("Поле пустое, введите номер прибора!!!", "Внимательнее!", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
  
  Поскролим чуть наверх... Ах, да, к этой муйне:
  
  if ((textBox2.Text != "") && (textBox3.Text != "") && (Convert.ToInt32(textBox2.Text) < Convert.ToInt32(textBox3.Text)))
  
  Замечательно! Все совершенно ясно!
  Ответить
- Pauchok-Anaynckiy 27.11.2010 02:48 # 0
  
  Вот думаю, выложить, что-ли все эти все куски в отдельные говнокоды-топики?..... Да такой картины наверно не будет.
  Ответить
- andremacareno 27.11.2010 15:01 # 0
  
  row.Нокод
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

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

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

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