Typedef指针是一个好主意吗?


75

我查看了一些代码,发现约定是将指针类型转换为

SomeStruct* 

进入

typedef SomeStruct* pSomeStruct;

有什么好处吗?


1
它在C语言中很常见。因为您试图隐藏它是指针的事实。这是您传递给库中所有接口的对象。在C ++中,虽然不是闻所未闻,但并不常见且不鼓励使用。
马丁·约克2009年

4
关于此一般主题的有趣文章:对话:仲夏夜之疯狂 另请参阅Linux内核编码样式,以获取“不要使用typedef指针(也不要为结构或联合提供typedef)”的极端观点。
乔纳森·莱夫勒

Answers:


108

当指针本身可以视为“黑匣子”时,即内部数据与代码无关的一条数据时,这可能是适当的。

本质上,如果您的代码永远不会取消引用指针,而只是将其传递给API函数(有时通过引用),则typedef不仅会减少*代码中的s数量,而且还会向程序员建议指针不应真的不喜欢。

如果需要的话,这也使得将来更容易更改API。例如,如果更改为使用ID而不是指针(反之亦然),则现有代码不会中断,因为从一开始就不应该取消指针的引用。


9
真实示例:在OS X中,CoreFoundation框架广泛使用了此技术,将它们称为“不透明”数据类型。
hbw

3
同样在OS X中:pthread_t是不透明的类型。类型定义为指向“结构_opaque_pthread_t”的指针,该结构本身是不透明的字节数组。
亚当·罗森菲尔德2009年

13
反例:FILE * fp?实际上,我同意你的看法,但是有很多相反的先例。
乔纳森·莱夫勒

11
'当然,如果执行此操作,则不会将不透明类型命名为“ pFoo”。
MSalters

3
@MattJoiner:之所以FILE*没有进行类型定义是因为不透明的指针,是为了允许诸如getc()和的宏putc()直接操作FILE结构,并且在大多数情况下避免了函数调用的开销。
chqrlie

74

根据我的经验。隐藏' *'使代码难以阅读。


1
邓诺对此。我的意思是,如果您有一个要说的指针,Device并且您typedef指向Device_Ptr(而不是必须使用Device *)或类似指针,则它非常可读。我看到很多成熟的,相当大的库可以做到这一点。尤其是在C ++中,如果添加模板,则可以省去很多打字的麻烦。
rbaleksandar

@rbaleksandar再次确认Device *是不透明的句柄是可以的。
Alex

作为C n00b,我认为C的指针语法是可憎的-只有那些已经花了10年以上的时间来学习喜欢它的不幸灵魂所喜欢。我每天都会接受myTypePtrmyTypeRef超过那颗星星。:P
Christoffer Bubach

28

我唯一在typedef中使用指针的时间是在处理指向函数的指针时:

typedef void (*SigCatcher(int, void (*)(int)))(int);

typedef void (*SigCatcher)(int);

SigCatcher old = signal(SIGINT, SIG_IGN);

否则,我发现它们比帮助更令人困惑。


剔除声明是指向signal()函数而不是信号捕获器的指针的正确类型。可以SigCatcher通过编写以下内容使它更清晰(使用上面的更正类型):

 typedef SigCatcher (*SignalFunction)(int, SigCatcher);

或者,声明signal()函数:

 extern SigCatcher signal(int, SigCatcher);

也就是说,aSignalFunction是指向带有两个参数(anint和a SigCatcher)并返回a的函数的指针SigCatcher。并且signal()它本身是一个函数,它接受两个参数(anint和a SigCatcher)并返回a SigCatcher


这种类型定义的标点较少吗?“ typedef void SigCatcher(int,void(*)(int))(int)”
sigjuice,2009年

没有; gcc说“错误:SigCatcher声明为返回函数的函数”。
乔纳森·莱夫勒

我只是遇到了更简单的类型定义的signal()声明freebsd.org/cgi/man.cgi?query=signal
sigjuice 2010年

@Sigjuice:即使没有转到BSD页面(直到编辑材料后才开始这样做),我仍然看到答案存在巨大问题,这些问题现在已修复。BSD调用的内容sig_t与my匹配SigCatcher,是的,它signal()极大地简化了声明。
乔纳森·勒夫勒

1
凉。带有显式*的等效声明看起来不错,恕我直言。typedef void SigCatcher(int); extern SigCatcher *信号(int,SigCatcher *);
sigjuice 2010年

16

这可以帮助您避免一些错误。例如下面的代码:

int* pointer1, pointer2;

pointer2不是int *,它是简单的int。但是使用typedef不会发生:

typedef int* pInt;
pInt pointer1, pointer2;

它们现在都是int *


