Сравнение 2 массивов

@Icaab · Регистрация: 28.01.2018

Author24 — интернет-сервис помощи студентам

Здравствуйте.Не могу понять,что здесь за ошибка в сравнении 2 массивов в 52 строке:

C++

#include <iostream>
#include <cstring>
#include <Windows.h>
 
using namespace std;
 
struct TRAIN
{
    char Number;
    int Time;
    char Final_Destination;
};
 
 
 
int main()
{
    setlocale( LC_ALL, "Russian");
    
    //Заносим данные в ограниченный массив
    TRAIN trn[8];
    char Numb[5];
    int i;
    
    for(int i=0; i<8; i++)
    {
        cout <<"Ввод номера поезда:"<<endl;
        cin >> trn[i].Number;
        cout <<"Ввод пункта назначения: "<<endl;
        cin >> trn[i].Final_Destination;
        cout <<"Ввод времени отправления: "<<endl;
        cin >> trn[i].Time;
        cout <<"\n";
    }
    
    
    cout << "Сортировка но номерам рейсов";
    
    for(int i=0; i<8; i++)
            
                cout << trn[i].Number <<"\v";
                cout << trn[i].Time <<"\v";
                cout << trn[i].Final_Destination <<"\v";
                cout <<"\n";
            
            
    cout << "Ввести номер рейса:"<<endl;
    cin >> Numb;
    
    for(int i=0; i<8; i++)
    
        if(strcmp(Numb,trn[i].Number) == 0)
        cout << trn[i].Time <<"\n";
        cout << trn[i].Final_Destination <<"\n";
    
    
    system("pause");    
    return 0;
 
}

@TrickyBestia · 16.09.2020, 12:16

Icaab, ну, вы строку с символом сравниваете. Можно привести char к (char*)

@AnyKey · 16.09.2020, 12:17

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

struct TRAIN { char Number;

где тут указатель?

@Icaab · 16.09.2020, 12:27 **[ТС]**

Я сделал, как вы сказали, но теперь при выводе,когда я ввожу лишь номер поезда - программа думает и завершается

C++

#include <iostream>
#include <cstring>
#include <Windows.h>
 
using namespace std;
 
struct TRAIN
{
    char* Number;
    int Time;
    char Final_Destination;
};
 
 
 
int main()
{
    setlocale( LC_ALL, "Russian");
    
    //Заносим данные в ограниченный массив
    TRAIN trn[8];
    char Numb[5];
    int i;
    cout <<"Ввод данных вручную\n"<<endl;
    
    
    for(int i=0; i<8; i++)
    {
        cout <<"Ввод номера поезда:"<<endl;
        cin >> trn[i].Number;
        cout <<"Ввод пункта назначения: "<<endl;
        cin >> trn[i].Final_Destination;
        cout <<"Ввод времени отправления: "<<endl;
        cin >> trn[i].Time;
        cout <<"\n";
    }
    
    cout << "Сортировка но номерам рейсов";
    
    for(int i=0; i<8; i++)
            
                cout << trn[i].Number <<"\v";
                cout << trn[i].Time <<"\v";
                cout << trn[i].Final_Destination <<"\v";
                cout <<"\n";
            
            
    cout << "Ввести номер рейса:"<<endl;
    cin >> Numb;
    
    for(int i=0; i<8; i++)
    
        if(strcmp(Numb,trn[i].Number) == 0)
        cout << trn[i].Time <<"\n";
        cout << trn[i].Final_Destination <<"\n";
    
    
    system("pause");    
    return 0;
 
}

@TrickyBestia · 16.09.2020, 13:26

Icaab, в отладчике смотрите. Скорее всего ошибка где-то вылетает. exit code не равен нулю.

Добавлено через 3 минуты
Icaab, скорее всего нарушение прав доступа к памяти. Вы нигде не инициализируете TRAIN.Number. Он по умолчанию в каждом экземпляре структуры будет равен 0 (или мусор, не помню)

Nishen · 16.09.2020, 14:07

C++

#define _CRT_SECURE_NO_WARNINGS
 
#include <iostream>
 
 
 
struct Train {
 
    char* number_;
    int time_;
    char* destination_;
 
    Train() :
        number_(nullptr), time_(0), destination_(nullptr) { }
 
    Train(const char* number, int time, const char* destination)
        : number_(nullptr), time_(time), destination_(nullptr) { 
    
        setNumber(number);
        setDestination(destination);
 
    }
 
