Классы C++, Объектно-ориентированное программирование

Конкретная причина того, что ничего не печатается, заключается в том, что в setstring p является копией указателя имени, а не ссылкой на него. Попробуйте изменить подпись setstring на:

void setstring(const char* s, char*& p);

(обратите внимание на &).

См. другие ответы для других значительных ошибок в коде - если эти проблемы не будут устранены, вы, вероятно, получите сбои или странное поведение.

И если целью кода не является просто изучение динамических массивов, используйте вместо этого std::string :-).

Если вы хотите правильно программировать объектно-ориентированное программирование на C++, вам, возможно, следует избегать прямого управления указателями, а использовать строковый класс STL и использовать ссылки вместо указателей. Тогда вам должно быть легче, и ваш источник должен выдавать правильный результат.

В противном случае проверьте свой конструктор класса person, массив имен имеет всего 3 элемента, но вы ссылаетесь на 4-й (индексы 0-2!). Также в методе setstring() вы не выделяете достаточно места для завершающего '\0' в массиве!

Для начинающих:

name=new char[3];
strcpy(name,"NILL");
name[3]='\0';

имя состоит из трех элементов — вы обращаетесь с ним так, как будто оно состоит из пяти. В том же духе:

p=new char[strlen(s)];

должно быть:

p=new char[strlen(s) + 1];

На самом деле эта функция совершенно неверна — предполагается, что вы копируете строку в «имя», а не в таинственный параметр «p». Если вам действительно нужна эта функция, то "p" должен быть указателем на указатель или ссылкой на указатель.

Вы думали об использовании std::string из STL?

Хотя первая проблема, которую я вижу, заключается в следующем.

person::person()
{
name=new char[3];
strcpy(name,"NILL");
name[3]='\0';
}

Вы выделяете массив символов, размер массива составляет 3 символа, затем вы копируете в него «NILL» с помощью strcpy, поэтому вы заполняете массив всеми символами, но без нулевого терминатора \0. «NILL» — это константная строка, и она имеет нулевой терминатор, который является неявным, но не отображается, например, «NILL\0». \0 — это управляющий символ, который используется для обозначения конец строки. Затем у вас есть индекс за пределами при доступе к 3-му элементу массива имен, когда размер вашего массива равен 3.

Чтобы помочь вам найти другие части, которые могут работать неправильно, вот несколько ссылок на используемые вами функции.

strcpy
strlen

strcpy скопирует всю строку из одного буфера в другой, включая управляющий символ нулевого конца.

Strlen вернет количество символов между началом строки и конечным нулевым символом. Хотя он не будет считать нулевой символ символом. Вот почему вы увидите предложения с strlen(string)+1 для включения нулевого терминатора.

Хотя у вас все хорошо, продолжайте в том же духе, вы поймете эти маленькие хитрости при использовании функций стандартной библиотеки.

Предположим, что s имеет 5 символов. Затем new char[strlen(s)]; выделяет память из 5 символов. Тогда p[strlen(s)]='\0' эквивалентно p[5]=0. Это очень очень плохо. p[5] не существует.

void person::setstring(const char* s, char*p) 
{ 
if(s!=NULL) 
{ 
delete[] p; 
p=new char[strlen(s)]; 
strcpy(p,s); 
p[strlen(s)]='\0';    //<<<< LOOK! Out of range
} 
} 

Это скорее C, чем C++ код. Существует эквивалент С++:

#include <string>
#include <iostream>

class person
{
    std::string name_;
public:
    person();
    //preferable design is to set object invariant in appropriate constructor
    explicit person(const std::string &name);
    std::string get_name() const;
    void set_name(const std::string &name);
    void print()const;
};
person::person()
 : name_("NILL")
{}

person::person(const std::string &name)
: name_(name)
{}

std::string person::get_name() const
{
    return name_;
}

void person::set_name(const std::string &name)
{
    name_ = name;
}
void person::print()const
{
    std::cout<<"Name: "<<name_<<std::endl;
}

int main()
{
    person person1;
    person1.set_name("Zia ur Rahman");
    person1.print();

    //this is better design decision
    //because after constructor we have valid object
    person person2("Zia ur Rahman");
    person2.print();
    std::cin.get();
    return 0;
}