这是识别在实施过程中发现的派生需求的经典示例。我建议你考虑写一个函数来支持它。
改变
if (theString == "Seven" || "seven" || "7")
{
//....
(这不是有效的 c++,因为 if 条件始终为真)
改成
if (0 == compare(theString, "Seven", "seven", "7")
{
//....
并声明和实现类似的东西
// return 0 when theString matches at least one patX
// else return -1
int compare(const std::string& theString,
const char* pat0, // no default, must provide
const char* pat1 = nullptr,
const char* pat2 = nullptr,
const char* pat3 = nullptr,
const char* pat4 = nullptr
/* add as many patX as you desire */)
{
if (0 == theString.compare(pat0)) return 0; // found a match
//
// ignore nullptr patterns
if (nullptr != pat1) && (0 == theString.compare(pat1)) {
return(0);
}
if(nullptr != pat2) && (0 == theString.compare(pat2)) {
return(0);
}
// ...
// when get to end with no match
return (-1); // indicate no match to any of patterns
}
我实际上更喜欢以下内容。上面有点像 strstr(),它使用了 std::string 的更多功能
int compare(const std::string& theString, // what you are given
const std::string& patterns) // concatenated list of search patterns
{
//..
}
你调用这个
if (0 == compare(theString, "Seven seven SEVEN 7")
{
//
实现必须梳理空格分隔的模式..但这并不难,并且可以很容易地在循环中实现,因此您希望测试的比较次数没有限制。
什么时候应该考虑创建一个新函数来支持新的派生需求?
当我能确定 3 种或更多用途时,我会创建新功能。
祝你好运。
找到了我几年前写的一些代码,修复了它,添加了演示......
代码可以编译并且似乎可以运行,但测试很少。
- 我创建了最小的包装——一个虚拟命名空间(dtb - 用于 d___ 的工具箱)和一个虚拟类(T471_t - 测试 471)
dtb::T471_t 提供私有方法供您查看。
size_t 比较(const std::string s, std::string patterns)
-
size_t grep(const std::string pfn, const std::string 模式,
std::ostream& an_ostream = std::cout)
-- 使用 compare()
-
size_t cpuinfoGet()
-- 使用 grep
-- “wc /cpuinfo”(ubuntu 上的一个“众所周知的文件”)在我的 2 核机器上报告 54 行,更多内核,更多行
-
size_t coreCountGet()
-- 使用 grep()
-- 创建nullDev来抑制grep正常输出
#include <chrono>
#include <fstream>
#include <iostream>
#include <sstream>
#include <string>
#include <cassert>
// 'compressed' chrono access --------------vvvvvvv
typedef std::chrono::high_resolution_clock HRClk_t; // std-chrono-hi-res-clk
typedef HRClk_t::time_point Time_t; // std-chrono-hi-res-clk-time-point
typedef std::chrono::microseconds US_t; // std-chrono-microseconds
using namespace std::chrono_literals; // support suffixes like 100ms
// examples:
//
// Time_t start_us = HRClk_t::now();
//
// auto duration_us = std::chrono::duration_cast<US_t>(HRClk_t::now() - start_us);
// auto count_us = duration_us.count();
// or
// std::cout << " complete " << duration_us.count() << " us" << std::endl;
namespace dtb
{
class T471_t
{
const std::string dashLine = (" --------------------------------------------------------------");
public:
T471_t() = default;
~T471_t() = default;
int exec()
{
std::cout << "\n cpuinfoGet()\n" << dashLine << std::endl;
(void)cpuinfoGet(); // uses grep which uses compare
std::cout << dashLine << std::endl;
// count of lines which contain "processor" in linux file "/proc/cpuinfo"
std::cout << "\n\n " << coreCountGet()
<< " cores on this system. (coreCountGet())\n\n" << std::endl;
return(0);
}
private: // methods
// returns std::string::npos when none of the patterns found,
// else returns index of earliest found patterns of space delimited substr in
size_t compare (const std::string& s,
std::string patterns) // pass by value
{
size_t found = std::string::npos;
size_t patCount = 0;
std::stringstream ssPat(patterns + ' '); // load patterns into ram based stringstream
// ^^^^^ -- getline() can cause eof() in this
const char delim = ' '; // see getline()
do
{
if(0 == patterns.size()) break; // no patterns to search for, kick out
if(0 == s.size()) break; // no string in which to search, kick out
do {
std::string pat;
(void)std::getline(ssPat, pat, delim); // extract 1 space delimited pattern
if(false == ssPat.good())
{
if(ssPat.eof()) break; // quitely exit, a normal op
// let user know of patten problem
std::cerr << "\n err pattern extract: " << patterns
<< " (" << patCount << ')' << std::endl;
break;
}
patCount += 1;
//trimLeadingWhiteSpace(patterns); // tbr
//trimTrailingWhiteSpace(patterns); // tbr
if(0 == patterns.size()) break; // no more patterns
// search s for pat
found = s.find(pat);
if(found != std::string::npos) break; // one of the patterns found in s
} while(1); // repeat until 1 found or all patterns tried
}while(0);
return(found);
} // size_t compare (const std::string& s, std::string patterns)
size_t grep(const std::string pfn,
const std::string patterns, // concatenated list of search patterns
std::ostream& an_ostream = std::cout) // redirectable
{
size_t foundCount = 0;
an_ostream << " grep (" << pfn << ", [" << patterns
<< "] )" << std::endl;
do
{
std::ifstream infile(pfn);
if(!infile.is_open())
{
an_ostream << pfn << " not found.\n" << std::endl; // tbr - std::cerr?
break; // skip over file op's (no close needed)
}
do
{
if(infile.eof()) break; // file empty?
std::string lineIn;
(void)getline(infile, lineIn); // default delimiter is \n
if (0 == lineIn.size()) continue; // empty line?
size_t found = compare(lineIn, patterns); // any of patterns in lineIn?
if(std::string::npos != found) // found at least one pattern
{
an_ostream << " " << lineIn << std::endl; // found it, print it
foundCount += 1;
}
// else no pattern found - continue until eof of inFil
} while(1);
infile.close();
}while(0);
return(foundCount);
} // size_t grep(const std::string pfn, const std::string patterns, std::ostream& an_ostream = std::cout)
// space delimited list of ---^^^^^^^^
size_t cpuinfoGet()
{
size_t count = grep("/proc/cpuinfo", // pfn
"bogomips model name processor"); // patterns to search for
std::cout << "\n info lines: " << count << std::endl;
return(count);
} // size_t cpuinfoGet(void)
size_t coreCountGet()
{
// create a no-ouptput output
std::ofstream nullDev; // and do not open
nullDev.setstate(std::ios_base::badbit); // set error, ignore errors, do not close
size_t retVal = grep(std::string("/proc/cpuinfo"),
std::string("processor"), // line count of "processor" is core count
nullDev); // nullDev -- no output
return(retVal);
} // size_t coreCountGet()(void)
}; // class T471_t
} // namespace dtb
int main(int /*argc*/, char** /*argv[]*/)
{
Time_t start_us = HRClk_t::now();
int retVal = -1;
{
dtb::T471_t t471;
retVal = t471.exec();
}
auto duration_us = std::chrono::duration_cast<US_t>(HRClk_t::now() - start_us);
std::cout << " FINI " << duration_us.count() << " us" << std::endl;
return(retVal);
}
输出:
cpuinfoGet()
--------------------------------------------------------------
grep (/proc/cpuinfo, [bogomips model name processor] )
processor : 0
model : 75
model name : AMD Athlon(tm) 64 X2 Dual Core Processor 5000+
bogomips : 4809.67
processor : 1
model : 75
model name : AMD Athlon(tm) 64 X2 Dual Core Processor 5000+
bogomips : 4809.67
info lines: 8
--------------------------------------------------------------
2 cores on this system. (coreCountGet())
FINI 829 us