Fehler beim Kompilieren

[ich (Gast)]

Hallo,

ich bekomme in PlatformIO folgenden Fehler beim Kompilieren:

.pio\build\d1_mini\src\main.cpp.o: In function `lcdml_menu_display_h()':
main.cpp:(.text._Z20lcdml_menu_display_hv+0x0): multiple definition of `lcdml_menu_display_h()'
.pio\build\d1_mini\src\display_utils.cpp.o:display_utils.cpp:(.text._Z20lcdml_menu_display_hv+0x0): first defined here


.pio\build\d1_mini\src\main.cpp.o: In function `lcdml_menu_clear_h()':
main.cpp:(.text._Z18lcdml_menu_clear_hv+0x0): multiple definition of `lcdml_menu_clear_h()'
.pio\build\d1_mini\src\display_utils.cpp.o:display_utils.cpp:(.text._Z18lcdml_menu_clear_hv+0x0): first defined here


.pio\build\d1_mini\src\main.cpp.o: In function `lcdml_menu_control_h()':
main.cpp:(.text._Z20lcdml_menu_control_hv+0x0): multiple definition of `lcdml_menu_control_h()'
.pio\build\d1_mini\src\display_utils.cpp.o:display_utils.cpp:(.text._Z20lcdml_menu_control_hv+0x0): first defined here
.pio\build\d1_mini\src\main.cpp.o:(.bss.lcd+0x0): multiple definition of `lcd'
.pio\build\d1_mini\src\display_utils.cpp.o:(.bss.lcd+0x0): first defined here
.pio\build\d1_mini\src\main.cpp.o:(.bss.LCDML_1+0x0): multiple definition of `LCDML_1'
.pio\build\d1_mini\src\display_utils.cpp.o:(.bss.LCDML_1+0x0): first defined here
.pio\build\d1_mini\src\main.cpp.o:(.bss.LCDML+0x0): multiple definition of `LCDML'
.pio\build\d1_mini\src\display_utils.cpp.o:(.bss.LCDML+0x0): first defined here

CPU: ESP8266

display.h

#ifndef DISPLAY_H
#define DISPLAY_H

#include <Arduino.h>
#include "config.h"
#include <LCDMenuLib2.h> //  https://github.com/ich423/LCDMenuLib2.git 
#include <Wire.h>
#include <LiquidCrystal_I2C.h>

extern void(*ptr_lcdml_menu_display)();
extern void(* ptr_lcdml_menu_clear)();
extern void(* ptr_lcdml_menu_control)();

void lcdml_menu_display_h() {
    //lcdml_menu_display();
}

void lcdml_menu_clear_h() {
    //lcdml_menu_display();
}

void lcdml_menu_control_h() {
    //lcdml_menu_display();
}

LiquidCrystal_I2C lcd(0x27, _LCDML_DISP_cols, _LCDML_DISP_rows);
LCDMenuLib2_menu LCDML_1(255, 0, 0, NULL, NULL); // root menu element (do not change)
LCDMenuLib2 LCDML(LCDML_1, _LCDML_DISP_rows, _LCDML_DISP_cols, lcdml_menu_display_h, lcdml_menu_clear_h, lcdml_menu_control_h);

class displayUtils {
  public:
    displayUtils();
    
};

#endif

display_utils.cpp (gekürzt)

#include <Arduino.h>
#include "display.h"
#include <LiquidCrystal_I2C.h>
#include <LCDMenuLib2.h> //https://github.com/Jomelo/LCDMenuLib2


displayUtils::displayUtils(){
}

// =====================================================================
//
// Output function
//
// =====================================================================

void lcdml_menu_control()
{
  // If something must init, put in in the setup condition
  if(LCDML.BT_setup()) {
    // runs only once
  }
/*
  // check if new serial input is available
  if (Serial.available()) {
    // read one char from input buffer,
   switch (Serial.read())
    {
      case _LCDML_CONTROL_serial_enter:  LCDML.BT_enter(); break;
      case _LCDML_CONTROL_serial_up:     LCDML.BT_up();    break;
      case _LCDML_CONTROL_serial_down:   LCDML.BT_down();  break;
      case _LCDML_CONTROL_serial_left:   LCDML.BT_left();  break;
      case _LCDML_CONTROL_serial_right:  LCDML.BT_right(); break;
      case _LCDML_CONTROL_serial_quit:   LCDML.BT_quit();  break;
      default: break;
    }
  }*/
}

/* ******************************************************************** */
void lcdml_menu_clear()
/* ******************************************************************** */
{
  lcd.clear();
  lcd.setCursor(0, 0);
}

/* ******************************************************************** */
void lcdml_menu_display()
/* ******************************************************************** */
{
  // update content
  // ***************
  if (LCDML.DISP_checkMenuUpdate()) {
    // clear menu
    // ***************
    LCDML.DISP_clear();

    // declaration of some variables
    // ***************
    // content variable
    char content_text[_LCDML_DISP_cols];  // save the content text of every menu element
    // menu element object
    LCDMenuLib2_menu *tmp;
    // some limit values
    uint8_t i = LCDML.MENU_getScroll();
    uint8_t maxi = _LCDML_DISP_rows + i;
    uint8_t n = 0;

    }
  }


  if(LCDML.DISP_checkMenuCursorUpdate())
  {
    // init vars
    uint8_t n_max             = (LCDML.MENU_getChilds() >= _LCDML_DISP_rows) ? _LCDML_DISP_rows : (LCDML.MENU_getChilds());
    uint8_t scrollbar_min     = 0;
    uint8_t scrollbar_max     = LCDML.MENU_getChilds();
    uint8_t scrollbar_cur_pos = LCDML.MENU_getCursorPosAbs();
    uint8_t scroll_pos        = ((1.*n_max * _LCDML_DISP_rows) / (scrollbar_max - 1) * scrollbar_cur_pos);

    }

    // display scrollbar
    if (_LCDML_DISP_cfg_scrollbar == 1) {
      if (scrollbar_max > n_max) {
        //set scroll position
        if (scrollbar_cur_pos == scrollbar_min) {
          // min pos
          lcd.setCursor((_LCDML_DISP_cols - 1), 0);
          lcd.write((uint8_t)1);
        } else if (scrollbar_cur_pos == (scrollbar_max - 1)) {
          // max pos
          lcd.setCursor((_LCDML_DISP_cols - 1), (n_max - 1));
          lcd.write((uint8_t)4);
        } else {
          // between
          lcd.setCursor((_LCDML_DISP_cols - 1), scroll_pos / n_max);
          lcd.write((uint8_t)(scroll_pos % n_max) + 1);
        }
      }
    }
  }
}

void(* ptr_lcdml_menu_display)() = &lcdml_menu_display;
void(* ptr_lcdml_menu_clear)() = &lcdml_menu_clear;
void(* ptr_lcdml_menu_control)() = &lcdml_menu_control;

[Jürgen W. (Firma: MED-EL GmbH) (wissenwasserj)]

Du definierst oben eine Funktion 'lcdml_menu_control_h' und gibst in der 
Funktion 'LCDML' ebendiese Funktion als Datentyp für ein Argument an.
Kann so nicht laufen.

[Jörg W. (dl8dtl) (Moderator)]

ich schrieb:

> void lcdml_menu_display_h() {
>     //lcdml_menu_display();
> }

Komplette Funktionsdefinitionen (statt nur Deklarationen) gehören nicht 
in ein Headerfile.

(Ausnahme: "static inline".)

> LiquidCrystal_I2C lcd(0x27, _LCDML_DISP_cols, _LCDML_DISP_rows);

Was soll das denn in einem Headerfile?

Das ist ja weder Definition noch Deklaration, sondern ein 
Funktionsaufruf?!

Der hat dort rein gar nichts zu suchen. Pack sowas an den Anfang von 
main().

> void(* ptr_lcdml_menu_display)() = &lcdml_menu_display;
> void(* ptr_lcdml_menu_clear)() = &lcdml_menu_clear;
> void(* ptr_lcdml_menu_control)() = &lcdml_menu_control;

Globale Variablendefinitionen schreibt man üblicherweise an den Anfang 
einer Datei, nicht ans Ende.