warning int format pointer arg

Hi guys im trying to compile this code in ansi c and i keep getting a warning when using the printf statement when trying to print the phone number from the struct it gives me a

warning: int format pointer arg what am i doing wrong in the printf statement

Code:

#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#define  PHONEMAX     9
#define  AGEMAX       20
#define  ADDRESSMAX   30
void flush_stdin();

 struct personDetails
{
   int phone    [PHONEMAX];
   int age      [AGEMAX];
   char gender;
   char address [ADDRESSMAX];


};


int main (void)
{
   int valid;
   struct personDetails pd;
   
 
   do
   {
   
   printf("please enter an integer phone number\n\n!");  
  
 
   if((fscanf(stdin, "%d", pd.phone)==1))
   {
     printf("you have entered an int\n\n");
     valid = 1;
   }  
   else
   {
     flush_stdin();
     valid  = 0;
   }
  
   }
   while(!valid);

   printf("Your phone number is:%d", pd.phone);
   return EXIT_SUCCESS;

}


/* begin of flush_stdin 05-8-31 19:30 */
void flush_stdin( void )
{
	if ( !feof(stdin) ) {
                int c;
		while( ( c=getchar() ) != '\n' && c != EOF )
			;
	}
} /* end of flush_stdin */

What you are declaring with int phone[PHONEMAX] is an array of 9 separate integer numbers.

The fscanf coincidentally "works" because it's expecting the address of the int, and when you specified pd.phone, the "phone" array decayed to a pointer, so you got the address of it.

When you did the printf further down, you should have done pd.phone[0] because you wrote the int into the first element of the phone array (9 numbers).

I expect this is not what you meant.

It is better to store phone numbers as strings. People will inevitably try to put in symbols, leading zeroes, aribtrary amounts of area code. All this stuff makes it inappropriate for an integer.

If you really wanted to store a phone number as an int, change it to simply int phone; and change the fscanf line to &(pd.phone).

As said above if you want to store it as telephone no. as int then declare phone no. as long or double.it would be better if you store as string(array of char).you can easily convert it to integer by subtracting its ascii of zero from that

Originally Posted by cbastard

As said above if you want to store it as telephone no. as int then declare phone no. as long or double.

I don't know about you, but my phone number doesn't have a decimal point in it.


Quzah.

Originally Posted by quzah

I don't know about you, but my phone number doesn't have a decimal point in it.
Quzah.

I've seen this on TV that last 3-4 years, since the net became so popular.

Code:

555.555.5555

Well, you can't store 555.555.5555 in a float, either. But maybe you can without an area code, like 555.5555.

Not to mention that some phone numbers have significant leading zeros which all number formats are going to trim away for you.

And I think many European phone numbers are alphanumeric.