При вводе данных в объект программа аварийно завершается

@Ann1984 · Регистрация: 09.10.2018

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

Здравствуйте. В процессе выполения лаб.работы столкнулась с такой проблемой. Вот мое задание: создать файлы main.h и main.cpp. В файле main.h определить с помощью ключевого слова class объект Ship, описать конструктор объекта, аргументы которого будут инициализировать все данные объекта. Описать конструктор объекта по умолчанию (без аргументов), проинициализировав все данные. Описать в объекте функцию void Print() с открытым уровнем доступа (public), которая будет выводить данные на экран.
В main.cpp определить конструктор объекта, инициализирующий все данные объекта значениями аргументов. В теле конструктора используем функцию strcpy (стр1, стр2) для копирования строки наименования корабля. Затем определить функцию void SHIP::Print(). Далее создать главную функцию программы int main(). Внутри нее создать объект SHIP, указав все значения данных объекта. Вывести данные объекта на экран, вызвав функцию Print. Далее создать динамический объект SHIP с помощью обычного конструктора и оператора new. Вывести данные объекта на экран и удалить его с помощью оператора delete. Затем написать функцию ввода данных в объект с клавиатуры void Person::Input(). (у меня не получилось ее сразу написать в .cpp и я поступила с ней аналогично Print). И затем создать объект SHIP с помощью конструктора по умолчанию и ввести данные в объект с клавиатуры, вызвав функцию Input (у меня это строчки 40-43). Вот тут у меня ничего не получается, VS выдает ошибку программы и просто закрывается.
Прошу помощи у Вас.

main.h

C++

using namespace std;
 
#include <iostream>
class Ship
{
private:
    char* name;
    int displacement;
    char* type;
 
public:
    Ship (const char* valueName, int valueDisplacement, const char* valueType); // конструктор с параметрами
    
    Ship () // конструктор по умолчанию
    {
        name = 0; 
        displacement = 0; 
        type = 0;
    }
 
    void Print(); // функция вывода данных на экран
 
    void Input(); // функция ввода данных с клавиатуры
};

main.cpp

C++

using namespace std;
#define _CRT_SECURE_NO_WARNINGS
 
#include <cstdlib>
#include <string>
#include <iostream>
#include "main.h"
 
 
Ship::Ship(const char* valueName, int valueDisplacement, const char* valueType) //конструктор с параметрами
{
    name = new char[strlen(valueName) + 1];
    strcpy(name, valueName);
    displacement = valueDisplacement;
    type = new char[strlen(valueType) + 1];
    strcpy(type, valueType);
};
 
void Ship:: Print() // функция вывода данных на экран
{
    cout << "\nНаименование\t" << name << "\nВодоизмещение\t" << displacement << "\nТип\t\t" << type << endl;
}
 
void Ship:: Input () // функция ввода данных с клавиатуры
{
    cin >> name >> displacement >> type ;
}
 
int main()
{
    setlocale(LC_ALL, "RU");
     Ship firstship("Титаник", 52310, "пассажирский");
     firstship.Print();
 
     Ship *Ship1;
     Ship1 = new Ship("Аврора", 6731, "крейсер");
     Ship1->Print();
     delete Ship1;
    
    Ship *Ship2;
    Ship2 = new Ship();
    Ship2->Input();
    Ship2->Print();
    
}

@oleg-m1973 · 26.12.2019, 10:00

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

Вот тут у меня ничего не получается, VS выдает ошибку программы и просто закрывается.

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

using namespace std;
#include <iostream>

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

using namespace std;
#define _CRT_SECURE_NO_WARNINGS
#include <cstdlib>
#include <string>
#include <iostream>
#include "main.h"

Для начала убери using namespace std; из main.h, а в main.cpp сделай его после инклудов

@zss · 26.12.2019, 10:07

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

Ship2 = new Ship();
Ship2->Input();

Здесь Вы создаете объект с нулевыми указателями
и в методе input пытаетесь ввести данные в объекты, имеющий нулевые адреса.

C++

void Ship::Input() // функция ввода данных с клавиатуры
{
    if(name!=nullptr)
        cin >> name >> displacement >> type;// !!!!! размеры останутся старые
    else
    {
        char namet[100];
        int dt;
        char typet[100];
        cin >> namet >> dt >> typet;
 
        name = new char[strlen(namet) + 1];
        strcpy(name, namet);
        displacement = dt;
        type = new char[strlen(typet) + 1];
        strcpy(type, typet);
    }
}

@zayats80888 · 26.12.2019, 10:08

Ann1984,

C++

void Ship::Input() // функция ввода данных с клавиатуры
{
    char buf[256]{};
 
    if (cin >> buf)
    {
        delete[] name;
        name = new char[strlen(buf) + 1];
        strcpy(name, buf);
    }
 
    cin >> displacement;
 
    if (cin >> buf)
    {
        delete[] type;
        type = new char[strlen(buf) + 1];
        strcpy(type, buf);
    }
}

@politoto · 26.12.2019, 10:12

Кратко проблема в следующем:

C++

#include <iostream>
int main()
{
    char* name = 0; //  конструктор по умолчанию
 
    std::cin >> name; //  Ship::Input()
}

@DrOffset · 26.12.2019, 10:13

Ann1984, у вас в задании нет ничего о том, какими должны быть поля объекта. Поэтому для вас лучше всего и понятнее будет не использовать самостоятельный контроль за памятью для полей, а использовать std::string.

C++

#include <string>
 
class Ship
{
private:
    std::string name;
    int displacement;
    std::string type;
 
public:
    Ship (const char* valueName, int valueDisplacement, const char* valueType); // конструктор с параметрами
    
    Ship () // конструктор по умолчанию
    {  }
 
    void Print(); // функция вывода данных на экран
 