13
如果您将*触摸指标1而不是触摸“ int”,则声明会更有意义的是,pointer2不是指针,而只是一个int。
dreamlax

3
我认为C程序员可能会争辩说,在声明指针变量时使用解引用运算符更像是一种习惯用法。为了进行检查,在K&R中,他们说:“'int * ip'旨在用作助记符;它表示表达式'* ip'是一个int。”
hbw

2
但是每个人都知道这种语法,无论如何一次最好声明和初始化一个变量……我很少写int * pointer1,* pointer2; 因为在下一行中,您有两个具有未定义值的漂亮指针。
丹尼尔·达拉纳斯

2
@Daniel Daranas:int *p1 = NULL, *p2 = NULL;
马特·

2
用更多的不良实践来捍卫不良实践的典型示例。绝对不要在一行上声明多个变量,因为从来没有理由这样做。这只会导致错误。整个“指针接触”辩论都是胡说八道,因为如果您首先编写好的代码,则位置*无关紧要。
伦丁2015年

15

我的回答是明确的“否”。

为什么?

好吧,首先,您只需将一个字符替换*为另一个字符p。那是增益。仅凭这一点就可以避免这样做,因为做多余的事情总是毫无意义的。

其次,这是重要的原因,*进行这意味着不好躲。如果我将某些东西传递给这样的功能

void foo(SomeType bar);

void baz() {
    SomeType myBar = getSomeType();
    foo(myBar);
}

我不希望通过将的含义myBar更改为foo()。毕竟,我是按照价值传递的,所以foo()只有看到myBar权利的副本吗?不是什么时候SomeType别名就意味着某种指针!

这适用于C指针和C ++智能指针:如果您隐藏它们是指向用户的指针这一事实,则会造成不必要的混乱。因此,请不要为指针加上别名。

(我相信类型定义指针类型的习惯只是一种误导的尝试,即隐藏一个程序员http://wiki.c2.com/?ThreeStarProgrammer拥有多少颗星星。)


+1。另一个原因是void *vp=/*...*/; foo(vp);或不会提供类型系统保护foo(0);。(用于枚举的Typedef和用于永远不会显示为数字的事物的数字typedef具有类似的问题)。
PSkocik

5

这是风格问题。您在Windows头文件中经常看到这种代码。尽管他们倾向于全部使用大写字母,而不是使用小写字母p作为前缀。

我个人避免使用typedef。让用户明确地说他们想要Foo *比PFoo更清楚。Typedef现在最适合使STL可读:)

typedef stl::map<stl::wstring,CAdapt<CComPtr<IFoo>> NameToFooMap;

5

它(像很多答案一样)取决于。

在C语言中,这很常见,因为您试图掩盖对象是指针。您试图暗示这是所有函数都操作的对象(我们知道它是下面的指针,但它表示您正在操作的对象)。

MYDB   db = MYDBcreateDB("Plop://djdjdjjdjd");

MYDBDoSomthingWithDB(db,5,6,7);
CallLocalFuc(db); // if db is not a pointer things could be complicated.
MYDBdestroyDB(db);

MYDB下方可能是某个对象的指针。

在C ++中,这不再是必需的。
主要是因为我们可以通过引用传递东西,并且方法被合并到类声明中。

MyDB   db("Plop://djdjdjjdjd");

db.DoSomthingWithDB(5,6,7);
CallLocalFuc(db);   // This time we can call be reference.
db.destroyDB();     // Or let the destructor handle it.

5

没有。

当你与之融合时,它将使你的生活痛苦不堪 const

typedef foo *fooptr;
const fooptr bar1;
const foo *bar2

bar1bar2相同类型的吗?

是的,我只是引用赫伯·萨特的大师。她说了很多实话。;)

-编辑-

将链接添加到引用的文章。

http://www.drdobbs.com/conversationsa-midsummer-nights-madness/184403835


您能否为您提到的文章添加链接?此外,他是一个男人!:-)
Fabio说恢复Monica的时间为

1
@Azat Ibrakov这是次要的观点,但是虽然Herb绝对是一个男人,但他写的大师是女性。因此,用女性来称呼她是适当的。是的,我知道她实际上是Herb的另一个自我,但是如果您阅读了整个系列,包括前20篇左右文章中讲述的故事,那么让她成为自己的人就很有意义。
dgnuff

4

Typedef用于使代码更具可读性,但是将指针作为Typedef会增加混乱。最好避免使用typedef指针。


4

假定感兴趣的语言是C进行了讨论。尚未考虑C ++的分支。

对未标记的结构使用aa指针typedef

定义为指针的结构的大小问题对使用typedeffor(结构)指针提出了有趣的看法。

考虑无标签的混凝土(不是不透明的)结构类型定义:

typedef struct { int field1; double field2; } *Information;

