Thread: function problem

this is the code i am trying to run and i keep getting am error on the line
if (values[i]>=50 && values[i] <=100)
which says: subscript requires array or pointer type
I am lost at what to do...
Please help...
here is the whole code
thanks much

Code:

#include <stdio.h>
#include <stdlib.h>

#define TRUE 1
#define FALSE 0

#define SIZE 5
main(){

	int values [SIZE]={0};
	int count;
	int i;
	int valid;
int result;
	count=0;
result=0;



	for (i=0; i<SIZE; i++){
		printf("Enter an integer (50-100):\n");
		scanf("%i",&values[i]);
	
		valid =isValid(values, i);
		//unique=isUnique();	
	}

	system("pause");
}


int isValid (int input) {
	int result,i;
int values ();
	if (values[i]>=50 && values[i] <=100)

		result= TRUE;


	else 
		result= FALSE;
	return result;
}

i tried doing what you said and i am still not getting it.
this is my first time taking a c class so I am having a ilttle more problems
here is the new code
please help

Code:

#include <stdio.h>
#include <stdlib.h>

#define TRUE 1
#define FALSE 0

#define SIZE 5

//prototype
int isValid(int number);


main(){

	int values [SIZE]={0};
	int count;
	int i;
	int valid;
int result;
	count=0;
result=0;



	for (i=0; i<SIZE; i++){
		printf("Enter an integer (50-100):\n");
		scanf("&#37;i",&values[i]);
	
		valid =isValid(values, i);
		//unique=isUnique();
		printf("%i",result);
	}

	system("pause");
}

int isValid(values[i]{
	int result;
	if (values[i]>=50 && values[i] <=100)
		result= TRUE;

	else 
		result= FALSE;
	return result;
}

this is what i have
no luck though...

Code:

#include <stdio.h>
#include <stdlib.h>

#define TRUE 1
#define FALSE 0

#define SIZE 5

//prototype
int isValid(values[i]);


main(){

	int values [SIZE]={0};
	int count;
	int i;
	int valid;
	int result;
	count=0;
	result=0;



	for (i=0; i<SIZE; i++){
		printf("Enter an integer (50-100):\n");
		scanf("%i",&values[i]);

		valid =isValid(values, i);
		//unique=isUnique();
		printf("%i",result);
	}

	system("pause");
}

int isValid(int input){
	int result;
	if (input>=50 && input <=100)
		result= TRUE;

	else 
		result= FALSE;
	return result;
}

Try This code....

Code:

#include <stdio.h>
#include <stdlib.h>
#define TRUE 1
#define FALSE 0
#define SIZE 5
//prototype
int isValid(int *, int);
int main(){

	int values [SIZE]={0};
	int count;
	int i;
	int valid;
   int result;
	count=0;
result=0;



	for (i=0; i<SIZE; i++){
		printf("Enter an integer (50-100):\n");
		scanf("&#37;i",&values[i]);
	
		valid =isValid(values, i);
		//unique=isUnique();
		printf("%i\n",valid);
	}

	system("pause");
}

int isValid(int* values, int i){
	int result;
	if ((values[i]>=50) && (values[i]<=100))
		result= TRUE;
	else 
		result= FALSE;
	return result;
}

i know you said that
main()
should be
int main(void)
but we still have not learned in class about it, so I can not use it
Is there anything else that I might be able to use?

there was two error in code...
1.Function declaration
2.printf(“&#37;d”,result);----> it should be printf(“%d”,valid);

check your function declaration ..there should be two argumnets..
int isValid(int *, int);
because you are passing the address of the array not the value..
valid =isValid(values, i);

hiiii vart acctualy main problem in the..
printf("&#37;i",result); statement because he/she has define it as 0 in main so it wll always print the same value whatever the value return by the function

yaa ..we can correct by both way either change in function or change in call declaration