Генерация данных класса из вектора

Question

Пожалуйста, подумайте над этим вопросом C ++:

#include <cstdio>
#include <iostream>
#include <vector>
using namespace std;

class ParseURL{
    private:
        int qualify; // 0 means we qualify
    public:
        string node;
        string service;
        bool primary;
        bool health;
        string epoch;
        // methods
        ParseURL(string URL);
        ~ParseURL();
};


ParseURL::ParseURL(string URL){
    vector<string> g2 = {"node", "service", "primary", "health", "epoch"};

    for (string tag : g2){
        auto found = URL.find(tag);
        if ( found != string::npos ){
            auto cut_from = found + 1 + tag.size() ;
            auto meh = URL.substr(cut_from, URL.substr(cut_from).find("&") );
            // is there a way we can avoid these lines below?
            if (tag.find("node",0) == 0){
                this->node = meh;
            } else if (tag.find("service",0) == 0 ){
                this->service = meh;
            } else if (tag.find("epoch",0) == 0) {
                this->epoch = meh;
            } else if (tag.find("health",0) == 0){
                this->health = (meh == "OK");
            } else if (tag.find("primary",0) == 0 ){
                this->primary == (this->node == meh);
            }
        }
    }

}

ParseURL::~ParseURL(){
    cout << "Tearing Down the class\n";
}
int main(){
    char req[] = "GET /register?node=hostrpi3&service=potatoservice&primary=node1&health=OK&epoch=1559345106 HTTP";
    string Request = req;
    Request = Request.substr(Request.find_first_of(" ") );
    Request = Request.substr(0, Request.find(" HTTP"));
    ParseURL *a = new ParseURL(Request);

    cout.width(12);
    cout << "a->node: " << a->node << endl;
    cout.width(12);
    cout << "a->service: " << a->service << endl;
    cout.width(12);
    cout << "a->epoch: " << a->epoch << endl;
    cout.width(12);
    cout << "a->health: " << a->health << endl;
    cout.width(12);
    cout << "a->primary: " << a->primary << endl;

    delete(a);

    return 0;
}

Мне нужно представить объект на основе URL Query Strings, нужные мне теги представлены в векторе g2, и, как вы можете видеть, я делаю камеры данных для ParseURL из этих тегов.

О / п выглядит так:

   a->node: hostrpi3
a->service: potatoservice
  a->epoch: 1559345106
 a->health: 1
a->primary: 0
Tearing Down the class

Хотя эта версия работает, она чувствует, что ее можно значительно улучшить. Несмотря на то, что все это есть в векторе, я постоянно делаю tag.find для каждого из этих атрибутов. Надеюсь, должен быть лучший путь. Не могли бы вы указать в правильном направлении? Спасибо!

Редактировать 1

Спасибо, я обновил конструктор, следуя советам:

ParseURL::ParseURL(string URL){
    char tags[10][100] = {"node", "service", "primary", "health", "epoch"};

    int i = 0;
    for (auto tag : tags){
        auto found = URL.find(tag);
        if ( found != string::npos ){
            auto cut_from = found + 1 + strlen(tag);
            auto tag_info = URL.substr(cut_from, URL.substr(cut_from).find("&") );
            switch (i){
                case 0:
                    this->node = tag_info; 
                    break;
                case 1:
                    this->service = tag_info; 
                    break;
                case 2:
                    this->primary = (this->node == tag_info); 
                    break;
                case 3:
                    this->health = (tag_info == "OK"); 
                    break;
                case 4:
                    this->epoch = tag_info; 
                    break;
            }
        }
    i++;
    }
}

bash, например, предлагает косвенную ссылку на переменные, то есть одна переменная может содержать имя другой переменной:

$ cat indirect.sh 
#!/bin/bash 

values="value1 value2"
value1="this is value1"
value2="this is value2"

for i in $values; do 
    echo "$i has value ${!i}"
done

$ ./indirect.sh 
value1 has value this is value1
value2 has value this is value2

Я надеялся, что для подобных случаев существует нечто подобное, тем не менее, я усвоил здесь ценные уроки. благодарю вас!

Answer 1

Я считаю, что вопрос можно разложить:

• парсинг URL и затем
• присвоение результатов элементам данных объекта.

regex предоставляет компактное и надежное решение для извлечения пар значений ключей URL (параметров) в карту:

#include <iostream>
#include <iterator>
#include <map>
#include <string>
#include <regex>

using namespace std;

int main()
{
    map<string, string> keyValues;
    string s = "GET /register?node=hostrpi3&service=potatoservice&primary=node1&health=OK&epoch=1559345106 HTTP";
    regex keyValueRegex("[?&](\\w+)=(\\w+)");
    auto pairsBegin = std::sregex_iterator(s.begin(), s.end(), keyValueRegex);
    auto pairsEnd = std::sregex_iterator();

    for (std::sregex_iterator i = pairsBegin; i != pairsEnd; ++i)
    {
        std::smatch match = *i;
        keyValues[match[1]] = match[2];
    }

    for (auto p : keyValues)
    {
        cout << p.first << " " << p.second << endl;
    }
}

дает вывод:

epoch 1559345106
health OK
node hostrpi3
primary node1
service potatoservice

Другой подход (для очень детального анализа URL) - использование библиотеки cpp-netlib (предлагается для включения Boost).

Answer 2

Предполагая, что вы хотите сохранить отдельные члены класса (а не одно хранилище значений ключей), вы можете использовать статическую таблицу строк и указателей на члены:

vector<pair<string,string ParseURL::*>> keys={{"node",&ParseURL::node},…};

Но здесь это не работает, потому что типы разные, а обработка не одинакова для каждого.Вместо этого, для такого рода почти параллелизма , синтаксически оптимальным выбором часто является лямбда :

const auto get=[&URL](const string &tag) {
  auto found = URL.find(tag);
  if(found == string::npos) return "";
  auto cut_from = found + 1 + tag.size() ;
  return URL.substr(cut_from, URL.substr(cut_from).find("&") );
};
node=get("node");
service=get("service");
epoch=get("epoch");
health=get("health")=="OK";
primary=get("primary")==node;

Этот стиль имеет то преимущество, что определенно инициализирует health и primary.При наличии достаточных искажений с помощью частного конструктора его можно поместить в список инициализатора члена, что даже лучше.

Answer 3

Я не совсем понял, для чего вы хотите использовать это косвенное обращение в вашем примере, но вот как это можно сделать:

vector<string> values{"value1", "value2"}
map<const string, const string> fields_and_values{{"value1", "this is value1"}, {"value2", "this is value2"}};
for (value: values)
     cout << value << " have value " << fields_and_values[value] << endl;

Этот код позволяет динамически добавлять дополнительные поля.Если у вас есть заранее определенное количество полей, это излишне, и простой массив подойдет.то есть

array<const string, 2> values{"this is value1", "this is value2"};
for (int i = 0; i < 2; ++i)
    cout << "value" << i << " have value " << values[i] << endl;

Надеюсь, что отвечает на ваш вопрос.

В целом о вашем коде:

Почему вы перешли с vector<string> на char[10][100]?(особенно если у вас есть только 5 строк, и ни одна из них не занимает 100 символов?) array<string, 5> - это подход к вашему делу.

Кроме того, не используйте new и delete .. Нет смысла выделять кучу, просто используйте ParseURL parse(request) (и даже если для этого была веская причина - используйте unique_ptr).

И, наконец, если вы не хотите просматривать строку несколько раз, ища разные ключевые слова, вы можете использовать регулярное выражение.