How to parse a string into vector of tuples with boost spirit?

Question

#include <boost/spirit/include/qi.hpp>
#include <boost/fusion/include/adapt_struct.hpp>
#include <boost/fusion/adapted/std_tuple.hpp>

#include <string>
#include <iostream>
#include <tuple>
#include <vector>

using tpl = std::tuple<int, double, std::string>;

boost::spirit::qi::rule<std::string::iterator, tpl> parse_into_tuple = 
    boost::spirit::qi::int_ >> ',' >> 
    boost::spirit::qi::double_ >> ',' >>
    boost::spirit::lexeme[+boost::spirit::qi::char_ - ';'];

boost::spirit::qi::rule<std::string::iterator, std::vector<tpl>> 
    parse_into_vec = parse_into_tuple % ';';


int main()
{
    std::string s = "1,5.4,abc xyz;2,91.05,qwe jkl";    
    std::vector<tpl> v;
    bool b = boost::spirit::qi::parse(
    s.begin(), s.end(), parse_into_vec, v, boost::spirit::qi::space);
    std::cout << std::boolalpha << b << '\n';
    std::cout << v.size() << '\n';
    for(const auto& t: v)
    {
        std::cout << std::get<0>(t) << ", " << std::get<1>(t) << ", " << std::get<2>(t) << '\n';
    }
}

The output now is "true" and 0 (size of the vector). My expected output is size 2. Furthemore, if I use phrase_parse instead of parse, it doesn't compile. What is my mistake and how do I achieve the expected result?

Answer 1

Fix the rule definition to correctly define the attribute signature as tpl() instead of tpl:

Live On Coliru

#include <boost/fusion/adapted/std_tuple.hpp>
#include <boost/fusion/include/adapt_struct.hpp>
#include <boost/spirit/include/qi.hpp>

#include <iostream>
#include <string>
#include <tuple>
#include <vector>

using tpl = std::tuple<int, double, std::string>;

boost::spirit::qi::rule<std::string::iterator, tpl()> parse_into_tuple =
    boost::spirit::qi::int_ >> ',' >> boost::spirit::qi::double_ >> ',' >>
    boost::spirit::lexeme[+boost::spirit::qi::char_ - ';'];

boost::spirit::qi::rule<std::string::iterator, std::vector<tpl>() > parse_into_vec = parse_into_tuple % ';';

int main() {
    std::string s = "1,5.4,abc xyz;2,91.05,qwe jkl";
    std::vector<tpl> v;
    bool b = boost::spirit::qi::parse(s.begin(), s.end(), parse_into_vec, v);
    std::cout << std::boolalpha << b << '\n';
    std::cout << v.size() << '\n';
    for (const auto &t : v) {
        std::cout << std::get<0>(t) << ", " << std::get<1>(t) << ", " << std::get<2>(t) << '\n';
    }
}

Prints

true
1
1, 5.4, abc xyz;2,91.05,qwe jkl

BONUS

• Fixing issue with white-space skipping in the way you likely expected (see Boost spirit skipper issues)
• Hiding the skipper choice inside the grammar
• Adding debug support
• Simplifying debug output
• Error checking (also consider >> qi::eoi in the grammar/parse expression)

Live On Coliru

//#define BOOST_SPIRIT_DEBUG
#include <boost/fusion/adapted/std_tuple.hpp>
#include <boost/spirit/include/qi.hpp>
namespace qi = boost::spirit::qi;

using tpl = std::tuple<int, double, std::string>;
using tpls = std::vector<tpl>;

template <typename It = std::string::const_iterator>
struct grammar : qi::grammar<It, tpls()> {
    grammar() : grammar::base_type(start) {
        using namespace qi;

        tuple_ = int_ >> ',' >> double_ >> ',' >> lexeme[+~char_(';')];
        vec_   = tuple_ % ';';

        start  = skip(space) [ vec_ ];

        BOOST_SPIRIT_DEBUG_NODES((start)(vec_)(tuple_))
    }
  private:
    qi::rule<It, tpls()> start;

    using Skipper = qi::space_type;
    qi::rule<It, tpls(), Skipper> vec_;
    qi::rule<It, tpl(), Skipper> tuple_;
};

int main() {
    grammar<> const g;
    for(std::string const s : {
            "1,5.4,abc xyz;2,91.05,qwe jkl",
            "1,5.4,abc xyz;2,91.05,qwe jkl; trailing garbage",
            "1,    \n5.4, abc xyz;",
            })
    {
        auto f = s.begin(), l = s.end();

        std::vector<tpl> v;
        if (parse(f, l, g, v))
        {
            std::cout << v.size() << '\n';
            for (const auto &t : v) {
                std::cout << boost::fusion::as_vector(t) << "\n";
            }
        } else {
            std::cout << "Parse failed\n";
        }

        if (f!=l) {
            std::cout << "Remaining unparsed: '" << std::string(f,l) << "'\n";
        }
    }
}

Prints

2
(1 5.4 abc xyz)
(2 91.05 qwe jkl)
2
(1 5.4 abc xyz)
(2 91.05 qwe jkl)
Remaining unparsed: '; trailing garbage'
1
(1 5.4 abc xyz)
Remaining unparsed: ';'

Answer 2

Omit the boost::spirit::qi::space from the call to parse, you don't have space as a seperator.

You can also drop the lexeme[] directive, because you aren't parsing in with a skip.

phrase_parse requires the rule have a compatible Skipper, which you don't have