Список говнокодов пользователя sbs

Всего: 6

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

−95

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18

public void CopyToListControl(ListControl ListControlElement, List<string> v)
        {
            if (ListControlElement is ComboBox)
            {
                ((ComboBox)ListControlElement).Items.Clear();
                for (int i = 0; i < v.Count; i++) ((ComboBox)ListControlElement).Items.Add(v);
            }
            if (ListControlElement is ListBox)
            {
                ((ListBox)ListControlElement).Items.Clear();
                for (int i = 0; i < v.Count; i++) ((ListBox)ListControlElement).Items.Add(v);
            }
            if (ListControlElement is CheckedListBox)
            {
                ((CheckedListBox)ListControlElement).Items.Clear();
                for (int i = 0; i < v.Count; i++) ((CheckedListBox)ListControlElement).Items.Add(v);
            }
        }

Функция нового класса для подключения к форме. На форме может быть любой из 3 списочных элементов. Подводим к общему классу ListControl
Но так писать нельзя:
public void CopyToListControl(ListControl ListControlElement, List<string> v)
{
ListControlElement.Items.Clear();
for (int i = 0; i < v.Count; i++) ListControlElement.Items.Add(v);
}

а дальше в коде будет несколько методов по тому же принципу, по 3 if в каждой функции (бла-бла-бла is бла-бла-бла)
+ ещё приведение типа каждый раз

sbs, 16 Февраля 2016

Комментарии (9)

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

+138

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21

const bool ShowOkIfAllIsCool = true;

book OK()
{
    return ShowOkIfAllIsCool;
}

bool GetFileExists(string filePath)
{
     return File.Exists(filePath);
}

int GetLengthFileName(string filePath)
{
    return filePath.Lenght;
}

int SetMaxValue(int maxValue)
{
    _OPENWINDOWSAFTERINITIALIZATION = maxValue;
}

Программированием занимаюсь много лет, а так и не могу понять, для чего создавать подобные методы в 1 строку и константы заглавными буквами огромной длины? Их люди пишут или это автоматически созданный код какой-то прогой? В чужих программах такое часто можно встретить.

sbs, 25 Декабря 2013

Комментарии (37)

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

+129

1
2
3
4

string fileName = Path.GetFileName(f);
string fileExt = Path.GetExtension(f);
string i=fileName.IndexOf(fileExt);
if (i > -1) fileName = fileName.Remove(i, fileExt.Length);

Получение имени файла

sbs, 25 Декабря 2013

Комментарии (12)

Pascal / Говнокод #13162

+99

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

Я в 2006 году учился работать на Дельфи.

var
  Form1: TForm1; f:file of byte; rb,re:array[0..6,0..10] of byte;
        rb1,re1:array[1..6] of byte;h:shortint;go:boolean;
        tm:array[0..6,0..1,0..10] of record
                h,m:byte;
                end;
...

procedure prnt1;
var t:string;
begin
{Вывод времени уроков и перемен на форму. Начало 0,1. 11 уроков 2-23}
str(rb1[h],t);
form1.edit1.text:=t;
str(re1[h],t);
form1.edit2.text:=t;
str(rb[h,0],t);
form1.edit3.text:=t;
str(re[h,0],t);
form1.edit4.text:=t;
str(rb[h,1],t);
form1.edit5.text:=t;
...
form1.edit21.text:=t;
str(re[h,9],t);
form1.edit22.text:=t;
str(rb[h,10],t);
form1.edit23.text:=t;
end;

...
procedure prnt2;
var t1,t2,t3,t4:string;a1,a2,b1,b2,c,d:integer;
begin
{Вывод списка уроков из текстового файла на форму}
val(form1.edit1.text,a1,c);
val(form1.edit2.text,a2,c);
rb1[h]:=a1;
re1[h]:=a2;

{1 урок}
b1:=a1;
b2:=a2;
val(form1.edit3.text,d,c);
rb[h,0]:=d;
b2:=b2+d;
if b2>=60 then
        begin
        b1:=b1+1;
        b2:=b2-60;
        end;
str(a1,t1);
str(a2,t2);
if a2<10 then t2:='0'+t2;
str(b1,t3);
str(b2,t4);
if b2<10 then t4:='0'+t4;
tm[h,0,0].h:=a1;
tm[h,0,0].m:=a2;
tm[h,1,0].h:=b1;
tm[h,1,0].m:=b2;
form1.label15.Caption:=t1+':'+t2+' - '+t3+':'+t4;
val(form1.edit4.text,d,c);
re[h,0]:=d;
b2:=b2+d;
if b2>=60 then
        begin
        b1:=b1+1;
        b2:=b2-60;
        end;

{2 урок}
a1:=b1;
a2:=b2;
val(form1.edit5.text,d,c);
rb[h,1]:=d;
b2:=b2+d;
if b2>=60 then
        begin
        b1:=b1+1;
        b2:=b2-60;
        end;
str(a1,t1);
str(a2,t2);
и т.д. (так 11 уроков и перемен по 30 строк)

