Ошибка в конструкторе класса Book

@matviiv · Регистрация: 09.02.2021

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

Здравствуйте ,не могу исправить ошибку ,буду благодарен за помощь:
Unhandled exception at 0x79E5E625 (ucrtbased.dll) in ФФ.exe: 0xC00000FD: Stack overflow (parameters: 0x00000001, 0x00EC2FFC).
Book.cpp:

C++

#include "Book.h"
 
string Book::GetAuthor()
{
    return author;
}
 
string Book::GetTitle()
{
    return title;
}
 
Book* Book::GetList()
{
    return list_of_references;
}
 
void Book::Set(string a, string t)
{
    author = a;
    title = t;
}
 
void Book::ShowAuthorAndTitle()
{
    cout << author << " " << title << endl;
}
 
void Book::ShowList()
{
    for (int i = 0; i < number; i++) {
        list_of_references[i].ShowAuthorAndTitle();
    }
}
 
Book::Book()
{
    author = "No author";
    title = "No title";
    list_of_references[number-1].Set("No", "No");
}
 
Book::~Book()
{
    delete[]list_of_references;
}

Book.h:

C++

#pragma once
#include <iostream>
#include <string>
using namespace std;
class Book
{
private:
    string author;
    string title;
    int number=1;
    Book *list_of_references = new Book[number];
public:
    string GetAuthor();
    string GetTitle();
    Book* GetList();
    void Set(string a, string t);
    void ShowAuthorAndTitle();
    void ShowList();
    Book();
    ~Book();
};

@Folian · 12.09.2021, 21:47

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

C++

