Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C/C++: Общие вопросы > вернуть ссылку на объект


Автор: 4epT 16.11.2009, 20:08
Добрый вечер! имеется класс (className) в нем поля с обычными типами (int, wstring). И имеется класс в котором есть вектор (vector<className>). Нужно из метода вернуть отдельный объект, и что изменения сделанные в main отразились в самом векторе.

есть такое метод

Код

className& get(int index)
{
        return this->lst[index];
}


такой код не работает, как мне объяснили вот тут "this->lst[index];" вызывает конструктор копирования по умолчанию, и уже возвращается компия объекта. Как изменить метод ?

Конструктор копирования я не переопределял.

Автор: zim22 16.11.2009, 20:21
Цитата(4epT @  16.11.2009,  19:08 Найти цитируемый пост)
такой код не работает,

всё работает
Код

#include <vector>

struct className {
  className() : x(0) { }
  className &operator=(int value) {
    x = value;
    return *this;
  }
  int x;
};

std::vector<className> lst(10);
className& get(int index)
{
  return lst[index];
}
int main()
{
  get(5) = 99;
  return 0;
}

Автор: 4epT 16.11.2009, 20:28
то есть что бы заработало нужно просто перегрузить оператор равно ?

Автор: SenkraD 16.11.2009, 20:29
если ты не определял конструктор копирования, то его создаст компилер.
Два, не вызывается там конструктор копирования, если только переменная lst не
кустомный класс с кривым оператором []

Автор: zim22 16.11.2009, 20:35
Цитата(4epT @  16.11.2009,  19:28 Найти цитируемый пост)
то есть что бы заработало нужно просто перегрузить оператор равно ?

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

Автор: 4epT 16.11.2009, 20:42
спасибо! Буду искать где ошибся

Автор: SenkraD 16.11.2009, 20:43
4epT, выложи минимальный приммер воспроизводящий ошибку - поможем

Автор: 4epT 16.11.2009, 21:01
Хорошо.

Вот h файл класса.

Код

#pragma once
#include "Nationality.h";
#include "Human.h"
#include "Footballer.h"
#include "Coach.h"
#include "Doctor.h"
#include "FootballTeam.h"

#include <string>
#include <vector>

using namespace std;

class FootballerTeamDB
{
private:
    
public:
    static vector<FootballTeam> lst;
    FootballerTeamDB();

    static FootballTeam getFootballerTeamOnIndex(int index);
    static FootballTeam& getFootballerTeamOnId(int id);
    static int addFootballerTeam(FootballTeam ft);//Если команда с таким названием уже существует вернется -1, иначе индекс с которым был добавлен элемент
    static void deleteFootballerTeamOnIndex(int index);
    static int getIndexFootballerTeamOnTitle(wstring title);
    static void updateFootballerTeamOnIndex(FootballTeam ft, int index);
    static int getSize();

    //Генерирование базы данных футбольных команд
    static void generateShahter();
    static void generateDynamo();
    static void generateMetallurg();
    static void generateDnepr();
    
    ~FootballerTeamDB();
};



а вот cpp файл.

Код

FootballTeam& FootballerTeamDB::getFootballerTeamOnId(int id)
{
    for(int i = 0; i < lst.size(); i++)
    {
        if(lst[i].getId() == id)
        {
            return lst[i];
        }
    }

    FootballTeam res(L"NULL");
    return res;
}


это именно тот метод

вот файл main:

Код

#pragma once
#include "Nationality.h";
#include "Human.h"
#include "Footballer.h"
#include "Coach.h"
#include "Doctor.h"
#include "FootballTeam.h"
#include "FootballerTeamDB.h"
#include "League.h"
#include "LeagueDB.h"
#include "linkClass.h"
#include "linkClassDB.h"

#include <iostream>

using namespace std;

int main()
{
    Nationality::init();
    
    FootballerTeamDB::generateDynamo();

    FootballTeam ft = FootballerTeamDB::getFootballerTeamOnId(0);

    ft.setTitle(L"Ololo");

    FootballerTeamDB::getFootballerTeamOnId(0).printShortly();
    ft.printShortly();

    return 0;
}


