Ошибка при написании класса: error LNK2019: ссылка на неразрешенный внешний символ

@RayPas · Регистрация: 18.01.2013

Author24 — интернет-сервис помощи студентам

Код:

C++

int _tmain(int argc, _TCHAR* argv[])
{
    Fraction x, x2, y, y2, fl, res, c;
    
    cout << "..." << endl;
    x.setFraction();
    
    cout << "..." << endl;
    y.setFraction();
 
    cout << "Умножение" << endl;
    res = x * y;
    res.ShowFraction();
 
...
}

Описание методов:

C++

void Fraction :: setFraction()
     { int numerator;
        char c;
        int denominator;
        
        
        cin >> numerator;
        cin >> c;
        cin >> denominator;
        m_Numerator = numerator;
        m_Denominator = denominator;
}
 
 Fraction Fraction :: operator* (const Fraction a) {
    Fraction tmp; //результат умножения двух дробей
    tmp.m_Numerator = m_Numerator*a.m_Numerator;
    tmp.m_Denominator = m_Denominator*a.m_Denominator;
    return tmp;
};

и при компиляции выдает ошибку при участии строки "res = x * y;", т.е. вводится дробь нормально, а дальше такая петрушка, в чем проблема? ах да, и пишет, что-то типа "error LNK2019: ссылка на неразрешенный внешний символ..."

@ShadowFirst · 12.02.2014, 14:32

А вы в аргументе перегруженного оператора принимайте ссылку, то есть:

C++

1	Fraction Fraction :: operator* (const Fraction a) {

заменить на

C++

1	Fraction Fraction :: operator* (const Fraction &a) {

@dzrkot · 12.02.2014, 14:33

по-моему из-за того что оператор= не перегружен

@RayPas · 12.02.2014, 14:44 **[ТС]**

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

по-моему из-за того что оператор= не перегружен

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

Добавлено через 20 секунд

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

А вы в аргументе перегруженного оператора принимайте ссылку, то есть:

C++

1	Fraction Fraction :: operator* (const Fraction a) {

заменить на

C++

1	Fraction Fraction :: operator* (const Fraction &a) {

не сработало

@dzrkot · 12.02.2014, 14:45

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

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

именно
вообще если выложите весь код то будет проще)

@ShadowFirst · 12.02.2014, 14:45

Кстати да, = тож нужно перегрузить, как то не подумал об этом сразу.

@RayPas · 12.02.2014, 15:02 **[ТС]**

нужно в Res положить рез-т умножения дробей;
попытался перегрузить присваивание... теперь получается нужно этот метод применить к строчке Res = x*y, но как?

C++

Fraction Fraction :: operator= (const Fraction &a) {
     R = a.R;
     return *this;
 }

R определил как объект класса Fraction...

Добавлено через 8 минут
да, все дело в этом операторе...
оставил только x*y - проект строится без ошибок

@dzrkot · 12.02.2014, 15:06

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

Fraction Fraction :: operator= (const Fraction &a) { R = a.R; return *this; }

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

R определил как объект класса Fraction...

...подумайте тогда над значением а.R ...))
вы не правы, надо создать нвоый обект и вернуть его копию
т.е.

C++

Fraction R();
R.парамерты=a.параметры
return R; 
//примерно так

Ilot · 12.02.2014, 15:10

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

@RayPas · 12.02.2014, 15:13 **[ТС]**

C++

#pragma once
 
namespace Program {
class Fraction
{
private:
    int m_Numerator;
    int m_Denominator;
    int R;
public:
    Fraction(void);
 
 
    int getNumerator() { return m_Numerator; }
    int getDenominator() { return m_Denominator; }
    void setFraction();
    void ShowFraction() const;
    void Fraction :: Control(int &denominator, int  &numerator);
    
    Fraction operator= (const Fraction &a);
    Fraction operator * (const Fraction &a);
    ~Fraction(void);
};
}

C++

#include "stdafx.h"
#include "Fraction.h"
#include <iostream>
 
using namespace std;
 
namespace Program {
 
Fraction::Fraction(void)
{
}
 
 
Fraction::~Fraction(void)
{
}
 
void Fraction :: setFraction()
     { int numerator;
        char c;
        int denominator;
        
        
        cin >> numerator;
        cin >> c;
        cin >> denominator;
        m_Numerator = numerator;
        m_Denominator = denominator;
}
 
 Fraction Fraction :: operator* (const Fraction &a) {
    Fraction tmp; //результат умножения двух дробей
    tmp.m_Numerator = m_Numerator*a.m_Numerator;
    tmp.m_Denominator = m_Denominator*a.m_Denominator;
    return tmp;
};
 
// Fraction Fraction :: operator= (const Fraction &a) {
    // R = a.R;
     //return *this;
// }
 
void Fraction :: ShowFraction() const {
    cout << this -> m_Numerator << '/' << this -> m_Denominator << endl;
}
 
 
void Fraction :: Control(int &denominator, int  &numerator) {
     if (denominator == 0) {cout << "error!try again!"<<endl;}
     if (numerator % denominator != 0) {cout << "error!try again!"<<endl;}
}
 
 
}

ну и главная часть такая же, как выше написал

Ilot · 12.02.2014, 15:24

А вы указали, что используете свое пространство имен?

C++

1	using namespace Program;

@dzrkot · 12.02.2014, 15:27

C++

 Fraction& Fraction :: operator= (const Fraction &a) {
     m_Numerator= a.m_Numerator;
     m_Denominator=a.m_Denominator;
     return *this;
 }

прототип подправьте тоже

@RayPas · 12.02.2014, 15:28 **[ТС]**

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

А вы указали, что используете свое пространство имен?

C++

1	using namespace Program;

да, только таким образом

C++

1	using Program:: Fraction;

и хмм, он заработал!!!
Но! Получается, что R нельзя определять как объект класса, потому что при введении дробей в R забивается мусор, и при отладке он выводит этот мусор через разделенную черту дроби...как быть?

@dzrkot · 12.02.2014, 15:30

Не по теме:

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

Код C++
Fraction R();
R.парамерты=a.параметры
return R;
//примерно так

глупость я написал походу сначала)

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

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