class Book
{
...
    Book *list_of_references = new Book[number];

Самозацикленное бесконечое.

@matviiv · 12.09.2021, 21:51 **[ТС]**

Folian, Хмм да теперь понял,только как исправить?Создавать клас Book отдельно и етот клас в Library?

@Folian · 12.09.2021, 21:57

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

Book отдельно и етот клас в Library?

Да.

@matviiv · 13.09.2021, 11:46 **[ТС]**

Folian, Вот сделал через 2 класса,но тоже ошибки,возможно в классах ошибка где-то а может в main не правильно обьявляю как-то,не могли б посмотреть?
Library.cpp:

C++

#include "Library.h"
 
void Library::SetList(int n, Book* list)
{
    number = n;
    list_of_references = list;
}
 
Book* Library::GetList()
{
    return list_of_references;
}
 
void Library::ShowList()
{
    for (int i = 0; i < number; i++) {
        list_of_references[i].ShowAuthorAndTitle();
    }
}
 
Library::~Library()
{
    delete[]list_of_references;
}

Book.cpp:

C++

#include "Book.h"
 
string Book::GetAuthor()
{
    return author;
}
 
string Book::GetTitle()
{
    return title;
}
 
 
void Book::SetAuthorAndTitle(string a, string t)
{
    author = a;
    title = t;
}
 
void Book::ShowAuthorAndTitle()
{
    cout << author << " " << title << endl;
}
 
 
Book::Book()
{
    author = "No author";
    title = "No title";
}

Library.h:

C++

#pragma once
#include <iostream>
#include <string>
#include "Book.h"
using namespace std;
class Library
{
private:
    int number ;
    Book* list_of_references = new Book[number];
public:
    void SetList(int n, Book* list);
    Book* GetList();
    void ShowList();
    ~Library();
};

Book.h:

C++

#pragma once
#include <iostream>
#include <string>
 
using namespace std;
class Book
{
private:
    string author;
    string title;
 
public:
    string GetAuthor();
    string GetTitle();
    void SetAuthorAndTitle(string a, string t);
    void ShowAuthorAndTitle();
    Book();
};

Source.cpp:

C++

#include <iostream>
#include <string>
#include "Book.h"
#include "Library.h"
 
using namespace std;
int main() {
    Book* b = new Book[2];
    for (int i = 0; i < 2; i++) {
        b[i].SetAuthorAndTitle("Example", "Example");
        b[i].ShowAuthorAndTitle();
    }
    Library l;
        l.SetList(2, b);
        l.ShowList();
    delete[]b;
    return 0;
}

@long399 · 13.09.2021, 12:20

Я думаю вам нужно убрать из заголовочного файла класса Library выделение памяти в 10 строке. Обычно, выделение памяти делается в конструкторе, а ее освобождение в деструкторе (как вы и делаете).

@XLAT · 13.09.2021, 12:22

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

только как исправить?

C++

/// Library.h:
#pragma once
#include <iostream>
#include <string>
//#include "Book.h"
using namespace std;
 
class Book{};
 
class Library
{
private:
    int number ;
    Book* list_of_references;
public:
    void SetList(int n, Book* list){}
    Book* GetList(){return nullptr;}
    void ShowList(){}
    ~Library(){delete[] list_of_references;}
    Library(int n): number(n), list_of_references(new Book[number]){} ///<<<---!
};
 
///----------------------------------------------------------------------------|
/// Тест.
///----------------------------------------------------------------------------:
int main()
{   Library library(10);
}

@matviiv · 13.09.2021, 12:29 **[ТС]**

XLAT, а как тогда заполнить етот список литератури и вивести

@long399 · 13.09.2021, 12:34

Думаю, можно добавить для класса библиотеки такие конструкторы:

C++

// Library.h
...
Library(int);
Library(int, Book*);
...

C++

// Library.cpp
...
Library::Library(int n)
{
    number = n;
    list_of_references = new Book[number];
}
 
Library::Library(int n, Book* list)
{
    number = n;
    list_of_references = new Book[number];
    for (int i = 0; i < number; i++) {
        list_of_references[i] = list[i];
    }
}
...

C++

int main() {
    Book* b = new Book[2];
    for (int i = 0; i < 2; i++) {
        b[i].SetAuthorAndTitle("Example", "Example");
        b[i].ShowAuthorAndTitle();
    }
    Library l1(2);
    l1.ShowList();
    
    Library l2(2, b);
    l2.ShowList();
    delete[]b;
    return 0;
}

@XLAT · 13.09.2021, 12:49

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

а как тогда заполнить етот список литератури и вивести

1. добавить метод для ручного ввода в цикле по каждому экземпляру книги.
2. добавить метод для ввода из файла в цикле по каждому экземпляру книги.
3. добавить метод для заполнения рандомными значениями в цикле по каждому экземпляру книги.
4. придумать другие источники, например, из памяти компутера.

@matviiv · 13.09.2021, 12:53 **[ТС]**

Я максимально запутался:
Source.cpp:

C++

#include <iostream>
#include <string>
#include "Book.h"
#include "Library.h"
 
using namespace std;
int main() {
    Book* b = new Book[2];
    for (int i = 0; i < 2; i++) {
        b[i].SetAuthorAndTitle("Example", "Example");
        b[i].ShowAuthorAndTitle();
    }
    Library l1(2);
    l1.ShowList();
 
    Library l2(2, b);
    l2.ShowList();
    delete[]b;
    return 0;
}

Library.h:

C++

#pragma once
#include <iostream>
#include <string>
#include "Book.h"
using namespace std;
class Library
{
private:
    int number ;
    Book* list_of_references = new Book[number];
public:
    void SetList(int n, Book* list);
    Book* GetList() { return nullptr; }
    void ShowList();
    ~Library();
    Library(int n );
    Library(int n, Book*list);
};

Library.cpp:

C++

#include "Library.h"
 
void Library::SetList(int n, Book* list)
{
    number = n;
    list_of_references = list;
}
 
Book* Library::GetList()
{
    return nullptr;
}
 
void Library::ShowList()
{
    for (int i = 0; i < number; i++) {
        list_of_references[i];
    }
}
 
Library::~Library()
{
    delete[]list_of_references;
}
 
Library::Library(int n)
{
    number = n;
    list_of_references = new Book[number];
}
 
Library::Library(int n, Book*list)
{
    number = n;
    list_of_references = new Book[number];
    for (int i = 0; i < number; i++) {
        list_of_references[i] = list[i];
    }
}

Book.cpp:

C++

#include "Book.h"
 
string Book::GetAuthor()
{
    return author;
}
 
string Book::GetTitle()
{
    return title;
}
 
 
void Book::SetAuthorAndTitle(string a, string t)
{
    author = a;
    title = t;
}
 
void Book::ShowAuthorAndTitle()
{
    cout << author << " " << title << endl;
}
 
 
Book::Book()
{
    author = "No author";
    title = "No title";
}

Book.h:

C++

#pragma once
#include <iostream>
#include <string>
 
using namespace std;
class Book
{
private:
    string author;
    string title;
 
public:
    string GetAuthor();
    string GetTitle();
    void SetAuthorAndTitle(string a, string t);
    void ShowAuthorAndTitle();
    Book();
}

;

Добавлено через 21 секунду
XLAT, я исправлял код под ваш, и почему-то много ошибок

@XLAT · 13.09.2021, 13:01

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

XLAT, я исправлял код под ваш, и почему-то много ошибок

хм,
я вижу всё тож же самый код:

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

Library.h:
...
Book* list_of_references = new Book[number];

исправьте ...

@matviiv · 13.09.2021, 13:20 **[ТС]**

XLAT, ну допустим вот:
Library.h:

C++

#pragma once
#include <iostream>
#include <string>
#include "Book.h"
using namespace std;
class Library
{
private:
    int number ;
    Book* list_of_references;
public:
    void SetList(int n, Book* list);
    Book* GetList() { return nullptr; }
    void ShowList();
    ~Library();
    Library(int n );
    Library(int n, Book*list);
};

Добавлено через 3 минуты
XLAT, а как добавить метод считивание с клавиатури и как еще можно поправить код?

@long399 · 13.09.2021, 13:28

как вариант:

C++

// Library.h
...
void fillList();
...

C++

// Library.cpp
...
void Library::fillList()
{
    for (int i = 0; i < number; i++) {
        // ввод author для i-ой книги
        // ввод title для i-ой книги
    }
}
...

C++

// main.cpp
...
Library l1(2);
l1.fillList();
...

@matviiv · 13.09.2021, 13:35 **[ТС]**

long399, Видимо немного не так понял:

C++

void Library::FillList()
{
    for (int i = 0; i < number; i++) {
        string a, t;
        cin >> a;
        cin >> t;
        list_of_references[i].SetAuthorAndTitle(a, t);
    }
}

@XLAT · 13.09.2021, 13:39

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

ну допустим вот:

ошибки компиляции есть?

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

а как добавить метод считивание с клавиатури

C++

class Library
{
private:
    ...
public:
    void input_user()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < nimber; ++i)
        {   std::getline(std::cin, p[i].author);
            std::getline(std::cin, p[i].title );
        }
    }
    
    void info()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < nimber; ++i)
        {   std::cout << p[i].author << " - " << p[i].title << '\n';
        }
    }
    ...

