Go Down

Topic: Error function Arduino nano (Read 933 times) previous topic - next topic

Oso57

Hello all :)

I Have this function:
(Dividing the number of seconds  ( TT long variable) into hours, minutes and seconds)

//*********** CALCULO DEL TIEMPO TOTAL  ******************************************

String TiempoTotal (unsigned long Cantidad){
     
      unsigned long TT = Cantidad * 10;           
     
      unsigned long Remanente1 = (TT % 3600);
      unsigned long H = (TT - Remanente1)/3600;
      unsigned long S = Remanente1 % 60;
      unsigned long M = ( Remanente1 - S) / 60;
      String Hs= String(H);
      if (H < 10) Hs= "0" + Hs;
      String Min = String(M);
      if (M < 10) Min= "0" + Min;
      String Seg = String(S);
      if (S < 10) Seg= "0" + Seg;
     
     
     
      return  " "  + Hs + "Hs. " + Min + "m. "+ Seg + "s.    ";
     
   }


When I invoke with :  lcd.print ( TiempoTotal (Disparos)); (ex Disparos=100)  this function break mi Arduino nano and make it crazy.... :~
What am I doing wrong?

Thanks in advance  OSO from Argentina!  :)


PaulS

Quote
What am I doing wrong?

Using the String class.

semicolo

#2
Feb 22, 2013, 08:37 pm Last Edit: Feb 23, 2013, 03:36 pm by semicolo Reason: 1
This works on an arduino 2009, the rest of the sketch I used to test is:

Code: [Select]

void setup() {
  // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
  // Print a message to the LCD.
//  lcd.print("hello, world!");
  lcd.print ( TiempoTotal (1000));
}


void loop() {
}

Oso57


Quote
What am I doing wrong?

Using the String class.

Hi...
How make it Please?
Thanks.

Arrch



Quote
What am I doing wrong?

Using the String class.

Hi...
How make it Please?
Thanks.


C-style strings and sprintf()

Oso57


This works on an arduino 2009, the rest of the sketch I used to test is:

void setup() {
  // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
  // Print a message to the LCD.
//  lcd.print("hello, world!");
  lcd.print ( TiempoTotal (1000));
}


void loop() {
}




Hello Semicolo, thanks for your response.

Here a sample of  program :

#include <LiquidCrystal.h>

LiquidCrystal lcd(7, 6, 5, 4, 3, 2);
// initialize the library with the numbers of the interface pins

void setup() {
  lcd.begin(16, 2);
 
}

void loop() {
 
lcd.clear();
lcd.print ( TiempoTotal (1000));
delay(1000);
}

String TiempoTotal (unsigned long Cantidad){
   
     unsigned long TT = Cantidad;          
   
     unsigned long Remanente1 = (TT % 3600);
     unsigned long H = (TT - Remanente1)/3600;
     unsigned long S = Remanente1 % 60;
     unsigned long M = ( Remanente1 - S) / 60;
     String Hs= String(H);
     if (H < 10) Hs= "0" + Hs;
     String Min = String(M);
     if (M < 10) Min= "0" + Min;
     String Seg = String(S);
     if (S < 10) Seg= "0" + Seg;
   
     return  " "  + Hs + "Hs. " + Min + "m. "+ Seg + "s.    ";
   
  }


The display lcd show some strange signs, then shows nothing more ...
will display problem ?
May be because I use pins 6 and 7? (for others applications works fine)
Can you help me?

Thanks!


PaulS

Why not move the lcd.print() to TiempoTotal()? Then, TiempoTotal() does not need to return a value. Use multiple lcd.print() calls in TiempoTotal(), so you don't need to use any Strings.

semicolo

Does the liquidcrystal->hello_world example work?

Nick Gammon

@Oso57: How to use this forum


Please edit your post, select the code, and put it between [code] ... [/code] tags.

You can do that by hitting the # button above the posting area.
http://www.gammon.com.au/electronics

Nick Gammon

Please note that, at present, the String library has bugs as discussed here and here.

In particular, the dynamic memory allocation used by the String class may fail and cause random crashes.

I recommend reworking your code to manage without String. Use C-style strings instead (strcpy, strcat, strcmp, etc.).

Alternatively, install the fix described here:
  http://arduino.cc/forum/index.php/topic,145765
http://www.gammon.com.au/electronics

Oso57

Hello all...
I've finally solved the problem, thanks to everyone for the tips.

Nic : malloc.c not solve the bug in this case.
Arrch:  sprintf() does not work very well, for example, sprintf( Result," %02u   %02u   %02 ",A,B,C) or
sprintf( Result," %d   %d   %d ",A,B,C) ,  does not work well.
PaulS:  Was a good idea to move the lcd.print() to TiempoTotal().


Finally my function code was well :
Code: [Select]

int TiempoTotal (unsigned long Cantidad){
     
      char Hs[10];
      char Min[10];
      char Seg[10];
      char Respuesta[16];
     
      unsigned long TT = Cantidad * Retardo;           
     
      unsigned long Remanente1 = (TT % 3600);
      unsigned long H = (TT - Remanente1)/3600;
      unsigned long S = Remanente1 % 60;
      unsigned long M = ( Remanente1 - S) / 60;
      sprintf(Hs," %02uHs. ",H);           
      sprintf(Min,"%02um. ",M); 
      sprintf(Seg,"%02us. ",S); 
      //sprintf(Respuesta,"%d:%d:%d  ", H,M,S);
      //lcd.setCursor(0, 2);
      //lcd.print (Respuesta);
   
      lcd.setCursor(0, 2);
      lcd.print (  Hs );
      lcd.setCursor(7, 2);
      lcd.print ( Min );
      lcd.setCursor(12, 2);
      lcd.print ( Seg );
   
      return 0;
}


Thanks again to all

Greeting Oso57 from Argentina,.

PaulS

Quote
Arrch:  sprintf() does not work very well, for example, sprintf( Result," %02u   %02u   %02 ",A,B,C) or
sprintf( Result," %d   %d   %d ",A,B,C) ,  does not work well.

What was the result of the two calls? The first one contains an invalid format specifier. The second assumes ints, not long (signed or unsigned).

Though why H, M, and S are longs in your code is a mystery. The range of values for M and S are 0 to 60. Those values would fit in a byte just as easily as in a 4-byte long.

Arrch


Arrch:  sprintf() does not work very well, for example, sprintf( Result," %02u   %02u   %02 ",A,B,C) or
sprintf( Result," %d   %d   %d ",A,B,C) ,  does not work well.


Good to see it's working, but sprintf() works just fine when you use it correctly.

Oso57

#13
Feb 23, 2013, 05:05 pm Last Edit: Feb 23, 2013, 05:07 pm by Oso57 Reason: 1


Arrch:  sprintf() does not work very well, for example, sprintf( Result," %02u   %02u   %02 ",A,B,C) or
sprintf( Result," %d   %d   %d ",A,B,C) ,  does not work well.


Good to see it's working, but sprintf() works just fine when you use it correctly.


PaulS, Arrrch ...

sprintf( Result," %02u", A ) 

Since A a long value works fine for me, this padded with leading zero if the value is less than 10) 

I understand that the code is not optimal, but is the result of looking again and again and again the solution of the problem, it did not and still do not know that the bug with Strings ().
Years ago I program in Visual Basic, but C is new to me
I will learn more about  sprintf() ( for me Visual basic is more, more simple. Ex: Result=Format( A ,"00") )
I will learn more C.

Thanks for your help.


Go Up