Повысьте странность - PullRequest
       28

Повысьте странность

0 голосов
/ 30 ноября 2010

Я пробую Fusion и обнаружил что-то очень странное ... Вот код ... Я выделил проблемный код с помощью // ############ УСТРАНЕНИЕ НЕИСПРАВНОСТЕЙ #### ##

#include <tr1/cstdint>
#include <tr1/functional>
#include <string>
#include <iostream>

//  #define FUSION_MAX_VECTOR_SIZE 64

#define BOOST_MPL_LIMIT_STRING_SIZE 128

#include <boost/type_traits.hpp>
#include <boost/mpl/string.hpp>
#include <boost/fusion/algorithm.hpp>
#include <boost/fusion/tuple.hpp>
#include <boost/fusion/container/vector.hpp>
#include <boost/fusion/container/generation.hpp>
#include <boost/fusion/container/generation/vector_tie.hpp>

typedef std::tr1::int32_t int32;

typedef std::tr1::int64_t int64;

template < class type_const_ref >
struct remove_const_reference
{
    typedef typename boost::remove_reference < type_const_ref >::type type_const;
    typedef typename boost::remove_const < type_const >::type type;
};

template < class T >
class MetaClass;

namespace fusion = boost::fusion;

template < class T >
struct ConstRefFieldMap
{
    typedef typename MetaClass < T >::FieldNames FieldNames;
    typedef typename MetaClass < T >::ConstRefFields ConstRefFields;
    typedef typename boost::fusion::result_of::zip < FieldNames const, ConstRefFields const >::type type;
};

template < class T >
static typename MetaClass < T >::FieldNames fieldNames()
{
    return typename MetaClass < T >::FieldNames();
}

template < class T >
static typename MetaClass < T >::ConstRefFields constRefFields(T const &obj)
{
    return MetaClass < T >::constRefFields(obj);
}

template < class T >
static typename ConstRefFieldMap < T >::type const constRefFieldMap(T const &obj)
{
    return boost::fusion::zip(fieldNames < T >(), constRefFields(obj));
}

class Currency
{
    private:
        typedef MetaClass < Currency > Meta;

        friend class MetaClass < Currency >;

    private:
        std::string m_isoCode;

        int32 m_rank;

    public:
        Currency(std::string const &isoCode, int32 const rank)
        : m_isoCode(isoCode)
        , m_rank(rank)
        {
        }

        std::string const& getIsoCode() const
        {
            return m_isoCode;
        }

        int32 const getRank() const
        {
            return m_rank;
        }

    private:
        void setIsoCode(std::string const &isoCode)
        {
            m_isoCode = isoCode;
        }

    public:
        void setRank(int32 rank)
        {
            m_rank = rank;
        }
};

template <>
class MetaClass < Currency >
{
    public:
        typedef Currency data_type;

    public:
        typedef std::string IsoCodeType;

        typedef int32 RankType;

        typedef boost::fusion::vector <
            boost::mpl::string < 'i', 's', 'o', 'C', 'o', 'd', 'e' >
        ,   boost::mpl::string < 'r', 'a', 'n', 'k' >
        > FieldNames;

        typedef boost::fusion::vector <
            IsoCodeType &
        ,   RankType &
        > MutableRefFields;

        typedef boost::fusion::vector <
            IsoCodeType const &
        ,   RankType const &
        > ConstRefFields;

        static MutableRefFields mutableRefFields(Currency &obj)
        {
            return MutableRefFields(obj.m_isoCode, obj.m_rank);
        }

        static ConstRefFields constRefFields(Currency const &obj)
        {
            return ConstRefFields(obj.m_isoCode, obj.m_rank);
        }

};

template < class T, class U >
static typename ConstRefFieldMap < T >::type const constRefFieldMapTest(T const &obj, U const &u)
{
    return boost::fusion::zip(fieldNames < T >(), u);
}

