Локальные переменные или прямое обращение к get-объекта класса?

@Ghost Writter · Регистрация: 09.12.2015

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

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

Например:

C++

void Array::operator +(Array value)
{
    int sizeFirst = getSize();
    int sizeSecond = value.getSize();
 
    if(sizeFirst > sizeSecond)
    {
 
    }
    else if(sizeFirst < sizeSecond)
    {
 
    }
    else
    {
 
    }
}

или

C++

void Array::operator +(Array value)
{
    if(getSize() > value.getSize())
    {
 
    }
    else if(getSize() < value.getSize())
    {
 
    }
    else
    {
 
    }
}

Ну, и почему? Склоняюсь к первому варианту, но, возможно, что разницы не будет и оптимальным вариантом будет второй пример.

@Ghost Writter · 09.01.2016, 08:43 **[ТС]**

nord_v, displayMessage, конкретно, второй скрин последняя функция.

nord_v · 09.01.2016, 08:44

Пример, к моим словам.

@Ghost Writter · 09.01.2016, 08:46 **[ТС]**

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

nord_v · 09.01.2016, 08:51

Сообщение от Ghost Writter

конкретно, второй скрин последняя функция.

А не задумывался: почему только в последней? Почему во всех, до последней, обходились без getCourseName()?

Добавлено через 2 минуты

Сообщение от Ghost Writter

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

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

nord_v · 09.01.2016, 08:58

Вот тебе, для начала (Солтер Н.А., Клепер С.Дж. "С++ для профессионалов").

@Ghost Writter · 09.01.2016, 08:59 **[ТС]**

Всм. обходились?

Там только один метод, устанавливающий значение названия курса. И один метод, который его получает. Вон, даже конструктор использует set метод, потому что это удобно и избавляет от повторений, как минимум. Так же и действует get метод.

В дальнейшем не буду обращать внимание на твои посты, тупо игнорировать их. Ты даже текст прочитать не в состоянии.

nord_v · 09.01.2016, 09:04

Вот тебе Прата. Ещё?

nord_v · 09.01.2016, 09:06

Не по теме:

Сообщение от Ghost Writter

В дальнейшем не буду обращать внимание на твои посты, тупо игнорировать их. Ты даже текст прочитать не в состоянии.

Ты меня сильно огорчил. С такими как ты общаться - себя не уважать. Продолжай дальше тупить.

Croessmah · 09.01.2016, 10:02

Ghost Writter, использование того или иного зависит сугубо от Вас, от логики работы, и т.д.

@hoggy · 09.01.2016, 10:30

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

Вот тебе, для начала (Солтер Н.А., Клепер С.Дж. "С++ для профессионалов").

C++

1	double getValue();

C++

1	void setValue(string inString);

C++

double SpreadsHeetCell::getValue()
{
    return (mValue);
}

плохой код.

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

Вот тебе Прата

C++

1	void Show();

плохой код.

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

Ещё?

жгите.

Croessmah · 09.01.2016, 10:47

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

плохой код

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

@hoggy · 09.01.2016, 11:09

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

Эта книга "Для профессионалов",
на самом деле "Почти для самых маленьких".

да я то этот момент понимаю.
как и тот, что в букварях целенаправленно совершаются допущения,
что бы упростить материал для новичков.

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

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

не нужно её воспринимать как религиозного лидера

дык, и я о том же.

суть проста:
не нужно апеллировать к букварям, как к неким авторитетам,
которые вам истину покажут,
и приучат к элегантному стилю.

вот если бы ребята Макконелла приводили
в качестве аргументации, или Роббинсона,
вот тогда бы тема была бы намного интереснее.

@ct0r · 09.01.2016, 12:30

О нет! Только не вызовы публичных методов (да-да, мне лень писать "функция-член", ну и что) из недр класса!

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

Если нам нужно вызывать публичные методы изнутри, то либо мы в этот класс впихуем невпихуемое, которое по логике должно быть снаружи. Либо эта логика должна быть в отдельном приватном методе. Есть вариант вообще этот метод, если он может обойтись только публичными членами класса, сделать свободной функцией.

@mporro · 09.01.2016, 15:04

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

@Ghost Writter 53 / 53 / 19 Регистрация: 09.12.2015 Сообщений: 215
	09.01.2016, 08:43 [ТС]	21
	nord_v, displayMessage, конкретно, второй скрин последняя функция. 0

nord_v 337 / 185 / 80 Регистрация: 22.08.2013 Сообщений: 724
	09.01.2016, 08:44	22
	Пример, к моим словам. 0

@Ghost Writter 53 / 53 / 19 Регистрация: 09.12.2015 Сообщений: 215
	09.01.2016, 08:46 [ТС]	23
	nord_v, я больше верю людям, которые живут в этой среде, чем пустозвонным словам без аргументов. 0

nord_v 337 / 185 / 80 Регистрация: 22.08.2013 Сообщений: 724
	09.01.2016, 08:58	25
	Вот тебе, для начала (Солтер Н.А., Клепер С.Дж. "С++ для профессионалов"). Миниатюры 0

