一个相当糟糕的函数
来源:互联网 发布:ui和美工前端 编辑:程序博客网 时间:2024/05/04 04:09
今天在坛里面见到一个函数,看了心寒。
代码是这样的,
#include <stdio.h>
#include <stdlib.h>
void fun(float *p1,float *p2,float *s)
{
s=(float*)calloc(1,sizeof(float)) ;
*s=*p1+*p2++;
}
fun函数有如下问题:
1、要求float*型参数p1,p2和s。而计算过程根本用不上,实际上只需使用float类型的p1和p2就足够了的,更不要说p2++了。
2、要求的参数s所指的空间没有使用,而calloc申请了另一块空间,计算的结果也没有传回到原来的空间,这样参数s就变成纯粹的一种摆设,而更严重的是这样子很容易让人产生误会,比如LZ。
3、使用calloc()分配了一块个数为1的空间,须知calloc()是用来产生数组并且初始化的函数,相比malloc()效率低的多,从后面也看不出有进行初始化的必要。有简单直接的malloc()不用,简直浪费!
4、使用calloc()申请分配的空间在使用完没有释放,在出了fun()函数之后便再也无法联系得到了,成了孤魂野鬼。
不知道这个函数是什么?老手写来骗人的?还是新手写来练手的?是新手也绝对不要这样写程序。我想这是个十足的糟糕函数!
- 一个相当糟糕的函数
- 糟糕的一个月!
- 郁闷,一个糟糕的团体
- D3DXLoadMeshFromX是个很糟糕的函数
- 一个糟糕透顶的自由职业者项目
- 一个糟糕的Model层设计
- 如何成为一个糟糕的程序员
- 如何成为一个糟糕的程序员
- 一个相当全的vimrc配置文件
- 一个相当不错的日期选择器[透明]
- 一个相当不错的Eclipse RCP实例
- 一个相当完整的build.xml
- 一个相当牛X的日历
- 关于VS一个相当奇怪的问题!!!
- 一个相当经典的AVR中断程序
- 我是一个相当浮躁的人
- 一个有趣的测试,相当神奇!
- 设计极其糟糕的select函数
- Item5: Explicitly disallow the use of compiler-generated functions you do not want--若不想使用编译器自动生成的函数,就该明确拒绝
- 福布斯公布09年全球富豪排行榜 盖茨重回榜首
- j2me http 获取网页源码
- UDP包是怎么丢的
- 在Ubuntu下装sourcenavigator6.0
- 一个相当糟糕的函数
- IE6与IE7中,innerHTML获取param的区别
- 学习 HeroKu 的架构设计
- 並發UDP發送,引起丟包如何處理?
- sniff原理
- 进程与线程的亲缘性
- 向MTK上移植浏览器(二)
- linux errno code
- 传智播客JDBC_规范和封装jdbc程序代码学习笔记