{Сохранение в базу данных под названием zvonok.ini :-) }
val(form1.edit1.text,rb1[h],d);
val(form1.edit2.text,re1[h],d);
val(form1.edit3.text,rb[h,0],d);
val(form1.edit4.text,re[h,0],d);
val(form1.edit5.text,rb[h,1],d);
val(form1.edit6.text,re[h,1],d);
val(form1.edit7.text,rb[h,2],d);
val(form1.edit8.text,re[h,2],d);
val(form1.edit9.text,rb[h,3],d);
val(form1.edit10.text,re[h,3],d);
и так до 11-го урока

Программа для подачи сигналов секретарю, что пора дать звонок на урок или на перемену.

sbs, 12 Июня 2013

Комментарии (476)

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

+134

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32

class Slovo
    {
        public string Word;
        public int Count;
        public Slovo(string slovo, int kolichestvo)
        {
            Word = slovo;
            Count = kolichestvo;
        }
    }

    class Program
    {
        static void Main(string[] args)
        {
            string s = "Один два четыре восемь число один и два дают вместе восемь а один умножить на один будет один";
            List<string> word = new List<string>();
            List<int> count = new List<int>();
            string[] word_collection = s.ToLower().Split(' ');
            foreach (string str in word_collection) if (!word.Contains(str))
                {
                    word.Add(str);
                    count.Add(1);
                }
                else count[word.IndexOf(str)]++;
            List<Slovo> spisok = new List<Slovo>();
            for (int i = 0; i < word.Count; i++) spisok.Add(new Slovo(word[i], count[i]));
            spisok.Sort((a, b) => b.Count - a.Count);
            foreach (Slovo slovo in spisok) Console.WriteLine(slovo.Word.PadRight(12) + slovo.Count.ToString());
            Console.ReadKey();
        }
    }

Найти в строке самые частоповторяющиеся слова

sbs, 10 Июня 2013

Комментарии (33)

Pascal / Говнокод #6917

+92

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35

var
    CommHandle:integer;
    DCB:TDCB;
    Stat:TComStat;
    Kols,TransMask,Errs:DWord;
    Ovr:TOverlapped;
    s:string;
    i:integer;

{Чтение канала n}
function ReadChannel(n:integer):string;
begin
inc(n,32);                                                                                         //код запроса для прибора (это по спецификации прибора)
s:=chr(n);                                                                                         //символ запроса
WriteFile(CommHandle,s[1],length(s),Kols,@Ovr);                   //оправили этот код в прибор
TransMask:=0;                                                                               //маска любого события в порте
Ovr.hEvent:=CreateEvent(nil, True, False, nil);                          //хз
repeat
if not WaitCommEvent(CommHandle, TransMask, @Ovr) then                                                       //если приём закончен не до конца, то...
  if GetLastError = ERROR_IO_PENDING then WaitForSingleObject(Ovr.hEvent, INFINITE);           //при условии "хз" подождать ещё чуть-чуть

ClearCommError(CommHandle,Errs,@Stat);                                          //скинуть длину байт из буфера порта в переменную Stat
Kols:=Stat.cbInQue;                                                                                    //выяснить длину символов
until (Kols=1) or (Kols=14);                                                                        //закончить, если в канале пусто (1) или данные есть (14)

SetLength(s,Kols);                                                                                       //установить переменную s длиной Kols
ReadFile(CommHandle,s[1],Kols,Kols,@Ovr);                                         //прочитать данные из порта в s, обнулив буфер
if length(s)=1 then ReadChannel:='Пусто';                                             //дальше просто идёт обработка полученных данных
if length(s)=14 then ReadChannel:=copy(s,5,7);
end;



{Основной код}
for i:=1 to 60 do Value:=ReadChannel(i);                                         //ПОЛУЧИТЬ ДАННЫЕ ИЗ ПОРТА n

Имеется прибор с хранением данных в 60 каналах. Связь осуществляется по RS-232.
Чтобы получить данные, нужно отправить № кода нужного канала. В ответ придут данные этого канала.
Если канал пустой, то приходит 1 байт, если данные есть, то 14 байт.
Создана функция ReadChannel(n) с запросом данных из канала n.

ПРОБЛЕМА:
Прочитать все 60 каналов удаётся иногда 3 раза, иногда 15 раз. Но обязательно когда-нибудь программа зависнет на
WaitCommEvent(CommHandle, TransMask, @Ovr), так ничего и не получив из порта.
Если между командами
WriteFile и WaitCommEvent
поставить sleep(50) //9600 бод = 1200 б/с, то бишь 12 мс на 1 байт
то всё пашет. Но это не дело же привязываться ко времени?
Как избавиться от зависания на WaitCommEvent, чтобы процесс продолжился?
Если прибор не получил байт, из-за чего не ответил, как это проверить? "Данных нет уже 0,5 с, значит и не будет; отправлю этот байт ещё раз".

sbs, 10 Июня 2011

Комментарии (13)

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