Есть примерно такой код

SomeClass::Func()
{
   memder->SomeFunction();
}


member::SomeFunction()
{
 // SomeClass  - SingleTone
  SomeClassHolder()->Instance()->SomeOtherFunction();
}

Раньше был один поток и такая конструкция работала, теперь появилась необходимость распределить нагрузку на несколько потоков, следовательно конструкция типа SomeClassHolder()->Instance()->SomeOtherFunction() не подходит. пока вижу 2 варианта решения.

1)Написать Singletone — который в зависимости от того из какого потока его вызвали будет возвращать соответствующий указатель.
— количество потоков константно не изменяется во время работы программы.
2)Передавать нужный объект в качестве параметра SomeOtherFunction(SomeClass &someclass);

Интересно может есть какиенить подводные камни которых я не вижу или другие интересные решения ?

Здравствуйте, ё-лка, Вы писали:

ЁЛ> SomeClassHolder()->Instance()->SomeOtherFunction();
Еще можно сделать SomeClass потоко-безопасным, и пусть все потоки работают с одним экземпляром.
Если нельзя — то потоково-локальный синглтон самое лучшее ИМХО, потребуется минимум изменений в существующем коде

Здравствуйте, ё-лка, Вы писали:

ЁЛ>1)Написать Singletone — который в зависимости от того из какого потока его вызвали будет возвращать соответствующий указатель.
ЁЛ>— количество потоков константно не изменяется во время работы программы.
Очень неудобно отлаживать.

ЁЛ>2)Передавать нужный объект в качестве параметра SomeOtherFunction(SomeClass &someclass);

ИМО, это вариант получше будет.

Здравствуйте, AndrewJD, Вы писали:

ЁЛ>>2)Передавать нужный объект в качестве параметра SomeOtherFunction(SomeClass &someclass);
AJD>ИМО, это вариант получше будет.
+1
С одной поправкой.
Передавать нужно не SomeClass &someclass, а Context* context.
Где Context примерно такого вида:

struct Context
{
    SomeClass* const someclass;
};

Тогда если понадобится протащить еще и SomeElseClass то просто добавляем его в контекст.
И вобще Re[3]: Singleton против static class c# 2

Автор: WolfHound
Дата: 16.05.06

А людей которые говорят что синглетоны рулят не слушай ибо синглетоны почти всегда создают проблемы (например в твоем случае) и никогда ничего не дают.

Посмотри thread local storage: TlsAlloc , TlsSetValue, TlsGetValue

Здравствуйте, WolfHound, Вы писали:

WH>С одной поправкой.
WH>Передавать нужно не SomeClass &someclass, а Context* context.

WH>А людей которые говорят что синглетоны рулят не слушай ибо синглетоны почти всегда создают проблемы (например в твоем случае) и никогда ничего не дают.

Сейчас как раз занимаюсь тем что избавляюсь от синглетона который намертво связал несколько разных компонентов.
Так народ чтобы хоть как-то их по отдельности протестировать во всю использовал нездоровые хаки с дефайнами, неймспейсами и включением одного cpp файла в другой.

Сначала я хотел отрихтовать реализацию синглетона, чтобы можно было подпихнуть mock объект. Но теперь пришел к выводу что дешевле будет выкинуть его нах. и протянуть контест через пяток классов.
Так что, кто ищет изощренный трах — синглетоны ваш выбор!

Здравствуйте, WolfHound, Вы писали:

Спасибо.

Здравствуйте, WolfHound, Вы писали:
WH>А людей которые говорят что синглетоны рулят не слушай ибо синглетоны почти всегда создают проблемы (например в твоем случае) и никогда ничего не дают.
В некоторых случаях singleton вполне уместен, была ветка

Автор: Graf Alex
Дата: 14.06.07

.

2 ё-лка:
Singleton в рамках потока реализуется с помощью thread local storage (TLS), qqqqq уже написал.

Здравствуйте, Пётр Седов, Вы писали:

ПС>В некоторых случаях singleton вполне уместен, была ветка

Автор: Graf Alex
Дата: 14.06.07

.
Конкретнее.