atof error

[bazzano]

Hey guys does anyone know why in this function i get an error at line 401 i think its at the atof function

void addStock(TennisStoreType* ts)
{
  char tmpDescription[DESCRIPTION_MAX + 1];
  char tmpunitPricePtr[10];
  char tmpstockLevelPtr[PRICE_COLWIDTH];
  
  float tmpPrice;
  int tmpStock[STOCKLEVEL_MAX];
  int finished = FAILURE;

  StockNodeType* curStock = ts -> headStock;
  StockNodeType* newStock;
  StockNodeType* prevStock;
 
   do
  {
     printf("Description: (1-40 characters)\n");
     fgets(tmpDescription, DESCRIPTION_MAX + 2, stdin);

     /* check the range of surname */

     if(tmpDescription[strlen(tmpDescription) -1] != '\n');
     {
        printf("Description too long less that 12\n\n");
     }     

     /* fails if empty line is pressed*/

     if(tmpDescription[0] == '\n')
     {
        printf("\t\n*****back to main menu*****");
        finished = FAILURE;
     }

  }while(tmpDescription[strlen(tmpDescription)-1] != '\n');

  tmpDescription[strlen(tmpDescription)-1] = '\0';
  
  /**** Unit Price ****/

   do
  {
     printf("UnitPrice: (0.05-999.95)\n");
     fgets(tmpunitPricePtr, POSTCODE_LEN + 2, stdin);

     /* fails if empty line is pressed*/

     if(tmpunitPricePtr[0] == '\n')
     {
        printf("\t\n*****back to main menu*****");
        finished = FAILURE;
     }


     /* check the range of the unit pice */

     if(tmpunitPricePtr[strlen(tmpunitPricePtr) -1] != '\n')
     {
        printf("The unit price is too long!\n\n");
        readRestOfLine();
     }

     else
     {
        tmpunitPricePtr[strlen(tmpunitPricePtr) -1] = '\0';
        finished = SUCCESS;
  
     }

     tmpPrice[10] = atof(tmpunitPricePtr);
     
     /***** stock level *****/
      do
  {
     printf("Stock Level: (1 - 100)\n");
     fgets(tmpstockLevelPtr, STOCKLEVEL_MAX + 2, stdin);

     /* fails if empty line is pressed*/

     if(tmpstockLevelPtr[0] == '\n')
     {
        printf("\t\n*****back to main menu*****");
        finished = FAILURE;
     }


     /* check the range of the stock */

     if(tmpstockLevelPtr[strlen(tmpstockLevelPtr) -1] != '\n')
     {
        printf(" Stock level needs to be between (0 -100)\n\n");
        readRestOfLine();
     }

     else
     {
        tmpstockLevelPtr[strlen(tmpstockLevelPtr) -1] = '\0';
        finished = SUCCESS;
  
     }

     tmpStock[STOCKLEVEL_MAX] = atoi(tmpstockLevelPtr);
     
     

  }while(finished == FAILURE);


     

  }while(finished == FAILURE);
 


}

[swoopy]

> tmpPrice[10] = atof(tmpunitPricePtr);
This should be:
tmpPrice = atof(tmpunitPricePtr);

But instead of atof(), I believe strtod() is recommended, because I don't believe atof() is standard, or does any error checking.

[Ancient Dragon]

atof() is an ansi standard c function. One nice feature of strtod() is that you can pass it a big string that contains a mixture of numeric and non-numeric fields. strtod() will return a pointer to the character that stopped the scan, and your program can use it for other operations without having to rescan the whole string to find it.

Of course the other nice feature of strtod() is that it will detect data overflow, which I don't think atof() does.