我的任务是输入一个文本,输出必须是包含 2, 3, 4...n 个字母的单词数,如下所示:
你好,这是我的鼠标。 2个 0 1个 2
符号 ‘ ‘, ‘,’, ‘;’ 和 ‘.’ 算作单词分隔符。 你能帮我完成任务吗?我有这段代码,但你能帮我修复错误吗?
#include <iostream>
#include <string>
using namespace std;
int main()
{
string text="";
int size, maxword=0;
getline(cin, text);
std::string bb=text;
size=text.size();
int b=size;
int v=size, br=0, t=0, c=0;
int i=0, a[1000]={0};
do
{
if(text[i]==32 || text[i]==44 || text[i]==59 || text[i]==46)
{
a[i]++;
text.erase (0,i+1);
size=size-i-1;
i=0;
}
else
{
i++;
}
}
while(i<size);
a[text.size()]++;
//maxword=broqt bukvi na nai golqmata vsichki bukvi, do predposlednata duma ot teksta
for(int k=0;k<b;k++)
{
if(text[k]==32 || text[k]==44 || text[k]==59 || text[k]==46)
{
maxword=k;
br++;
}
if(br==1) break;
}
//t=broqt bukvi na poslednata duma ot teksta
for(int k=b-1;k>=0;k--)
{
if(c>0) break;
else
{
if(bb[k]==32 || bb[k]==44 || bb[k]==59 || bb[k]==46)
{
for(int m=k+1;m<b;m++)
{
t++;
}
c++;
}
}
}
//sravnqva t i maxword, za da vidi broqt bukvi na nai golqmata duma ot teksta
if(maxword<t) maxword=t;
for(int y=2;y<maxword+1;y++)
{
cout<<a[y]<<endl;
}
system("pause");
return 0;
}
最佳答案
我建议使用 std::map这里不是固定大小的大数组 (a[1000]
) - 以简化程序,使其更安全(无需越界检查)并节省内存。最棒的是:它可以像数组一样使用 []
运算符
你的第二个和第三个循环是多余的,不是必需的——只需要一个循环来计算长度(另一个循环在程序结束时输出长度)。
还要注意一些事情:
您可以不使用 ascii ' '
、';'
、'.'
和 ','
常量只需使用字符常量 - 它更容易理解并提高可读性。 size_t是一种方便的无符号类型,可以保存任何数组/字符串的长度——在正常情况下永远不能对大小/长度进行签名(负长度)——所以最好在那里使用无符号类型。
清晰的变量名,如 wordCount
使代码比 b
或 a
更容易阅读。所以我不得不对代码进行相当多的更改,因为我无法真正理解您在那里尝试做什么。
考虑一下:
#include <iostream>
#include <string>
#include <map>
int main(int argc, const char * argv[]){
std::string text="";
size_t pos = 0, count_chars = 0, max_chars = 0;
std::map<size_t, size_t> wordCount;
std::getline(std::cin, text);
do{
if((text[pos] == ' ') || (text[pos] == ';') || (text[pos] == ',') || (text[pos]=='.')){
if(count_chars > max_chars) max_chars = count_chars;
wordCount[count_chars]++;
count_chars = 0;
}else{
count_chars++;
}
pos++;
}while(pos < text.size());
for(size_t i = 2 ; i < max_chars + 1; i++){
std::cout << wordCount[i] << std::endl;
}
system("pause");
return 0;
}
输入:
Hello, whis is my mouse.
输出:
2
0
1
2
但要小心:如果您输入的不是鼠标后的最终 .
,它不会以这种方式识别最后一个词 - 但这很容易修复:您可以这样做
if(count_chars>0) wordCount[count_chars]++;
在第一个循环(do-while 循环)之后。如果您对此有任何疑问,请随时提问。
关于C++文本分隔和计数单词,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/20339201/