Adding a map of string vector and different datatype vector

Question

As suggested in https://stackoverflow.com/a/44884240/3782963. I created a method of return type std::map<std::string, Spikes::data> get_data1(). I create a struct called data:

struct data {

            std::vector<Eigen::MatrixXd> samples;
            std::vector<std::string> sample_names;
            int index = -1;
        };

and placed it in my namespace and out of the class FileReader.

My method looks like:

std::map<std::string, Spikes::data> FileReader::get_data1() {
    std::map<std::string, Spikes::data> content;

    std::vector<Eigen::MatrixXd> samples;
    std::vector<std::string> sample_names;

    for (auto i : list_dir()) {
        if (contains_number(i)) {
            samples.push_back(load_csv(location + "/" + i));
            sample_names.push_back(i);
        }
    }

    content.emplace(std::make_pair("samples", Spikes::data{samples,0}));
    content.emplace(std::make_pair("sample_names", Spikes::data{sample_names,1}));

    return content;

And in my main.cpp I am calling this method as:

for (auto i:a.get_data1()["samples"].samples){
        std::cout<<i;
    }

But I get the following errors:

/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.cpp: In member function 'std::map<std::basic_string<char>, Spikes::data> Spikes::FileReader::get_data1()':
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.cpp:154:69: error: no matching function for call to 'Spikes::data::data(<brace-enclosed initializer list>)'
     content.emplace(std::make_pair("samples", Spikes::data{samples,0}));
                                                                     ^
In file included from /cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.cpp:3:0:
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note: candidate: Spikes::data::data()
     struct data {
            ^
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note:   candidate expects 0 arguments, 2 provided
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note: candidate: Spikes::data::data(const Spikes::data&)
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note:   candidate expects 1 argument, 2 provided
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note: candidate: Spikes::data::data(Spikes::data&&)
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note:   candidate expects 1 argument, 2 provided
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.cpp:155:79: error: no matching function for call to 'Spikes::data::data(<brace-enclosed initializer list>)'
     content.emplace(std::make_pair("sample_names", Spikes::data{sample_names,1}));
                                                                               ^
In file included from /cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.cpp:3:0:
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note: candidate: Spikes::data::data()
     struct data {
            ^
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note:   candidate expects 0 arguments, 2 provided
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note: candidate: Spikes::data::data(const Spikes::data&)
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note:   candidate expects 1 argument, 2 provided
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note: candidate: Spikes::data::data(Spikes::data&&)
/cygdrive/c/Users/aksha/Box Sync/MyDrive/Projects/Akshay-GitHub/libSpikes/libSpikes/src/Core/FileReader.h:13:12: note:   candidate expects 1 argument, 2 provided

I have no clue as to what is going on.

I have even tried doing:

content["samples"] = Spikes::data{samples,0};
content["sample_names"] = Spikes::data{sample_names,1};

I still get the same error. Any help would be highly appreciated.

Dean Seo · Accepted Answer · 2017-07-05 03:40:59Z

1

As suggested in https://stackoverflow.com/a/44884240/3782963. I created a method of return type std::map get_data1().

The suggested answer clearly states that you need to implement constructors for data:

struct data 
{
    // constructors... <- here

    std::vector<Eigen::MatrixXd> samples;
    std::vector<std::string> names;
    int index = -1;
};

@SamVarshavchik described the reason well.

I assume your intention would be having constructors as follows:

// Constructors
data(std::vector<Eigen::MatrixXd> && rhs, int idx) 
{
    samples = std::move(rhs);
    index = idx;
}

data(std::vector<std::string> && rhs, int idx) 
{
    samples = std::move(rhs);
    index = idx;
}

content.emplace(std::make_pair("samples", Spikes::data{ std::move(samples), 0}));
content.emplace(std::make_pair("sample_names", Spikes::data{ std::move(sample_names), 1}));

edited Jul 5, 2017 at 3:40

answered Jul 4, 2017 at 3:08

Dean Seo

5,7433 gold badges36 silver badges53 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Dean Seo Over a year ago

@akshay You put those 2 constructors in data, and replace your emplacing with the last 2 lines.

Akshay Over a year ago

Awesome! I was able to get it. But I am not able to get the data unless I do something like this - for (auto i:a.get_data1()){ for (auto b:i.second.sample_names){ std::cout<<b; } } Is there a was to loop over the content by name?

Dean Seo Over a year ago

@akshay Do you mean key by by name (as in Key-Value in map)? Then Have you tried i.first?

Akshay Over a year ago

aam! something like i:a.get_data1()["samples"].samples, if I loop over this the IDE is able to detect that there is an Eigen matrix, but using that is giving me huge lines of errors! I did - for (auto i:a.get_data1()["samples"].samples){ std::cout<<i; }

Dean Seo Over a year ago

@akshay for(auto & i : a.get_data1()) { for(auto & item : i.second.samples) { std::cout << item } };

|

Sam Varshavchik · Accepted Answer · 2017-07-04 02:58:13Z

1

When you use value initialization for a simple structure/aggregate/etc...:

Spikes::data{samples,0}

You must initialize every member of this structure.

Your data class contains three members, the first two of them are vectors, and the third class member an int. Obviously initializing a std::vector of std::strings using a plain integer 0 is not going to go anywhere. This 0 will be interpreted as a value to initialize the second member of your struct, since it's the second value in the initialization list. The second member of your struct is a std::vector of std::strings, and the compiler will, essentially, attempt to initialize the second member accordingly, something like this:

std::vector<std::string> sample_names{0};

This is what your attempt at initialization boils down to, and this is not going to go very far...

You cannot arbitrarily skip class members, when using value initialization. You must initialize every class member, in order. In this case, your likely intent is probably something along these lines:

Spikes::data{samples,{},0}

Thusly initializing the second vector with an empty sub-initialization list. Similarly, your second attempt at initialization:

Spikes::data{sample_names,1}

It suffers from the same problem. Your likely intentions here are probably:

Spikes::data{{}, sample_names,1}}

i.e. initializing the first std::vector member with an empty initialization list, and then initializing the second std::vector accordingly.

answered Jul 4, 2017 at 2:58

Sam Varshavchik

119k6 gold badges109 silver badges164 bronze badges

2 Comments

Akshay Over a year ago

So doing content.emplace(std::make_pair("samples", Spikes::data{samples,{},0})); and content.emplace(std::make_pair("sample_names", Spikes::data{{},sample_names,1})); should solve the problem right. but I get the same error.

Sam Varshavchik Over a year ago

That should be insert instead of emplace.

Collectives™ on Stack Overflow

Adding a map of string vector and different datatype vector

2 Answers 2

6 Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related