-1

所以今天我决定用向量写一个简单的直方图,但是当我写这个程序并编译它时,输出显示一个永无止境的“*”。

#include <iostream>
#include <conio.h>
#include <ios>
#include <vector>
#include <algorithm>


using std::cout;
using std::vector;
using std::cin;
using std::endl;



int main()
{
int x;
double k;
cout<<"How many range would you like = ";
    cin>>x;
vector<double> number; //All data was being stored here
cout<<"Input the number to be included in histogram = ";
while(cin>>k)
number.push_back(k);


x = x*10;
sort(number.begin(), number.end());


for(int i=0;i<x;i+=10) // Problem is on this loop statement, but I can't fix it.
{
    vector<double>::size_type u = 0;
    cout<<i<<"-"<<i+9<<" = "; //The range, Intended to put the "*" besides it.
    while(u < number.size())
     {
        if( number[u]<=i+9 && number[u]>=i)
        cout<<"*"; //For showing how many numbers on that range
        else
            u++;

    }
    cout<<endl;
}

getch();
return 0;
  }

没有显示错误,只有第一个范围内的“”永无止境。(例如,当程序编译时,我输入数据并显示“0-9 =** * **** " 并且它只是继续重复而不转到下一行。有什么建议可以解决这个问题吗?编辑:我之前尝试过使用迭代器,但它与这个具有相同的输出。

4

2 回答 2

1

所以,在你的while循环中......

if( number[u]<=i+9 && number[u]>=i)
    cout<<"*";

你有那个代码。问题是,你只在这个循环中增加你uelse一部分。因此,如果您进入循环并且此条件返回,它将永远存在。您将永远陷入循环执行中,因为您没有更改此处的值。if elsewhilewhileiftruetruewhilecout<<"*";u

于 2013-10-08T15:43:40.303 回答
1

这是问题所在:

if( number[u]<=i+9 && number[u]>=i)
    cout<<"*"; //For showing how many numbers on that range
else
    u++;

一旦你找到一个范围内的数字,就else意味着你不会移动到下一个数字;你会一直呆在那里,打印*,永远。

您可以简单地删除else; 或者您可以将其构建为一个for循环,这可能会使它更明显正确:

for (size_t u = 0; u < number.size(); ++u)

或者,自 2011 年以来,一种新型for循环:

for (double n : number) {
    if (n < i+10 && n >= i) {
        cout << "*";
    }
}

我还冒昧地修复了逻辑,例如,9.5不被排除在外。

于 2013-10-08T15:48:03.993 回答