成员的详细信息与讨论完全相切;重要的是这不是像这样的不透明类型typedef struct tag *tag;(并且您不能通过不typedef带标签的来定义这种不透明类型)。

提出的问题是“如何找到该结构的大小”?

简短的答案是“仅通过类型的变量”。没有可用于的标签sizeof(struct tag)sizeof(*Information)例如,您不能有用地编写,并且sizeof(Information *)是指向指针类型的指针的大小,而不是结构类型的大小。

实际上,如果要分配这样的结构,则只能通过动态分配(或模拟动态分配的替代技术)来创建。无法创建称为指针的结构类型的局部变量Information,也无法创建static结构类型的文件范围(全局或)变量,也无法嵌入此类结构(例如与指向此类结构的指针相反)转换为另一种结构或联合类型。

您可以-必须-写:

Information info = malloc(sizeof(*info));

除了将指针隐藏在中之外typedef,这是一个好习惯-如果info更改类型,则大小分配将保持准确。但是在这种情况下,这也是获取结构大小和分配结构的唯一方法。并且没有其他方法可以创建该结构的实例。

这有害吗?

这取决于您的目标。

这不是不透明的类型-当指针类型为typedef'd时,必须定义结构的详细信息。

这是只能与动态内存分配一起使用的类型。

这是一种无名的类型。指向结构类型的指针具有名称,但是结构类型本身没有名称。

如果要强制执行动态分配,这似乎是一种方法。

但是,总的来说,它比启蒙更容易引起混乱和焦虑。

概要

通常,typedef定义一个无标签结构类型的指针是一个坏主意。


3

如果这样做,由于编译器将读取以下内容,则将无法创建const pSomeStruct的STL容器:

list<const pSomeStruct> structs;

list<SomeStruct * const> structs;

由于元素不可分配,因此这不是合法的STL容器。

看到这个问题


当然可以,但是永远也不会写list<const ordinary_type>,因为它无论如何都不会起作用,因此不会造成混乱。

1
我不确定您的意思ordinary_typelist<const SomeStruct*>const SomeStruct*CopyAssignable完全一样有效。
Dan Hook

1

Win32 API几乎对每个结构(如果不是全部)都执行此操作

POINT => *LPPOINT
WNDCLASSEX => *LPWNDCLASSEX
RECT => *LPRECT
PRINT_INFO_2 => *LPPRINT_INFO_2

保持一致很不错,但是我认为它并没有增加任何优雅。


这是因为Win32是C API,并且在C语言而不是C ++中很常见
Martin York

2
这个问题同时被标记为C和C ++,您的意思到底是什么?
dreamlax

1
应该注意的是,这也不是C语言中的常见做法,Win API几乎是唯一一个执行此操作的著名C库。它还使用匈牙利表示法。这是一个非常晦涩的图书馆风格,不是您应该提倡的标准。
隆丁

1
这种表示法可以追溯到30年前Windows的起源,该Windows是Intel 16位分段架构。LP代表长指针,简写表示法LPPOINT ptr;不过是方便的缩写POINT FAR *ptr;FAR扩展到far,然后扩展到__far。所有这一切都相当微不足道,但过去更糟。
chqrlie

1

typedef的目的是隐藏实现细节,但是typedef-ingpointer属性隐藏太多,使代码更难以阅读/理解。所以请不要那样做。


如果要隐藏实现细节(通常是一件好事),请不要隐藏指针部分。以标准FILE接口的原型为例:

FILE *fopen(const char *filename, const char *mode);
char *fgets(char *s, int size, FILE *stream);

在这里,fopen返回一个指向某些结构的指针FILE(您不知道其实现细节)。也许FILE不是一个很好的例子,因为在这种情况下,它可以使用某些pFILE类型来隐藏它是指针的事实。

pFILE fopen(const char *filename, const char *mode);
char *fgets(char *s, int size, pFILE stream);

但是,这只会起作用,因为您永远不会弄乱直接指向的内容。根据我的经验,在您键入一些指针并在某些地方修改代码的那一刻变得很难阅读。


0

前一段时间,我会对这个问题回答“否”。现在,随着智能指针的兴起,指针不再总是用星号“ *”定义。因此,关于类型是否为指针没有明显的认识。

所以现在我要说:只要明确指出它是“指针类型”,就可以使用typedef指针。这意味着您必须专门为其使用前缀/后缀。不,例如,“ p”不是足够的前缀。我可能会选择“ ptr”。


除非您不打算将其用作指针,否则可以将其命名为您的名字。如果仅用于传递内容(例如C FILE *),那么它到底有什么关系呢?
David Thornley,2009年

好点子。我尝试一些优点添加到已表示,对typedef'ing指针利弊的列表...
伯努瓦
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.