    Train(const Train& other)
        : number_(nullptr), time_(other.time_), destination_(nullptr) {
 
        setNumber(other.number_);
        setDestination(other.destination_);
 
    }
 
    ~Train() {
 
        delete[] number_;
        delete[] destination_;
 
    }
 
    Train& operator=(const Train& other) {
 
        if (this == &other)
            return *this;
 
        setNumber(other.number_);
        setDestination(other.destination_);
        time_ = other.time_;
 
        return *this;
 
    }
 
    void setNumber(const char* number) {
 
        std::size_t size = std::strlen(number) + 1;
        
        delete[] number_;
        number_ = new char[size];
        std::strcpy(number_, number);
 
    }
 
    void setDestination(const char* destination) {
 
        std::size_t size = std::strlen(destination) + 1;
 
        delete[] destination_;
        destination_ = new char[size];
        std::strcpy(destination_, destination);
 
    }
 
    void setTime(int time) {
 
        time_ = time;
 
    }
 
};
 
 
 
int main() {
 
    const std::size_t numTrains = 2;
    Train trains[numTrains];
    char buffer[256];
    int time;
 
    std::cout << "Manually input:\n\n";
 
    for (std::size_t i = 0; i < numTrains; ++i) {
 
        std::cout << "Number of train: ";
        std::cin >> buffer;
        trains[i].setNumber(buffer);
 
        std::cout << "Destination: ";
        std::cin >> buffer;
        trains[i].setDestination(buffer);
 
        std::cout << "Departure time: ";
        std::cin >> time;
        trains[i].setTime(time);
 
        std::cout << '\n';
 
    }
 
    std::cout << "\nSorting by train numbers...";
    for (std::size_t i = 0; i < numTrains - 1; ++i) {
 
        for (std::size_t j = i + 1; j < numTrains; ++j) {
 
            if (std::strcmp(trains[i].number_, trains[j].number_) > 0) {
 
                Train temp = trains[i];
                trains[i] = trains[j];
                trains[j] = temp;
 
            }
 
        }
 
    }
 
    std::cout << "\nResult:\n";
    for (std::size_t i = 0; i < numTrains; ++i) {
 
        std::cout << "Number of train - " << trains[i].number_  
            << "\n\t" << "Destination - " << trains[i].destination_
            << ", time - " << trains[i].time_ << '\n';
 
    }
 
 
    system("pause");
    return 0;
 
}

@Icaab · 16.09.2020, 14:32 **[ТС]**

Большое вам спасибо!Код конечно не прост для моего понимания(некоторые бы моменты подписать,за что отвечают), но вы проделали прекрасную работу.

Nishen · 16.09.2020, 15:58

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

некоторые бы моменты подписать,за что отвечают

Какие некоторые, показывай.

@TrickyBestia Пора на C++? 370 / 264 / 99 Регистрация: 10.04.2020 Сообщений: 1,275
	16.09.2020, 12:16	2
	Icaab, ну, вы строку с символом сравниваете. Можно привести char к (char*) 0

@AnyKey 264 / 183 / 87 Регистрация: 03.05.2020 Сообщений: 790
	16.09.2020, 12:17	3
	Сообщение от Icaab struct TRAIN { char Number; где тут указатель? 0

@TrickyBestia Пора на C++? 370 / 264 / 99 Регистрация: 10.04.2020 Сообщений: 1,275
	16.09.2020, 13:26	5
	Icaab, в отладчике смотрите. Скорее всего ошибка где-то вылетает. exit code не равен нулю. Добавлено через 3 минуты Icaab, скорее всего нарушение прав доступа к памяти. Вы нигде не инициализируете TRAIN.Number. Он по умолчанию в каждом экземпляре структуры будет равен 0 (или мусор, не помню) 0

@Icaab 1 / 1 / 0 Регистрация: 28.01.2018 Сообщений: 58
	16.09.2020, 14:32 [ТС]	7
	Большое вам спасибо!Код конечно не прост для моего понимания(некоторые бы моменты подписать,за что отвечают), но вы проделали прекрасную работу. 0

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	16.09.2020, 15:58	8
	Сообщение от Icaab некоторые бы моменты подписать,за что отвечают Какие некоторые, показывай. 0

Опции темы

Сравнение 2 массивов

Решение