这个头文件怎么了

Question

这是一个c ++函数头文件。 它提供了大量的随机错误。 我知道这很明显，但是我以前从未只制作没有类的头文件。 它没有链接的cpp文件。

#include <vector>
#include <sstream>
#include <string>

#ifndef SPLIT_H
#define SPLIT_H
 void std::vector<std::string> &split(const std::string &s, char delim, std::vector<std::string> &elems);
 void std::vector<std::string> split(const std::string &s, char delim);
#endif

void std::vector<std::string> &split(const std::string &s, char delim, std::vector<std::string> &elems) {
    std::stringstream ss(s);
    std::string item;
    while(std::getline(ss, item, delim)) {
        elems.push_back(item);
    }
    return elems;
}


void std::vector<std::string> split(const std::string &s, char delim) {
    std::vector<std::string> elems;
    return split(s, delim, elems);
}

Answer 1

似乎您正在返回两个值， void和vector<std::string> 。 尝试删除函数开头的void 。

Answer 2

除了@Default的（正确和有效）观察之外，如果要将这些函数定义放在标头中，则几乎可以肯定地希望将它们标记为inline 。 否则，如果/如果您将标头包含在多个源文件中，并尝试将它们链接在一起，则会违反一个定义规则。 您的链接器可能会允许它，但是当然不能保证。 @You的建议是显而易见的替代方案：只需将类定义放在标头中，然后将函数定义放在它们自己的源文件中。

Answer 3

1. void std::vector<std::string> &是错误的。 您是说const std::vector<std::string> &还是只是std::vector<std::string> & ？
2. 如果将此头文件包含在多个源文件中，则可能会收到有关重新定义符号的链接器错误。 您应该在源文件中定义功能或将其内联。