0

Бывает когда пишишь код и тебе требуется улучшить его читабельность, но не всегда ясно, правильно ли это или нет!

Так вот хочу узнать у Вас, нужно ли использовать рефакторинг при написание кода?!

Допустим такой код:

static string Desktop = Environment.GetFolderPath(Environment.SpecialFolder.Desktop);

Студия рекомендует использовать рефакторинг, то есть инкапсулировать поле (и продолжить использовать свойства)

После рефакторинга:

public static string Desktop1
    {
        get
        {
            return Desktop;
        }

        set
        {
            Desktop = value;
        }
    }

Теперь он возвращает функцию для Desktop,нужно ли использовать такой подход?

2
  • Рефакторинг не используют при написании кода.
    – MihailPw
    18 мая 2017 в 18:25
  • @AGS17, Да, это уже после написания)
    – GooliveR
    18 мая 2017 в 19:37

2 ответа 2

2

Ок, вы меня уговорили, перечислять все в комментариях не удобно, поэтому пусть будет ответом, несмотря на "холиварность" вопроса, средней степени тяжести.

Все поля должны быть приватными.

Это требование ООП, поэтому даже не хочу обсуждать. Если требуется предоставить доступ к полю уровня protected, public или internal, то использование для этого свойств обязательно.

Когда нужны private свойства?

В общем случае private свойства абсолютно бесполезны, т.к. они не могут ограничить доступ к полю - уровень доступа у них одинаковый, ниже уже некуда. Исключение составляют автоматические свойства, но их возможность ограничения доступа к полю связана не с уровнем доступа, а тем фактом, что имена полей не известны программисту, да и компилятор их генерирует только в момент компиляции.

Но один случай полезного применения приватных свойств все же есть - это вычисляемые свойства за которыми нет конкретных полей или наоборот - совокупность зависимых полей. Другой вопрос, что для этих целей можно использовать обычные методы, но тут уже нужно смотреть на семантику использования и сложность кода аксессоров.

Более подробно про преимущества свойств, когда их применять нужно, а когда не нужно можно почитать в ответе @VladD - Для чего нужны свойства?


В приведенном вами примере, куда более ценно - явно указать модификатор private перед полем. Это избавит читающего ваш код от необходимости вспоминать/изучать уровень доступа по-умолчанию для членов класса.

7
  • Я правильно понял что публичные свойства лучше не использовать в том же классе, где вызывается какой-либо метод который присваивает это свойство) ? То есть это лучше использовать через другой класс и уже от туда обращаться?!
    – GooliveR
    18 мая 2017 в 20:36
  • @ArteS с чего вы это взяли? Публичное свойство также может иметь вычисляемые аксесоры или содержать логику валидации, почему бы к нему не обратиться в методе того же класса, если это следует из логики метода.
    – rdorn
    18 мая 2017 в 20:39
  • в каком случае лучше не использовать аксессоры get set? И в каких даже очень требуется?! Поясните пожалуйста, хочу понять просто)
    – GooliveR
    18 мая 2017 в 20:47
  • 1
    @AlexanderPetrov я считаю, что лучше сознательно нарушать запрет, когда знаешь зачем, чем считать, что так можно делать везде. Понятно, что ради производительности, когда все уперлось именно в нее, и глобальные переменные с безусловными переходами задействовать не грех. Но тут ведь как - опытный и так знает когда без этого не обойтись и чем грозит, а новичку лучше обозначить границу, за которую ходить не надо, дорастет - поймет сам. Так что пока оставлю ответ как есть.
    – rdorn
    19 мая 2017 в 12:19
0

Если вы используете это поле только внутри методов своего класса (а у вас оно приватное), то данная инкапсуляция не нужна.

Если вас интересует тема автоматического рефакторинга кода, то лучше взгянуть в сторону плагина для Visual Studio под названием Resharper. Он имеет бОльшую функциональность и благодаря нему новичку очень легко приучить себя писать хороший код.

12
  • 2
    Хм, но у автора до проведения авторефакторинга было всё-таки приватное статическое поле, следовательно только для "улучшения читаемости" не стоит инкапсулировать это поле в свойство, изменяя при этом уровень доступа для последнего.
    – Nikita
    18 мая 2017 в 18:45
  • Я где-то вычитал что не рекомендуется использовать публичные статические поля, что для них нужно использовать публичные свойства с аксессорами get;set; хочу понять просто нужно ли оно или нет)
    – GooliveR
    18 мая 2017 в 19:09
  • На примере у меня есть след код: foreach (var files in new DirectoryInfo(Desktop).EnumerateFiles("","")) но ведь правильнее же будет использовать публичное свойства например: foreach (var files in new DirectoryInfo(Desktop1).EnumerateFiles("","")) ?! Ваша точка зрения?
    – GooliveR
    18 мая 2017 в 19:19
  • 1
    @ArteS у вас исходное поле приватное (не указан ни один из модификаторов => по-умолчанию приватное) - в данном случае переход к свойству исключительно на усмотрение автора, лично я бы не стал менять, на читаемость это не повлияет никак, а время обращения, пусть и незначительно, но увеличится из-за обращения через дополнительный метод
    – rdorn
    18 мая 2017 в 19:57
  • 1
    @rdorn, И пусть обращается дольше чем обычно, но ведь пусть будет правильно чем не правильно)) Доли секунды ничего не изменят.
    – GooliveR
    18 мая 2017 в 20:22

Ваш ответ

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

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