
平台论坛博客文库

› 论坛 › 程序设计 › C/C++ › error: taking address of temporary [-fpermissive]

[C++] error: taking address of temporary [-fpermissive] [复制链接]

时间看来

白手起家

论坛徽章:: 0

电梯直达

1楼 [收藏(0)] [报告]

发表于 2012-09-20 21:02 |只看该作者 |倒序浏览

本帖最后由时间看来于 2012-09-20 21:16 编辑

///**
//欢迎转载，请注明出处。
//
//Flyweight模式
//
//
//官方描述：
//
//意图
// 运用共享技术有效地支持大量细粒度的对象。
//
//适用性
// 1、一个应用程序使用了大量的对象。
// 2、完全由于使用大量的对象，造成很大的存储开销。
// 3、对象的大多数状态都可变为外部状态。
// 4、如果删除对象的外部状态，那么可以用相对较少的共享对象取代很多组对象。
// 5、应用程序不依赖于对象标识。由于Flyweight 对象可以被共享，对于概念上明显有别的对象，标识测试将返回真值。
//
//
//我的描述：
//
// 在开始设计类的时候，很自然的根据现实情况把某些属性作为类的一部分，例如设计一个图书的管理系统，每本书都有出版社、
//编号、作者、价钱等的属性，所以这个书的类可以这样设计：
//
//class Book
//{
//public:
// string GetPublish() {return m_publishCompany;}
// string GetWriter() {return m_writer;}
// int GetBookID() {return m_bookID;}
// int GetPrice() {return m_price;}
// string GetName() {return m_name;}
//
// void SetPublish(string &s) {m_publishCompany = s;}
// void SetWriter(string &s) {m_writer = s;}
// void SetBookID(int id) {m_bookID = id;}
// void SetPrice(int price) {m_price = price;}
// void SetName(string &s) {m_name = s;}
//
//private:
// string m_publishCompany; // 出版社
// string m_writer; // 作者
// int m_bookID; // 书籍编号
// int m_price; // 价钱
// string m_name; // 书名
//};
//
// 这非常自然。但是随着系统的开发和试运行，这种设计就表现出一些不足的地方了，例如一般图书会非常多，成千上万本；
//如果每本书都耗费多一些空间，那整个系统所耗费的空间是不可想象的。所以，我们就要想办法来解决这个问题。
// 然后高明的面向对象设计者就想出了这个办法，把一些相同的属性（把它叫做“享元”）提取出来，并用一个表来管理（池），
//这就是Flyweight模式的思想所在。好了，我们现在来看看示意图：
//
//
//
//
//
// 这里的ConcreteFlyweight就是需要共享的属性（享元），例如上面的出版社和作者，这里的UnsharedConcreteFlyweight就是
//不用共享的属性（书籍编号和价钱）。
//
// 想象一下系统里面Book类的使用：
// 1、先看看不用共享模式：
//
// Book book1, book2, book3;
// book1.SetPublish("机械工业出版社");
// book1.SetWriter("候捷");
// book1.SetBookID(0000);
// book1.SetPrice(20);
// book1.SetName("C++好野");
//
// book2.SetPublish("人民邮电出版社");
// book2.SetWriter("候捷");
// book2.SetBookID(0001);
// book2.SetPrice(30);
// book2.SetName("C++是好劲");
//
// book3.SetPublish("机械工业出版社");
// book3.SetWriter("一雨田");
// book3.SetBookID(0002);
// book3.SetPrice(50);
// book3.SetName("C++无得顶，我是铁头功...");
//
// 这里有两个“机械工业出版社”和两个“候捷”，重复了。如果使用共享模式的话，这里的浪费就可以避免了。
// 别看这里只有两个，想象一下成千上万本书时的空间浪费...
//
// 2、使用共享模式，把出版社和作者两个属性作为享元(Fly~~Weight)
// PublishFlyweightFactory pff;
// WriterFlyweightFactory wff;
// Book book1, book2, book3;
//
// book1.SetPublish(pff.GetPublish("机械工业出版社")->GetName());
// book1.SetWriter(wff.GetWriter("候捷")->GetName());
// book1.SetBookID(0000);
// book1.SetPrice(20);
// book1.SetName("C++好野");
//
// book2.SetPublish(pff.GetPublish("人民邮电出版社")->GetName());
// book2.SetWriter(wff.GetWriter("候捷")->GetName());
// book2.SetBookID(0001);
// book2.SetPrice(30);
// book2.SetName("C++是好劲");
//
// book3.SetPublish(pff.GetPublish("机械工业出版社")->GetName());
// book3.SetWriter(wff.GetWriter("一雨田")->GetName());
// book3.SetBookID(0002);
// book3.SetPrice(50);
// book3.SetName("C++无得顶，我是铁头功...");
//
// 为什么使用了PublishFlyweightFactory和WriterFlyweightFactory之后就可以节省空间了呢？
//
// 奥妙就在于GetPublish和GetWriter的实现：
//
// PublishFlyweight* GetPublish(string key)
// {
// PublishFlyweight *p;
// mapPublish::iterator it;
// it = mapPublish.find(key);
//
// // 存在这个Writer
// if(it != mapPublish.end() )
// p = it;
//
// else
// {// 插入这个PublishFlyweight
//
// p = new PublishFlyweight(key);
// mapPublish[key] = p;
// }
//
// return p;
// }
//
// GetWriter的实现大同小异，这里就不列出来了，请看详细代码里的实现。
//*/
// Flyweight.cpp
#pragma warning(disable: 4786)
#include <iostream>
#include <map>
#include <string>
#include <cstring>
using namespace std;
class Book
{
public:
string GetPublish() {return *m_publishCompany;}
string GetWriter() {return *m_writer;}
int GetBookID() {return m_bookID;}
int GetPrice() {return m_price;}
string GetName() {return m_name;}
void SetPublish(string *s) {m_publishCompany = s;}
void SetWriter(string *s) {m_writer = s;}
void SetBookID(int id) {m_bookID = id;}
void SetPrice(int price) {m_price = price;}
void SetName(string s) {m_name = s;}
private:
string *m_publishCompany; // 出版社
string *m_writer; // 作者
int m_bookID; // 书籍编号
int m_price; // 价钱
string m_name; // 书名
};
class PublishFlyweight
{
public:
PublishFlyweight(string s)
{
m_name = s;
}
string GetName()
{
return m_name;
}
private:
string m_name;
};
class PublishFlyweightFactory
{
public:
PublishFlyweight* GetPublish(string key)
{
PublishFlyweight *p;
map<string, PublishFlyweight*>::iterator it;
it = mapPublish.find(key);
// 存在这个出版社
if(it != mapPublish.end() )
{
// 这里可能有点难懂，请查阅ＳＴＬ的帮助文档
// 其实second就是指 map<string, PublishFlyweight*> 的 PublishFlyweight*
p = (*it).second;
cout << "已经有这个出版社: " << p->GetName() << " 你节省了" << strlen(p->GetName().c_str()) << "字节的空间" << endl;
}
else
{// 插入这个PublishFlyweight
p = new PublishFlyweight(key);
mapPublish[key] = p;
}
return p;
}
private:
map<string, PublishFlyweight*> mapPublish;
};
class WriterFlyweight
{
public:
WriterFlyweight(string s)
{
m_name = s;
}
string GetName()
{
return m_name;
}
private:
string m_name;
};
class WriterFlyweightFactory
{
public:
WriterFlyweight* GetWriter(string key)
{
WriterFlyweight *p;
map<string, WriterFlyweight*>::iterator it;
it = mapWriter.find(key);
// 存在这个Writer
if(it != mapWriter.end() )
{
// 这里可能有点难懂，请查阅ＳＴＬ的帮助文档
// 其实second就是指 map<string, WriterFlyweight*> 的 WriterFlyweight*
p = (*it).second;
cout << "已经有这个作者名字: " << p->GetName() << " 你节省了" << strlen(p->GetName().c_str()) << "字节的空间" << endl;
}
else
{// 插入这个PublishFlyweight
p = new WriterFlyweight(key);
mapWriter[key] = p;
}
return p;
}
private:
map<string, WriterFlyweight*> mapWriter;
};
void ShowBookInfo(Book book)
{
cout << "书名：" << book.GetName() << endl;
cout << "编号：" << book.GetBookID() << endl;
cout << "价钱：" << book.GetPrice() << endl;
cout << "出版：" << book.GetPublish() << endl;
cout << "作者：" << book.GetWriter() << endl;
cout << endl;
}
int main()
{
PublishFlyweightFactory pff;
WriterFlyweightFactory wff;
Book book1, book2, book3;
book1.SetPublish( &(pff.GetPublish("机械工业出版社")->GetName()) );
book1.SetWriter( &(wff.GetWriter("候捷")->GetName()) );
book1.SetBookID(0000);
book1.SetPrice(20);
book1.SetName(string("<<C++好野>>"));
ShowBookInfo(book1);
book2.SetPublish( &(pff.GetPublish("人民邮电出版社")->GetName()) );
book2.SetWriter( &(wff.GetWriter("候捷")->GetName()) );
book2.SetBookID(0001);
book2.SetPrice(30);
book2.SetName(string("<<C++是好劲>>"));
ShowBookInfo(book2);
book3.SetPublish( &(pff.GetPublish("机械工业出版社")->GetName()) );
book3.SetWriter( &(wff.GetWriter("一雨田")->GetName()) );
book3.SetBookID(0002);
book3.SetPrice(50);
book3.SetName(string("<<C++无得顶，我是铁头功...>>"));
ShowBookInfo(book3);
return 0;
}

