一个相当糟糕的函数

今天在坛里面见到一个函数，看了心寒。

代码是这样的，

 

#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()函数之后便再也无法联系得到了，成了孤魂野鬼。

不知道这个函数是什么？老手写来骗人的？还是新手写来练手的？是新手也绝对不要这样写程序。我想这是个十足的糟糕函数！