当从 C++11 的 std::vector 开始遍历到倒数第二个元素时，应该采用哪种首选风格？
std::vector<const char*> argv;
std::string str;
应该使用更类似于C ++的方法吗？
for (const auto& s: decltype(argv)(argv.begin(), argv.end()-1)) {
str += std::string(s) + ' ';
}
还是更传统的方式更好？
for (size_t i = 0; i < argv.size() - 1; ++i) {
str += std::string(argv[i]);
}
请不要写这个：
for (const auto& s: decltype(argv)(argv.begin(), argv.end()-1)) {
decltype(argv) 是一个 vector， 这将复制大量元素...只是为了避免迭代其中一个元素？这非常浪费。for (size_t i = 0; i < argv.size() - 1; ++i) {
这段代码存在微妙的问题，因为size()是无符号的。所以如果argv恰好为空，argv.size() - 1将会是一个极大的数字，导致你实际上访问了所有这些无效的数组元素，从而导致未定义的行为。
对于迭代器，如果argv.begin() == argv.end()，那么你将无法从end()获取前一个迭代器，因为end()没有前一个迭代器。此时，end() - 1、prev(end())和--end()都已经是未定义的行为。此时，我们甚至无法推断循环将会做什么，因为我们甚至没有一个有效的范围。
我建议使用以下方式：
template <typename It>
struct iterator_pair {
It b, e;
It begin() const { return b; }
It end() const { return e; }
};
// this doesn't work for types like raw arrays, I leave that as
// an exercise to the reader
template <typename Range>
auto drop_last(Range& r)
-> iterator_pair<decltype(r.begin())>
{
return {r.begin(), r.begin() == r.end() ? r.end() : std::prev(r.end())};
}
这使您可以进行以下操作：
for (const auto& s : drop_last(argv)) { ... }
drop_last()始终提供有效范围），从名称上也很清楚它的作用。init，尽管在 C++ 中，这个名称经常用于相当反模式的代码设计。 - biplldecltype的一个通用经验法则是，一般情况下应该避免使用它，特别是在模板之外。我自己花了相当长的时间才理解那段代码。永远不应该在可能被轻松解释为函数调用的上下文中使用decltype。 - Omnifariousdecltype。在这种情况下，我建议使用using或typedef声明以及decltype，这样您就有了类型的本地名称，并使用该名称。此外，更喜欢{...}初始化语法，因为它也更难与函数调用混淆。 - Omnifarious我认为第一种选项有些笨拙难读，但由于这更多是个人偏好的问题，因此我提出了一种替代方法，避免手写循环（并假设 argv.size() >= 1），可能更好的地方在于它减少了打字错误和索引错误的可能性。
#include <numeric>
std::string str;
if (!argv.empty())
str = std::accumulate(argv.begin(), std::prev(argv.end()), str);
accumulate。 - NathanOliver+1，但我会诱惑将 if(!argv.empty()) 语句放入代码中，这样对于那些比阅读文字更擅长阅读代码的人来说就更清晰了。 :-) - Galikgsl::span（等待C++20将其作为 std::span 可能有点长）或类似方法来访问所需的子范围。template <class T>
constexpr gsl::span<T> drop_last(gsl::span<T> s, gsl::span<T>::index_type n = 1) noexcept
{ return s.subspan(0, std::min(s.size(), n) - n); }
for (auto s : drop_last(argv)) {
// do things
}
std::for_each函数。std::for_each(argv.begin(), std::prev(argv.end()), [&](const auto& s){ str += s; });
看起来您正在尝试在容器中输出元素，并在它们之间留有空格。另一种编写方式是：
const char* space = ""; // no space before the first item
for (const char* s : argv) {
str += space;
str += s;
space = " ";
}
针对您的特定示例，您可以使用Google Abseil。
str_join.h标头中的代码示例：
std::vector<std::string> v = {"foo", "bar", "baz"};
std::string s = absl::StrJoin(v, "-");
EXPECT_EQ("foo-bar-baz", s);
EXPECT_EQ 是 Google Test 的宏，意思是 s 等于 "foo-bar-baz"
argv.size() 转换为 int 并使用普通的 for 循环。这种方法在大约 99% 的情况下都能正常工作，易于阅读，不需要深入了解 C++ 的玄学知识。for(int i=0; i<(int)argv.size() - 1; i++)
如果您的容器有超过20亿个元素，您可能会提前知道此情况，这种情况下，只需使用size_t并使用特殊情况检查argv.size() == 0以防止下溢。
std::prev(v.end())而不是v.end()-1。这更符合惯用法，并适用于其他容器。main函数。int main()
{
std::set<int> s = {1, 2, 3, 4};
for (const auto& item: decltype(s)(s.begin(), std::prev(s.end())))
{
std::cout << item << std::endl;
}
std::vector<int> v = {10, 20, 30};
for (const auto& item: decltype(v)(v.begin(), std::prev(v.end())))
{
std::cout << item << std::endl;
}
}
for循环。for (auto iter = argv.begin(); iter != std::prev(argv.end()); ++iter )
{
str += *iter + ' ';
}
或者使用std::for_each。
std::for_each(argv.begin(), std::prev(argv.end()),
[](std::string const& item) { str += item + ' '; });
argv.end()-1和/或argv.size()-1是否有效，否则不要这样做。 - Richard Critteni + 1 < argv.size()- bipllsize()是无符号的，而0ULL - 1非常大？ - chew socksargv的最后一个元素是nulltpr，因为它在execvp(argv[0], (char* const*)&argv[0])中发生了。 - chew socks