SPIFFS - Ecriture des fichiers - incompréhension

Version imprimable

Bonjour à tous,
Je butte sur une incompréhension de ma part, certainement sur le langage C++:
Après avoir chargé

Code:

1
2
#include "FS.h"
#include <SPIFFS.h>

Je crée une routine d'écriture de fichier:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
//***************** Ecriture SPIFFS **********************************
void ecritureFichier(fs::FS &fs, const char *dir, const char *message){
  sBt.printf("Ecriture du fichier: %s\r\n", dir);
  File fic = fs.open(dir, FILE_WRITE);
  if(!fic){
    sBt.println("Pas de fichier");
    return;
  }
  if(fic.print(message)){
    sBt.println("Fichier ecrit");
  } else {
    sBt.println("Erreur d'ecriture");
  }
  fic.close();
}

puis pour y accéder j'ai une autre routine:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
//*************Programmation alarme journalière par bluetooth**************
void enregAlarmJour(char *al){
  if(!(al[2]==':' && al[5]=='-')){
    sBt.println(msg[3]);
    return;
  }
  hAlarmJ= (al[0]-ASCII0) * 10 + (al[1]-ASCII0);    //conversion ASCII en int
  mAlarmJ= (al[3]-ASCII0) * 10 + (al[4]-ASCII0);
  msgAlarmJ="";
  for(int i=6;i<LGMSG;i++){
    if(al[i]=='\n') break;
    msgAlarmJ += al[i];
  }
  msgAlarmJ.trim();
  char rep;
  do{
    sBt.println("Enregistrement ? O/N");
    while(!sBt.available()){}
    rep=sBt.read();
  }while(!(rep=='o' || rep=='O' || rep=='n' || rep=='N'));
  if(rep=='o' || rep=='O'){
    ecritureFichier(SPIFFS,"/AlarmJ.txt",al);
    delay(1000);
  }
}

Là j'obtiens un message positionné sur la contenu du fichier 'al' à l'appel de la procédure d'écriture:

Citation:

ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]

Je ne comprends pas ce message. Pourquoi dans la fonction d'écriture la variable 'message' est déclarée 'const' ?

Merci d'une aide
Michel

03/02/2023, 15h30
Jay M

Note: ne postez pas des bouts de code et postez l'erreur complète, ça fait perdre du temps à tout le monde

quel est le type de msgAlarmJ ? une instance de la classe String?

ou est exactement le souci dans le code?
03/02/2023, 17h05
mormic

Merci de vous interessez à mon problème.

Mon prog fait actuellement 600 lignes et le coeur de mon problème se situe sur ces bouts de code.
msgAlarmJ est bien une instance de la classe String puisque je l'initialize à "" dans la routine; puis je lui applique la fonction .trim().
Le soucis se situe sur l'instruction

Code:

ecritureFichier(SPIFFS,"/AlarmJ.txt",al);

sur la variable al.

Par contre je viens de retenter une compilation; et sans rien modifier les erreurs ont disparues ??
Je travaille sur VS Code et le seul message d'erreur que j'avais est celui affiché lorsque je positionne le curseur sur le soulignement d'erreur.

De temps en temps VS Code a des réactions bizarres. Je viens de charger l'ESP32 et tout fonctionne correctement.

Désolé mais je crois vous avoir alerté pour rien.
03/02/2023, 22h52
Jay M

OK - juste par acquis de conscience, quand vous appeliez

Code:

void enregAlarmJour(char *al)

est-ce que pour tester vous aviez mis un texte fixe, du genre enregAlarmJour("10:20 du texte"); ?

