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

[Vladkp]

Нужно вывести на экран информацию о поездах, дата отправления которых введена с клавиатуры; если таких поездов нет, то вывести соответствующее сообщение.
Какую бы дату не ввел постоянно пишет что "таких поездов нет" как исправить??

#include <iostream>
#include <string>
#include <conio.h>
 
using namespace std;
struct vremya {
    int day;
    int month;
    int year;
    int hour;
    int min;
    int sec;
}
; struct TIMETABLE
{
    char Nazv[100];
    int NUMR;
    vremya Date;
    vremya Time;
};
 
int main() {
    setlocale(LC_ALL, "");
    int n;
    char ch;
    TIMETABLE RASP[100];
    TIMETABLE Rasp2[100];
    TIMETABLE vved;
    int counter = 0;
 
#pragma region ВВОД МАССИВА СТРУКТУР
 
    cout << "Введите кол-во интересующих вас поездов ";
    cin >> n;
    for (int i = 0; i < n; i++) {
        cout << "введите место назначения на англ ";
        cin >> RASP[i].Nazv;
        cout << "введите номер поезда ";
        cin >> RASP[i].NUMR;
        cout << "введите дату отправления(день/месяц/год) : ";
        cin >> RASP[i].Date.day >> ch >> RASP[i].Date.month >> ch >> RASP[i].Date.year;
        cout << "введите время отправлени (час/мин/сек) : ";
        cin >> RASP[i].Time.hour >> ch >> RASP[i].Time.min >> ch >> RASP[i].Time.sec;
    }
#pragma endregion
 
#pragma region СОРТИРОВКА
 
    for (int i = 0; i < n; i++) {
        for (int j = 1; j < n; j++) {
            if (strcmp(RASP[j - 1].Nazv, RASP[j].Nazv) > 0) {
                swap(RASP[j - 1].Nazv, RASP[j].Nazv);
            }
        }
    }
 
#pragma endregion
 
#pragma region ВЫВОД СОРТИРОВКИ 
 
    for (int i = 0; i < n; i++) {
        cout << "Упорядочение записей по номерам поездов" << endl;
        cout << RASP[i].NUMR << endl;
 
#pragma endregion
 
#pragma region ИНФОРМАЦИЯ О ПОЕЗДАХ, ДАТА КОТОРЫХ ВВЕДЕНА С КЛАВИАТУРЫ
        bool flag = true;
        cout << "Введите дату(день/месяц/год) отправления поезда ";
        cin >> vved.Date.day >> ch >> vved.Date.month >> ch >> vved.Date.year;
        cout << "Время(час/мин/сек) отправления поезда ";
        cin >> vved.Time.hour >> ch >> vved.Time.min >> ch >> vved.Time.sec;
        for (int i = 0; i < n; i++) {
            if (vved.Date.day == RASP[i].Date.day &&
                vved.Date.month == RASP[i].Date.month &&
                vved.Date.year == RASP[i].Date.year &&
                vved.Time.hour < RASP[i].Time.hour) {
                flag = false;
                Rasp2[counter] = RASP[i];
                counter++;
            }
        }
        if (flag == false) {
            for (int i = 0; i < counter; i++) {
                cout << "номер поезда " << Rasp2[i].NUMR << endl;
                cout << "направление  " << Rasp2[i].Nazv << endl;
                cout << "Время отправления " << Rasp2[i].Time.hour <<
                    ch << Rasp2[i].Time.min << ch
                    << Rasp2[i].Time.sec << endl;
            }
        }
        else {
            cout << "Поездов на такое время к сожалению нет! ";
        }
 
#pragma endregion
 
        _getch();
    }
}

[Serge_Bliznykov]

ну, вроде бы всё в порядке (кроме сортировки - она явно с большим багом!!).

но, может быть, Вы вводите неверные данные для поиска?
Ведь день, месяц, год у Вас сравниваются точно, а вот час, указанный в поиске, должен быть СТРОГО МЕНЬШЕ, чем у поезда (вы проверяете vved.Time.hour < RASP[i].Time.hour ) (а минуты и секунды вообще непонятно для чего вводятся, они в поиске вообще не проверяются).

[Vladkp]

Сообщение от Serge_Bliznykov

ну, вроде бы всё в порядке (кроме сортировки - она явно с большим багом!!).

но, может быть, Вы вводите неверные данные для поиска?
Ведь день, месяц, год у Вас сравниваются точно, а вот час, указанный в поиске, должен быть СТРОГО МЕНЬШЕ, чем у поезда (вы проверяете vved.Time.hour < RASP[i].Time.hour ) (а минуты и секунды вообще непонятно для чего вводятся, они в поиске вообще не проверяются).

что и как подправить? а то не очень шарю честно говоря )

