Управление общими ресурсами между классами? - PullRequest
Новая
       6

Управление общими ресурсами между классами?

5 голосов
/ 23 марта 2010

Представьте, что у меня есть несколько компонентов Viewer, которые используются для отображения текста, и у них есть несколько режимов, которые пользователь может переключать (разные предустановки шрифтов для просмотра текста / двоичного / шестнадцатеричного). Каков наилучший подход для управления общими объектами - например, шрифты, диалог поиска и т. Д.? Я подумал, что статический класс с лениво инициализированными объектами будет в порядке, но это может быть неправильной идеей. 

static class ViewerStatic
{
    private static Font monospaceFont;
    public static Font MonospaceFont
    {
        get
        {
            if (monospaceFont == null)
                //TODO read font settings from configuration
                monospaceFont = new Font(FontFamily.GenericMonospace, 9, FontStyle.Bold);
            return monospaceFont;
        }
    }

    private static Font sansFont;
    public static Font SansFont
    {
        get
        {
            if (sansFont == null)
                //TODO read font settings from configuration
                sansFont = new Font(FontFamily.GenericSansSerif, 9, FontStyle.Bold);
            return sansFont;
        }
    }
}

Ответы [ 2 ]

1 голос
/ 25 марта 2010

Для элементов, которые вы хотите создать один раз, а затем повторно использовать, есть два соответствующих шаблона: Singleton и Cache. Если вы будете использовать этот предмет вечно, синглтон в порядке. Память, выделенная этому экземпляру, никогда не будет очищена. Если вы будете повторно использовать элемент некоторое время, но, возможно, эта функция не будет использоваться в течение нескольких дней, я предлагаю использовать кэш. Затем память может быть очищена, когда элемент больше не используется.

Если вы используете Singleton, вы, вероятно, захотите просто инициализировать шрифты напрямую, а не использовать шаблон Lazy init. Для меня шрифты звучат довольно просто и вряд ли ошибятся. Однако, если элемент может потерпеть неудачу во время создания (возможно, из-за отсутствующего файла шрифта или чего-то еще), ленивый шаблон по крайней мере позволяет повторить попытку в следующий раз. Вы не сможете восстановить статический инициализатор позже, даже если он потерпит неудачу, без перезапуска всего приложения. Будьте осторожны, чтобы ограничить эти попытки!

Наконец, имя вашего класса "ViewerStatic" вызывает озабоченность. Существует анти-паттерн, известный как объект «Бог». Я называю это «ведром». Если вы создадите это, материал придет. Вскоре вы найдете все виды вещей, которые выбрасывают в ведро. Ваш класс ViewerStatic станет огромным. Было бы лучше иметь класс с именем «FontFlyWeights», а затем еще один с именем «ConstantStrings» или «SystemDialogFactory» ... и т. Д.

1 голос
/ 23 марта 2010

Мне кажется, что это нормально, но действительно ли это необходимо? Простой подход состоит в том, чтобы просто создавать новые шрифты и диалоговые окна, когда они вам нужны, а затем утилизировать их, если необходимо, и позволить сборщику мусора очистить их.

Проверили ли вы, чтобы увидеть, есть ли у простого подхода заметная стоимость, которая стоит добавить сложность кэширования общих объектов?

...