Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: C++ доступ к членам класса через this->
Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > Программирование
sigmaN
May 14 2018, 07:23
Один мой товарищ( как говорится, не к ночи будет помянут сей язык, бывший PHPшник) в C++ коде для AVR8 везде лепит this->

Вот пример его кода https://pastebin.com/hqyYcQYh

Аргументирует он это тем, что так якобы понятнее, что это член класса.
Хотя, конечно-же, это решается нэймингом.

В общем-то на C++ так никто не пишет, но хотелось бы привести весомые аргументы почему именно.

Первый аргумент, который я привел: this это указатель. Используя его мы подрезаем ноги оптимизатору, который, как я понимаю, на много аккуратнее относится к оптимизации indirect access
И может быть в каких-то простых случаях он этот this и упразднит, но тем не менее.

Может быть у кого-то еще есть аргументы? Мне даже самому стало интересно. Уже который год эта тема у нас с ним регулярно поднимается. Прям холливар Сишника и бывшего ПХПшника biggrin.gif
AlexRayne
May 14 2018, 07:45
Цитата(sigmaN @ May 14 2018, 10:23) *
Может быть у кого-то еще есть аргументы? Мне даже самому стало интересно. Уже который год эта тема у нас с ним регулярно поднимается. Прям холливар Сишника и бывшего ПХПшника biggrin.gif

тоже встречаю такой код. имхо, это подстелить соломку, дуть на воду после молока - как раз защищает от ошибочного нейминга. если объявить в функции локальное имя, перекрывающее поле класса, то получим непонятку.
если класс большой, то обнаружить что локальная переменная перекрыла поле, может быть не всегда тривиально.
novikovfb
May 14 2018, 07:47
ИМХО, ставить везде this-> равносильно избыточному написанию скобок в выражениях: не мешает компилятору, но в случае перебора текст становится громоздким.
Сергей Борщ
May 14 2018, 08:09
Согласен со всеми - с одной стороны защищает от сокрытия имени локальной перемеенй (последние ГЦЦ умеют ругаться по этому поводу), с другой стороны захламляет исходник как скобки в и без того очевидных местах:
QUOTE
-- Осмелюсь доложить, господин фельдкурат взаправду не
ошибся. Когда я был на действительной, меня освободили от
военной службы из-за идиотизма, общепризнанного идиотизма. По
этой причине отпустили из полка двоих: меня и еще одного,
капитана фон Кауница. Тот, господин поручик, идя по улице,
одновременно, извините за выражение, ковырял пальцем левой руки
в левой ноздре, а пальцем правой руки -- в правой. На учении он
каждый раз строил нас, как для церемониального марша, и
говорил: "Солдаты... э-э... имейте в виду... э-э... что
сегодня... среда, потому что... завтра будет четверг... э-э..."
Kabdim
May 14 2018, 09:10
расскажите ему о (*this).something .
k155la3
May 14 2018, 09:17
Видать человек при переходе на С(++) походил по своим, исконным, граблям и решил их не убирать, а устранять последствия (спец-шлем this->).
Сейчас и граблей уже нет, и забыто зачем это делалось. А привычка осталась.

Возможно связано с тем, что в приведенном коде вся реализация класса находится в h-файле.
А кода в cpp модуле с ф-ми в виде
Код
uint8_t TimeClass::GetSecond(void) {  }
не наблюдается. Соотв-но под вопросом информированность об операторе :: и
определение области видимости вообще. Как с этим делом в PHP - не знаю, не плавал.
tabr
May 14 2018, 09:50
Объясню, почему я так делаю...
Код:
Код
class ClassA
{
public:
void Set(uint8_t some_var)
{
//some actions
}
}
class ClassB
{
public:
void Set(uint8_t some_var)
{
//some actions
}
void BlaBlaBla(*ClassA var, uint8_t some_var)
{
if (/*something*/)
{
var->Set(some_var);
this->Set(some_var);//вот тут правильнее в целом смотрится именно this, а не "Set(some_var);"
}
}
}

