1

我正在尝试学习一些 c++，首先我创建了一些方法来处理控制台的输出和读取。我有两个主要问题，在代码中标记，操作/访问通过引用传入的字符串的 std::vector 中的值。

下面的方法接受一个问题（标准字符串）来询问用户和一个向量标准字符串，其中包含用户认为可接受的响应。为了学习，我还想访问向量中的字符串并更改其值。

std::string My_Namespace::My_Class::ask(std::string question, std::vector<std::string> *validInputs){
    bool val = false;
    std::string response;
    while(!val){
        //Ask and get a response
        response = ask(question);
        //Iterate through the acceptable responses looking for a match
        for(unsigned int i = 0; i < validInputs->size(); i++){
            if(response == validInputs->at(i)){
                ////1) Above condition always returns true/////
                val = true;
                break;
            }
        }
    }
//////////2) does not print anything//////////
println(validInputs->at(0)); //note the println method is just cout << param << "\n" << std::endl
//Really I want to manipulate its value (not the pointer the actual value)
//So I'd want something analogous to validInputs.set(index, newVal); from java
///////////////////////////////////////////
}

几个额外的问题：

3）我在向量上使用 .at(index) 来获取值，但我已经读到应该使用 [] 来代替，但是我不确定它应该是什么样子（validInputs[i] 没有' t 编译）。

4）我假设由于不需要深拷贝，因此像上面那样传递指向向量的指针是一种很好的做法，有人可以验证吗？

5) 我听说 ++i 在循环中比 i++ 更好，是真的吗？为什么？

4

4 回答 4

2

3)在这种情况下，使用atand应该没有显着差异。operator[]请注意，您有一个指向向量的指针，而不是一个向量（也不是对向量的引用），因此您将不得不使用(*validInputs)[i]或validInputs->operator[](i)使用运算符重载。validInputs->at(i)如果您不想使用这些其他方法中的任何一种，则使用很好。（如果参数超出数组边界，该at方法将抛出异常，而当参数超出数组边界时，该operator[]方法具有未定义的行为。由于operator[]跳过边界检查，如果您知道一个事实，i它会更快在向量的范围内。如果不确定，请使用at并准备捕获异常。）

4）指针很好，但引用会更好。而且，如果您不修改方法中的向量，则最好使用对 const 向量的引用 ( std::vector<std::string> const &)。这可确保您不会被传递空指针（引用不能为空），同时还确保您不会意外修改向量。

5）通常是。 i++是后自增，表示必须复制原值，然后自i增，返回原值的副本。 ++i递增i然后返回i，所以它通常更快，尤其是在处理复杂的迭代器时。编译器unsigned int应该足够聪明，可以意识到预增量会很好，但是++i如果您不需要i.

于 2013-01-18T19:56:10.990 回答
2

我会使用对 const 的引用，并且std::find. 请注意，我还通过引用获取字符串（否则它会被深度复制）：

std::string My_Class::
ask (const std::string& question, const std::vector<std::string>& validInputs)
{
    for (;;) {
        auto response = ask (question);
        auto i = std::find (validInputs.begin (), validInputs.end (), response);

        if (i != validInputs.end ()) {
             std::cout << *i << '\n'; // Prints the value found
             return *i;
        }
    }
}

如果您不了解代码，请阅读迭代器。当然，如果您需要，请随时提出其他问题。

于 2013-01-18T20:00:28.963 回答
1

我不打算讨论第 1 点和第 2 点，因为我们不知道您在做什么，我们甚至看不到askand的代码println。

我在向量上使用 .at(index) 来获取值，但我读过应该使用 [] 来代替，但是我不确定它应该是什么样子（validInputs[i] 无法编译）。

下标访问和at成员函数是不同的东西。它们为您提供了相同的东西，即对索引元素的引用，但如果您传递越界索引，它们的行为会有所不同：at将引发异常，同时[]会调用未定义的行为（就像内置数组一样）。在指针上使用[]有点难看，(*validInputs)[i]但您确实应该尽可能避免使用指针。

我假设由于不需要深层复制，因此如上所述传递指向向量的指针是一种很好的做法，有人可以验证吗？

深拷贝是不必要的，但指针也是如此。你需要一个参考，const因为我认为你不应该修改那些：

ask(std::string const& question, std::vector<std::string> const& validInputs)

我听说 ++i 在循环中比 i++ 更好，这是真的吗？为什么？

在一般情况下它是正确的。这两个操作不同，++i递增i并返回新值，同时i++递增i但返回递增前的值，需要临时保持并返回。对于ints 这几乎无关紧要，但对于潜在的胖迭代器，如果您不需要或不关心它的返回值，则预增量更有效，也是更好的选择。

于 2013-01-18T20:01:09.717 回答
0

要回答问题 1 和 2，我们可能需要更多信息，例如：您是如何初始化 validInputs 的？问的来源是什么？

3）首先取消引用指针，然后索引向量：

(*validInputs)[i]

4）参考被认为是更好的风格。特别是代替从不为 NULL 的指针。

5）对于整数，没关系（除非您评估表达式的结果）。对于其他对象，使用重载的 ++ 运算符（例如迭代器）可能会更好地使用 ++i。但实际上，对于 ++ 运算符的内联定义，它可能会针对相同的代码进行优化。

于 2013-01-18T20:00:13.927 回答