首页
学习
活动
专区
圈层
工具
发布
首页
学习
活动
专区
圈层
工具
MCP广场
社区首页 >问答首页 >删除C++中向量中的第一个最小数字(并保持顺序)

删除C++中向量中的第一个最小数字(并保持顺序)
EN

Stack Overflow用户
提问于 2019-11-02 14:17:44
回答 3查看 378关注 0票数 0

我有一个包含{1, 0, 0, 0, 0, 0 ,0}vector<int> number_vector。我需要迭代这个number_vector,例如4次,并在每次迭代中删除第一个最小的数字,即在第一次迭代中,我将删除索引1处的值0,在下一次迭代中,我将删除索引1处的0,依此类推。

代码语言:javascript
运行
复制
int n = 7;

int d = 4;

vector<int> number_vector{1, 0, 0, 0, 0, 0 ,0};

for (int counter = 0; counter < n - d; counter++)
{
    int index = distance(number_vector.begin(), min_element(number_vector.begin(), number_vector.end()));

    if (index != number_vector.size() - 1)
    {
        number_vector[index] = move(number_vector.back());
    }

    number_vector.pop_back();

    // number_vector.erase(number_vector.begin() + index);
}

问题是,如果我运行上面的代码,在最后number_vector应该有{1, 0, 0, 0},而它应该有{1, 0, 0},对于其他情况,如n = 4d = 2number_vector{3, 7, 5, 9},最终的number_vector具有正确的值,即79。有什么建议吗?

EN

回答 3

Stack Overflow用户

发布于 2019-11-02 14:28:39

首先,你要迭代三次,而不是四次。其次,如果向量不是必需的,您可以只使用map并弹出前端迭代器,因为它总是最低的值。最后,不需要交换或距离,只要擦除min_element的结果,如果它不是无效的。

票数 1
EN

Stack Overflow用户

发布于 2019-11-02 15:03:10

当您使用n=7和d=4从0迭代到n-d时,您将从counter=0迭代到计数器< 7-4,即3。因此,您的循环将使用值0、1和2迭代3次。这将从number_vector中删除三个零。因此,您的代码的行为符合预期。

我认为你想要的是从0迭代到d,而且你使用索引会使代码变得不必要的复杂。您可以直接使用迭代器,如下所示。

代码语言:javascript
运行
复制
for (int counter = 0; counter < d; counter++)
{   
    *min_element(number_vector.begin(), number_vector.end()) = *number_vector.rbegin();
    number_vector.pop_back();
}   
票数 1
EN

Stack Overflow用户

发布于 2019-11-02 15:48:06

你的d是不是代表了向量的最小pop次数?

然后将counter < n - d修改为counter < d,它将具有{1, 0, 0}

11.04如果您想保留订单,您可以修改

代码语言:javascript
运行
复制
for (int counter = 0; counter < d; ++counter)
{
    auto iter = min_element(number_vector.begin(), number_vector.end());
    number_vector.erase(iter);
}

PS:std::list也许是更好的选择?

票数 1
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/58668724

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档