复制代码

编译信息：

**** Build of configuration Debug for project DesignPattern_one ****
**** Internal Builder is used for build ****
g++ -O0 -g3 -Wall -c -fmessage-length=0 -o Flyweight.o ..\Flyweight.cpp
..\Flyweight.cpp:137:0: warning: ignoring #pragma warning [-Wunknown-pragmas]
..\Flyweight.cpp: In function 'int main()':
..\Flyweight.cpp:283:65: error: taking address of temporary [-fpermissive]
..\Flyweight.cpp:284:53: error: taking address of temporary [-fpermissive]
..\Flyweight.cpp:291:65: error: taking address of temporary [-fpermissive]
..\Flyweight.cpp:292:53: error: taking address of temporary [-fpermissive]
..\Flyweight.cpp:299:65: error: taking address of temporary [-fpermissive]
..\Flyweight.cpp:300:55: error: taking address of temporary [-fpermissive]
Build error occurred, build is stopped
Time consumed: 1459 ms.

复制代码

首先，既然网友晒出的代码，应该是经过测试过的。难道是因为我使用的编译器，会产生出这个错误！
这是怎么回事呢？
这个临时变量是指的哪个？
该如何修改呢？我想了哈，没有搞定~

谢谢您的回答！

文库|博客

linux_c_py_php

