VPF::вернуть ссылку на объект - Форум программистов

4epT

Дата 16.11.2009, 20:08 (ссылка)

(голосов:2)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

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

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

Код


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

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

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

zim22

Дата 16.11.2009, 20:21 (ссылка)

(голосов:2)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(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;
}

--------------------

C++ FAQ Lite

4epT

Дата 16.11.2009, 20:28 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

то есть что бы заработало нужно просто перегрузить оператор равно ?

SenkraD

Дата 16.11.2009, 20:29 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 933
Регистрация: 3.2.2006
Где: Украина::Киев

Репутация: 2
Всего: 23

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

--------------------

Имеющий язык - да не убоится спросить!

zim22

Дата 16.11.2009, 20:35 (ссылка)

(нет голосов)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(4epT @ 16.11.2009, 19:28

)

то есть что бы заработало нужно просто перегрузить оператор равно ?

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

--------------------

C++ FAQ Lite

4epT

Дата 16.11.2009, 20:42 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

спасибо! Буду искать где ошибся

SenkraD

Дата 16.11.2009, 20:43 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 933
Регистрация: 3.2.2006
Где: Украина::Киев

Репутация: 2
Всего: 23

4epT, выложи минимальный приммер воспроизводящий ошибку - поможем

--------------------

Имеющий язык - да не убоится спросить!

4epT

Дата 16.11.2009, 21:01 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

Хорошо.

Вот 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 (ссылка)

(нет голосов)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(4epT @ 16.11.2009, 20:01

)

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

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

--------------------

C++ FAQ Lite

4epT

Дата 16.11.2009, 22:38 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

Ну насчет локального объекта понял, только возвращается не он ... а именно тот объект что нужен.

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

zim22

Дата 16.11.2009, 22:51 (ссылка)

(голосов:2)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(4epT @ 16.11.2009, 21:38

)

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

Код


int *fcn() {
  return 0;
}

void fcn() {
  return;
}

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

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

Это сообщение отредактировал(а) zim22 - 17.11.2009, 10:15

--------------------

C++ FAQ Lite

4epT

Дата 16.11.2009, 22:55 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

нет, я имею ввиду когда нужно вернуть объект ... и такого объекта нет в векторе.

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

Код


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

mes

Дата 16.11.2009, 23:02 (ссылка)

(голосов:1)

Загрузка ...

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 144
Всего: 250

Цитата(4epT @ 16.11.2009, 21:55

)

нет, я имею ввиду когда нужно вернуть объект ... и такого объекта нет в векторе.

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

--------------------

http://opendots.net

4epT

Дата 16.11.2009, 23:07 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

оки ... и я так и не понял, почему у меня не работает мой код ?

zim22

Дата 17.11.2009, 10:21 (ссылка)

(нет голосов)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(4epT @ 16.11.2009, 22:07

)

я так и не понял, почему у меня не работает мой код ?

я так понял, что здесь изменяется значение объекта.

Цитата(4epT @ 16.11.2009, 20:01

)

ft.setTitle(L"Ololo");

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

--------------------

C++ FAQ Lite

4epT

Дата 17.11.2009, 10:30 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

Цитата(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 (ссылка)

(нет голосов)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(4epT @ 17.11.2009, 09:30

)

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

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

--------------------

C++ FAQ Lite

4epT

Дата 17.11.2009, 10:45 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

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

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

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

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

zim22

Дата 17.11.2009, 10:52 (ссылка)

(нет голосов)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(4epT @ 17.11.2009, 09:45

)

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

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

--------------------

C++ FAQ Lite

4epT

Дата 17.11.2009, 11:10 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

Почему ? Я ведь вернул ссылку! И теперь в промежуточная переменная ссылается на переменную из вектора ...

zim22

Дата 17.11.2009, 11:12 (ссылка)

(нет голосов)

Загрузка ...

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69

Цитата(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;
}

Это сообщение отредактировал(а) zim22 - 17.11.2009, 11:13

--------------------

C++ FAQ Lite

xvr

Дата 17.11.2009, 14:59 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 60
Всего: 223

Ошибка в main, в строке 24:

Код


    FootballTeam ft = FootballerTeamDB::getFootballerTeamOnId(0);

там должно быть

Код


    FootballTeam & ft = FootballerTeamDB::getFootballerTeamOnId(0);

ссылку то ты вернул, а вот проинициализировал ею в объект, а не ссылку.

4epT

Дата 17.11.2009, 22:33 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 784
Регистрация: 13.8.2007

Репутация: нет
Всего: 3

Спасибо вам =)) получилось!!

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »