Это функция моей программы, которая продолжает сбой. Целью функции является поиск по векторам и отображение того, что они оценивают, и частоты, которая считывается в соответствующие векторы. Я предполагаю, что мой цикл while, но я не уверен. Векторы правильно считываются.
void searchNames(vector<string> girlsname,vector<int> girlsfeq,vector<string> boysname,vector<int> boysfeq){
string name1;
string name;
int SIZE = girlsname.size();
int SIZE2 = boysname.size();
while(name1 != "quit"){
int i=0;
int frank=find(girlsname.begin(), girlsname.end(), name1) - girlsname.begin();
int mrank=find(boysname.begin(), boysname.end(), name1) - boysname.begin();
name1.clear();
name.clear();
cout << "Please enter a name: ";
cin >> name1;
cout << endl;
while(i< SIZE && name1 !=girlsname[i++]){
frank++;
}
name.clear();
i=0;
while(i< SIZE2 && name1 !=boysname[i++]){
mrank++;
}
cout << left << setw(15) << "Name" << right << setw(10) << "Girl Rank" << right << setw(10) << "Frequency" << right << setw(10) << "Boy Rank"<< right << setw(10) << "Frequency" << endl;
if(frank != SIZE && mrank != SIZE2){
cout << left << setw(15) << name1 << right << setw(10) << frank << right << setw(10) << girlsfeq[frank] << right << setw(10) << mrank << right << setw(10) << boysfeq[mrank] << endl;
}
}
Во-первых, в любом из ваших циклов вы не меняете значение i
, поэтому они потенциально бесконечны.
Во-вторых, если я понимаю, что вы пытаетесь сделать, у вас есть парные векторы, и вы ищете индекс, по которому имя находится в первом векторе, а затем используйте его для получения значения из второго вектора. Вы можете использовать функцию std::find
для этого:
int frank = find(girlsname.begin(), girlsname.end(), name1) - girlsname.begin();
Теперь вы можете получить доступ к girlsfreq[frank]
. Если имя отсутствует в векторе, frank
равно размеру вектора, который вы должны проверить. Не забудьте добавить:
#include <algorithm>
Я не могу воспроизвести ваш крах. Вместо этого я могу воспроизвести бесконечный цикл. Присмотревшись к вашим в while
циклы, то становится очевидным: значение я никогда не меняется. Если он не может найти совпадение имен, он будет продолжать увеличивать значение frank
. Таким образом, исправление этой проблемы будет:
while(i < SIZE && name1 != name) {
name = girlsname[i];
frank++;
i++;
}
Но его можно улучшить: вам не нужно это задание. Кроме того, вы можете использовать преимущество приращения postfix:
while(i < SIZE && name1 != girlsname[i++]){
frank++;
}
При этом вы также удалите необходимость name
и name1
.
i
обратно в0
перед второй вwhile
цикла.