Помогите заполнить массив char из класса

[Farlep]

Добрый день.

Есть 3 файла

PhoneBook.h

#pragma once
#include <iostream>
#include <Windows.h>
#include <string.h>
#include <conio.h>

using namespace std;


class PhoneBook
{

private:

	char *FIO;
	char* MobPhone;

public:
	
	PhoneBook();
	PhoneBook(const char *_fio, const char* _mobphone);
	
	~PhoneBook();

	void ShowPhoneBook();

};

PhoneBook.cpp

#include "PhoneBook.h"

PhoneBook::PhoneBook()
{
	*FIO = 'Z';
	*MobPhone = '9';
}


PhoneBook::PhoneBook(const char* _fio, const char* _mobphone)
{
	FIO = new char[10];
	MobPhone = new char[10];
	*FIO = *_fio;
	*MobPhone = *_mobphone;
}

PhoneBook::~PhoneBook()
{
}

void PhoneBook::ShowPhoneBook()
{
	cout << "ФИО: " << FIO << endl;
	cout << "Тел: " << MobPhone << endl;

Menu.cpp

...

void MenuFun1()
{
	system("cls"); 	

	//gotoxy(25, 10);

	PhoneBook* p = new PhoneBook[3]
	{
		{"A", "9"},
		{"B", "8"},
		{"DD DD", "777"}
	};

	cout << "Телефонная книга" << endl;

	//вывод для теста контакта
	p[2].ShowPhoneBook();
	
        //деструктор
        delete[]p;

	_getch();
	system("cls");

}

...

Не могу никак понять, как скопировать полностью проинициализированный массив PhoneBook* p и вывести без мусора полностью данные {"DD DD", "777"}

Выводится первый символ и мусор.

[Farlep]

Проблема решена.

Все оказалось просто. Вспомнил, что char нужно прогнать циклом и в конце добавить '\0'

for (size_t i = 0; i < 5; i++)
	{
		*(FIO + i) = *(_fio + i);
		if (i == 4)
			*(FIO + i + 1) = '\0';
	}

Дальше буду делать подсчет символов и создавать массив [кол-во + 1 (под '\0')]
И прогонять количество в цикле согласно этому значению.

Вот только не получается передавать по ссылке, что бы уйти от разыменовывания (

Вот это, не ругается

PhoneBook* p = new PhoneBook[3]
	{
		{"A", "9"},
		{"B", "8"},
		{"DD DD", "777"}
	};

только когда так скармливаю

PhoneBook::PhoneBook(const char* _fio, const char* _mobphone)


Как считаете нужно делать так?

PhoneBook::~PhoneBook()
{
	delete []FIO;
	delete []MobPhone;
	
}

Ведь мы выделяем память в

PhoneBook::PhoneBook(const char* _fio, const char* _mobphone)
{
	FIO = new char[10];
	MobPhone = new char[10];
	*FIO = *_fio;
	*MobPhone = *_mobphone;
}

[Farlep]

Ну как то так, в итоге

Как то лучше это сделать можно было?

PhoneBook::PhoneBook(const char* _fio, const char* _mobphone)
{
	int count_fio, count_mobphone;
	count_fio = strlen(_fio);
	count_mobphone = strlen(_mobphone);	
	
	FIO = new char[count_fio + 1];
	MobPhone = new char[count_fio + 1];

	for (size_t i = 0; i < count_fio; i++)
	{
		FIO[i] = _fio[i];
		if (i == count_fio - 1)
			FIO[i + 1] = '\0';
	}

	for (size_t i = 0; i < count_mobphone; i++)
	{
		MobPhone[i] = _mobphone[i];
		if (i == count_mobphone - 1)
			MobPhone[i + 1] = '\0';
	}
}

[Farlep]

Ушел от циклов и сократил код

PhoneBook::PhoneBook(const char* _fio, const char* _mobphone)
{
	int count_fio, count_mobphone;
	count_fio = strlen(_fio);
	count_mobphone = strlen(_mobphone);	
	
	FIO = new char[count_fio + 1];
	MobPhone = new char[count_fio + 1];

	strcpy(FIO, _fio);

	strcpy(MobPhone, _mobphone);

}

[Алексей1153]

Farlep, https://onlinegdb.com/ZL9xv9LMc

[taras-proger77]

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

Как считаете нужно делать так?

Код:

PhoneBook::~PhoneBook()
{
delete []FIO;
delete []MobPhone;

}

Ведь мы выделяем память в

Выделил – освободи. И проверь перед освобождением.