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

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

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

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

+124

private void HideActiveButtons()
{
    try
    {

    }
    catch (Exception)
    {
        throw;
    }
}

Откопано в форме Windows Forms в страшном legacy-коде.

Запостил:

someone, 09 Июня 2013

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

taburetka 09.06.2013 22:10 # −8

показать все, что скрытоЕсли бы не было (Exception) то может использоваться как дыра в безопасности, кстати. Не CLR исключения имеют другой уровен доступа.
Ответить
- taburetka 10.06.2013 09:33 # −12
  
  показать все, что скрытоКакое единодушие в минусовании. http://msdn.microsoft.com/en-gb/bb264489.aspx
  Ответить
  - someone 10.06.2013 09:35 # −9
    
    показать все, что скрытоДа это бот развлекается.
    Ответить
  - roman-kashitsyn 10.06.2013 09:37 # −10
    
    показать все, что скрытоТут бото-минусовательные рейсы школоты проходят, не обращайте внимания.
    Ответить
  - neeedle 10.06.2013 10:51 # −14
    показать все, что скрытоМне кажется минусуют из-за того, что не в
    (Exception)
    дело.
    Ответить

neeedle 10.06.2013 06:52 # −13

показать все, что скрытоНастоящий шарпей должен вписывать трай-кетч везде, где только можно, а еще лучше по 4 раза, а кто не согласен, тот — нубье, неумеющее в computer science.

Ответить

someone 10.06.2013 07:03 # −10

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

try {
	try {
		try {
			try {
				
			} catch (Exception e) {
				log.Error(e);
			}
		} catch (Exception e) {
			if (e is COMException) {
				// Ёлки, ну давно надо вычистить зависимость от этого кривого дельфи-компонента
				Marshal.FinalReleaseComObject(pupkinController);
			}
			
			throw e;
		}
	} catch (Exception e) {
		try {
			transaction.Rollback();
			session.Close();
		} catch (Exception) {
			// Заткнись
		}
		
		// Что за кривой лог, в консоль не пишет
		Console.WriteLine(e.Message);
	}
} catch (Exception) {
	// Сюда уж точно не доберёмся!
}

Ответить

someone 10.06.2013 11:16 # −14

показать все, что скрытоКстати, тут ничего не вырезано. Тело try действительно пустое.
Ответить
- bormand 10.06.2013 11:43 # −9
  
  показать все, что скрытоПопробуем ничего не делать. Если не получилось - пусть выше разбираются.
  Ответить
  - eth0 10.06.2013 19:16 # −13
    
    показать все, что скрытоВ небесной канцелярии?
    Ответить
    - bormand 10.06.2013 19:49 # −13
      
      показать все, что скрыто[Оффтопик]Когда уже у Страйко прекратятся application halted by exception. Debug mode is off...[/Оффтопик]
      Ответить
      - inkanus-gray 10.06.2013 19:54 # −13
        
        показать все, что скрытоА мне нравится. Это создаёт вид серьёзности. Не то, что на других сайтах: «Warning: mysql_select_db() expects parameter 2 to be resource, boolean given in /srv/www/vhosts/mpak.cms/include/mpfunc.php(690) : runtime-created function on line 6»
        Ответить
        
        bormand 10.06.2013 20:04 # −12
        
        показать все, что скрытоДа тут дело не в сообщении, а в том, что лог при этом никто не читает, и устранять причину этого экцепшена никто не собирается ;(
        Ответить
kxKIHhJ 25.08.2021 04:57 # 0

Руслан судорожно вдохнул, борясь с нахлынувшей тоской. Чувство вины за то, что он не смог - или не захотел - вовремя понять чувства брата, держало его на месте, не позволяло двигаться вперёд. О создании семьи он даже не думал - благо, родители за тридевять земель, да и никогда особо не интересовались такими аспектами его жизни, так что отчитываться не перед кем.
Ответить

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

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

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

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