巨富豪门

论坛徽章:: 4

2楼 [报告]

发表于 2012-09-20 21:54 |只看该作者

操作了临时内存?

网友的代码难道就一定是正确的? 反思.

实战分享：从技术角度谈机器学习入门| 【大话IT】RadonDB低门槛向MySQL集群下战书 | ChinaUnix打赏功能已上线！ | 新一代分布式关系型数据库RadonDB知多少？

时间看来

白手起家

论坛徽章:: 0

3楼 [报告]

发表于 2012-09-20 22:03 |只看该作者

本帖最后由时间看来于 2012-09-20 22:21 编辑

回复 2# linux_c_py_php

谢谢你热心的回答。昨天你也解决了我的一个困惑，多谢！
嗯，自己运行下，改一改，理解得更好。

pff.GetPublish("机械工业出版社")，返回的是指针啊。
动态分配的内存，p = new WriterFlyweight(key);还没有delete啊。&(pff.GetPublish("机械工业出版社")->GetName())有什么问题呢？

linux_c_py_php

巨富豪门

论坛徽章:: 4

4楼 [报告]

发表于 2012-09-20 22:06 |只看该作者

&(pff.GetPublish("机械工业出版社")->GetName())

你自己都分析完了... 还没反应过来么-, -

GetPublish返回的是new出来的对象的话, 你没有留住那个内存的地址, 你就没机会释放掉它, 就是内存泄漏了.

时间看来

白手起家

论坛徽章:: 0

5楼 [报告]

发表于 2012-09-20 22:20 |只看该作者

回复 4# linux_c_py_php

在这个应用程序结束前，如果分配的内存不再用到，那才算泄漏吧。
pff.GetPublish("机械工业出版社")，得到分配的PublishFlyweight(key)，不能用吗？
我尝试return mapWriter[key];可还是不行。

印象中，应用程序结束后，泄漏的内存会被回收。所以这个小程序可以忽略这个问题吧，虽然不太好。