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

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

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

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

+133

public bool SelectUserGroup(string UserName,string GroupName)
{
    try
    {
        return true;
    }
    catch
    {
        return false;
    }
}

public KUser GetUserByKey(Guid Key)
{
    try
    {
        return new KUser();
    }
    catch
    {
        return new KUser();
    }
}

Особая защита от исключительных ситуаций. Взято из реального проекта.

Запостил:

Zergatul, 05 Мая 2011

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

Lure Of Chaos 05.05.2011 15:00 # 0

просто catch { тоже можно?
Ответить
- absolut 05.05.2011 15:13 # 0
  
  тогда уж finally
  Ответить
- absolut 05.05.2011 15:31 # 0
  "Хотя предложение catch можно использовать без аргументов для перехвата любого типа исключения, такой подход не рекомендуется."
  
  http://tinyurl.com/3fy2ez5
  Ответить
  - bugmenot 05.05.2011 15:56 # 0
    
    немного пугает машинный перевод для тех, кто на английском маны не читал, а на кривом русском и подавно не станет
    Ответить
TarasB 05.05.2011 16:19 # +3

Ржал в голос. Такого тупого кода давно не видел.
Ответить

guest 05.05.2011 17:21 # +9

try {
	return true;
} catch {
	try {
		return false;
	} catch {
		Console.WriteLine("залогинься сука");
	}
}

Ответить

guest 05.05.2011 17:52 # 0

Игра продолжается! Позвоните KUser!
Ответить
RaZeR 05.05.2011 21:29 # +3

|try
| {
| return new KUser();
| }
| catch
| {
| return new KUser();
| }
Юзера мне запили!
Ответить
inser 05.05.2011 23:50 # 0

Не верю, что это реальный код!!!
Ответить
- Zergatul 06.05.2011 01:45 # 0
  
  inser, код реальный, но использовались ли эти функции я не проверял
  Ответить
  - inser 06.05.2011 14:23 # +4
    bool GetTrue() { try { return true; } catch { return GetTrue(); } }
    
    Верни мне true, падла!!!
    Ответить
guest 16.05.2011 07:47 # +2

Только я вижу здесь заглушки? В первом методе перед return true появится обращение к БД с возможностью вызова исключения. Во втором методе вместо return new KUser() появится обращение к БД, возвращающее KUser, так же с возможностью исключения. Таким образом оба метода станут корректными, сейчас просто заглушки. Согласен, метод не самый изящный, но и не такой уж тупой.
Ответить
- inser 16.05.2011 14:49 # +1
  
  Тогда надо было написать TODO: тарам-пам-пам, чтобы не вводить никого в заблуждение.
  В этих блоках try / catch нет никакого смысла. По сему, это говнокод. Нельзя так оставлять в "реальном проекте".
  Возможно, программер пришёл из мира java, где принято каждую функцию начинать с try и заканчивать catch/finally, но это его не оправдывает.
  Ответить
- TauSigma 13.05.2013 14:15 # 0
  Для этого есть NotImplementedException.
  
  public bool SelectUserGroup(string UserName,string GroupName) { throw new NotImplementedException(); } public KUser GetUserByKey(Guid Key) { throw new NotImplementedException(); }
  Ответить
guest8 07.04.2019 23:04 # −999

показать все, что скрытоvanished
Ответить
guest8 08.04.2019 22:03 # −999

показать все, что скрытоvanished
Ответить
guest8 08.04.2019 22:06 # −999

показать все, что скрытоvanished
Ответить
guest8 08.04.2019 22:08 # −999

показать все, что скрытоvanished
Ответить
guest8 09.04.2019 10:42 # −999

показать все, что скрытоvanished
Ответить
guest8 09.04.2019 17:48 # −999

показать все, что скрытоvanished
Ответить
guest8 09.04.2019 17:50 # −999

показать все, что скрытоvanished
Ответить
guest8 09.04.2019 17:50 # −999

показать все, что скрытоvanished
Ответить

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

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

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

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