    void Input(); // функция ввода данных с клавиатуры
};

C++

Ship::Ship(const char* valueName, int valueDisplacement, const char* valueType) //конструктор с параметрами
    : name(valueName), displacement(valueDisplacement), type(valueType)
{ }
 
void Ship:: Print() // функция вывода данных на экран
{
    cout << "\nНаименование\t" << name << "\nВодоизмещение\t" << displacement << "\nТип\t\t" << type << endl;
}
 
void Ship:: Input () // функция ввода данных с клавиатуры
{
    cin >> name >> displacement >> type ;
}

Тогда работать будет тот Input, который у вас уже написан.

@politoto · 26.12.2019, 10:17

Так, несомненно, лучше, но тогда непонятно, зачем нужна strcpy()

@Ann1984 · 26.12.2019, 15:25 **[ТС]**

politoto , это у меня было в задании. Нужно было в теле конструктора использовать функцию strcpy (стр1, стр2) для копирования строки наименования корабля.

Добавлено через 1 минуту
oleg-m1973 , спасибо за замечание, исправила.

Добавлено через 11 минут
oleg-m1973 и zss, спасибо за развернутые ответы, но учитывая мой уровень знаний, я до таких схем просто не могла бы додуматься. И сейчас, когда Вы все расписали, понимаю со скрипом.

Добавлено через 6 минут
DrOffset ,

Ship::Ship(const char* valueName, int valueDisplacement, const char* valueType) //конструктор с параметрами
: name(valueName), displacement(valueDisplacement), type(valueType)

не могли бы Вы объяснить, что означает вторая строчка?

Добавлено через 19 минут
Теперь программа дает ввести данные, но в окне вывода "Наименование", "водоизмещение" и "тип" написаны верно, а выводимые данные "наименование" и "тип" - просто набор символов. Но у меня вроде включена функция setlocale.

@DrOffset · 26.12.2019, 15:35

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

не могли бы Вы объяснить, что означает вторая строчка?

Это список инициализации конструктора.

@nyaka_wai · 26.12.2019, 16:02

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

Теперь программа дает ввести данные, но в окне вывода "Наименование", "водоизмещение" и "тип" написаны верно, а выводимые данные "наименование" и "тип" - просто набор символов. Но у меня вроде включена функция setlocale.

Попробуйте

C++

1	setlocale(LC_CTYPE, "rus"); SetConsoleCP(1251); SetConsoleOutputCP(1251);

вместо простого

C++

1	setlocale(LC_CTYPE, "rus");

@Ann1984 · 27.12.2019, 08:48 **[ТС]**

nyaka_wai , у меня компилятор ругается, пишет, что "идентификатор SetConsoleCP не определен" и то же самое с SetConsoleOutputCP.

@DrOffset · 27.12.2019, 11:15

Ann1984, #include <windows.h>

@Ann1984 · 27.12.2019, 14:48 **[ТС]**

DrOffset,

Ann1984, #include <windows.h>

спасибо, теперь буду знать.

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	26.12.2019, 10:00	2
	Сообщение от Ann1984 Вот тут у меня ничего не получается, VS выдает ошибку программы и просто закрывается. Сообщение от Ann1984 using namespace std; #include <iostream> Сообщение от Ann1984 using namespace std; #define _CRT_SECURE_NO_WARNINGS #include <cstdlib> #include <string> #include <iostream> #include "main.h" Для начала убери using namespace std; из main.h, а в main.cpp сделай его после инклудов 0

@politoto Модератор 2121 / 991 / 169 Регистрация: 23.07.2018 Сообщений: 3,328 Записей в блоге: 3
	26.12.2019, 10:17	7
	Так, несомненно, лучше, но тогда непонятно, зачем нужна `strcpy()` 0

@Ann1984 0 / 0 / 0 Регистрация: 09.10.2018 Сообщений: 87
	26.12.2019, 15:25 [ТС]	8
	politoto , это у меня было в задании. Нужно было в теле конструктора использовать функцию strcpy (стр1, стр2) для копирования строки наименования корабля. Добавлено через 1 минуту oleg-m1973 , спасибо за замечание, исправила. Добавлено через 11 минут oleg-m1973 и zss, спасибо за развернутые ответы, но учитывая мой уровень знаний, я до таких схем просто не могла бы додуматься. И сейчас, когда Вы все расписали, понимаю со скрипом. Добавлено через 6 минут DrOffset , Ship::Ship(const char* valueName, int valueDisplacement, const char* valueType) //конструктор с параметрами : name(valueName), displacement(valueDisplacement), type(valueType) не могли бы Вы объяснить, что означает вторая строчка? Добавлено через 19 минут Теперь программа дает ввести данные, но в окне вывода "Наименование", "водоизмещение" и "тип" написаны верно, а выводимые данные "наименование" и "тип" - просто набор символов. Но у меня вроде включена функция setlocale. 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	26.12.2019, 15:35	9
	Сообщение от Ann1984 не могли бы Вы объяснить, что означает вторая строчка? Это список инициализации конструктора. 0

@Ann1984 0 / 0 / 0 Регистрация: 09.10.2018 Сообщений: 87
	27.12.2019, 08:48 [ТС]	11
	nyaka_wai , у меня компилятор ругается, пишет, что "идентификатор SetConsoleCP не определен" и то же самое с SetConsoleOutputCP. 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	27.12.2019, 11:15	12
	Ann1984, #include <windows.h> 1

@Ann1984 0 / 0 / 0 Регистрация: 09.10.2018 Сообщений: 87
	27.12.2019, 14:48 [ТС]	13
	DrOffset, Ann1984, #include <windows.h> спасибо, теперь буду знать. 0

При вводе данных в объект программа аварийно завершается

Решение