0

Есть код пользовательского TextBox (ReadOnlyTextBox), который размещается неоднократно на форме с помощью панели элементов:

using System;
using System.Text;
using System.Drawing;
using System.Windows.Forms;
using System.Runtime.InteropServices;

public class ReadOnlyTextBox : TextBox
{
    [DllImport("user32.dll")]
    static extern bool HideCaret(IntPtr hWnd);

    public ReadOnlyTextBox()
    {
        this.ReadOnly = true;
        this.BackColor = Color.White;
        this.GotFocus += TextBoxGotFocus;
        this.Cursor = Cursors.Arrow;
    }

    private void TextBoxGotFocus(object sender, EventArgs args)
    {
        HideCaret(this.Handle);
    }
}

Подскажите пожалуйста, как корректно освободить память от "this.GotFocus += TextBoxGotFocus;", надо прописывать "this.GotFocus -= TextBoxGotFocus;" в коде формы?

1
  • Можете ничего не делать, такая подписка не повлияет на возможность удаления элемента сборщиком мусора 19 сен 2018 в 5:25

2 ответа 2

1

Вам не нужно отменять подписку в этом случае.

Смотрите, подписки на события же не просто так мешают сборщику мусора - а потому что создают дополнительные ссылки.

Рассмотрим такой код:

a.Foo += b.Bar

После выполнения этого кода у объекта a окажется ссылка на объект b.

В каком случае эта ссылка помешает сборке мусора? В том случае когда время жизни объекта b должно быть меньше времени жизни объекта a.

В вашем же случае что объект a, что объект b - это один и тот же this. Такой обработчик события никогда не вызовет утечку памяти.

0

Можно просто по идее дописать Dispose, так как TextBox : TextBoxBase : Control : IDisposable

public class ReadOnlyTextBox : TextBox
{
  [DllImport("user32.dll")]
  static extern bool HideCaret(IntPtr hWnd);

  public ReadOnlyTextBox()
  {
    this.ReadOnly = true;
    this.BackColor = Color.White;
    this.GotFocus += TextBoxGotFocus;
    this.Cursor = Cursors.Arrow;
  }

  private void TextBoxGotFocus(object sender, EventArgs args)
  {
    HideCaret(this.Handle);
  }

  protected override void Dispose(bool disposing)
  {
    this.GotFocus -= TextBoxGotFocus;
    base.Dispose(disposing);
  }
}
1
  • Спасибо, попробую!
    – olga
    19 сен 2018 в 5:18

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.