Question

#include<stdio.h>
#include "amicablenumber.h"

int i,j;
struct amicable         
{
    int **amicablePair;
    int size;
};

main()
{

int startnum = 250;
int endnum = 1000;
struct amicable* ami;

ami = getAmicablePairs(startnum, endnum);   

printf("{");
for(int i = 0; i<ami->size; i++)
{
printf("{%d, %d}",ami->amicablePair[i][0], ami->amicablePair[i][1]);
}
printf("}");
}

amicable *getAmicablePairs(int startnum,int endnum)
{
    int size=0;
    int sumfactors(int);
    amicable record;
 for(i=startnum;i<=endnum;i++)
 {
 for(j=endnum;j>=startnum;j--)
 {
 if((sumfactors(i)==j)&&(sumfactors(j)==i) && (i!=j))
 {
  record.amicablePair[size][0]=i;
  record.amicablePair[size][1]=j;

  size++;
 }}}
 record.size=size;
 return record;
 }

  int sumfactors(int number)
 {
 int sum=0;
 for(i=1;i<number;i++)
 {
 if(number%i==0)
 sum +=i;
 }
 return sum;
 }

在上面的代码我得到一个错误

无法将amicable转换为友好*作为回报

Answer 1

声明

getAmicablePairs返回指向amicable的指针：

amicable *getAmicablePairs(...)

但您尝试返回amicable：

return record;

而不是指向一个指针。

注意一个“明显”的修复，即返回指向record的指针：

return &record;

将无效，因为只要getAmicablePairs返回，您就会返回指向即将消失的变量的指针。相反，您需要使用malloc创建记录并返回该记录;像这样的东西：

amicable *record = (amicable*) malloc(sizeof(amicable));

您需要将所有record.更改为record->。

另请注意，您正在写入结构的amicablePair成员而不分配它 - 这将导致崩溃。您需要malloc amicablePair以及amicable。

Answer 2

你正在返回一个（友好的*） - 一个指向友好的指针，但是你的函数创建了一个（友好的）（不是一个pinter）。

而不是声明

amicable record;

你需要这样做（或同等的）：

amicable *record = (amicable *) malloc(sizeof(amicable));

然后通过“record-＆gt;”访问而不是“记录。”

注意：使用上述方法，您需要在完成上述分配后释放（）。

函数返回结构中的错误

2 个答案: