7

Есть классы, которые наследуются от одного абстрактного класса, у которого есть не виртуальная функция для работы со строкой:

protected string NormalizeString(string s)
{
    return s.Trim().ToUpper();
}

Нужно ли эту функцию делать статической? Когда функции нужно делать статическими, а когда обычными, если они не используют поля объекта?

1 ответ 1

11
  1. Да, я бы сделал эту функцию статической.
  2. Принцип очень простой.

    • Если функция относится к конкретному объекту, она должна быть нестатической. Пример: получение температуры с конкретного термометра — явно нестатическая фукция.
    • Если функция относится ко всем объектам данного типа, то она скорее всего должна быть статической. Пример: получение диапазона температур для класса термометров — явно статическая функция.
    • Если функция не относится ни к какому объекту, то есть является по существу утилитой (как у вас), и не сильно связана с функциональностью класса, её надо бы вынести в отдельный статический класс. Но часто, конечно, лень. Пример: перевод температуры из градусов Фаренгейта в градусы Цельсия — явная функция-утилита.

    (Существуют и пограничные случаи, когда не так очевидно, к какому из случаев относится функция. В этих случаях полагайтесь на интуицию или совет более опытных коллег.)

5
  • 3
    Я бы добавил, что если функция - утилита, без состояния, напрямую не относится к логике приложения (как у автора), то её можно сделать расширяющим методом и поместить в класс, специально предназначенный для строковых утилит. И ещё неалохо было бы проверить параметр на Null
    – tym32167
    4 июл 2017 в 9:20
  • 1
    @tym32167 проверка на null есть в коде методов, которые вызывают эту функцию перед ее вызовом. Если выносить функцию в отдельный статический класс, то, пожалуй, стоит сделать проверку.
    – Kunoichi
    4 июл 2017 в 9:23
  • 2
    Пограничный случай хорошо иллюстрирует Equals. И решение довольно простое - статическая + экземплярная реализации, причем экземплярная внутри использует статическую.
    – rdorn
    4 июл 2017 в 11:05
  • @rdorn: Хороший пример, спасибо!
    – VladD
    4 июл 2017 в 11:47
  • @tym32167: Согласен, обычно так и делаю.
    – VladD
    4 июл 2017 в 11:48

Ваш ответ

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

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