Yeungpok Pyin Ответов: 4

Почему я нажимаю строковый индекс вне диапазона?

Я получаю строковый индекс из ошибки диапазона, запустив этот код.

int main() {

	//Read file
	string filename = "";
	std::cout << "Welcome to the Bank system!\n";
	std::cout << "Please input the file name of the data file: ";
	std::cin >> filename;
	ifstream myfile(filename);
	
	int i = 0; // number of char
	int j = 0; // number of column
	int k = 0; // number of ac
	char c = 'a';
	bool skip = false;
	string temp = "";
	string pwhold;
	string amounthold;
	string acnohold;
	string telnohold;
	string namehold;
	if (myfile.is_open()) {
		while (getline(myfile, temp)) {
			pwhold = "";
			amounthold = "";
			acnohold = "";
			telnohold = "";
			namehold = "";
			j = 0;
			skip = false;
			i = 0;	
			while (!skip) {
				char c = temp[i];
				if (c == '\n') {
					skip = true;
					break;
				}
				else if (c == '\t') {	
					j++;
				}
				else {
					switch (j) {
					case 0: acnohold = acnohold + c; break;
					case 1: namehold = namehold + c; break;
					case 2: telnohold = telnohold + c; break;
					case 3: pwhold = pwhold + c; break;
					case 4: amounthold = amounthold + c; break;
					default: break;
					}
				}
				i++;
			}	
			system("pause");
			temp = " ";
			db1[k].pw = stoul(pwhold, nullptr, 0);
			db1[k].deposit = stod(amounthold, nullptr);
			db1[k].telno = stoul(telnohold, nullptr, 0);
			db1[k].acno = stoul(acnohold, nullptr, 0);
			db1[k].name = namehold;
			k++;
		}
	}


Используемый текстовый файл:
567123456	Chan Tai Man	90112233	3942681528	1234.56
567112233	Wong Mei Lai	60123456	2670544664	10000.12
567555555	Lau Tak Wah	69876543	3418631757	888888.88
567987654	Wong Siu Ming	60123456	3752537240	1300000.12


Почему возникает проблема и как ее устранить?

Что я уже пробовал:

Я пробовал гуглить ответ, но все же...

4 Ответов

Рейтинг:
2

Rick York

Ошибка выхода индекса за пределы диапазона означает, что индекс массива превысил допустимый диапазон. Я бы предположил, что это связано с доступом к db1, но я не могу быть уверен, так как ваш код не включает его объявление.

Я рекомендую использовать вектор для db1 и его метод push_back, и вам больше не придется беспокоиться об этом, если вы можете.

Рейтинг:
1

CPallini

Цитата:
если (c == '\n') {
пропустить = true;
перерыв;
}
Вышеприведенное условие никогда не выполняется, потому что getline удаляет символ новой строки.
(Видеть std::getline - cppreference.com[^]: "b) следующий доступный входной символ-delim, как проверено Traits::eq(c, delim), и в этом случае символ-разделитель извлекается из входных данных, но не добавляется к str").
Изменение:
if (i >= temp.size()) {
         skip = true;
         break;
       }


Пожалуйста, обратите внимание, что вы также можете
любой
или

Stefan_Lang

Хорошо подмечено. Я прочитал сразу после этого, потому что не помнил, откуда шла струна...

Есть 5.

CPallini

Спасибо!

Рейтинг:
1

Patrice T

Цитата:
Почему я нажимаю строковый индекс вне диапазона?

Что произойдет, если строка не заканчивается на \n ?

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Обратная сторона этого решения:
- Это DIY, вы один отслеживаете проблему и находите ее корни, которые ведут к решению.
Положительная сторона этого решения:
- Это также отличный инструмент обучения, потому что он показывает вам реальность, и вы можете увидеть, какие ожидания соответствуют реальности.

Вторичное воздействие
- Вы будете гордиться тем, что сами находите Жуков.
- Ваши навыки обучения улучшатся.

Вы должны довольно быстро найти, что не так.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.

Рейтинг:
0

OriginalGriff

Компиляция не означает, что ваш код верен! :смеяться:
Подумайте о процессе разработки как о написании электронного письма: успешная компиляция означает, что вы написали письмо на правильном языке - например, на английском, а не на немецком, - а не то, что письмо содержало сообщение, которое вы хотели отправить.

Итак, теперь вы входите во вторую стадию разработки (на самом деле это четвертая или пятая, но вы перейдете к более ранним стадиям позже): тестирование и отладка.

Начните с рассмотрения того, что он делает, и как это отличается от того, что вы хотели. Это важно, потому что это дает вам информацию о том, почему он это делает. Например, если программа предназначена для того, чтобы позволить пользователю ввести число, а он удваивает его и печатает ответ, то если ввод / вывод был таким:

Input   Expected output    Actual output
  1            2                 1
  2            4                 4
  3            6                 9
  4            8                16
Тогда совершенно очевидно, что проблема заключается в бите, который удваивает его - он не прибавляет себя к себе или умножает его на 2, он умножает его на себя и возвращает квадрат входного сигнала.
Таким образом, вы можете посмотреть на код, и очевидно, что он находится где-то здесь:
int Double(int value)
   {
   return value * value;
   }

Как только у вас появится идея, что может пойти не так, начните использовать отладчик, чтобы выяснить, почему. Поместите точку останова в первую строку метода и запустите приложение. Когда он достигнет точки останова, отладчик остановится и передаст управление вам. Теперь вы можете запускать свой код построчно (так называемый "одноступенчатый") и просматривать (или даже изменять) содержимое переменных по мере необходимости (черт возьми, вы даже можете изменить код и повторить попытку, если вам это нужно).
Подумайте о том, что должна делать каждая строка кода перед ее выполнением, и сравните это с тем, что она действительно делала, когда вы использовали кнопку "шаг вперед" для выполнения каждой строки по очереди. Он сделал то, что вы ожидали? Если да, то переходите к следующей строке.
Если нет, то почему? Чем это отличается?
Надеюсь, это поможет вам определить, в какой части этого кода есть проблема и в чем она заключается.
Это навык, и его стоит развивать, поскольку он помогает вам как в реальном мире, так и в развитии. И, как и все навыки, он только улучшается при использовании!