Так же если в методе я буду всё-таки вынужден использовать this для доступа к членам текущего класса, то комбинация
Код
void SomeFunction(void)
{
//some code
}
class SomeClass
{
void SomeFunction(void)
{
//some another code
}
void SomemethodA(void)
{
}
void SomemethodB(void)
{
}
void SomemethodC(void)
{
this->SomeFunction();
SomemethodA();
SomemethodB();
SomeFunction();//чё происходит?
}
}

выглядит не очень. Как-то не читаемо, т.к. оно означает что где-то (НАВЕРНОЕ) есть функция с таким же названием, но это не точно.
Такая ситуация может возникнуть, если оспользуешь какие-нить сишные либы, к примеру (мне неизвестные), и имя твоего метода ВНЕЗАПНО совпало с именем... ну вы поняли...
ИМХО чем использовать ИНОГДА this (что я и делал сначала), я принял волевое xD решение использовать его ВСЕГДА - именно для повышения удобочитаемости кода.
Forger
May 14 2018, 10:49
© "Масло масляное"

В крайне редких случаях использовал.
Но потом по прошествии некоторого времени когда вижу "this->" в своем старом коде, то стараюсь исправить при первой же возможности.
Другими словами с помощью "this->" я помечал костыли в своем коде.

Поэтому, если весь код в итоге состоит из таких вот "костылей", то стоит задуматься, не пора ли "пациента" в конце концов вылечить и не мучаться wink.gif

Были времена, когда поля класса помечал префиксом "_" подчеркивание. Но, к счастью, своевременно отказался, поскольку читать такой код сложно - глаза "спотыкаются" на ровном месте.
esaulenka
May 14 2018, 11:50
Цитата(sigmaN @ May 14 2018, 10:23) *
Может быть у кого-то еще есть аргументы?


У меня два предложения.
1) поменьше думать за оптимизатор. this, не this.. Пока не доказано, что оно заметно влияет на производительность, заморачиваться не стоит.
2) Давать членам класса (да вообще, всему!) осмысленные имена, чтобы было понятно, что это, и откуда.

Сам пишу без всяких this->, и очень страдаю, когда пишу self.бла-бла() :-)
Nixon
May 14 2018, 14:02
Да какие аргументы, в первой же строчке написано почему - человек пришел из php. Вы еще кода перешедших с питона не видели sm.gif
jcxz
May 14 2018, 14:06
Цитата(sigmaN @ May 14 2018, 10:23) *
Первый аргумент, который я привел: this это указатель. Используя его мы подрезаем ноги оптимизатору, который, как я понимаю, на много аккуратнее относится к оптимизации indirect access
И может быть в каких-то простых случаях он этот this и упразднит, но тем не менее.

А какая разница оптимизатору: явное указание this или его неявное подразумевание? Один фиг это косвенное обращение через указатель. Ему должно быть одинаково.
k155la3
May 14 2018, 17:33
Цитата(tabr @ May 14 2018, 12:50) *
Код: . . .

Код
void SomeFunction(void)
{
..//some code  AAAA
}

class SomeClass
{
...void SomeFunction(void)
...{
...//some another code  BBBB
...}
...void SomemethodA(void)
...{
...}
...void SomemethodB(void)
...{
...}
...void SomemethodC(void)
...{
.....this->SomeFunction();
.....SomemethodA();
.....SomemethodB();
.....SomeFunction();//чё происходит?   - тоже, что и this->SomeFunction();  some code  BBBB
---------------
    ::SomeFunction();   // +++   some code  AAAA
---------------
...}

}
Y/N ?
Forger
May 14 2018, 19:35
Если не ошибаюсь, то локальные методы/поля имеют приоритет над глобальными методами (функциями)/переменными.
Другими словами: чем уже область видимости, тем больше преимущество (если тут можно применить такой термин).
Речь про одинаковые названия.
Но дабы не гадать как себя поведет тот или иной компилятор, стоит вообще избегать таких ситуаций.
Поскольку найти потом подобные ошибки будет крайне сложно!
jcxz
May 15 2018, 05:28
Цитата(Forger @ May 14 2018, 22:35) *
Если не ошибаюсь, то локальные методы/поля имеют приоритет над глобальными методами (функциями)/переменными.
Другими словами: чем уже область видимости, тем больше преимущество (если тут можно применить такой термин).

