如果字符串向量包含 char 'p'，我如何检查 c++

Posted 2023-02-22

【中文标题】如果字符串向量包含 char \'p\'，我如何检查 c++

【英文标题】：How do I check in c++, if a vector of strings contains char 'p'如果字符串向量包含 char 'p'，我如何检查 c++

【发布时间】：2019-09-12 11:37:23

【问题描述】：

1) 假设我有一个Wizards 的向量v（Wizard 有一个名字、姓氏、字符串向量，其中包括他/她参加的科目，以及他/她所属的房子）

2) 我有一个空向量cpy，我想复制那些参加某个主题的巫师，其中有一个字母“p”。

就我而言，我只想复制 Laura，因为她参加体育运动，这是唯一包含“p”的主题。

//wizard.cpp
Wizard::Wizard(string name, string lastname, Vector<string> subjects, Haus haus) :
  namename, lastnamelastname, subjectssubjects, haushaus

  if (name.empty() || lastname.empty() )
    throw runtime_error("name or lastname wrong");
  


string Wizard::get_name() const 
  return name;


string Wizard::get_lastname() const 
  return lastname;


Vector<string> Wizard::get_subjects() const 
  return subjects;


Haus Wizard::get_haus() const 
  return haus;


Vector<Wizard> v;
Wizard harry("Harry", "Potter", "magic", "music", Haus::Gryffindor);
Wizard ron("Ron", "Weasley", "magic", "dancing", Haus::Gryffindor);
Wizard hermione("Hermione", "Granger", "magic", "defence", Haus::Gryffindor);
Wizard laura("Laura", "Someone", "running", "sports", Haus::Slytherin);

v.push_back(harry);
v.push_back(ron);
v.push_back(hermione);
v.push_back(laura);


Vector<Wizard> cpy;

// v is the original vector of all wizards

copy_if(v.begin(), v.end(), back_inserter(cpy), [](const Wizard& w) 
  return(any_of(w.get_subjects().begin(), w.get_subjects().end(), [](const string& s) 
    return s.find('p') != string::npos;
   ));
 );

我最终得到退出代码 11

【问题讨论】：

最重要的是：韦斯莱！

w.get_subjects() 是按引用返回还是按值返回？提交minimal reproducible example。

没有看到Wizard的定义，无法准确回答。不过，我自己也是个巫师，我的占卜技巧告诉我get_subjects 按值返回一个容器。这是真的吗？

@kawillzocken 实际上首先比较两个迭代器是 UB。但我和 Angew 都没有解释是有原因的：(a) 我们不希望奖励发布不完整的问题，并且 (b) 我们不知道问题尚未解决，并且 (c) 答案不在 cmets 部分。

它仍然不是minimal reproducible example。我们确实有足够的信息来回答这个问题（事实上，正如您所见，我们之前已经猜到了足够多），但您确实应该为您的问题提供minimal reproducible example。

【参考方案1】：

你在任何地方都使用值，包括get_subjects()的返回类型。

因此，下面的两个迭代器：

w.get_subjects().begin(), w.get_subjects().end()

指的是向量的完全独立的、不相关的副本。

将迭代器与两个不相关的向量进行比较具有未定义的行为，这永远不会起作用。

相反，您的访问器应该通过 (const) 引用返回。

【参考方案2】：

首先声明函数get_subjectslike

const Vector<string> & Wizzard::get_subjects() const 
  return subjects;

否则在此算法调用中

any_of(w.get_subjects().begin(), w.get_subjects().end(),...);

begin 和 end 返回不同范围（向量）的迭代器。