#include<windows.h>
#include<stdio.h>
unsigned int rhythm[]={
   
    0x188,0x188,0x1B8,0x188,0x20B,0x1EE,
   
    0x188,0x188,0x1B8,0x188,0x24B,0x20B,
   
    0x188,0x188,0x310,0x293,0x20B,0x1EE,
   
    0x1B8,0x2BA,0x2BA,0x293,0x20B,0x24B,
   
    0x20B
   
};
unsigned int delay[]={

0x177,0x07D,0x1F4,0x1F4,0x3E8,

0x177,0x07D,0x1F4,0x1F4,0x3E8,

0x177,0x07D,0x1F4,0x1F4,0x1F4,0x1F4,0x3E8,

0x177,0x07D,0x1F4,0x1F4,0x1F4,0x3E8,

};

void play_music();//奏乐

int main(int argc,char*argv[])

 {

    puts ("happy birthday!\n");
   
    play_music();
   
    return 0;

    }
void play_music()

{
   
    int i=0;
   
    for(;i<25;++i)

{
    Beep(rhythm[i]*5,delay[i]);
   
    if(i==24)
   
    {
        Sleep(500);
        
        i=0
    ;}
        }   
}

硬编码貌似没啥研究意义 不通用

delay[]定义不完整，还缺2个元素。

那要如何改正

{

    puts ("happy birthday!\n");
   
    play_music();
   
    return 0;

    }
我记得编译的时候，是说这里的函数返回值错了

你这个void play_music()哪来的返回值？

rhythm数组有25个元素，delay只有23个元素。for(;i<25;++i)遍历的时候i=23的时候delay数组下标越界了。你的delay里面是不是少写了两个元素哦？？？