Очень опасно сиё. Ибо достаточно потом чуток изменить список аргументов вызова член-метода и забыть в одном из вызовов поправить этот список. И получить неожиданный эффект biggrin.gif
Стараюсь не делать глобальных функций и член-методов с одинаковыми именами без реальной необходимости.
Forger
May 15 2018, 05:35
Цитата(jcxz @ May 15 2018, 08:28) *
Очень опасно сиё.

А я с этим и не спорю. И потому нигде не применяю.

Цитата
Стараюсь не делать глобальных функций и член-методов с одинаковыми именами без реальной необходимости.

А я вообще не использую ничего глобального. Одна из причин - как раз именно эта путаница в именах и никому не нужные сложности с именованием.
AlexandrY
May 15 2018, 05:42
Цитата(jcxz @ May 15 2018, 08:28) *
Очень опасно сиё. Ибо достаточно потом чуток изменить список аргументов вызова член-метода и забыть в одном из вызовов поправить этот список. И получить неожиданный эффект biggrin.gif
Стараюсь не делать глобальных функций и член-методов с одинаковыми именами без реальной необходимости.

Пральна.
Если делать программы на 1000 строк кода то можно не делать одинаковых имен.
А если у вас тонны стороннего кода, то вы даже проверить толком не можете какие у вас есть глобальные имена.
PHP-шники это хорошо знают потому как имеют дела с монструозными CMS-ами.
Дело тут не в языке, а в масштабах.

Вторая причина в IDE.
Если программист владеет регулярными выражениями и одним кликом может добавить или удалить все this, то он и не заморачавается где у него и что стоит.
А если писать в notepad-е, то да, любой знак не в том месте вызывает раздражение.
Сергей Борщ
May 15 2018, 07:51
QUOTE (jcxz @ May 15 2018, 08:28) *
Стараюсь не делать глобальных функций и член-методов с одинаковыми именами

QUOTE (Forger @ May 15 2018, 08:35) *
А я вообще не использую ничего глобального.
Сначала заклевали неизвестного автора за безобидный this, потом насыпали своих фобий про глобальную область видимости sm.gif
Forger
May 15 2018, 10:00
Цитата(Сергей Борщ @ May 15 2018, 10:51) *
Сначала заклевали неизвестного автора за безобидный this, потом насыпали своих фобий про глобальную область видимости sm.gif

Судя по автору (число постов, статус), не удивлюсь, что целью этой темы был очередной "холивар" wink.gif
Естественно подключились соответствующие "персонажи" ...
Это кто ж откажется-то от участия в таком "событии" ?!
k155la3
May 15 2018, 11:07
Эт не мы. Все с англичан началось, с ихним This_IS sm.gif
Сергей Борщ
May 15 2018, 11:08
QUOTE (Forger @ May 15 2018, 13:00) *
Судя по автору (число постов, статус), не удивлюсь, что целью этой темы был очередной "холивар" wink.gif
Он сразу сказал "мопед не мой" sm.gif. Не путайте автора вопроса и автора кода.
Forger
May 15 2018, 11:19
Цитата(Сергей Борщ @ May 15 2018, 14:08) *
Он сразу сказал "мопед не мой" sm.gif. Не путайте автора вопроса и автора кода.

Именно так и появляются холивары: "безобидный" вброс и .... "пошла моча по трубам" wink.gif
sigmaN
May 15 2018, 15:42
Никакого троллинга и холливара
У меня с другом состоялся реальный разговор на вполне конкретную тему(причем неоднократно), к обсуждению которой я решил подключить форумчан....
Forger
May 15 2018, 16:07
Цитата(sigmaN @ May 15 2018, 18:42) *
Никакого троллинга и холливара

Все еще впереди - пока идет всего лишь 2я страница темы ... :D
Сергей Борщ
May 15 2018, 21:28
QUOTE (k155la3 @ May 15 2018, 14:07) *
Эт не мы. Все с англичан началось, с ихним This_IS sm.gif

Ладно, подкину... Английский, вроде как, начинается с "Ландан из зе кэпитал оф зэ Грит Британ". sm.gif
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2024 Invision Power Services, Inc.