LINUX.ORG.RU

а что метод str() у stringstream отменили?

документацию к stringstream слабо прочитать?

anonymous
()

Можно вернуть ссылку на stringstream.

Но только это тоже нафиг не нужно.

burivuh
()
Ответ на: комментарий от Selecter

>А пояснить?

функция объявленная в теле класса является inline функцией, нет нужды писать рядом inline

anonymous
()


1. все-таки round(const std::string &unit). зачем вам лишняя локальная копия и расходы на неё?
2. вообще, зачем указатель типа округления передавать в виде строки? перечисление будет понятнее и существенно дешевле.
3. man strstr -> постарайтесь не использовать такие имена.
4. во избежании путаницы, криво разводимой через this->h и иже с ними, все-таки не стоит перекрывать имена членов структуры локальными переменными. и читается плохо и легко внести ошибку из-за невнимательности.
5. s/stringstream/ostringstream. вы же из него не читаете, правда?
6. как уже было сказано, подобные операторы реализовывать в виде inline несколько расточительно :)
7. #include <cstdlib>
...
99. посмотреть, как похожий конкретный класс реализован в Страуструпе :)

// wbr

klalafuda ★☆☆
()
Ответ на: комментарий от anonymous

Это я знаю. Но чтобы не забыть, я указываю inline (если это надо) и у методов, и у функций.

Selecter ★★★★
() автор топика
Ответ на: комментарий от Selecter

> 0. Такая задача не заслуживает такой пристальной критики.

afaiu это отнюдь не задача, а человек просто учится -> вполне заслуживает, почему нет? :)

// wbr

klalafuda ★☆☆
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.