@Ghost Writter 53 / 53 / 19 Регистрация: 09.12.2015 Сообщений: 215
	09.01.2016, 08:59 [ТС]	26
	Всм. обходились? Там только один метод, устанавливающий значение названия курса. И один метод, который его получает. Вон, даже конструктор использует set метод, потому что это удобно и избавляет от повторений, как минимум. Так же и действует get метод. В дальнейшем не буду обращать внимание на твои посты, тупо игнорировать их. Ты даже текст прочитать не в состоянии. 0

nord_v 337 / 185 / 80 Регистрация: 22.08.2013 Сообщений: 724
	09.01.2016, 08:51	24
	Сообщение от Ghost Writter конкретно, второй скрин последняя функция. А не задумывался: почему только в последней? Почему во всех, до последней, обходились без getCourseName()? Добавлено через 2 минуты Сообщение от Ghost Writter я больше верю людям, которые живут в этой среде, чем пустозвонным словам без аргументов. Я не знаю, какие тебе аргументы нужны, голову просто включи. Скринов, с подтверждением моих слов, я тебе гору могу выложить. 0

nord_v 337 / 185 / 80 Регистрация: 22.08.2013 Сообщений: 724
	09.01.2016, 09:04	27
	Вот тебе Прата. Ещё? Миниатюры 0

nord_v
	09.01.2016, 09:06 #28
	Не по теме: Сообщение от Ghost Writter В дальнейшем не буду обращать внимание на твои посты, тупо игнорировать их. Ты даже текст прочитать не в состоянии. Ты меня сильно огорчил. С такими как ты общаться - себя не уважать. Продолжай дальше тупить. 0

Croessmah Неэпический 17869 / 10634 / 2054 Регистрация: 27.09.2012 Сообщений: 26,736 Записей в блоге: 1
	09.01.2016, 10:02	29
	Ghost Writter, использование того или иного зависит сугубо от Вас, от логики работы, и т.д. 0

Croessmah Неэпический 17869 / 10634 / 2054 Регистрация: 27.09.2012 Сообщений: 26,736 Записей в блоге: 1
	09.01.2016, 10:47	31
	Сообщение от hoggy плохой код Эта книга "Для профессионалов", на самом деле "Почти для самых маленьких". Мало чем отличается от других учебных пособий, поэтому то и код соответствующий. Конечно, из нее есть что почерпнуть, но не нужно её воспринимать как религиозного лидера 0

@hoggy 8739 / 4317 / 960 Регистрация: 15.11.2014 Сообщений: 9,760
	09.01.2016, 11:09	32
	Сообщение от Croessmah Эта книга "Для профессионалов", на самом деле "Почти для самых маленьких". да я то этот момент понимаю. как и тот, что в букварях целенаправленно совершаются допущения, что бы упростить материал для новичков. понятно, что "для профессионалов" - это просто жульнический маркетинг. Сообщение от Croessmah не нужно её воспринимать как религиозного лидера дык, и я о том же. суть проста: не нужно апеллировать к букварям, как к неким авторитетам, которые вам истину покажут, и приучат к элегантному стилю. вот если бы ребята Макконелла приводили в качестве аргументации, или Роббинсона, вот тогда бы тема была бы намного интереснее. 2

@ct0r Игогошка! 1801 / 708 / 44 Регистрация: 19.08.2012 Сообщений: 1,367
	09.01.2016, 12:30	33
	О нет! Только не вызовы публичных методов (да-да, мне лень писать "функция-член", ну и что) из недр класса! Это же лишний гемор и кладезь ошибок. Когда я меняю публичный метод, я хочу думать (и в первую очередь думаю) о том, как изменится из-за этого поведение внешней среды, но я вообще не хочу думать о том (и могу упустить), какая при этом фигня может приключиться внутри моего класса. Как еще пример - есть какой-то класс. Я не хотел, но нужда заставила сделать его потокобезопасным. Дедлайны все прошли уже год назад, я быстренько втыкаю в начало всех методов по мьютексу. А глубоко в классе, в одном редко используемом приватном методе, запрятан вызов публичного. Нужно ли говорить о том, что произойдет, если в публичном методе мы лочим мьютекс (нерекурсивный), по цепочке вызываем приватный, который вызывает публичный, в котором опять этот мьютекс лочим? Если нам нужно вызывать публичные методы изнутри, то либо мы в этот класс впихуем невпихуемое, которое по логике должно быть снаружи. Либо эта логика должна быть в отдельном приватном методе. Есть вариант вообще этот метод, если он может обойтись только публичными членами класса, сделать свободной функцией. 1

@mporro 306 / 101 / 18 Регистрация: 04.07.2014 Сообщений: 571
	09.01.2016, 15:04	34
	Если метод класса никак внутренние данные не использует, его немедленно из методов нужно убирать, ибо это не метод, а некоторая функция, в математическом смысле, над экземплярами класса, коих можно написать счётное число. 0