[Serge_Bliznykov]

Сообщение от Vladkp

что и как подправить?

простите, а что Вы хотите исправить: сортировку или выдачу данных о поездах?

если сортировку, то попробуйте заменить

swap(RASP[j - 1].Nazv, RASP[j].Nazv);

на

swap(RASP[j - 1], RASP[j]);

если же выдачу - то там, насколько я понимаю, нужно просто данные вносить правильно.

что не работает?

[Vladkp]

Сообщение от Serge_Bliznykov

простите, а что Вы хотите исправить: сортировку или выдачу данных о поездах?

если сортировку, то попробуйте заменить
на

swap(RASP[j - 1], RASP[j]);

если же выдачу - то там, насколько я понимаю, нужно просто данные вносить правильно.

что не работает?

смотри, я вот тут вроде подправил код, но если ввожу 1 поезд то все хорошо, а если 2 то сортировка как то не правильно начинает работать и после информации о 2 поездах номер время и тд еще раз показывает номер время и тд. Т.е по 2 раза то же самое. Как исправить?

#include <iostream>
#include <string>
#include <conio.h>

using namespace std;
struct vremya {
	int day;
	int month;
	int year;
	int hour;
	int min;
	int sec;
}
; struct TIMETABLE
{
	char Nazv[100];
	int NUMR;
	vremya Date;
	vremya Time;
};

int main() {
	setlocale(LC_ALL, "");
	int n;
	char ch;
	TIMETABLE RASP[100];
	int counter = 0;

#pragma region ВВОД МАССИВА СТРУКТУР

	cout << "Введите кол-во интересующих вас поездов ";
	cin >> n;
	for (int i = 0; i < n; i++) {
		cout << "введите место назначения на англ ";
		cin >> RASP[i].Nazv;
		cout << "введите номер поезда ";
		cin >> RASP[i].NUMR;
		cout << "введите дату отправления(день/месяц/год) : ";
		cin >> RASP[i].Date.day >> ch >> RASP[i].Date.month >> ch >> RASP[i].Date.year;
		cout << "введите время отправления (час/мин/сек) : ";
		cin >> RASP[i].Time.hour >> ch >> RASP[i].Time.min >> ch >> RASP[i].Time.sec;
	}
#pragma endregion

#pragma region СОРТИРОВКА

	for (int i = 0; i < n; i++) {
		for (int j = 1; j < n; j++) {
			if (strcmp(RASP[j - 1].Nazv, RASP[j].Nazv) > 0) {
				swap(RASP[j - 1].Nazv, RASP[j].Nazv);
			}
		}
	}

#pragma endregion

#pragma region ВЫВОД СОРТИРОВКИ 

	for (int i = 0; i < n; i++) {
		cout << "Упорядочение записей по номерам поездов" << endl;
		cout << RASP[i].NUMR << endl;

#pragma endregion

#pragma region ИНФОРМАЦИЯ О ПОЕЗДАХ, ДАТА КОТОРЫХ ВВЕДЕНА С КЛАВИАТУРЫ
		cout << "Введите дату (день/месяц/год) отправления поезда : ";
		; int privet;
		bool found = false;
		cin >> privet;
		for (int i = 0; i < n; i++) {
			if (RASP[i].Date.day == privet) {
				cout << " " << RASP[i].Nazv << " № " << RASP[i].NUMR << "  " << RASP[i].Date.day << " " << RASP[i].Date.month << " " << RASP[i].Date.year << " " << RASP[i].Time.hour << " " << RASP[i].Time.min << " " << RASP[i].Time.sec << " " << endl;
				found = true;




			}
			if (found == false) {
				cout << "Поездов на такую дату к сожалению нет! ";
			}
			system("pause");

		}
	}
}

[Vladkp]

upup

[Croessmah]

А почему при сортировке Вы свопаете только название поезда?

[Vladkp]

Сообщение от Croessmah

А почему при сортировке Вы свопаете только название поезда?

а что еще свапать надо?

[Croessmah]

Сообщение от Vladkp

а что еще свапать надо?

По логике - все данные о поезде.
Иначе получается, что было:
Имя_поезда_1 Данные_поезда_1
Имя_поезда_2 Данные_поезда_2
А после сортировки получилось:
Имя_поезда_2 Данные_поезда_1
Имя_поезда_1 Данные_поезда_2
т.е. данные уже неверные.

Плюс вот это:

			if (found == false) {
				cout << "Поездов на такую дату к сожалению нет! ";
			}

Надобно вынести за пределы цикла.

[Serge_Bliznykov]

Сообщение от Vladkp

а что еще свапать надо?

мой пост #4
(от слов "если сортировку, то попробуйте заменить" и ниже по тексту).