l'origine du message est là:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
//*****************boucle principale****************
void loop() {
  char tempo ;
  char commande[50];
  //****gestion des commandes Bluetooth****
  if(sBt.available()){
    tempo = sBt.read();                   //lit le premier caractère
    int i = 0;
    while(sBt.available()){
      commande[i] = sBt.read();           // lit la commande complète sans tempo
      i++;
    }
    if((tempo=='A' || tempo=='a')){       //commande d'affichage
      analyseComAffic(commande[0]);       //envoi la commande d'affichage
    }   
    else if(tempo=='F' || tempo=='f'){    //c'est une commande de fonction
      analyseComFct(commande);            //envoi la commande complète
      matrice.displayClear();
    }
    else {                                //erreur de commande
      sBt.println(commande);
      sBt.println(msg[0]);                //"cette commande n'existe pas"
    }
  }
  while(sBt.available()){tempo=sBt.read();}   //pour vider le buffer de sBt
  delay(50);

puis là:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
//*************** Analyse de la commande de fonction ************************
void analyseComFct(char *codeRecu){
  char cd = codeRecu[0];        //commande de la fonction
  for(int i=1; i<LGMSG;i++){    //i=1 pour démarrer l'analyse après le cd
    if(codeRecu[i] == '\n'){
      break;    //si code reçu est un  NUL on sort de la boucle 
    }
    codeRecu[i-1] = codeRecu[i];    //décalage de 1 caractère pour éliminer la 'cd'
  }
  switch (cd){
    case 'S': case 's': ecritureFichier(SPIFFS,"/ssid.txt",codeRecu);
    break;
    case 'P': case 'p': ecritureFichier(SPIFFS,"/pass.txt",codeRecu);
    break;
    case 'H': case 'h': enregAlarmHebdo(codeRecu);        //format: Hhh:mm+J-'texte'   
    break;                                                //pour J dimanche=1, lundi=2, mardi=3, etc
    case 'O': case 'o': enregAlarmeOccas(codeRecu);       //format: Ohh:mm-'texte'
    break;
    case 'J': case 'j': enregAlarmJour(codeRecu);
    break;
    case 'B': case 'b': enregPub(codeRecu);
    break;
    default:
    break;
  }
}

Donc je ne viens pas d'un String mais de tableau de caractères.

04/02/2023, 17h06
Jay M

ça ne m'en dit pas plus...

pour revenir à ce point

Citation:

Envoyé par mormic

Là j'obtiens un message positionné sur la contenu du fichier 'al' à l'appel de la procédure d'écriture:

Code:

ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]

Je ne comprends pas ce message. Pourquoi dans la fonction d'écriture la variable 'message' est déclarée 'const' ?

on a ce message quand on passe une chaîne (string literal) du genre "Hello" à une fonction qui attend unchar * et pas un const char *. Une string literal est constant et donc ne peut être convertie en variable. Il en va de même si on utilise .c_str() sur une String puisque ça retourne un const char *.

Vous aviez sans doute quelque chose comme cela qui trainait dans votre code
04/02/2023, 18h17
mormic

L'appel initial des procédures se fait en passant un char *

Code:

analyseComFct(commande);

puis toujours en char *

Code:

case 'J': case 'j': enregAlarmJour(codeRecu);

enfin encore en char*

Code:

ecritureFichier(SPIFFS,"/AlarmJ.txt",al);

et pour finir je devrais transmettre un const char *. Là je pourrais comprendre qu'il y a une erreur. Mais maintenant
la compilation est bonne et pourtant c'est une variable que je transmets ??

Pourquoi la routine d'écriture nécessite de mettre un const char * ?
04/02/2023, 18h43
Jay M

Code:

Pourquoi la routine d'écriture nécessite de mettre un const char * ?

quand on met const dans un paramètre pointeur dans une fonction, ça veut dire que la fonction ne touchera pas à sa valeur. Un truc variable peut toujours être catégorisé constant mais l'inverse n'est pas vrai.

certaines fonctions cependant sont mal écrites et ont oublié de mettre le const, ce qui fait que si dans une fonction bien écrite (où vous avez mis const parce qu'il n'y a pas de modification) vous invoquez une fonction mal écrite qui attend un char * au lieu d'un const char *, alors vous aurez le message d'erreur

C'est dommage que le compilateur n'ait pas été plus bavard sur le lieu du problème, l'important c'est que vous ne l'ayez plus (si vous pouvez faire une recompilation complète du projet peut-être que vous reverrez le message d'erreur)
05/02/2023, 09h18
mormic

Merci Jay M pour tous ces éclaircissements.
Votre aide nous est toujours aussi précieuse pour mener à bien nos petits projets.
Bien cordialement
Michel