函数的使用

紫气东升

我最近接手了一个同事的项目工程，我看他的程序总感觉有些地方是多余的，可以删掉。但是不知道会不会对其他地方有影响。比如以下：
#include "led.h"

void LED_Event(void);     //这个定义我总觉得是多余的

void LED_Event(void)
{
      if (Mic_Mute == 1)
      {
             LedOff(B_LED);
       }
      else
       {
             ......
       }
}

int main(void)
{
    while(1)
    {
          ......
          LED_Event();    //调用
          ......
     }
}

holy404290384

这个是声明，函数体是定义，你如果将函数体放在调用者（此处是main）的后面，你就必须先申明，否则编译会提示未定义。此处可以删除。

紫气东升

holy404290384 发表于 2019-3-8 16:19
这个是声明，函数体是定义，你如果将函数体放在调用者（此处是main）的后面，你就必须先申明，否则编译会提 ...

你说的这点我知道，我只是担心删了会不会影响程序的性能之类的

holy404290384

紫气东升 发表于 2019-3-8 17:12
你说的这点我知道，我只是担心删了会不会影响程序的性能之类的

顶多就是影响编译的速度，与性能无关

Psheng

不影响

1208

去掉不影响，你可以测试下嘛

usb123

多试试就知道啦。实践出真知

edmund1234

void LED_Event(void);     //这个定义我总觉得是多余的

没这一条， 其它C文件就调用不了LED_Event（）， 如果你确定其它C文件确实不会调用LED_Event， 那就删呗

这类声明应该是放在led。h， 但好奇的是你为什么要删它？ 它又不占内存

Acuity

这是个人良好的编码习惯吧，函数、变量使用步骤，声明->定义->使用。

d1z1y2

这个是代码风格的问题，我一般都是这么写的，个人认为这种写法比较规范，当函数比较多的时候，前面全部写上声明，这样一看就知道这个文件局部函数有多少，而且我一般还会加上static，如果要给外部调用，就放在.h里面，局部的我是不会放在.h里面的，个人看法

323232

没准一开始  你同事是将这个函数放在主函数下面的，然后才有的声明，完事他又把函数移上去又忘记删除声明了

lvkanger

Acuity 发表于 2019-3-8 23:41
这是个人良好的编码习惯吧，函数、变量使用步骤，声明->定义->使用。

同意!!!!