Расширение данных

[Igors]

Добрый день

Объект имеет обширный набор "параметров" организованных в виде дерева. Доступ к параметру осуществляется по уникальному ключу (ID), напр (псевдокод)

Код

C++ (Qt)
const int DIFFUSE_ID = 2070;
...
Param * param = object->GetParam(DIFFUSE_ID);

Сам нод дерева (param) нужен редко, зато очень часто его значение, напр

Код

C++ (Qt)
double value = object->GetParamValue(DIFFUSE_ID, currentTime);

Таких вызовов тысячи. Все работает, все хорошо. Но вот понадобилось дерево сделать динамическим, т.е. узлы могут добавляться и удаляться. С точки зрения дерева это выглядит так

...
  MATERIAL_ID     // Нод "материал" (все как сейчас)
    DIFFUSE_ID
    LUMA_ID
    ....
  MATERIAL_ID2     //  Еще один нод "материал" (теперь их может быть много)
    DIFFUSE_ID
    LUMA_ID
...

Как лучше всего адаптировать имеющийся (не маленький) код к этому изменению?

Спасибо

[qate]

не совсем понятно что меняется при удалении и добавлении узла
ведь есть указатель на объект и по нему получить можно параметр
чтото еще изменилось ?

[Igors]

не совсем понятно что меняется при удалении и добавлении узла
ведь есть указатель на объект и по нему получить можно параметр
чтото еще изменилось ?

Ткперь DIFFUSE_ID уже не является однозначным ключом. Напр Что должен делать объект чтобы получить значение не из первого а из второго узла "материал"?

[Bepec]

Ну, или добавить ещё 1 параметр как "родителя", или сделать привязку к текущему узлу.

Как я вижу без изменения основных вызовов( добавляется только установка текущего параметра)

Код:

const int MATERIAL_ID = 2070;
...
// получаем DIFFUSE_ID из MATERIAL_ID
object->setCurrentParam(MATERIAL_ID);
double value1 = object->GetParamValue(DIFFUSE_ID, currentTime);

// получаем DIFFUSE_ID из MATERIAL_ID1
object->setCurrentParam(MATERIAL_ID1);
double value2 = object->GetParamValue(DIFFUSE_ID, currentTime);

Или же вариант с добавлением параметра

Код:

const int MATERIAL_ID = 2070;
...
// получаем DIFFUSE_ID из MATERIAL_ID
double value1 = object->GetParamValue(MATERIAL_ID, DIFFUSE_ID, currentTime);

// получаем DIFFUSE_ID из MATERIAL_ID1
double value2 = object->GetParamValue(MATERIAL_ID1, DIFFUSE_ID, currentTime);

Ну ещё можно зафигачить GetParamValue со значением по умолчанию для поддержки старого кода. Т.е. до первого найденного параметра DiffuseId

PS других вариантов, без масштабных изменений не вижу.

[Igors]

Число параметров нода "материал" - где-то от 100 до 200 (точно не мерял). Число вызовов этих параметров - тысячи. При этом механизм GetParamValue используется повсеместно, не только для материала.

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

Код:

object->setCurrentParam(MATERIAL_ID);
double value1 = object->GetParamValue(DIFFUSE_ID, currentTime);

Нет той самой надежности

Код:

// получаем DIFFUSE_ID из MATERIAL_ID1
double value2 = object->GetParamValue(MATERIAL_ID1, DIFFUSE_ID, currentTime);

Не пресекается старый вызов. И кто ответит за MATERIAL_ID1? Это уже не константа

 

[Bepec]

Если, как вы говорите, констант больше нет - надо всё переписывать.

Ну потому что кирпичей нет, а из треугольников домик не построишь.

[Igors]

Если, как вы говорите, констант больше нет - надо всё переписывать.

Понятно что надо, вопрос каким образом. На что менять существующие вызовы?

[Igors]

Я пока вижу такую схемку

1) Переименовать все константы нода "материал", напр X_DIFFUSE_ID вместо DIFFUSE_ID, это вызовет массовые ошибки компилятора, ну и хорошо

2) Править ошибки заменяя

Код

C++ (Qt)
double value = object->GetParamValue(DIFFUSE_ID, currentTime);

На

Код

C++ (Qt)
double value = object->GetMaterialValue(materialIndex, X_DIFFUSE_ID, currentTime);

Ну и по месту откуда-то брать тот materialIndex. В самом объекте мапы

Код

C++ (Qt)
double CObject::GetMaterialValue( uint materialIndex, uint  paramIndex, double currentTime )
{
  Q_ASSERT(materialIndex < m_MaterialMaps.size());
  int ID = m_MaterialMaps[materialIndex].value(paramIndex); // получаем реальное уникальное ID
  Q_ASSRERT(ID);
  return GetParamValue(id, currentTime);
}

При создании/удалении материалов парить m_MaterialMaps их владельцев. Критикуем, пинаем

Да, и неужели нет такого паттерна? 

[Bepec]

Паттерны это дерево. Вы счас к нему всё и сводите.
А переделка с одного паттерна на другой к паттернам не относится

PS у вас нечего критиковать, вы привели мой второй вариант

[Igors]

Паттерны это дерево.

Думал, может человек повзрослел (после длительного отсутствия). Какое там - все та же трещетка, абы чего написать, авось что-то да в масть. Идите отсюда, вертихвост эдакий 

[Авварон]

Думал, может человек повзрослел (после длительного отсутствия). Какое там - все та же трещетка, абы чего написать, авось что-то да в масть. Идите отсюда, вертихвост эдакий 

О_о

[Racheengel]

а если сделать ключи в виде "пути"?

objectPtr->GetParamValue("MATERIAL_ID2/DIFFUSE_ID");

[Bepec]

Так это будут те же ключи
И проблему igors с тем, что ему переписывать всё придётся не исправят

to Igors я тебя тоже, !адекват, люблю

[Igors]

а если сделать ключи в виде "пути"?

objectPtr->GetParamValue("MATERIAL_ID2/DIFFUSE_ID");

Проблема та же - где взять MATERIAL_ID2? (в смысле чему оно соответствует) Только здесь еще отягощенная строками.

Владелец - конечно объект. Юзер указывает что желает еще материал, может еще и еще (хотя и редко). Ну накидать их всех в дерево - дело несложной техники, имеющиеся структуры данных это позволяют. Но как их затем удобно использовать?

[Racheengel]

Проблема та же - где взять MATERIAL_ID2? (в смысле чему оно соответствует)

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

const QByteArray MATERIAL_ID2 = "Material2";

(например)