Класс Person

@Tantay · Регистрация: 28.04.2012

Добрый день, форумчане. Решал задание из книги С.Прата, №2 из главы 10. Пусть имеется определение простого класса в нем используется объект string и символьный массив. Напишите программу, которая дополнит реализацию за счет предоставления кода для пока еще не определенных методов. В программе, использующей класс, должны также присутствовать вызовы трех возможных конструкторов (без аргументов, с одним аргументом, с двумя
аргументами) и двух методов отображения. Вот мой код, он компилится, но отображает не так как нужно. Обьясните пожалуйста в чем моя ошибка?

Person.h

C++

#ifndef PERSON_H_
#define PERSON_H_
#include <string>
 
class Person
{
private: 
    static const int LIMIT = 25;
    std::string lname;
    char fname[LIMIT];
 
public:
    Person() {lname=""; fname[0]='\0';};
    Person(const std::string & ln, const char* fn="Heyyou");
    //Person(const std::string & ln, const char* fn);
    ~Person(void);
 
    void Show() const;
    void FormalShow() const;
};
 
#endif

Person.cpp

C++

#include "Person.h"
#include <iostream>
 
 
//Person::Person()
//{
//  //fname[LIMIT] = '\0';
//  //lname = "";
//}
 
Person::Person(const std::string & ln, const char* fn)
{
    lname = ln;
//  strncpy(Person::fname, fn, LIMIT-1);
    fname[LIMIT] = '\0'; 
}
 
 
Person::~Person(void){}
 
void Person::Show() const
{
    using namespace std;
    cout << fname << " " << lname << endl;
}
 
void Person::FormalShow() const
{
    using namespace std;
    cout << lname << ", " << fname << endl;

"UsesPerson.cpp"

C++

#include <iostream>
#include "Person.h"
 
int main()
{
    {
        using namespace std;
 
        Person one;
        Person two = Person ("Smythecraft");
        Person three ("Dimwiddy", "Sam");
 
        one.Show();
        one.FormalShow();
 
        cout <<endl;
        two.Show();
        two.FormalShow();
        
        cout <<endl;
        three.Show();
        three.FormalShow();
        cout <<endl;
    }
    system("pause");
}

@Hugra · 24.08.2013, 17:10

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

Вот мой код, он компилится, но отображает не так как нужно. Обьясните пожалуйста в чем моя ошибка?

У тебя в твоем конструкторе, переменной fname первому символу присваивается символ "конец строки" (\0).

C++

1	fname[LIMIT] = '\0';

замени свою строку на это:

C++

    
#include <cstring> // для работы strncpy()
...
strncpy(fname,fn,LIMIT);
fname[LIMIT-1] = '\0';

@Tantay · 24.08.2013, 17:58 **[ТС]**

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

замени свою строку на это:

C++

1
2
3

#include <cstring>
...
strncpy(fname,fn,LIMIT);

ругается на это (

@Hugra · 24.08.2013, 19:09

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

[/CPP]

ругается на это (

Что ты и где заменил?

@Tantay · 24.08.2013, 19:24 **[ТС]**

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

Что ты и где заменил?

C++

#include "Person.h"
#include <iostream>
#include <cstring> 
 
 
 
Person::Person(const std::string & ln, const char* fn)
{
    lname = ln;
    strncpy(Person::fname, fn, LIMIT);
    fname[LIMIT-1] = '\0'; 
}
 
 
Person::~Person(void){}
 
void Person::Show() const
{
    using namespace std;
    cout << fname << " " << lname << endl;
}
 
void Person::FormalShow() const
{
    using namespace std;
    cout << lname << ", " << fname << endl;

@Olivеr · 24.08.2013, 19:34

полно ошибок. проще переписать

C++

#include <iostream>
#include <string>
#include <cstring>
 
class Person
{
private:
    std::string lname;
    char *fname;
 
public:
    Person() : Person("{null}", "{null}") {}
 
    Person(const std::string &ln)
        : Person()
    {
        lname = ln;
    }
 
    Person(const std::string &ln, const char *fn)
        : lname(ln), fname( new char [strlen(fn) + 1] )
    {
        strcpy(fname, fn);
    }
 
    ~Person()
    {
        delete [] fname;
    }
 
    void Show() const
    {
        std::cout << fname << " " << lname << std::endl;
    }
 
    void FormalShow() const
    {
        std::cout << lname << ", " << fname << std::endl;
    }
};
 
 
int main()
{
    Person one;
    Person two("Smythecraft");
    Person three("Dimwiddy", "Sam");
 
    one.Show();
    one.FormalShow();
 
    std::cout << std::endl;
    two.Show();
    two.FormalShow();
 
    std::cout << std::endl;
    three.Show();
    three.FormalShow();
    std::cout << std::endl;
    system("pause");
}

Добавлено через 41 секунду
http://ideone.com/3DOjoH

@Hugra · 24.08.2013, 20:00

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

C++

#include "Person.h"
#include <iostream>
#include <cstring> 
 
 
 
Person::Person(const std::string & ln, const char* fn)
{
    lname = ln;
    strncpy(Person::fname, fn, LIMIT);
    fname[LIMIT-1] = '\0'; 
}
 
 
Person::~Person(void){}
 
void Person::Show() const
{
    using namespace std;
    cout << fname << " " << lname << endl;
}
 
void Person::FormalShow() const
{
    using namespace std;
    cout << lname << ", " << fname << endl;

C++

1	strncpy(Person::fname, fn, LIMIT);

и

C++

1	strncpy(fname, fn, LIMIT);

есть разница?

@Tantay · 24.08.2013, 20:32 **[ТС]**

Сообщение от Olivеr

полно ошибок. проще переписать

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

Добавлено через 1 минуту

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

Код C++

C++

1	strncpy(Person::fname, fn, LIMIT);

и

C++

1	strncpy(fname, fn, LIMIT);

есть разница?

да есть я не исправил, в посте. просто компилю я на другом компьютере, а в интернете сижу с другого. Но все равно ругается:
1>person.cpp(15): error C4996: 'strncpy': This function or variable may be unsafe. Consider using strncpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 11.0\vc\include\string.h(191): см. объявление "strncpy"

@Hugra · 24.08.2013, 20:46

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

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

Добавлено через 1 минуту

да есть я не исправил, в посте. просто компилю я на другом компьютере, а в интернете сижу с другого. Но все равно ругается:
1>person.cpp(15): error C4996: 'strncpy': This function or variable may be unsafe. Consider using strncpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 11.0\vc\include\string.h(191): см. объявление "strncpy"

В самом начале файла Person.cpp добавте

C++

1	#define _CRT_SECURE_NO_WARNINGS

@Tantay · 24.08.2013, 20:51 **[ТС]**

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

В самом начале файла Person.cpp добавте

C++

1	#define _CRT_SECURE_NO_WARNINGS

спасибо, заработало!)

@Hugra 50 / 50 / 9 Регистрация: 18.07.2013 Сообщений: 138 Записей в блоге: 1
	24.08.2013, 19:09	4
	Сообщение от Tantay [/CPP] ругается на это ( Что ты и где заменил? 0