int main()
{
    Currency const EUR("EUR", 500);
    using boost::fusion::any;

    {
        std::cout << boost::fusion::at_c < 0 >(constRefFields(EUR)) << " : " << boost::fusion::at_c < 1 >(constRefFields(EUR)) << std::endl;
        ConstRefFieldMap < Currency >::type const &fm = boost::fusion::zip(fieldNames < Currency >(), constRefFields(EUR));
// ############ TROUBLE HERE ######
//        ConstRefFieldMap < Currency >::type const &fm = constRefFieldMap(EUR);
// ############ TROUBLE HERE ######
        {
            {
                typedef boost::fusion::result_of::at_c < ConstRefFieldMap < Currency >::type, 0 >::type field_value_type;
                field_value_type const v = boost::fusion::at_c < 0 >(fm);
                typedef boost::fusion::result_of::at_c < field_value_type, 0 >::type field_name_type;
                field_name_type const n = boost::fusion::at_c < 0 >(v);
                typedef boost::fusion::result_of::at_c < field_value_type, 1 >::type field_data_type;
                field_data_type const d = boost::fusion::at_c < 1 >(v);
                std::cout << boost::mpl::c_str < remove_const_reference < field_name_type >::type >::value << " : " << d << std::endl;
            }

            {
                typedef boost::fusion::result_of::at_c < ConstRefFieldMap < Currency >::type, 1 >::type field_value_type;
                field_value_type const v = boost::fusion::at_c < 1 >(fm);
                typedef boost::fusion::result_of::at_c < field_value_type, 0 >::type field_name_type;
                field_name_type const n = boost::fusion::at_c < 0 >(v);
                typedef boost::fusion::result_of::at_c < field_value_type, 1 >::type field_data_type;
                field_data_type const d = boost::fusion::at_c < 1 >(v);
                std::cout << boost::mpl::c_str < remove_const_reference < field_name_type >::type >::value << " : " << d << std::endl;
            }
        }
    }
}

Я получаю значения мусора или SIGSEGV, если я использую функцию constRefFieldMap (). Если я вызываю boost :: fusion :: zip напрямую, он работает отлично. Вот вывод ...

EUR : 500
isoCode : EUR
rank : 500

Я уже рассматривал этот вопрос ранее ... Я сталкиваюсь с той же проблемой здесь ???

РЕДАКТ. 1:

Представляя пример того, что я пытаюсь сделать ...

На самом деле ... Я пытаюсь написать такой код.

MetaObject < Currency const > EUR_META(make_meta_object(EUR));
std::cout  << get_field < std::string >("isoCode", EUR_META.constRefFieldMap()) << std::endl;

MetaObject < Currency > GBP_META(make_meta_object(GBP));
MutableRefFieldMap < Currency >::type const &fm = GBP_META.mutableRefFieldMap();
std::cout  << set_field("rank", fm, 497) << std::endl;

Средства доступа и модификаторы, которые я могу вызывать по именам полей ...

Я планирую написать анализатор спирта для анализа JSON & XML и создания объектов ... с некоторой помощью моих генераторов кода. Основная идея состоит в том, чтобы избежать генерации кода синтаксического анализа для каждого объекта, но только для тех объектов, которые используются, и, следовательно, уменьшить двоичный размер. У меня сейчас тысячи объектов.

У меня сейчас это работает.

1 Ответ

7 голосов
/ 01 декабря 2010

К сожалению, этот вопрос не привлек больше внимания, но я думаю, что большой кусок кода не сильно помог (это и тот факт, что метапрограммирование шаблонов не так популярно).

В любом случае, вы правы, это похожая проблема.

Проблема в том, что views в fusion не копируют аргументы, они только сохраняют ссылки на них. Это позволяет вам изменять исходные аргументы по их промежуточному соединению.

Проблема в том, что в C ++ вам разрешено привязывать временную ссылку к const-ссылке, а временная жизнь увеличивается до той же, что и у ссылки. Однако это поведение не является переходным, что вызвало кучу неприятностей. (Clang попытается диагностировать эти ситуации, к сожалению, первый патч не удался: p)

Итак, ваша проблема находится в одной строке:

return boost::fusion::zip(fieldNames < T >(), constRefFields(obj));
  • fieldNames<T>() создает временный объект, связанный с const-ссылкой, его время жизни увеличивается до конца выражения: ;
  • когда вы возвращаете представление, временное время жизни истекло, вы держитесь за висячую ссылку

Быстрое исправление: у fieldNames<T>() есть локальная статическая переменная и возвращается ссылка на эту переменную, это исправит проблему времени жизни.

Я до сих пор не понял, что вы пытались сделать, поэтому я не могу дать "более разумный" совет:)

...