Discussion :

[herrflick13]

Bonjour et bonne année à tous.

j'ai écrit un petite bibliothèque pour mon Arduino. lorsque je mets, les fichiers .cpp et .h dans le répertoire de mon sketch, tout va bien. Mais lorsque je les installe en tant que librairie, j'ai une erreur de compilation, dont voici le texte :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\sketch\Tools.cpp.o (symbol from plugin):(.text+0x0): first defined here
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\libraries\Tools\Tools.cpp.o (symbol from plugin): In function `lcd':
(.text+0x0): multiple definition of `Tools::spaces(int)'
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\sketch\Tools.cpp.o (symbol from plugin):(.text+0x0): first defined here
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\libraries\Tools\Tools.cpp.o (symbol from plugin): In function `lcd':
(.text+0x0): multiple definition of `Tools::cls()'
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\sketch\Tools.cpp.o (symbol from plugin):(.text+0x0): first defined here
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\libraries\Tools\Tools.cpp.o (symbol from plugin): In function `lcd':
(.text+0x0): multiple definition of `Tools::erase()'
C:\Users\pasca\AppData\Local\arduino\sketches\F72C5994B7661A9E78E69ECE879FBFB2\sketch\Tools.cpp.o (symbol from plugin):(.text+0x0): first defined here
collect2.exe: error: ld returned 1 exit status
 
exit status 1

je ne comprends pas pourquoi...

Voici une partie de ma déclaration dans le fichier .h

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
class Tools
{
 
public:
 
  void setup();
  void cls();
  void erase();
  String spaces(int nombreEspaces);
  void writeln(String nouvelleLigne);
  void write(String ajouterALaDerniereLigne);
//  void write(bool booleen);
  void write(byte octet);
  void write(char caractere);
  void write(double flottantDoublePrecision);
//  void write(float Flottant);
  void write(int entier);
  void write(long entierLong);
private:
   bool   flagWrite;   
   String ligne[nombreLignes];
   String tamponLigne;
};

Si vous pouviez m'éclairer, ce serait vraiment sympa. Je ne suis pas un pro du c++, je débute alors soyez indulgents.

Merci

[Jay M]

Peut être vous avez un souci de déclaration double. Le .h devrait ressembler à cela avec une garde contre la double importation

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#ifndef TOOLS_H
#define TOOLS_H
 
class Tools {
  public:
    // Constructeur
    Tools();
 
    // Méthodes ...
    void xxx();
};
 
#endif

et le .cpp serait

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#include "Tools.h"
#include "Arduino.h"
 
// Constructeur
Tools::Tools() {
  // ...
}
 
// Méthodes
void Tools::xxx() {
  // ...
}

votre .ino serait

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
#include "Tools.h"
 
void setup() {}
void loop() {}

[herrflick13]

Bonsoir à tous,

finalement, j'ai hérité de l'objet que j'utilisais : LiquidCrystal_I2C. Mais je me trouve face à un gros problème, j'utilise deux méthodes 'write' ou 'writeln' où j'écris des chaines de caractères. Mais impossible de définir plusieurs méthodes portant le même nom avec un paramètre diffèrent plutôt qu'une chaine, un entier, un booléen, un flottant, mais rien n'y fait. Comme je débute en c++, la partie du code permettant d'afficher une chaine fonctionne très bien et fais ce que je lui demande mais j'ai été obligé de mettre en commentaire mes autres méthodes.
Pouvez vous m'aider à résoudre mon problème ?

[jackk]

Bonjour,

Mais impossible de définir plusieurs méthodes portant le même nom avec un paramètre diffèrent plutôt qu'une chaine, un entier, un booléen, un flottant, mais rien n'y fait.

Ce n'est pas normal, car le C++ l'autorise.
Il faudrait voir le code complet de la classe ainsi que les messages d'erreur.

[Jay M]

oui il faut poster le code là - vous avez dû rater une étape.

[herrflick13]

j'ai un problème d'initialisation de mon tableau, voici la partie du code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
class Tools : public LiquidCrystal_I2C {
private:
 
  bool   flagWrite;   
  // String ligne[nombreLignes];
  char** ligne;
  String tamponLigne;
  void _write(String value);      
  uint8_t nombreLignes;
  uint8_t nombreColonnes;    
 
public:
  // Constructeur     
  Tools(uint8_t lcd_Addr,uint8_t lcd_cols,uint8_t lcd_rows) : LiquidCrystal_I2C(lcd_Addr,lcd_cols,lcd_rows) {
    ligne = new char*[lcd_rows];
    for (int i = 0; i < lcd_rows; i++)  ligne[i] = new char*[lcd_cols];
    nombreLignes = lcd_cols;
    nombreColonnes = lcd_rows;    
  };

le but est de créer un tableau de chaine de caractères fixe dépendant des paramètres du constructor.

Si vous pouviez m'éclairer, ce serait vraiment sympa. Je débute en C++, alors un peu d'indulgence s'il vous plait. Merci.

[Jay M]

on pourrait faire un truc comme cela sans doute (ne pas oublier le destructeur pour ne pas avoir de fuite mémoire)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
class Tools : public LiquidCrystal_I2C {
private:
    uint8_t nombreLignes;
    uint8_t nombreColonnes;
    char** lignes;  // Tableau dynamique de chaînes de caractères
 
public:
    Tools(uint8_t lcd_Addr, uint8_t lcd_cols, uint8_t lcd_rows) 
        : LiquidCrystal_I2C(lcd_Addr, lcd_cols, lcd_rows), nombreLignes(lcd_rows), nombreColonnes(lcd_cols) {
        lignes = new char*[nombreLignes];
        for (uint8_t i = 0; i < nombreLignes; ++i) {
            lignes[i] = new char[nombreColonnes + 1]; // +1 pour le caractère nul de fin de chaîne
        }
    }
 