@matviiv · 13.09.2021, 13:42 **[ТС]**

XLAT, ошибок нет, а вот не виводит результат

Добавлено через 1 минуту
XLAT, в вашем коде ошибки на author и title

@XLAT · 13.09.2021, 13:50

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

в вашем коде ошибки на author и title

сделайте их друзьями:

C++

#pragma once
#include <iostream>
#include <string>
//#include "Book.h"
using namespace std;
 
class Book
{
    string author;
    string title;
    friend class Library; ///<<<--- Вот так!
 
public:
 
};
 
class Library
{
private:
    int number ;
    Book* list_of_references;
public:
    void input_user()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < number; ++i)
        {   std::getline(std::cin, p[i].author);
            std::getline(std::cin, p[i].title );
        }
    }
 
    void info()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < number; ++i)
        {   std::cout << p[i].author << " - " << p[i].title << '\n';
        }
    }
 
    void SetList(int n, Book* list){}
    Book* GetList(){return nullptr;}
    void ShowList(){}
    ~Library(){auto& p = list_of_references; delete[] list_of_references;}
    Library(int n): number(n), list_of_references(new Book[number]){}
};
 
///----------------------------------------------------------------------------|
/// Тест.
///----------------------------------------------------------------------------:
int main()
{   Library library(2);
            library.input_user();
            library.info      ();
}

Добавлено через 5 минут
c колокольни ооп'а лучше так:

C++

#pragma once
#include <iostream>
#include <string>
//#include "Book.h"
using namespace std;
 
class Book
{
    string author;
    string title;
    void input_user()
    {   std::getline(std::cin, author);
        std::getline(std::cin, title );
    }
    void info()
    {   std::cout << author << " - " << title << '\n';
    }
    friend class Library;
 
public:
 
};
 
class Library
{
private:
    int number ;
    Book* list_of_references;
 
public:
    void input_user()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < number; ++i)
        {   p[i].input_user();
        }
    }
 
    void info()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < number; ++i)
        {   p[i].info();
        }
    }
 
    void SetList(int n, Book* list){}
    Book* GetList(){return nullptr;}
    void ShowList(){}
    ~Library(){auto& p = list_of_references; delete[] list_of_references;}
    Library(int n): number(n), list_of_references(new Book[number]){}
};
 
///----------------------------------------------------------------------------|
/// Тест.
///----------------------------------------------------------------------------:
int main()
{   Library library(2);
            library.input_user();
            library.info      ();
}

@matviiv · 13.09.2021, 14:14 **[ТС]**

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

@XLAT · 13.09.2021, 14:21

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

без друга то никак не исправить?

C++

#pragma once
#include <iostream>
#include <string>
//#include "Book.h"
using namespace std;
 
class Book
{
    string author;
    string title;
 
public:
    void input_user()
    {   std::getline(std::cin, author);
        std::getline(std::cin, title );
    }
    void info()
    {   std::cout << author << " - " << title << '\n';
    }
};
 
class Library
{
private:
    int number ;
    Book* list_of_references;
public:
        void input_user()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < number; ++i)
        {   p[i].input_user();
        }
    }
 
    void info()
    {   auto& p = list_of_references;
        for(size_t i = 0; i < number; ++i)
        {   p[i].info();
        }
    }
 
    void SetList(int n, Book* list){}
    Book* GetList(){return nullptr;}
    void ShowList(){}
    ~Library(){auto& p = list_of_references; delete[] list_of_references;}
    Library(int n): number(n), list_of_references(new Book[number]){}
};
 
///----------------------------------------------------------------------------|
/// Тест.
///----------------------------------------------------------------------------:
int main()
{   Library library(2);
            library.input_user();
            library.info      ();
}

кроме того раз у вас уже есть гетеры:

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

string GetAuthor();
string GetTitle();

то можно добавить и сетеры:

C++

1 2	void SetAuthor(); void SetTitle();

но я уже добавил:
это в качестве сетера:

C++

1	void input_user();

@matviiv 429 / 312 / 129 Регистрация: 09.02.2021 Сообщений: 1,096 Записей в блоге: 1
	13.09.2021, 13:42 [ТС]	17
	XLAT, ошибок нет, а вот не виводит результат Добавлено через 1 минуту XLAT, в вашем коде ошибки на author и title 0

@matviiv 429 / 312 / 129 Регистрация: 09.02.2021 Сообщений: 1,096 Записей в блоге: 1
	13.09.2021, 14:14 [ТС]	19
	XLAT, а если брать мой код и без друга то никак не исправить? 0

@matviiv 429 / 312 / 129 Регистрация: 09.02.2021 Сообщений: 1,096 Записей в блоге: 1
	12.09.2021, 21:51 [ТС]	3
	Folian, Хмм да теперь понял,только как исправить?Создавать клас Book отдельно и етот клас в Library? 0

@Folian 1710 / 1110 / 337 Регистрация: 25.01.2019 Сообщений: 2,910
	12.09.2021, 21:57	4
	Сообщение от matviiv Book отдельно и етот клас в Library? Да. 1

@long399 Модератор 2841 / 1792 / 932 Регистрация: 16.10.2013 Сообщений: 5,161 Записей в блоге: 15
	13.09.2021, 12:20	6
	Я думаю вам нужно убрать из заголовочного файла класса Library выделение памяти в 10 строке. Обычно, выделение памяти делается в конструкторе, а ее освобождение в деструкторе (как вы и делаете). 1

@matviiv 429 / 312 / 129 Регистрация: 09.02.2021 Сообщений: 1,096 Записей в блоге: 1
	13.09.2021, 12:29 [ТС]	8
	XLAT, а как тогда заполнить етот список литератури и вивести 0

@XLAT Just Do It! 4028 / 2422 / 640 Регистрация: 23.09.2014 Сообщений: 7,806 Записей в блоге: 2
	13.09.2021, 12:49	10
	Сообщение от matviiv а как тогда заполнить етот список литератури и вивести 1. добавить метод для ручного ввода в цикле по каждому экземпляру книги. 2. добавить метод для ввода из файла в цикле по каждому экземпляру книги. 3. добавить метод для заполнения рандомными значениями в цикле по каждому экземпляру книги. 4. придумать другие источники, например, из памяти компутера. 0

@XLAT Just Do It! 4028 / 2422 / 640 Регистрация: 23.09.2014 Сообщений: 7,806 Записей в блоге: 2
	13.09.2021, 13:01	12
	Сообщение от matviiv XLAT, я исправлял код под ваш, и почему-то много ошибок хм, я вижу всё тож же самый код: Сообщение от matviiv Library.h: ... Book* list_of_references = new Book[number]; исправьте ... 0

Ошибка в конструкторе класса Book

Решение