[Jérémy]

Bonsoir à tous,

Je ne comprends pas un truc !

Je suis obligé de déclarer une variable en INT alors que celle ci ne depassera jamais 255 ! sinon le calcul renvoyer est FAUX !!

Je m'explique :

Je recois par sms l'heure les minutes et les secondes , à la reception je les stock dans un tableau par INT Rx UART .

Ensuite je recupere chaques cases pour calculer le temps en secondes que je stock dans un unsigned long que j'appelle "chrono" .

Ensuite je souhaite envoyer ce "chrono" par texto donc je dois faire le sens inverse !

voila ce que ca donne .

Code : Tout sélectionner


char T_Chrono[15]={0};
unsigned int Heures, Minutes, secondes;

la partie calcul du "chrono" dans le main ( en mode simplifiée ):

Code : Tout sélectionner

       if ( F_chrono){
            F_chrono = 0;
            //--- récupération et calcul du Chrono en seconde
            Heures = ((T_Chrono[0]-48)*10) + (T_Chrono[1]-48);
            Minutes = ((T_Chrono[2]-48)*10) + (T_Chrono[3]-48);
            Secondes = ((T_Chrono[4]-48)*10) + (T_Chrono[5]-48);
            
            Chrono = ( (Heures*3600) + (Minutes*60) + Secondes );
           
               composition_message(1);
             }
         }

La fonction composition_message simplifiée aussi :

Code : Tout sélectionner

//------------------------------------------------------------------------------
void composition_message(char numero_message){

  char Compteur=0, Compteur2=0;
  char temp[30];

  F_envoi_ok=0;
  T_message[0] = 0;    // On commence la chaine par un NULL afin de pouvoir y ajouter la suite

  //--------------------    séléction et Création du message      -------------------
  switch (numero_message){

         case 1:{
                  strcat(T_message, "Valeurs recues:\r\n");
                  ShortToStr (Heures, temp);
                  strcat(T_message, temp );
                  strcat(T_message, "h");
                  ShortToStr (Minutes, temp);
                  strcat(T_message, temp );
                  strcat(T_message, "m");
                  ShortToStr (Secondes, temp);
                  strcat(T_message, temp );
                  strcat(T_message, "s\r\n");

                  LongWordToStr(Chrono, temp);
                  strcat(T_message, temp );
                  strcat(T_message, "s\r\n");
                  
                  strcat(T_message, "Voulez-vous le lancer ?");
                  strcat(T_message, 0);  // On termine par un terminateur
                }


dans cette configuration ça fonctionne .

Mais si je mets char Heures, Minutes, secondes; le résultat est mauvais et je ne comprends pas pourquoi !

merci à vous si vous avez une explication

[brunog]

Source du message        if ( F_chrono){
            F_chrono = 0;
            //--- récupération et calcul du Chrono en seconde
            Heures = ((T_Chrono[0]-48)*10) + (T_Chrono[1]-48);
            Minutes = ((T_Chrono[2]-48)*10) + (T_Chrono[3]-48);
            Secondes = ((T_Chrono[4]-48)*10) + (T_Chrono[5]-48);
            
            Chrono = ( (Heures*3600) + (Minutes*60) + Secondes );
           
               composition_message(1);
             }
         } 

Hello,
le langage C ne fait pas de surclassement automatique, ainsi par exemple si Heures est de type unsigned char, l'expression Heure * 3600 sera évaluée avec l'arithmétique unsigned char et le résultat ne sera pas celui attendu

[Jérémy]

J'avoue ne pas comprendre

Je stock ce résultat dans un unsigned long ! je pensais que cela ne posais pas de problème. Seulement si le stockage est inférieur au résultat !!

Mais apparemment j'étais dans l'erreur !

Donc la variable "Heures" doit être du type pour stocker son résultat après la multiplication , c'est bien ca ?

[brunog]

le type du membre de gauche d'une affectation ne détermine pas l'arithmétique du membre de droite

Source du message Donc la variable "Heures" doit être du type pour stocker son résultat après la multiplication , c'est bien ca ?

oui, ou sinon faire un type cast pour indiquer au compilateur l'arithmétique à utiliser et la nature du stockage du résultat intermédiaire, par exemple :

Code : Tout sélectionner

(unsigned int)Heures*3600

[Jérémy]

Ok !!

Beaucoup !

Je vais donc déclarer directement mes varaibles en unsigned intce sera plus simple pour moi !

[paulfjujo]

bonjour,


le compilateur MikroC sait gerer (un peu) les dépassement de format
ex:

Code : Tout sélectionner

unsigned char c1
unisgned int k;
c1=18;
k=c1*3600
result k=64800


mais si tu geres l'heure au format 24H au lieu de 12H AM PM
tu auras un probleme des 19H00

Code : Tout sélectionner


c1=19;
k=2846 

debordement du int car > 65535  .. là le compilo n'y peut rien 


mais l'usage d'un cast est une tres bonne habitude , au cas ou le compilo ( ou le programmeur ) s'y perdrait un peu
unsigned long L;
L=(unsigned long) c1 *3600;
...

[Jérémy]

pour ce complément d'infos

[Jérémy]

J'ai opté au final pour cette solution suite à vos conseils

Ainsi je déclare mes variables en char , car les valeurs ne dépassent jamais les 255

Code : Tout sélectionner

char Heures, Minutes, secondes; 

et ensuite je CAST seulement les "Heures" qui atteignent 99*3600=356400 au max ! donc en Long
ça donne :

Code : Tout sélectionner

Chrono = ( ((unsigned long)Heures*3600) + (Minutes*60) + Secondes ); 

Je test ca dés que je peux ! ( besoin du tel de ma femme qui est en train de dévaler les pistes de ski !)