Why is this code causing the Arduino Mega 2560 to crash???

[ALB10811Banner]

 static int DepositFunds(void){
  int i;
  double depositAmount;
  float currentBalance;
  char accountBalance[10];
  char amount[10];


    i = GetAccount();


    if(i <= arrayLength)
    {
      printf("Enter The Amount You Would Like to Deposit: \n");
      scanf("%s", &amount);
      depositAmount = atof(amount);
      currentBalance = userAccountBalance[i];
      userAccountBalance[i] = currentBalance + depositAmount;
      dtostrf(userAccountBalance[i],10,2,accountBalance);
      printf("********************************************** \n");
      printf("Credit Card Number: %s \n", userAccount[2][i]);
      printf("Deposit Amount: $%s \n", amount);
      printf("********************************************** \n");
      printf("New Account Balance: $%s \n", accountBalance);
      printf("********************************************** \n\n");
      return 0;
    }
    else
    {
      printf("Error!. Account Not Found.");
      return 1;
    }
}

[jimblumberg]

Why is this code causing the Arduino Mega 2560 to crash???

Have you run the program with your debugger?

Look at this snippet:

      printf("Enter The Amount You Would Like to Deposit: \n");
      scanf("%s", &amount);
      depositAmount = atof(amount);

First you don't need the ampersand when dealing with C-strings, the name of the variable is all you need.

Second, but why are you using a C-string? Why not just get the value as a double?

[Salem]

What are the declarations of all those global arrays?

> if(i <= arrayLength)
Array subscripts start from 0, so the normal test for validity is < N, not <= N.

Does the width you pass to dtostrf() specifically allow room for the \0 at the end?