    // Destructeur pour libérer la mémoire allouée
    ~Tools() {
        for (uint8_t i = 0; i < nombreLignes; ++i) {
            delete[] lignes[i];
        }
        delete[] lignes;
    }
};

mais on pourrait aussi utiliser un template C++ ou la classe est paramètrée par les dimension du LCD ce qui permet de créer un tableau en 2D avec ces valeurs.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
template <uint8_t nombreLignes, uint8_t nombreColonnes>
class Tools : public LiquidCrystal_I2C {
  private:
    char lignes[nombreLignes][nombreColonnes + 1];  // Tableau 2D statique de chaînes de caractères
 
  public:
    Tools(uint8_t lcd_Addr)
      : LiquidCrystal_I2C(lcd_Addr, nombreColonnes, nombreLignes) {}
};

c'est quand même plus simple

[herrflick13]

Merci pour ton aide, ça passe maintenant à la compilation, mais je ne sais pas comment affecter une valeur a l'une des lignes du tableau :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
class Tools : public LiquidCrystal_I2C {
private:
 
  bool   flagWrite;   
  // String ligne[nombreLignes];
  char** ligne;
  String tamponLigne;
  void _write(String value);      
  uint8_t nombreLignes;
  uint8_t nombreColonnes;    
 
public:
  // Constructeur     
  Tools(uint8_t lcd_Addr,uint8_t lcd_cols,uint8_t lcd_rows) : LiquidCrystal_I2C(lcd_Addr,lcd_cols,lcd_rows) {
    ligne = new char*[lcd_rows];
    for (uint8_t i = 0; i < lcd_rows; i++)  ligne[i] = new char[lcd_cols  + 1];
    nombreLignes = lcd_cols;
    nombreColonnes = lcd_rows;    
  };

voici la méthode d'initialisation :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
void Tools::cls() {
  //flagWrite = true;  
 
  for (int i = 0;i < nombreLignes; i++) ligne[i] = "\0";
 
  setCursor(0,0);
 
  print(spaces(nombreLignes * nombreColonnes));
 
  setCursor(0,nombreLignes);
}

et l'erreur :

C:\Users\pasca\Documents\Arduino\Writeln_LCD\Tools.cpp: In member function 'void Tools::cls()':
C:\Users\pasca\Documents\Arduino\Writeln_LCD\Tools.cpp:16:51: error: expected primary-expression before '=' token
for (int i = 0;i < nombreLignes; i++) ligne[i]* = "\0";

[jackk]

j'ai un problème d'initialisation de mon tableau, voici la partie du code :

Quel problème?
A la compilation?
A l'exécution?

[jackk]

A mince, je n'avais pas rafraichi la page et vu ton dernier message.

Le problème est qu'on ne peut pas affecter une "C string" dans une autre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ligne[i] = "\0";

est donc impossible.

Pour copier une chaine, il faut utiliser la fonction "strncpy"

[herrflick13]

Bonsoir,

j'ai un nouveau problème :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
void Tools::writeln(char* nouvelleLigne)

avec le message d'erreur :

C:\Users\pasca\Documents\Arduino\Writeln_LCD\Tools.h:48:37: warning: ISO C++ forbids declaration of 'writeln' with no type [-fpermissive]
Tools::writeln(char* nouvelleLigne);


pouvez vous m'éclairer, s'il vous plait ?

Merci

[jackk]

Tu nous montres l'entête de la définition de la fonction alors que le compilateur parle d'un problème dans la déclarationde writeln, qui se trouve à priori dans le fichier d'entête (.h).
Tu n'aurais pas oublié le "void" dans la déclaration de writeln?

[herrflick13]

Merci pour ton aide

le void ajouté il me donne un warning :

In file included from C:\Users\pasca\Documents\Arduino\Writeln_LCD\Tools.cpp:11:0:
C:\Users\pasca\Documents\Arduino\Writeln_LCD\Tools.h:48:8: warning: extra qualification 'Tools::' on member 'writeln' [-fpermissive]
void Tools::writeln(char* nouvelleLigne);


une idée ? Merci

[herrflick13]

j'avais laisser Tools:: dans le fichier .h, je continue et je vous tiens au courant,

Merci de tout cœur à tous, ça ne fait que quelques jours que je programme en C,

j'ai une fonction write qui est surchargée et la fonction principale s'appelle _write, on m'a dit que je ne respectais pas les recommandations pour le c++. Comment devrais-je nommer cette méthode ?

[jackk]

j'avais laisser Tools:: dans le fichier .h, je continue et je vous tiens au courant

C'est bon du coup?

j'ai une fonction write qui est surchargée et la fonction principale s'appelle _write, on m'a dit que je ne respectais pas les recommandations pour le c++. Comment devrais-je nommer cette méthode ?

Je ne comprends pas trop: qu'appelles-tu une fonction principale?
Le problème avec write, c'est qu'elle appartient déjà aux bibliothèques classiques du C/C++. Mais si cette fonction write appartient à une classe que tu as créée, aucun problème de te créer une fonction membre qui s'appelle write et de la surcharger.