Организация "утилей"

[Igors]

Добрый день

Вот опять возникла эта ситуация. Есть библиотечный класс который надо "обустроить" поудобнее. Напр сейчас класс с мета-данными

Код

C++ (Qt)
class Motion {
 ...
 std::map<std::string, std::string> metaData;

Мне нужно извлечь параметр по ключу "noise_scale", убедиться что значение корректно (флот), иначе использовать какой-то дефаулт. Такие операции массовые, ясно что нужно написать ф-ции утилиты (напр GetNoiseScale) чтобы всякий раз не возюкаться. Но где их поместить, как оформить?

Или даже более упрощенный пример: вот набегают полезные ф-ции для работы с пресловутым QString - ну и куда их девать? Наследоваться от QString явно нехорошо, тогда как?

Спасибо

[kai666_73]

Или даже более упрощенный пример: вот набегают полезные ф-ции для работы с пресловутым QString - ну и куда их девать? Наследоваться от QString явно нехорошо, тогда как?
Спасибо

Я бы в библиотеку вынес. Например, Utils, а в ней StringUtils.cpp/StringUtils.h с полезными экспортируемыми функциями...
Или я вопроса не понял?)

[Igors]

Я бы в библиотеку вынес. Например, Utils, а в ней StringUtils.cpp/StringUtils.h с полезными экспортируемыми функциями...

Хорошо, попробуем так и сделать

Код

C++ (Qt)
// MotionUtils.h
#ifndef MOTION_UTILS_H 
#define MOTION_UTILS_H 
 
namespace SomeLibrary {
  class Motion;
};
 
double GetNoiseScale( SomeLibrary::Motion * motion, double defaultValue = 0.0 );
void SetNoiseScale( SomeLibrary::Motion * motion, double value );
 
extern const double defaultNoiseScale;
 
#endif // MOTION_UTILS_H 

В принципе нормальный вариант, но достаточно ли убедителен вызов/использование просто "GetNoiseScale"? Напр слова "noise" и "scаle" могут быть и никак не связаны с "Motion"

[Old]

Если вы все равно передаете в функции указатель (ссылку) на Motion, т.е. явно привязываете ее к Motion, почему бы не расширить класс Motion такими методами? Для чего делать внешние геттеры/сеттеры, если можно их добавить в сам класс?

[Igors]

Если вы все равно передаете в функции указатель (ссылку) на Motion, т.е. явно привязываете ее к Motion, почему бы не расширить класс Motion такими методами? Для чего делать внешние геттеры/сеттеры, если можно их добавить в сам класс?

В том-то и дело что класс Motion библиотечный, поддерживается разработчиком либы, и время от времени обновляется, поэтому трогать его как минимум очень нежелательно. Напр добавить свои методы в QString можно, но во что обойдется (пере)сборка всякий раз?

[Old]

В том-то и дело что класс Motion библиотечный, поддерживается разработчиком либы, и время от времени обновляется, поэтому трогать его как минимум очень нежелательно. Напр добавить свои методы в QString можно, но во что обойдется (пере)сборка всякий раз?

Тогда это

В принципе нормальный вариант, но достаточно ли убедителен вызов/использование просто "GetNoiseScale"? Напр слова "noise" и "scаle" могут быть и никак не связаны с "Motion"

не проблема.

Можно сделать несколько функций для манипуляций с разными объектами:

Код

C++ (Qt)
double getNoiseScale( const SomeLibrary::Motion &motion, double defaultValue = 0.0 );
void setNoiseScale( SomeLibrary::Motion &motion, double value );
 
double getNoiseScale( const SomeLibrary::Poligon &poligon, double defaultValue = 0.0 );
void setNoiseScale( SomeLibrary::Poligon &poligon, double value );
 

Тогда для манипуляций с одинаковыми параметрами понадобиться одинаковые по имени функции.

[Old]

А если таких функций набирается много, можно сделать некий класс враппер:

Код

C++ (Qt)
class MotionHelper
{
public:
    MotionHelper( SomeLibrary::Motion &motion ) : m_motion( motion ) {}
 
    double getNoiseScale( double defVal = 0.0 ) const;
    void    setNoiseScale( double val );
 
private:
    SomeLibrary::Motion &m_motion;
};
 

[Igors]

не проблема.

Можно сделать несколько функций для манипуляций с разными объектами:

Все-таки я сейчас оборачиваю в namespace, а при использовании когда как.

Код

C++ (Qt)
double scale = MotionUtils::GetNoiseScale(motion);

Иной раз длинновато, но имя namespace заметно облегчает понимание. Того же можно добиться сделав класс со всеми методами static (а ля жаба), но тогда уже имя класса придется писать всегда

А если таких функций набирается много, можно сделать некий класс враппер:

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

Код

C++ (Qt)
double scale = MotionHelper(*motion).GetNoiseScale();

Так особого выигрыша нет

[Old]

Так особого выигрыша нет

Выигрыш будет при сложной настройке Motion:

Код

C++ (Qt)
MotionHelper m( motion );
m.setNoiseScale( 100500 );
m.setPos( ... );
m.setXXX();
 
// или
 
MotionHelper( motion ).setNoiseScale( 100500 ).setPos( ... ).setXXX();