Russian Qt Forum

Добрый день

Сначала сам код

C++ (Qt)
template <class T>
struct ID {
//...
	static ID<T> statID;
};
 
template<class T> ID<T> ID<T>::statID;
 
template<class T>
struct Object {
	Object( ID <T> & parent );
// ...
 
// data
	ID<T> mID;
};
 
struct Object_A : public Object<Object_A> {
// ...
};
 
struct Object_B : public Object<Object_B> {
//...
};
 
template<class T>
struct Container : public T {
// ...
};
 
template<class T>
struct Layer : public Container<T> {
//...
};
 
typedef Layer <Object_A> Layer_A;
typedef Layer <Object_B> Layer_B;
 

При переходе с gcc на clang эта конструкция заклинила - сначала на компиляции, а потом на линковке (unresolved statID). Я сделал "скелетик" - переписал структуру классов в тестовый проект, вот это и есть код выше. Впрочем мне это не помогло - тестовый почему-то линкуется отлично  :)

Вопрос не корректен..(

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

C++ (Qt)
struct Object_A : public Object<Object_A> 
 

Так делать можно только тогда, когда размер Object<T> известен на стадии компиляции (и не зависит от T). Иначе получите зацикливание..

Прошу пояснить:

- чем некорректен вопрос ???  Во всяком случае это пример из реальной практики

- в чем "говнокодистость"? Может Вас смутило struct - так это я написал, в оригинале там class, у каждого куча методов (даже есть virtual'ы), и все оформлено очень аккуратно. 

- в чем некорректность с точки зрения языка? Темплейт "на себя" - ну это даже я знаю :) Да, надо соблюдать осторожность, но так делать не запрещено.

- и наконец - ну а как же писать правильно? Вот Вы объявили это говнокодом (чужую корову легко охаять), ну хорошо. Допустим (просто допустим) Вы переделали этот код, создали крепкую архитектуру, в конце-концов - работает. Приходит другой знаток, и вот, так же как Вы, уверенно заявляет: "говнокод, снести нахрен". Или Вы думаете что с Вашим кодом этого никогда не случится? Потому что он будет куда легче и прозрачнее для понимания? Я в этом сильно сомневаюсь :) 

---------------

Понятно почему из сотни прочитавших проголосовало лишь двое (пока) проголосовало. Неудобно выбирать "долго вникать" - выходит не знаю языка, а сказать "легко" - неправда. Лично я, посмотрев на этот код 5-10 минут не смог осознать "конструкцию" и понять (ну хотя бы примерно) что он делает. Пришлось отложить свои дела (которые я считал важными) и углубиться в изучение. Через полтора-два дня начало доходить в чем был авторский замысел. Он совсем не глуп, и даже по-своему красив. А обычно реакция переделывающего не очень приятная, напр
(это в литературной форме :))

Ради ненужного ответа на ненужный вопрос вникать в шаблоны - нафиг надо. Попусту, без выгоды для себя, никто не любит утруждаться :)

Ага, значит утруждаться придется (еще и как) - так почему же Вы не нажали заготовленную бубочку "придется долго вникать", а вместо этого свистите про "ненужный" ответ/вопрос ? :)

-------

Код этот вполне успешно работал. Др программисты (в том числе и я) просто "юзали" его не вникая в подробности - в основном нужно было создавать контейнеры и добавлять в них эл-ты нужного типа, это делалось "по образцу" имеющегося кода. Расширение "инстанциации" (напр Object_C, Layer <Object_C>) не понадобилось ни разу. Зато неск других задач (гораздо бОльших) используют эту структуру как базовую. Поэтому снести/переписать ее практически нереально.

Поясняю:

Вы предлагаете поддерживать и сопровождать неработающий код. Какой в этом смысл?


Ну хотя бы в том, что он не работает) Потом, объясните смысл этой конструкции и как она в реальном коде используется:

C++ (Qt)
template <class T>
struct ID {
//...
	static ID<T> statID;
};
 

Вообще или конкретно здесь?
Поскольку я не знаю всего, что за этим кодом скрывается и как это используется, то что-либо посоветовать не могу)

Попробуйте заменить это

C++ (Qt)
struct ID {
//...
	static ID<T> statID;
};
 

на

C++ (Qt)
struct ID {
//...
	static boost::recursive_wrapper<ID<T>> statID;
};
 

А не ответил на опрос потому, что даже не пробовал его рассмотреть. Следовательно любой мой вывод будет недоказанным и необосновенным :)

Откуда такой вывод? См первый пост:
Ну знаете, если при каждой template-проблеме компилятора (а они сыпятся как из ведра), я стану "переписывать все"...

ID обеспечивает глобальный список объектов, по ID можно получить Object и наоборот. Если объект удален из глобального списка (или еще в него не вставлен), то его ID == statID

А в классе ID<T> где-нибудь ещё используется class T и если да, то как?

Да. конечно

C++ (Qt)
template <class T>
struct ID {
//...
        Object <T> * mObject;
        ID<T> * mPrev, * mNext;
 
	static ID<T> statID;
};

Ну и перекрытые операторы -> и др (в стиле шаред пойнтера)

Нет никакого зацикливания.

Есть нюансы, но в целом ничего особенного.
Довольно распространенная практика.

Самый большой косяк, который здесь может вылезти: вижал студия жрет только в путь.

Гцц/шланги могут обидиться, если базовый шаблон тронет лишнее.
Но это будет вполне себе читабельная ошибка компилятора, а не "зацикливание".




Пишите проще:


И раз уж пошла телега за шаблоны, тогда уж сразу по уму: