1. C# / Говнокод #21580

    −15

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    try 
{
	Stream stream = new FileStream(fileName, FileMode.Open, FileAccess.Read, FileShare.Read);
	buffer = new Buffer(stream, false);
	Init();
} 
catch (IOException)
{
	throw new Exception("Cannot open file " + fileName);
}

    Запостил: dm_fomenok, 26 Октября 2016

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

    • ava someone 27.10.2016 13:06 # 0

      А закрывать кто будет?
      Ответить

      • ava roman-kashitsyn 27.10.2016 13:19 # 0

        > А закрывать кто будет?

        Хейлсберг.
        Ответить

      • ava nihau 27.10.2016 15:44 # 0

        гц и диспоуз паттерн закроют, негоже барину с таким ебаться
        Ответить

        • ava someone 27.10.2016 18:18 # 0

          Где он здесь? Поток никто не закрывает до выхода переменной stream из области видимости, и using не используется.
          Ответить

          • ava Steve_Brown 27.10.2016 18:27 # +1 

            void Init()
{
  try
  {
    ...
  }
  finally
  {
    buffer.Stream.Close();
  } 
}

            Логично же.
            Ответить

            • ava guest 28.10.2016 10:52 # 0

              Логично было обернуть этой хуйней вызов инит, чтобы сразу было видно что логично.
              Ответить

            • ava dm_fomenok 28.10.2016 13:04 # 0 

              ~Buffer() 
{ 
	Close();
}
		
protected void Close() 
{
	if (!isUserStream && stream != null)
	{
		stream.Close();
		stream = null;
	}
}


              Вот и весь код закрытия. Я думаю, то говницо, ибо финализаторы -- медленно.
              Ответить

    • ava Lokich 31.10.2016 15:57 # +1 

      try 
{
using(Stream stream = new FileStream(fileName, FileMode.Open, FileAccess.Read, FileShare.Read)){
	buffer = new Buffer(stream, false);
	Init();
}
} 
catch (IOException)
{
	throw new Exception("Cannot open file " + fileName);
}
      не?
      Ответить

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