• State Not Answered
  • Locked Locked
  • Replies 4 replies
  • Subscribers 21 subscribers
  • Views 949 views
  • Users 0 members are here
2025 survey

We are running a survey to help us improve the experience for all of our members. If you see the survey appear, please take the time to tell us about your experience if you can.

Options
Related
How was your experience today?
This discussion has been locked.
You can no longer post new replies to this discussion. If you have a question you can start a new discussion

function point

Jarvee wang

flowing is error,Please help me!!

#include <stdio.h>
#define NUMBER 2


int max(int a[])
{
int z;
if(a[0]>a[1]) z=a[0]; else z=a[1];
return(z);
}



int process(void (* fun)(void *p),void *arg)
{
int z;
z=(int *)(* fun)((int *)arg);
return(z);
}



main()
{
int a[NUMBER];
int i;
int d;
for(i=0;i++;i<NUMBER) a[i]=i;

d=process(max,a);
while(1);
}

  • 0 

    #include <stdio.h>
#define NUMBER 2

int max(int a[])
{
    int z;
    if(a[0]>a[1]) z=a[0]; else z=a[1];
    return(z);
}

int process(void (* fun)(void *p),void *arg)
{
    int z;
    z=(int *)(* fun)((int *)arg);
    In the line above, the usage does not match the declaration.
        return(z);
}

main()
{
    int a[NUMBER];
    int i;
    int d;
    for(i=0;i++;i<NUMBER) a[i]=i;
    The loop will never be entered. It appears you have the loop termination expression and the loop iteration expression reversed.
        d=process(max,a);
    The first argument has a type mismatch, implicitly attempting to convert an int (*)(int *) to a void (*)(void *)
        while(1);
}

  • 0 in reply to Dan Henry

    Sorry. I said:

    It appears you have the loop termination expression and the loop iteration expression reversed.

    That should be test, not termination.

  • 0

    Your code is based on an invalid assumption, and it contains various bugs.

    Generally speaking, it's almost always an error to cast a function pointer to any other function pointer type.

    On top of that, the casts you do apply are wrong, here:

    int z;
z=(int *)(* fun)((int *)arg);

    The cast to (int *) is seriously wrong. You have to cast "fun" back to the real signature before you're allowed to use it for calling max(), which will fix the return type in the same step:

    
typedef int (*my_fp_type)(int *);

int z;
z=((my_fp_type)fun)((int *)arg);

  • 0 in reply to HansBernhard Broeker

    Having fixed these problems, be sure to search both the App Notes and the Knowledgebase - the limitations of the 8051 architecture put some serious constraints on the use of function pointers in C51.
    You will get some very obscure bugs if you don't carefully observe all the precautions...!

    Search this forum, too.