пытаюсь освоить ООП

[radistkmm]

Помогите плиз начинающему писаке)))

суть такова, я только начал учиться, пытаюсь освоить ООП и вот в написании небольшого кода у меня возник ступор дальше которого не могу двинуться, есть ошибки, а где не понимаю((((
собственно сам код
тут класс и метод класса

class bibl
{
public:
	int BookCount=0;

	unsigned int inv[50];
	char avtor[50][20];
	char nazva[50][20];
	unsigned int pages[50];
	unsigned int year[50];

	bibl(void);

	bibl(int _inv, char _avtor, char _nazva, int _pages, int _year);

	~bibl(void);

	void add();
	void del(int n);
	void show(int i);
};

метод класса

#include "stdafx.h"
#include <iostream>
#include "bibl.h"
#include <locale>
#include "windows.h"

using namespace std;

bibl::bibl(void)
{
}

bibl::bibl(int _inv, char _avtor, char _nazva, int _pages, int _year)
{
}

bibl::~bibl(void)
{
}


void bibl::add()
{
	BookCount++;
	cout << "vvedit inv nomer: ";
	cin >> inv[BookCount];

	cout << "vvedit avtor: ";
	cin >> avtor[BookCount];

	cout << "vvedit nazva: ";
	cin >> nazva[BookCount];

	cout << "vvedit kilkist storinok: ";
	cin >> pages[BookCount];

	cout << "vvedit rik vydannya: ";
	cin >> year[BookCount];
}

void bibl::del(int n)
{
	                                                                     
	BookCount--;
	cout << " inv nomer: " << inv[n];

	cout << "avtor: " << avtor[n];

	cout << "nazva: " << nazva[n];

	cout << "kilkist storinok: " << pages[n];

	cout << "rik vudanya: " << year[n];
}

void bibl::show(int i)
{
	for (int i = 0; i<BookCount; i++)
	{
		                                                                    
		cout << "inv nomer: " << inv[i];

		cout << "avtor: " << avtor[i];

		cout << "nazva: " << nazva[i];

		cout << "kilkist storinok: " << pages[i];

		cout << "rik vudanya: " << year[i];
	}
}

сильно только не смейтесь)))

______________________
Используйте тег [CODE] (кнопка с решеткой # в форме сообщения) при вставке кода на форум.

[p51x]

Ошибкак то какая? А так для размышления:
1. Зачем вам класс, если все поля в паблике?
2. Что вы собрались иницилизировать в библиотеке инфой об одной книге?
3. При добавлении почему вы начинаете писать в массивы с 1?
4. Почему удаление не удаляет по индексу, как следует из интерфеса, а с конца?
5. Почему нет проверки при добавлении на ваши 50? Почему нет проверки при удалении на индекс?
6. Зачем вы включаете неиспользуемые заголовки?
7. Воид не надо в С++ писать в параметрах
8. Зачем вам диструктор, если уничтожать в ручную тут ничего не надо?
9. Не стоит писать пустой дефолтный конструктор. Если нужен пишите =default.

[radistkmm]

Там должна быть не одна книга, что то типа базы данных, с возможностью забивать книги, а потом вызывать информацию о них.
Я еще плохо шарю во всем этом, вот и накидал такое...
А ошибок там достаточно, вот и пытаюсь разгрести, код запускается но дальше ад какой то..
Директивы то я уберу лишние, это фигня, у меня проблема с методом класса где то, данные ввести я могу, а когда их вызываю выдает кучу не нужных символов
Какую проверку Вы имеете в виду?
Вы имели ввиду здесь воид не нужно писать?
bibl::bibl(void)
{
}

bibl::bibl(int _inv, char _avtor, char _nazva, int _pages, int _year)
{
}

bibl::~bibl(void)

[Alex11223]

Ввод/вывод обычно не делают внутри таких классов. Делайте ввод снаружи и передавайте параметрами функций класса, а результат возвращайте и выводите снаружи что надо. Посмотрите на стандартные контейнеры (std::vector, ...).

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

Я еще плохо шарю во всем этом, вот и накидал такое...

Хватит оправдываться всякой фигней, лучше сообщение выше читайте до понимания.

[p51x]

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

Там должна быть не одна книга, что то типа базы данных, с возможностью забивать книги, а потом вызывать информацию о них.

Вы в конструктор библиотеки передеаете парамтры 1 книги, что вы этим хотите сказать с точки зрения логики, без программирования?

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

у меня проблема с методом класса где то, данные ввести я могу, а когда их вызываю выдает кучу не нужных символов

Я вам уже писал, заносите вы с 1, 0-й элемент не инициализирован. А выводите с 0.

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

Какую проверку Вы имеете в виду?

Я обе описал.

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

Вы имели ввиду здесь воид не нужно писать?

Да. Это в С есть разница между () и (void).

[radistkmm]

Хоть я так и не понял что к чему, но все равно спасибо)

[SAMOUCHKA]

а что тут не понятного? У тебя библиотека состоит из книг. Так опиши эту сущность, создай класс книга Book.
ну что там будет- название, автор, год. Все эти поля определи приватными. Наверное лучше сделать конструктор с параметрами ( название, автор, год). И функции геттеры. Если редактировать не надо, сеттеры не нужны.
Я бы в место массивов char использовал std::string.

Ну а далее нам твои книги надо поместить в библиотеку. Можно, для хранения использовать массив или вообще класс написать. Но зачем есть же стандартные контейнеры, вектор например.

std::vector<Book> bibl;

[radistkmm]

Получается мне нужно дописать еще один класс и описание к нему, правильно?
std::string это что? контейнер?

[Alex11223]

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

std::string это что? контейнер?

Вас в гугле забанили? (если да — используйте VPN)

Читайте и гуглите непонятные слова.

[SAMOUCHKA]

напиши класс книга (Book), а уже объекты этого класса помещай в библиотеку.

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

std::string это что? контейнер?

ну типа того. это тип данных для работы со строками. используй его вместо массива char[] гораздо удобнее