analizer 30.05.2011 20:36 talkonaut-iphone_5.88_67b1c873

пстач, а как ты относишься к тому чтобы в паттерне PImpl все методы класса-обёртки делать константными. Это ок?

c++, programming?
#sfith
re 16 comments !recommend ~bookmark Ssubscribe
1. jtootf 30.05.2011 20:47

зачем? если класс иммутабельный, то отношения к pimpl вопрос не имеет; если мутабельный, то это сокрытие возможных ошибок и дыра в системе типов

#sfith/1
rereply
2. analizer → jtootf /1 30.05.2011 20:49 talkonaut-iphone_5.88_67b1c873

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

#sfith/2
rereply
3. jtootf → analizer /2 30.05.2011 20:50

а в чём профит?

#sfith/3
rereply
4. analizer → jtootf /3 30.05.2011 20:55 talkonaut-iphone_5.88_67b1c873

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

#sfith/4
rereply
5. jtootf → analizer /4 30.05.2011 21:01

опять же, в чём профит, если эти const'ы ничего не гарантируют? с тем же успехом можно писать их в комментариях перед методом

#sfith/5
rereply
6. DZhon 30.05.2011 21:06 Psi+

Как верно заметили, _все_ методы могут и не обладать константным смыслом. Каждое поле с ключевым словом mutable должно <s>даваться потом и кровью</s> появляться после размышлений о том, не нарушается ли логическая константность.

#sfith/6
rereply
7. analizer → DZhon /6 30.05.2011 21:07 talkonaut-iphone_5.88_67b1c873

кто сказал mutable?

#sfith/7
rereply
8. jtootf → analizer /7 30.05.2011 21:10

а изобрази пример, пожалуйста. минимальный

#sfith/8
rereply
9. DZhon → analizer /7 30.05.2011 21:14 Psi+

http://pastebin.com/9Mv3tzrP так хочешь ? Так это вырывание костылей самому себе.

#sfith/9
rereply
10. analizer → jtootf /8 30.05.2011 21:15 talkonaut-iphone_5.88_67b1c873

pastebin.com/Yz9RN1LM

#sfith/10
rereply
11. DZhon → analizer /10 30.05.2011 21:16 Psi+

оукей, то есть константный интерфейс вносит изменения в реализацию, противоречие не видишь ? %) Pimpl какбе говорит "реализован посредством".

#sfith/11
rereply
12. analizer → DZhon /11 30.05.2011 21:18 talkonaut-iphone_5.88_67b1c873

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

#sfith/12
rereply
13. jtootf → analizer /10 30.05.2011 21:18

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

#sfith/13
rereply
14. jtootf → analizer /12 30.05.2011 21:19

то, что это компилируется — недостаток системы типов C++, и демонстрация слабости гарантий, предоставляемых ключевым словом const. не более того

#sfith/14
rereply
15. DZhon → analizer /12 30.05.2011 21:20 Psi+

Это, увы, проблема недостаточно въедливого компилятора (насколько я понимаю). Которой нагло пользуются.

#sfith/15
rereply
16. analizer → DZhon /15 30.05.2011 21:22 talkonaut-iphone_5.88_67b1c873

ты недооцениваешь его въедливость. См. #sfoff/4

#sfith/16
rereply
Do you really want to delete ?