结构体作为函数参数调用的问题

暮二木

首先是定义的结构体
struct individual
                      {    double  gene[a];                    /* 染色体 */
                        double  fitness;                   /* 个体适应度*/
                       double selectprob;
                       double sumprob;
                       };
struct individual pop[popsize+1];
然后是在Ag_Init（）这个函数里调用ag_adopt函数，其中ag_adopt定义为double ag_adopt(double *p,double *m,double *n)        ，不知道为什么这样不可以，一硬件仿真就会进入直接进入 void HardFault_Handler(void) ，有没有大佬会这个啊！
void Ag_Init(void)
{
    int ag_index=0;
    double max_fit = 0;
          double *pt,*mt,*nt;
          initialize();
    for(int i = 0; i < eranum; i++)
    {
        for(int j = 0; j < popsize; j++)
        {
                                           *pt=pop[j].gene[0];
                   *mt=pop[j].gene[1];
                   *nt=pop[j].gene[2];
            pop[j].fitness =  ag_adopt(pt,mt,nt);
            if(pop[j].fitness >= max_fit)
            {
                max_fit = pop[j].fitness;
                ag_index = j;
            }
        }
}

暮二木

感谢大家的帮助，问题已解决，除了指针的问题，还有数组的问题，数组的定义是没有问题的，因为我之前定义了两个6500的数组，加上遗传算法的迭代，导致数组溢出

Neil2436

a是宏？

LcwSwust

你的问题在于：指针未初始化，不知指向何方，却对其指向的地方赋值。
这帖子第13条
http://www.openedv.com/forum.php?mod=viewthread&tid=323935&extra=

moranyidui

一步一步调试，看从什么地方进入的HardFault_Handler

暮二木

Neil2436 发表于 2021-4-22 16:15
a是宏？

是的，#define a 3
#define popsize 50
#define eranum 25

暮二木

LcwSwust 发表于 2021-4-22 16:18
你的问题在于：指针未初始化，不知指向何方，却对其指向的地方赋值。
这帖子第13条
http://www.openedv.c ...

  double *pt,*mt,*nt,l,m,n;
          pt=&l;
          mt=&m;
          nt=&n;
//          initialize();
    for(int i = 0; i < eranum; i++)
    {
        for(int j = 0; j < popsize; j++)
        {
                                            *pt=pop[j].gene[0];
                    *mt=pop[j].gene[1];
                    *nt=pop[j].gene[2];
           pop[j].fitness =  ag_adopt(l,m,n);
你好，我改了一下，这样还是不对啊

姚先起

*pt这个指针没有给地址，或者是*pt=pop[j].gene[1]的方法有错，因为pt在你定义的时候只是指针而没分配地址，给未分配地址的指针赋值导致内存非法访问，
第一种方式：应该是pt=&pop[j].gene[1]；这样pt就有实际的地址了，pt指向gene【1】。
第二种方式：在定义*pt后加入，pt=（double *）malloc(sizeof(double);这样pt有了新的空间，然后*pt=pop[j].gene[1]是正确的；使用完记得free。

第一种方式 pt其实是gene的镜像，操作pt会导致gene发生改变，第二种方法操作pt不会影响gene的变化，因为是两个不同的内存单元了。
用指针首先要分清哪个是地址，哪个是值。

jinx

姚先起 发表于 2021-4-22 17:01
*pt这个指针没有给地址，或者是*pt=pop[j].gene[1]的方法有错，因为pt在你定义的时候只是指针而没分配地址 ...

你写的多，就感觉你是正确的。

edmund1234

*pt=pop[j].gene[0];
*mt=pop[j].gene[1];
*nt=pop[j].gene[2];

应该是

pt=&(pop[j].gene[0]);
mt=&(pop[j].gene[1]);
nt=&(pop[j].gene[2]);

暮二木

edmund1234 发表于 2021-4-22 17:49
*pt=pop[j].gene[0];
*mt=pop[j].gene[1];
*nt=pop[j].gene[2];

谢谢，就感觉很奇怪，我确实是这么写的，但还是在硬件调试的时候进入void HardFault_Handler(void)，然后把指针删掉就不会进入
void Ag_Init(void)
{
    int ag_index=0;
    double max_fit = 0;
          double *pt,*mt,*nt;
         
//          initialize();
    for(int i = 0; i < eranum; i++)
    {
        for(int j = 0; j < popsize; j++)
        {
                                            pt=&pop[j].gene[0];
                    mt=&pop[j].gene[1];
                    nt=&pop[j].gene[2];
//              pop[j].fitness =  ag_adopt(pt,mt,nt);
//            if(pop[j].fitness >= max_fit)
//            {
//                max_fit = pop[j].fitness;
//                ag_index = j;
//            }
        }

暮二木

姚先起 发表于 2021-4-22 17:01
*pt这个指针没有给地址，或者是*pt=pop[j].gene[1]的方法有错，因为pt在你定义的时候只是指针而没分配地址 ...

谢谢，但是如果用的keil的话应该不能用malloc动态分配地址吧，用第一种方法也还是会进入void HardFault_Handler(void)

moranyidui

暮二木 发表于 2021-4-22 17:56
谢谢，就感觉很奇怪，我确实是这么写的，但还是在硬件调试的时候进入void HardFault_Handler(void)，然后 ...

楼主有用吗？第一种写法，好像是给指向的内容赋值？

edmund1234

暮二木 发表于 2021-4-22 17:56
谢谢，就感觉很奇怪，我确实是这么写的，但还是在硬件调试的时候进入void HardFault_Handler(void)，然后 ...

如果说， 你这样屏蔽了就不进HardFault_Handler， 那问题是在ag_adopt函数内

edmund1234

暮二木 发表于 2021-4-22 17:56
谢谢，就感觉很奇怪，我确实是这么写的，但还是在硬件调试的时候进入void HardFault_Handler(void)，然后 ...

在HardFault_Handler 设个断点， 在断点停下来后， 在寄存器窗口， 把SP的值抄出来

然后把此地址贴到Memory窗口



然后在这窗口， 右键 选 Show Disassembler At Address。。。


然后把第七个INT抄出来的值贴上， 就可以看到是那一句触发HardFault异常

y284858

你指针赋值不对
void Ag_Init(void)
{
    int ag_index=0;
    double max_fit = 0;
          double *pt,*mt,*nt;//这里没有初始化
          initialize();
    for(int i = 0; i < eranum; i++)
    {
        for(int j = 0; j < popsize; j++)
        {
                                           *pt=pop[j].gene[0];//这里你赋值的内容是gene[0]的值而不是所在的地址，后面两个同理，如果你想传递地址需要用取地址符&取出地址或者把gene数组名当作指针用 pop[j].gene[0]的地址就是 pop[j].gene ，pop[j].gene[1]的地址就是pop[j].gene + 1
                   *mt=pop[j].gene[1];
                   *nt=pop[j].gene[2];
            pop[j].fitness =  ag_adopt(pt,mt,nt);
            if(pop[j].fitness >= max_fit)
            {
                max_fit = pop[j].fitness;
                ag_index = j;
            }
        }
}

暮二木

edmund1234 发表于 2021-4-22 18:20
如果说， 你这样屏蔽了就不进HardFault_Handler， 那问题是在ag_adopt函数内

不是，是把指针屏蔽了不进去HardFault_Handler，把ag_adopt屏蔽了没有影响

暮二木

moranyidui 发表于 2021-4-22 18:05
楼主有用吗？第一种写法，好像是给指向的内容赋值？

第一种应该是指向pop[j].gene的地址吧，我觉得哈

暮二木

y284858 发表于 2021-4-22 18:38
你指针赋值不对
void Ag_Init(void)
{

谢谢，其实地址和值都行，我主要是想把pop[j].gene[]传给ag_adopt,但是传的都不对

暮二木

edmund1234 发表于 2021-4-22 18:33
在HardFault_Handler 设个断点， 在断点停下来后， 在寄存器窗口， 把SP的值抄出来

然后把此地址贴到M ...

谢谢大佬，虽然你贴的图看不见，但是我试了一下，就是我硬件仿真时一进去就是全速运行，然后此时加断点，不管加到哪里都不会停止，然后停止仿真后才会到HardFault_Handler，就很奇怪，但是删掉指针就不会这样。

姚先起

暮二木 发表于 2021-4-22 17:58
谢谢，但是如果用的keil的话应该不能用malloc动态分配地址吧，用第一种方法也还是会进入void HardFault_H ...

那你试试注释掉ag_adopt，看看还会不会异常。

moranyidui

暮二木 发表于 2021-4-22 20:38
谢谢大佬，虽然你贴的图看不见，但是我试了一下，就是我硬件仿真时一进去就是全速运行，然后此时加断点， ...

断点从第一行打起，再运行

暮二木

moranyidui 发表于 2021-4-23 09:11
断点从第一行打起，再运行

有点问题

y284858

暮二木 发表于 2021-4-22 20:08
谢谢，其实地址和值都行，我主要是想把pop[j].gene[]传给ag_adopt,但是传的都不对

你传的是值但是在函数里面是把这个值当作指针来用，结果就会造成非法访问进入硬件错误中断你把这三句
*pt=pop[j].gene[0];
                   *mt=pop[j].gene[1];//编辑补充说明：之前没有注意你的指针变量名前面有*，你的指针还没有初始化地址，你的目的是把后面地址赋值给指针变量，那么你应当直接使用指针变量名，如果在变量名前加入*就是对指针指向的地址进行操作。你需要复习一下关于指针的知识了
                   *nt=pop[j].gene[2];

改成
pt=pop[j].gene ;
                   mt=pop[j].gene+1;
                   nt=pop[j].gene+2;
试试

QuiGonJinn

这属于最基本的编码能力范畴


首先double ag_adopt(double *p,double *m,double *n)函数中，ag_adopt函数是否要修改3个形参？如果不需要，请不要随便定义成指针。（ 看你后面的代码，这里很可能是不需要的 ）直接定义双精度对象就足够了。


其次，之前的朋友也说了*pt,*mt,*nt这三个指针定义后没有初始化，就直接间接引用赋值了。这等于对三个野指针的内容进行赋值，这是不对的。

再次，如果ag_adopt函数的形参一定要定义为指针，调用的时候直接填对应数组的地址就可以了，根本不需要先定义三个指针赋值完了后再填入形参，多此一举！
类似于这样 pop[j].fitness = ag_adopt( &( pop[j].gene[0] ), &( pop[j].gene[1] ), &( pop[j].gene[2] ) );



最后，新手上路，不要随便乱用指针，定义了指针后一定要初始化，调用指针之前一定要断言判断指针是否合法。

暮二木

y284858 发表于 2021-4-23 11:40
你传的是值但是在函数里面是把这个值当作指针来用，结果就会造成非法访问进入硬件错误中断你把这三句
*p ...

好的谢谢，我试一下

暮二木

QuiGonJinn 发表于 2021-4-23 12:24
这属于最基本的编码能力范畴

嗯嗯谢谢，感谢指导

暮二木

姚先起 发表于 2021-4-23 08:46
那你试试注释掉ag_adopt，看看还会不会异常。

感谢各位大佬，指针我注释掉了，但是我前面观察的不够仔细，今天我发现把结构体注释掉就不会有异常，但我觉得我结构体没有问题啊typedef struct
{
    double  gene[chro];                    /* 染色体 */
    double  fitness;                   /* 个体适应度*/
    double selectprob;
    double sumprob;
}individual;

individual pop[popsize];
individual newpop[popsize];
individual bestpop[eranum];
调用是pop.gene[]