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

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

+6

01
02
03
04
05
06
07
08
09
10
11

public DateTime Birth { get; }

public int Age
{
    get
   {
        var n = 0;
        while(Birth.AddYears(n + 1) < DateTime.Now) ++n;
        return n;
    }
}

Проверял работу у изворотливой студентки

gravejohn4, 28 Апреля 2020

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

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

+2

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

System.Action callback = null;

                        // TODO
                        if (true)
                        {
                            callback = () =>
                            {
                                GameLogic_TheFixer.I.SetGameState(GameLogic_TheFixer.GameAction.clean);
                            };
                        }
                        else
                        {

                        }
                        foamSpawner.onEndWork = callback;

Копаюсь в чьем-то легаси

recstazy, 09 Апреля 2020

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

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

0

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

public async Task<ActionResult> Enable2FA(string id)
        {
            if (string.IsNullOrEmpty(id))
            {
                return BadRequest("Invalid user Id");
            }

            User user = await userManager.FindByIdAsync(id);
            if (user == null)
            {
                return BadRequest(USER_NOT_FOUND_MESSAGE);
            }

            EnableAuthenticatorModel model = await LoadSharedKeyAndQrCodeUriAsync(user);
            return Ok(model);
        }

Что-то мне подсказывает что можно более логичнее написать сие кусок кода

horil97821, 08 Апреля 2020

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

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

+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

using System;
 
namespace Test
{
    public class HttpException : Exception
    {
        public HttpException(int status)
        {
            StatusCode = status;
        }
 
        public int StatusCode { get; set; }
    }
 
    class Program
    {
        static void TestCatch(int status)
        {
            try
            {
                throw new HttpException(status);
            }
            catch (HttpException ex) when (ex.StatusCode == 404)
            {
                Console.WriteLine("Not Found!");
            }
            catch (HttpException ex) when (ex.StatusCode >= 500 && ex.StatusCode < 600)
            {
                Console.WriteLine("Server Error");
            }
            catch (HttpException ex)
            {
                Console.WriteLine("HTTP Error {0}", ex.StatusCode);
            } 
        }
        static void Main(string[] args)
        {
            TestCatch(404);
            TestCatch(501);
            TestCatch(101);
        }
    }
}

https://ideone.com/zXstg3
Именно поэтому я за «C#».

gost, 27 Марта 2020

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

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

0

1
2
3
4
5
6
7
8

try
       {
            await storageClient.DownloadObjectAsync(Bucket, fileName, stream).ConfigureAwait(false);
       }
    catch(Exception ex)
       {
            throw new FileStorageException($"File '{fileName}' not found in a bucket '{Bucket}'", ex) { StatusCode = StatusCodes.Status404NotFound };
       }

Сеньйор дот нет девелопер

horil97821, 26 Марта 2020

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

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

+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

public class WindowEx : Window //...
{
        //hwnd окна
        public IntPtr Handle { get; set; }

        private const int GWL_EXSTYLE = (-20);
        private const uint WS_EX_TOPMOST = 0x00000008;

        public void SetTopmost()
        {
            SetTopmost(Handle);
        }

        public static void SetTopmost(IntPtr hWnd)
        {
            SetWindowLongPtr(hWnd, GWL_EXSTYLE, (IntPtr)((ulong)GetWindowLongPtr(hWnd, GWL_EXSTYLE) | WS_EX_TOPMOST));
        }

        public void UnSetTopmost()
        {
            UnSetTopmost(Handle);
        }

        public static void UnSetTopmost(IntPtr hWnd)
        {
            SetWindowLongPtr(hWnd, GWL_EXSTYLE, (IntPtr)((ulong)GetWindowLongPtr(hWnd, GWL_EXSTYLE) & ~WS_EX_TOPMOST));
        }

        private bool _topmost;
        public new bool Topmost
        {
            set
            {
                if (value)
                {
                    SetTopmost();
                }
                else
                {
                    UnSetTopmost();
                }
                _topmost = value;
            }
            get { return _topmost; }
        }
}

Тру окно. С тру топмостом. Бесполезно чуть менее, чем полностью:
1) Новый топмост не нужен. Оригинальное свойство через SetWindowPos сделает тоже самое (SetWindowPos может менять дополнительный стиль WS_EX_TOPMOST)
2) Handle (hwnd окна) c публичным set. Круто, чё. Следовало бы сделать public IntPtr Handle { get; private set; }

Так я писал код где-то лет 6 назад.

Janycz, 08 Марта 2020

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

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