должно поменять имя и в векторе lst (класса FootballerTeamDB) и в переменной ft.
а не меняет (

Автор: zim22 16.11.2009, 21:44
Цитата(4epT @  16.11.2009,  20:01 Найти цитируемый пост)
   FootballTeam res(L"NULL");    return res;

нельзя возвращать ссылку на локальный объект. который скорей всего и возвращается.

Автор: 4epT 16.11.2009, 22:38
Ну насчет локального объекта понял, только возвращается не он ... а именно тот объект что нужен.

и каким способом можно в С++ вернуть "ничего" ?

Автор: zim22 16.11.2009, 22:51
Цитата(4epT @  16.11.2009,  21:38 Найти цитируемый пост)
и каким способом можно в С++ вернуть "ничего" ?

Код

int *fcn() {
  return 0;
}

void fcn() {
  return;
}

int &fcn() {
  static int nothing = 0;
  return nothing;
}

***
интересно, кто это минусует мои сообщения... smile

Автор: 4epT 16.11.2009, 22:55
нет, я имею ввиду когда нужно вернуть объект ... и такого объекта нет в векторе.

что бы не писать 

Код

    FootballTeam res(L"NULL");
    return res;

Автор: mes 16.11.2009, 23:02
Цитата(4epT @  16.11.2009,  21:55 Найти цитируемый пост)
нет, я имею ввиду когда нужно вернуть объект ... и такого объекта нет в векторе.

по ссылке вернуть "ничего" нельзя, поэтому обязательно надо использовать только валидный диапазон или можно  возвращать по указателю.

Автор: 4epT 16.11.2009, 23:07
оки ... и я так и не понял, почему у меня не работает мой код ?

Автор: zim22 17.11.2009, 10:21
Цитата(4epT @  16.11.2009,  22:07 Найти цитируемый пост)
 я так и не понял, почему у меня не работает мой код ?

я так понял, что здесь изменяется значение объекта.
Цитата(4epT @  16.11.2009,  20:01 Найти цитируемый пост)
ft.setTitle(L"Ololo");

код функции в студию.

Автор: 4epT 17.11.2009, 10:30
Цитата(zim22 @ 17.11.2009,  10:21)
Цитата(4epT @  16.11.2009,  22:07 Найти цитируемый пост)
 я так и не понял, почему у меня не работает мой код ?

я так понял, что здесь изменяется значение объекта.
Цитата(4epT @  16.11.2009,  20:01 Найти цитируемый пост)
ft.setTitle(L"Ololo");

код функции в студию.

Код

void FootballTeam::setTitle(wstring aTitle)
{
    this->title = aTitle;
}


h файл класса FootballerTeam (объекты этого класса содержаться в векторе ...)

Код

#pragma once
#include "Coach.h"
#include "Doctor.h"
#include "Footballer.h"
#include "Human.h"
#include "Nationality.h"
#include <vector>
#include <string>

using namespace std;

class FootballTeam
{
private:
    static int currentID;

    wstring title;
    int id;
    Coach coach;
    vector<Doctor> doctors;
    vector<Footballer> footballers;
public:
    FootballTeam(wstring aTitle);
    
    void setCoach(Coach coach);
    void setTitle(wstring aTitle);
    int getId();
    void addDoctor(Doctor doc);
    void addFootballer(Footballer fb);
    wstring getTitle();
    Coach getCoach();
    Doctor getDoctorOnIndex(int index);
    Footballer getFootballerOnIndex(int index);
    
    void printShortly();
    void printDetails();

    ~FootballTeam();
};

Автор: zim22 17.11.2009, 10:38
Цитата(4epT @  17.11.2009,  09:30 Найти цитируемый пост)
void FootballTeam::setTitle(wstring aTitle)
{   
 this->title = aTitle;
}

проверь дебаггером. неужели title не присваивается новое значение?
и каким образом изменение этого title должно привести к изменению в lst?

Автор: 4epT 17.11.2009, 10:45
При запуске программы я заполняю статический вектор класса FootballerTeamDB, в промежуточную переменную возвращаю объект (из вектора), причем возвращается ссылка. на сколько я понимаю любые изменения в промежуточном объекте приведут к изменению в объекте вектора. Но не приводят ((

Зачем смотреть дебагером? если видно что у переменной ft значение title поменялось. Я ведь делаю принт в main файле ...

Добавлено через 2 минуты и 7 секунд
При запуске программы я заполняю статический вектор класса FootballerTeamDB, в промежуточную переменную возвращаю объект (из вектора), причем возвращается ссылка. на сколько я понимаю любые изменения в промежуточном объекте приведут к изменению в объекте вектора. Но не приводят ((

Зачем смотреть дебагером? если видно что у переменной ft значение title поменялось. Я ведь делаю принт в main файле ...

Автор: zim22 17.11.2009, 10:52
Цитата(4epT @  17.11.2009,  09:45 Найти цитируемый пост)
 в промежуточную переменную возвращаю объект (из вектора), причем возвращается ссылка. на сколько я понимаю любые изменения в промежуточном объекте приведут к изменению в объекте вектора. Но не приводят ((

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

Автор: 4epT 17.11.2009, 11:10
Почему ? Я ведь вернул ссылку! И теперь в промежуточная переменная ссылается на переменную из вектора ...

Автор: zim22 17.11.2009, 11:12
Цитата(4epT @  17.11.2009,  10:10 Найти цитируемый пост)
Почему ? Я ведь вернул ссылку!

нужно указать, что это ссылка была возвращена

Код

#include <vector>

std::vector<int> &fcn() {
  static std::vector<int> vi(10);
  return vi;
}
int main()
{
  std::vector<int> &v = fcn(); // !!! &v - ссылка
  v[2] = 99;
  int r = fcn()[2];

  return 0;
}

Автор: xvr 17.11.2009, 14:59
Ошибка в main, в строке 24:
Код

    FootballTeam ft = FootballerTeamDB::getFootballerTeamOnId(0);
там должно быть
Код

    FootballTeam & ft = FootballerTeamDB::getFootballerTeamOnId(0);
ссылку то ты вернул, а вот проинициализировал ею в объект, а не ссылку.

Автор: 4epT 17.11.2009, 22:33
Спасибо вам =)) получилось!!

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)