Thread: Error Trap for a 2nd decimal

I am wanting to add a second portion to a validation function for a float to trap if a 2nd decimal point is accidently entered.

Code:

int sValidFloatString(char *chFloat)
{
 	// Declare Variables for Function
 	float flValidEntry = 0.0;
 	int i = 0;
 	char chStringEnd = '\0';
 	char chDot = '.';
 
 	//Check each chararacter in string for validity
 	while(chFloat[i]!=chStringEnd)
 	{
  		//If entered character not a valid number, return invalid and exit function
  		if(isdigit(chFloat[i])!=0 || (chFloat[i]==chDot))
  		{
			//Valid entry
			flValidEntry = 1;
		}
  		else
  		{
   		//Invalid entry, end function
   		flValidEntry = 0;
		break;
  		}
 
  		//Movenext
  		i++;
 	}
 
    	return flValidEntry;
}
// ********** End Function ************

Example:
User input: 1.23.2
Output: Invalid Entry

Right now it just ignores the 2nd decimal point and the following digit.

Thank you in advance

ropewrench

Quzah,
Thank you for the suggestions. I have not quite gotten it yet.

Code:

// ********** Check for Valid Float Entry Function ***********
// Function checks users entry to see if it is valid integer value


int sValidFloatString(char *chFloat)
{
 	// Declare Variables for Function
 	float flValidEntry = 0.0;
 	int i = 0;
 	char chStringEnd = '\0';
 	char chDot = '.';
	int count = 0;
 	//Check each chararacter in string for validity
 	while(chFloat[i]!=chStringEnd)
 	{
  		//If entered character not a valid number, return invalid and exit function
  		if(isdigit(chFloat[i])!=0 || (chFloat[i]==chDot) && (chFloat[count] == chDot))
  		{
			if(chFloat[count]>1)
			{
				flValidEntry = 0;
				break;
			}
			else
			{
				//Valid entry
				flValidEntry = 1;
			}
			//Valid entry
			flValidEntry = 1;	
		}
  		else
  		{
   		//Invalid entry, end function
   		flValidEntry = 0;
		break;
  		}
 
  		//Movenext
  		i++;
  		count++;
 	}
 
    	return flValidEntry;
}
// ********** End Function ************

Quzah,
Thank you for your help. I got it to work pretty well. It is not the prettiest code in the world but it did the job. I would like to thank everyone on this forum, it has proven to be invaluable.

Code:

// ********** Check for Valid Float Entry Function ***********
// Function checks users entry to see if it is valid integer value


int sValidFloatString(char *chFloat)
{
 	// Declare Variables for Function
 	float flValidEntry = 0.0;
 	int i = 0;
 	char chStringEnd = '\0';
 	char chDot = '.';
	int count = 0;
	char *chTempDot;
 	//Check each chararacter in string for validity
 	while(chFloat[i]!=chStringEnd && chFloat[count]!=chStringEnd)
 	{
 		/*if (chFloat == chDot)
 		{
 			chTempDot = chFloat;
 			
 		}*/
  		//If entered character not a valid number, return invalid and exit function
  		if(isdigit(chFloat[i])!=0 || (chFloat[count] == chDot && count <=1))
  		{	
			//Valid entry
			flValidEntry = 1;	
			
		}	
  		else
  		{
   		//Invalid entry, end function
   		flValidEntry = 0;
		break;
  		}
 
  		//Movenext
  		i++;
  		count++;
 	}
 
    	return flValidEntry;
}
// ********** End Function ************