+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

private void MainDataGridCellEditEnding(object sender, DataGridCellEditEndingEventArgs e)
{
            if (e.Column == MainDataGrid.Columns[1])
                return;

            if (CheckComplianceWithIndentation)
                if ((NumberOfLeadingSpaces(Rows[e.Row.GetIndex()].OriginalText) != NumberOfLeadingSpaces(Rows[e.Row.GetIndex()].Translation)) && !Rows[e.Row.GetIndex()].Tags.Contains("I"))
                    Rows[e.Row.GetIndex()].Tags += "I";
                else if (NumberOfLeadingSpaces(Rows[e.Row.GetIndex()].OriginalText) == NumberOfLeadingSpaces(Rows[e.Row.GetIndex()].Translation))
                    Rows[e.Row.GetIndex()].Tags = Rows[e.Row.GetIndex()].Tags.Replace("I", "");

            if ((Rows[e.Row.GetIndex()].Translation.Trim() == "") && !Rows[e.Row.GetIndex()].Tags.Contains("N"))
                Rows[e.Row.GetIndex()].Tags += "N";
            else if (Rows[e.Row.GetIndex()].Translation.Trim() != "")
                Rows[e.Row.GetIndex()].Tags = Rows[e.Row.GetIndex()].Tags.Replace("N", "");

            //...
}

public void TagsInit()
{
            if (CheckComplianceWithIndentation)
                foreach (var hRow in Rows.Where(hRow => NumberOfLeadingSpaces(hRow.OriginalText) != NumberOfLeadingSpaces(hRow.Translation)))
                {
                    hRow.Tags += "I";
                }

            foreach (var row in Rows.Where(hRow => hRow.Translation == ""))
            {
                row.Tags += "N";
            }
}

Дано: C#, WPF, DataGrid, таблица.
Надо сделать: таблица имеет поля, в том числе и поле Tags, которое определяется полями Translation и OriginalText, а также настройками пользователя (CheckComplianceWithIndentation), нет бы его вынести в класс строки как геттер:
public string Tags => f(Translation, Original);

вместо:
private string _tags;
public string Tags
{
get => _tags;
set
{
_tags = value;
RaisePropertyChanged("Tags");
}
}

Так нет же, будем отлавливать изменения ячеек таблицы (MainDataGridCellEditEnding) и пересчитывать Tags. MainDataGrid.Columns[1] - это колонка с ними, прибито гвоздями в xaml: CanUserReorderColumns="False"

Эх, а еще и Trim вместо String.IsNullOrWhiteSpace.

А еще немного венгерской говнонотации (hRow).

Так я писал где-то лет 7 назад. Да, тогда стрелок не было, но они приведены в описании тупо для сокращения строк кода.

Janycz, 08 Марта 2020

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

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

+1

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

// Задача: Рисовать в консоли прогресс бар, на сколько дней текущий год завершился

static void ProgressBarOfElapsedDays(DateTime dateTime, bool isRefreshebleat = false)   // рефрешеБЛЕАТ! :))
{
    int daysOfYear = DateTime.IsLeapYear(dateTime.Year) ? 366 : 365;
    int dayInProcents = daysOfYear * 100 / 100;     // WAT?
    int currentDay = dateTime.DayOfYear * 100;
    int daysOfYearNotElapsed = daysOfYear * 100 - currentDay;  // Сложный матан
    int procent = 100 - (daysOfYear * 100 - currentDay) / dayInProcents; 
    int buffer = 0;

    if (isRefreshebleat)
    {
        if (Console.CursorTop >= 1 || Console.CursorLeft >= 1)
        {
            Console.SetCursorPosition(Console.CursorLeft - Console.CursorLeft, Console.CursorTop - Console.CursorTop); // когда Console.SetCursorPosition(0, 0) недостаточно
        }
    }

    // остальной код
}

isRefreshebleat и всё остальное :))

adoconnection, 06 Марта 2020

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

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

0

1
2
3
4
5
6

public static void ActivateChangeHotKeyButtonModeButtonClick(object sender, EventArgs e)
{
       TaskBarHidderForm.ChangeHotKeyPanel.Visible = true;
       TaskBarHidderForm.KeyButtons[0].Text = TaskBarHidder.HotKeyCodes[0].ToString();
       TaskBarHidderForm.KeyButtons[1].Text = TaskBarHidder.HotKeyCodes[1].ToString();
}

кодить под пивко заебись :)

pashaluk31, 21 Января 2020

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

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