Но! Получается, что R нельзя определять как объект класса, потому что при введении дробей в R забивается мусор, и при отладке он выводит этот мусор через разделенную черту дроби...как быть?

я выше отписал, поправьте перегрузку=, у меня всё заработало с тем что выше

@RayPas · 12.02.2014, 15:31 **[ТС]**

dzrkot, спасиб друг!) и, правда, все заработало!)
ох ты ж и это только умножение...самое легкое!)
просто, про перегрузку оператора присваивания, вообще не знал, классы буквально вчера начал изучать.

Ilot · 12.02.2014, 15:32

Вы R не инициализируете вообще. Вводите какую-то с...

@RayPas · 12.02.2014, 15:34 **[ТС]**

какое -то c это чаровский символ для имитации дроби, а R собсно переменная для реза...

Ilot $Эксперт по математике/физике$ 2044 / 1363 / 393 Регистрация: 16.05.2013 Сообщений: 3,498 Записей в блоге: 6
	12.02.2014, 15:32	16
	Вы R не инициализируете вообще. Вводите какую-то с... 0

@RayPas 2 / 0 / 0 Регистрация: 18.01.2013 Сообщений: 370
	12.02.2014, 15:34 [ТС]	17
	какое -то c это чаровский символ для имитации дроби, а R собсно переменная для реза... 0

@dzrkot zzzZZZ... 527 / 358 / 94 Регистрация: 11.09.2013 Сообщений: 2,041
	12.02.2014, 14:33	3
	по-моему из-за того что оператор= не перегружен 0

@dzrkot zzzZZZ... 527 / 358 / 94 Регистрация: 11.09.2013 Сообщений: 2,041
	12.02.2014, 14:45	5
	Сообщение от RayPas вы имеете в виду оператор присваивание нужно перегрузить? именно вообще если выложите весь код то будет проще) 0

@ShadowFirst 55 / 48 / 13 Регистрация: 31.10.2013 Сообщений: 166
	12.02.2014, 14:45	6
	Кстати да, = тож нужно перегрузить, как то не подумал об этом сразу. 0

Ilot $Эксперт по математике/физике$ 2044 / 1363 / 393 Регистрация: 16.05.2013 Сообщений: 3,498 Записей в блоге: 6
	12.02.2014, 15:10	9
	RayPas, вряд ли проблема в операторе присваения. Приведите весь код пожайлуста. Созданный по умолчанию работает примеоно так как вы написали выше. Повторюсь нужен код. 0

@dzrkot zzzZZZ... 527 / 358 / 94 Регистрация: 11.09.2013 Сообщений: 2,041
	12.02.2014, 15:30	14
	Не по теме: Сообщение от dzrkot Код C++ Fraction R(); R.парамерты=a.параметры return R; //примерно так глупость я написал походу сначала) Добавлено через 1 минуту Сообщение от RayPas Но! Получается, что R нельзя определять как объект класса, потому что при введении дробей в R забивается мусор, и при отладке он выводит этот мусор через разделенную черту дроби...как быть? я выше отписал, поправьте перегрузку=, у меня всё заработало с тем что выше 1

@RayPas 2 / 0 / 0 Регистрация: 18.01.2013 Сообщений: 370
	12.02.2014, 15:31 [ТС]	15
	dzrkot, спасиб друг!) и, правда, все заработало!) ох ты ж и это только умножение...самое легкое!) просто, про перегрузку оператора присваивания, вообще не знал, классы буквально вчера начал изучать. 0