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

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

+5

01
02
03
04
05
06
07
08
09
10
11
12
13

int tries = 0;
while (true)
{
	try
	{
		DoSomething();
	}
	catch
	{
		if (++tries > 3)
			throw;
	}
}

Оригинальный способ попытаться выполнить некий код, давая ему на это N попыток , а только потом упасть с ошибкой.

leon_mz, 02 Марта 2016

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

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

−2

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

[HttpPost]
        [Route("api/Pateikimai/SaveList")]
        public OperacijosRezultatas SaveList(List<PateikimasEditItem> models)
        {
            var atsakymas = new OperacijosRezultatas();
            int failCounter = 0;
            foreach (var model in models) {
                try {
                    PateikimaiBll.SaveOrUpdate(model);
                }
                catch (Exception ex) {
                    failCounter++;
                }
            }
            if (failCounter > 0)
            {
                atsakymas.SekmingaOperacija = false;
                atsakymas.PridetiKlaida("Ne visi įrašai buvo išsaugoti");
            }
            else {
                atsakymas.PridetiSekme("Pateikimų įrašas sėkmingai išsaugotas");
            }
            

            return atsakymas;
        }

Смесь литовского с английским. Try/catch, который не записывает в лог и юзается для того, чтобы в front-end'e не запариватся с масивами (повторяющееся элементы). Уже не говорю об отсутсвии транзакций и foreach'e в контроллере.

kontora, 02 Марта 2016

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

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

−1

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67

if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
if(Checksquare()){
              Cut(_point(x, y), true);
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}
}

Здесь уместно вспомнить пословицу "Семь раз отмерь,(в данном случае 32 раза) один раз отрежь"

sliper, 01 Марта 2016

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

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

−3

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

_sock = new Socket(SocketType.Stream, ProtocolType.Tcp);
var saea = new SocketAsyncEventArgs {RemoteEndPoint = new IPEndPoint(0x0100007f, 1338)};
var mre = new ManualResetEventSlim(false);
saea.Completed += (o, e2) => mre.Set();
_sock.ConnectAsync(saea);
mre.Wait();
mre.Reset();
var idBuf = new byte[8];
var idSeg = new ArraySegment<byte>(idBuf);
saea = new SocketAsyncEventArgs { BufferList = new[] { idSeg } };
saea.Completed += (o, e2) => mre.Set();
_sock.ReceiveAsync(saea);
mre.Wait();
Task.Run((Action) ReceivingLoop);
mre.Dispose();

Блядские сокеты в UWP. Майки лучше не смогли ничего придумать, как добавлять в язык с нормальной асинхронностью сокеты на КОЛЛБЕКАХ блять. В 2016. Коллбеки. Да они там ебанулись к хуям

cykablyad, 01 Марта 2016

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

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

+6

1
2
3
4

public string GetRegion(CallContext context)
{
      return "Регион";
}

hinduCoder, 29 Февраля 2016

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

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

+6

01
02
03
04
05
06
07
08
09
10

using System;
namespace Colculator
{
    public partial class Form1 : Form
    {
           bool BOOL = {
                 true,true,true,true,true,true,true,false};
           if(BOOL(new Random(1,8)){
                      Consosle.WriteLine("ДА");
}

Так делать нельзя!!!

sliper, 28 Февраля 2016

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

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

+1

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49

if ( text.Contains( "лет" ) || text.Contains( "год" ) )
{
	var match = Regex.Match( text, @"\d+", RegexOptions.Singleline );
	if ( !match.Success )
		return null;
	var date = DateTime.UtcNow;
	return date.AddYears( -Int32.Parse( match.Value ) );
}
else if ( text.Contains( "дн" ) || text.Contains( "ден" ) )
{
	var match = Regex.Match( text, @"\d+", RegexOptions.Singleline );
	if ( !match.Success )
		return null;
	var date = DateTime.UtcNow;
	return date.AddDays( -Int32.Parse( match.Value ) );
}
else if ( text.Contains( "месяц" ) )
{
	var match = Regex.Match( text, @"\d+", RegexOptions.Singleline );
	if ( !match.Success )
		return null;
	var date = DateTime.UtcNow;
	return date.AddMonths( -Int32.Parse( match.Value ) );
}
else if ( text.Contains( "час" ) )
{
	var match = Regex.Match( text, @"\d+", RegexOptions.Singleline );
	if ( !match.Success )
		return null;
	var date = DateTime.UtcNow;
	return date.AddHours( -Int32.Parse( match.Value ) );
}
else if ( text.Contains( "недел" ) )
{
	var match = Regex.Match( text, @"\d+", RegexOptions.Singleline );
	if ( !match.Success )
		return null;
	var date = DateTime.UtcNow;
	return date.AddHours( ( -Int32.Parse( match.Value ) ) * 7 );
}
else if ( text.Contains( "минут" ) )
{
	var match = Regex.Match( text, @"\d+", RegexOptions.Singleline );
	if ( !match.Success )
		return null;
	var date = DateTime.UtcNow;
	return date.AddMinutes( -Int32.Parse( match.Value ) );
}
return null;

Копипаста >_< До кучи ещё и с классической copy-paste ошибкой в логике.

HellBrick, 28 Февраля 2016

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

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

−1

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

if (p != null)
{
    Thread thread = new Thread(() =>
    {
        StaffList.App.Controls.Personal.PersonRec rec = new Controls.Personal.PersonRec();
        rec.DataContext = p;
        rec.Mode = StaffList.Controls.OperatingMode.Show;
        var win = new BaseWindow();
        win.Form = rec;
        win.ShowDialog();
    });

    thread.SetApartmentState(ApartmentState.STA);
    thread.Start();
}

Это мы так делаем немодальные окна.

kerman, 25 Февраля 2016

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

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

+10

1
2
3
4

public static string toFormat(string s, int u)
{
    return "                                                                       ".Substring(0, u * 4) + s +"\r\n";
}

Функция используется для форматирования файла класса, созданного автоматически.

deathguard771, 25 Февраля 2016

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

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

+2

01
02
03
04
05
06
07
08
09
10
11
12
13

if (t.IsFaulted)
{
    try
    {
        throw t.Exception.InnerException;
    }
    catch (ObjectDisposedException)
    {
        Dispose();
        return;
    }
    ...
}

cykablyad, 24 Февраля 2016

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

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