Необходима ли инкапсуляция

@olimpset · Регистрация: 12.12.2012

Author24 — интернет-сервис помощи студентам

Есть переменная (флаг) типа bool. Нужно ли сделать отдельные медоты, которые присваивают на true/false?

C#

public static void InitResume() {
            createResume = true;
        }
public static void InitPause() {
            createResume = false;
        }

Или проще напрямую присваивать: createResume = false; Как код более естетично смотрится?

@bax_tang · 27.09.2016, 15:42

Я бы использовал методы - нагляднее выглядит, проще для понимания, легче тестировать; потом, при необходимости, достаточно будет переписать только реализацию соответствующего метода и не задумываться об изменениях в местах его вызовов.

@Tsin · 27.09.2016, 16:03

Заведите пропертю и обращайтесь через неё

C#

1	public static bool CreateResume { get; set; }

или, если нужно оставить поле

C#

1	public static void SetResume(bool value) { createResume = value; }

@olimpset · 27.09.2016, 16:09 **[ТС]**

Да, этот вариант будет лучше. Меньше строк кода будет, в отличии от инкапсуляции. Тем более если так сделать:

C#

1	static bool CreateResume { get { return true; } set {} }

(по умолчанию будет true), то так всего одна строка. Я добиваюсь минимизации кода, убираю пустые строки, стараюсь небольшой метод, условие и т.д в одну строку.

@valera_21 · 27.09.2016, 16:48

Сообщение от olimpset

static bool CreateResume { get { return true; } set {} }

чтот мне кажется, что в таком случае всегда будет возвращать true,не?

@Tsin · 27.09.2016, 16:50

Сообщение от olimpset

в отличии от инкапсуляции

некорректно так говорить

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

Смотрите, чтоб не сказалось на читаемости потом

@evkaky · 27.09.2016, 16:58

Нужно ли сделать отдельные медоты, которые присваивают на true/false?

Нет.
Подправить имя, что бы по нему однозначно было понятно, что это переменная-флаг, и всё

C#

1	public static bool ResumeCreated;

Эстетично и практично

@BozKurt · 27.09.2016, 18:07

Сообщение от olimpset

Меньше строк кода будет, в отличии от инкапсуляции

Вообще-то свойства это и есть инкапсуляция оформленная в виде удобного сахара. В итоге, get и set всё равно развернуться в методы.

Сообщение от olimpset

Я добиваюсь минимизации кода, убираю пустые строки, стараюсь небольшой метод, условие и т.д в одну строку

Это не всегда полезно делать. Не увлекайся.

@olimpset · 27.09.2016, 18:26 **[ТС]**

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

@Tsin · 27.09.2016, 18:38

Сообщение от olimpset

хоть и будет сплошная каша из кода

Если вам нужна компактность, то просто сворачивайте методы, но кашу внутри не городите. Потом забудется, и будет сложно отлаживать, коли чего поломается. Код, прежде всего, пишется для людей, а только потом - для машин.

@Tsin 746 / 487 / 187 Регистрация: 30.12.2012 Сообщений: 1,278 Записей в блоге: 2
	27.09.2016, 18:38	10
	Сообщение от olimpset хоть и будет сплошная каша из кода Если вам нужна компактность, то просто сворачивайте методы, но кашу внутри не городите. Потом забудется, и будет сложно отлаживать, коли чего поломается. Код, прежде всего, пишется для людей, а только потом - для машин. 0

Необходима ли инкапсуляция

Решение

Решение

@bax_tang .NET senior 440 / 358 / 137 Регистрация: 23.09.2016 Сообщений: 980
	27.09.2016, 15:42	2
	Я бы использовал методы - нагляднее выглядит, проще для понимания, легче тестировать; потом, при необходимости, достаточно будет переписать только реализацию соответствующего метода и не задумываться об изменениях в местах его вызовов. 0

@valera_21 484 / 439 / 123 Регистрация: 05.01.2010 Сообщений: 1,848
	27.09.2016, 16:48	5
	Сообщение от olimpset `static bool CreateResume { get { return true; } set {} }` чтот мне кажется, что в таком случае всегда будет возвращать `true`,не? 0

@Tsin 746 / 487 / 187 Регистрация: 30.12.2012 Сообщений: 1,278 Записей в блоге: 2
	27.09.2016, 16:50	6
	Сообщение от olimpset в отличии от инкапсуляции некорректно так говорить Таким кодом вы будете запутывать клиентов класса. Они будут иметь возможность устанавливать значение поля, но по факту ничего происходить не будет. или запретите это, сделав private set, или подумайте как по-другому это исправить. Смотрите, чтоб не сказалось на читаемости потом 0

@BozKurt 307 / 284 / 102 Регистрация: 06.05.2014 Сообщений: 861
	27.09.2016, 18:07	8
	Сообщение от olimpset Меньше строк кода будет, в отличии от инкапсуляции Вообще-то свойства это и есть инкапсуляция оформленная в виде удобного сахара. В итоге, get и set всё равно развернуться в методы. Сообщение от olimpset Я добиваюсь минимизации кода, убираю пустые строки, стараюсь небольшой метод, условие и т.д в одну строку Это не всегда полезно делать. Не увлекайся. 0

@olimpset 1 / 1 / 1 Регистрация: 12.12.2012 Сообщений: 22
	27.09.2016, 18:26 [ТС]	9
	Но все же, хоть и будет сплошная каша из кода (без промежутков лишних), найти что то не составляет труда в коде, главное грамотно методы назвать, что бы потом знать что и где искать. 0