Мне нужно сделать программу, которая будет принимать входной файл чисел (integer.txt), который будет отсортирован по одному числу на строку, в вектор, а затем использовать алгоритм сортировки выбора, чтобы отсортировать числа в порядке убывания и записать их в выходной файл (sorted.txt). Я совершенно уверен, что что-то не так в моей функции selectionSort (), из-за которой цикл не получает правильные значения, потому что после тестирования cout я получаю совершенно неправильный вывод. Я уверен, что это глупость начинающего программиста.
vector<string> getNumbers()
{
vector<string> numberList;
ifstream inputFile ("integer.txt");
string pushToVector;
while (inputFile >> pushToVector)
{
numberList.push_back(pushToVector);
}
return numberList;
}vector<string> selectionSort()
{
vector<string> showNumbers = getNumbers();
int vectorMax = showNumbers.size();
int vectorRange = (showNumbers.size() - 1);
int i, j, iMin;
for (j = 0; j < vectorMax; j++)
{
iMin = j;
for( i = j; i < vectorMax; i++)
{
if(showNumbers[i] < showNumbers[iMin])
{
iMin = i;
}
}
if (iMin != j)
{
showNumbers[j] = showNumbers [iMin];
}
}
return showNumbers;
}
void vectorToFile()
{
vector<string> sortedVector = selectionSort();
int vectorSize = sortedVector.size();
ofstream writeTo;
writeTo.open("sorted.txt");
int i = 0;
while (writeTo.is_open())
{
while (i < vectorSize)
{
writeTo << sortedVector[i] << endl;
i += 1;
}
writeTo.close();
}
return;
}
int main()
{
vectorToFile();
}
vectorRange определен, но не используется.
В вашем selectionSort()
единственная команда, которая изменяет вектор:
showNumbers[j] = showNumbers [iMin];
Каждый раз, когда контроль достигает этой линии, вы затирать элемент вектора.
Вы должны научиться менять два значения, прежде чем даже думать о сортировке вектора.
Кроме того, ваши функции перепутаны. Если все, что вы хотите исправить, это selectionSort
, тогда вы должны быть в состоянии опубликовать это плюс main
это вызывает его с некоторыми тестовыми данными и отображает результат, но нет, все ваши функции вызывают друг друга. Учись отделять.
Также ваши имена переменных ужасны.