Вызов метода класса из main

@Athena · Регистрация: 17.03.2015

Как вызвать функцию plus в меин? пробовал вариант р3.=р3.plus(p1,p2), но не получилось

C++

#include "stdafx.h"
#include <iostream>
#include <conio.h>
using namespace std;
 
class Nexia {
protected: 
        int c,m;
public:
    Nexia(int clock, int minit);
    Nexia&plus(Nexia &p1, Nexia &p2);
    int Tget();
    int Mget();
};
 
int Nexia::Tget() { return c; }
int Nexia::Mget() { return m; }
Nexia(int clock, int minit) { c=clock; m=minit;}
 
Nexia&Nexia::plus (Nexia &p1, Nexia &p2) {
    Nexia p3;
    p3.c=p1.c+p2.c;
    p3.m=p2.m+p2.m;
    return p3;
}
 
void main(void)
{
    Nexia p1(10,45),p2(1,25), p3;
}

@DrOffset · 24.09.2016, 14:10

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

C++

Nexia&Nexia::plus (Nexia &p1, Nexia &p2) {
    Nexia p3;
    p3.c=p1.c+p2.c;
    p3.m=p2.m+p2.m;
    return p3;
}

Тут ошибка. Нельзя возвращать ссылку на локальную переменную, т.к. ее время жизни ограничено областью функции. Надо так:

C++

Nexia Nexia::plus (Nexia &p1, Nexia &p2) {
    Nexia p3;
    p3.c=p1.c+p2.c;
    p3.m=p2.m+p2.m;
    return p3;
}

Во-вторых, функция plus весьма странная. Если она принимает оба аргумента, то по логике она должна быть статической. Но что-то подсказывает, что все же имелось в виду такое:

C++

class Nexia {
protected: 
    int c,m;
public:
    Nexia(int clock, int minit);
    Nexia plus(Nexia const & p2) const;
 
    //......
};
 
Nexia Nexia::plus (Nexia const & p2) const {
    return Nexia(c + p2.c, m + p2.m);
}

Соответственно запись в main будет такой:

C++

1 2	//..... p3 = p1.plus(p2);

При этом, чтобы работал конструктор по умолчанию для p3 (строка 29 в исходном примере), его нужно предоставить. Например добавив параметры по умолчанию:

C++

class Nexia {
protected: 
    int c,m;
public:
    Nexia(int clock = 0, int minit = 0);
    Nexia plus(Nexia const & p2) const;
 
    //......
};

Ну и по мелочи.
* В строке 18 забыл квалифицировать имя. Надо так:

C++

1	Nexia::Nexia(int clock, int minit) { c=clock; m=minit;}

В конструкторе лучше пользоваться списком инициализации, вместо присваивания:

C++

1 2	Nexia::Nexia(int clock, int minit) : c(clock), m(minit) { }

* Геттеры должны быть const.

C++

class Nexia {
protected: 
    int c,m;
public:
//.....
    int Tget() const;
    int Mget() const;
};
 
int Nexia::Tget() const { return c; }
int Nexia::Mget() const { return m; }

* Помещение c и m в protected является сомнительным решением, особенно с учетом того, что для них есть геттеры.
* Функция main по стандарту возвращает int.

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

C++

class Nexia {
private: 
    int c, m;
public:
    Nexia(int clock = 0, int minit = 0);
    Nexia plus(Nexia const & p2) const;
    int Tget() const;
    int Mget() const;
};
 
int Nexia::Tget() const { return c; }
int Nexia::Mget() const { return m; }
 
Nexia::Nexia(int clock, int minit) 
    : c(clock), m(minit) 
{ }
 
 
Nexia Nexia::plus(Nexia const & p2) const {
    return Nexia(c + p2.c, m + p2.m);
}
 
int main()
{
    Nexia p1(10,45), p2(1,25), p3;
    
    p3 = p1.plus(p2);
}

@Athena · 24.09.2016, 14:40 **[ТС]**

* Геттеры должны быть const.

а почему?

@DrOffset · 24.09.2016, 14:47

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

а почему?

Ну например, сделаешь ты свой объект Nexia константой. И все, геттерами пользоваться не сможешь:

C++

    const Nexia p3 = p1.plus(p2);
    
    std::cout << p3.Tget() << std::endl; // будет ошибка, если Tget не const
    std::cout << p3.Mget() << std::endl; // будет ошибка, если Mget не const

Хотя по сути ничто этому не мешает. Ведь геттеры не меняют состояние объекта, а значит должны быть const.

Добавлено через 2 минуты
Athena, в общем, книжку вот посоветую, для старта: Стандарты программирования на C++.

@Athena 4 / 4 / 2 Регистрация: 17.03.2015 Сообщений: 47
	24.09.2016, 14:40 [ТС]	3
	* Геттеры должны быть const. а почему? 0

Опции темы

Вызов метода класса из main

Решение