3. qualified-id in declaration before '(' token

qualified-id in declaration before '(' token

  • B

    geht leider so nicht:

    | src/OV/OpenVarioMenu.cpp: In lambda function:
| src/OV/OpenVarioMenu.cpp:190:18: error: cannot bind non-const lvalue reference of type 'std::string&' {aka 'std::__cxx11::basic_string<char>&'} to an rvalue of type 'std::string' {aka 'std::__cxx11::basic_string<char>'}
|   190 |     SaveRotation("rotation=0", "0");
|       |                  ^~~~~~~~~~~~
| In file included from ../recipe-sysroot/usr/include/c++/11.3.0/string:55,
|                  from ../recipe-sysroot/usr/include/c++/11.3.0/bits/locale_classes.h:40,
|                  from ../recipe-sysroot/usr/include/c++/11.3.0/bits/ios_base.h:41,
|                  from ../recipe-sysroot/usr/include/c++/11.3.0/streambuf:41,
|                  from ../recipe-sysroot/usr/include/c++/11.3.0/bits/streambuf_iterator.h:35,
|                  from ../recipe-sysroot/usr/include/c++/11.3.0/iterator:66,
|                  from ./output/src/boost_1_81_0/boost/move/detail/iterator_traits.hpp:27,
|                  from ./output/src/boost_1_81_0/boost/move/detail/reverse_iterator.hpp:25,
|                  from ./output/src/boost_1_81_0/boost/intrusive/detail/reverse_iterator.hpp:16,
|                  from ./output/src/boost_1_81_0/boost/intrusive/list.hpp:28,
|                  from ./src/ui/window/custom/WList.hpp:28,
|                  from ./src/ui/window/ContainerWindow.hpp:29,
|                  from ./src/Form/Form.hpp:26,
|                  from ./src/Dialogs/WidgetDialog.hpp:26,
|                  from src/OV/OpenVarioMenu.cpp:26:
| ../recipe-sysroot/usr/include/c++/11.3.0/bits/basic_string.h:533:7: note:   after user-defined conversion: 'std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, const _Alloc&) [with <template-parameter-2-1> = std::allocator<char>; _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]'
|   533 |       basic_string(const _CharT* __s, const _Alloc& __a = _Alloc())
|       |       ^~~~~~~~~~~~
| src/OV/OpenVarioMenu.cpp:179:44: note:   initializing argument 1 of 'void ScreenRotationWidget::SaveRotation(std::string&, std::string&)'
|   179 | ScreenRotationWidget::SaveRotation(string &rotation, string &rotationvalue)
|       |
    0
  • B

    es funktioniert

    void
ScreenRotationWidget::SaveRotation(const string &rotation, const string &rotationvalue) <--hier const eingefügt
{
    File::WriteExisting(Path("/boot/config.uEnv"), (rotation).c_str());
    File::WriteExisting(Path("/sys/class/graphics/fbcon/rotate_all"), (rotationvalue).c_str());
}

    {
  AddButton("Landscape", [&](){  <--hier [&] statt []
    SaveRotation("rotation=0", "0");
    static constexpr const char *argv[] = {
      "/bin/sh", "-c", 
      "echo The screen was rotated to landscape", 
      nullptr 
    };

    RunProcessDialog(UIGlobals::GetMainWindow(),
                     UIGlobals::GetDialogLook(),
                     "Landscape", argv);
  });

    Danke allen für die Hilfe!!!

    0

  • geht leider so nicht: (...)

    Ja, das ist halt jetzt ein anderer, zusätzlicher Fehler.

    es funktioniert (...)

    [&] ist Overkill, [this] wie ich geschrieben habe sollte reichen.

    0
    B
     1 Antwort
  • B

    Wie so oft ist ein Problem nicht so trivial, wie es zu Beginn scheint. Das obige Funktioniert, hat aber den Nachteil, dass der String immer in die erste Zeile der Datei geschrieben wird. Meine Datei hat in etwa folgenden Inhalt:

    rotation=1
    timeout=10
    LANG=de_DE.UTF-8

    Ich muss es daher noch realisieren, dass der String in die Zeile geschrieben wird, in der der alte String stand. Sollte der String noch nicht vorhandene ein, soll er hinten angehängt werden. Ich habe den Hinweis bekommen, dass das mit folgendem Skript geht:
    https://github.com/XCSoar/XCSoar/blob/f829b8f394d2eb21c276d17f38b134bc3f9e7466/src/Profile/File.cpp

    Ich habe mir andere Dateien angeschaut, in denen Profile::LoadFile vorkommt, verstehe aber die Funktionsweise noch nicht. Nach meinem Verständnis muss ich ja den zu suchenden String und den Pfad angeben. Daher habe ich mir was in der Art gedacht:
    Profile::LoadFile("LANG*", "/boot/config.uEnv");

    Nach dem ich mir die Beispiel angeschaut habe bin ich mir ziemlich sicher, das it Blödsinn. Vielleicht kann mir jemand helfen die Funktion besser zu Verstehen.

    Danke!!

    0

  • Man lädt einmalig die Datei mit LoadFile und übergibt eine leere ProfileMap.
    Anschließend ändert man die Werte in der ProfileMap und speichert dann mit SaveFile wieder die gesamte Datei ab (man kann in einer Textdatei nicht direkt einzelne Zeilen ersetzen).

    0
    B
     1 Antwort
  • B

    @hustbaer Danke, ändere ich!!

    0
  • B

    @Th69 Danke!! Das hilft für das Verständnis sehr weiter!!!

    0
  • B

    Ich versuche mich gerade in das Thema ProfileMap einzuarbeiten. Ich versuche mal kleinschrittige Fragen zu stellen, um es besser verstehen zu können.

    Die Funktion LoadConfig müsste also ein map mit dem Namen configuration anlegen und jede Zeile der Datei /boot/config.uEnv auslesen. Dann wird z.B. dem key1 rotation und als Wert 1, dem key3 LANG und als Wert de_DE.UTF-8, etc zugewiesen. Habe ich das bis hierher korrekt verstanden?

    static void LoadConfig()
{
    Profile::Clear();
    Profile::LoadFile(configuration, "/boot/config.uEnv");
}

    Aufgerufen wird die Funktion mit:

    LoadConfig();
    0

  • Ja, du mußt eine Variable dafür anlegen:

    ProfileMap configuration;

    und dann kannst du (entsprechend Map.hpp) eine der Set-Funktionen dafür aufrufen, z.B.

    configuration.Set("rotation", 1);
    0
  • B

    Ich hoffe, ich hab es korrekt verstanden. Dann müsste die Funktion so aussehen:

    static void LoadConfig(&keyvalue, &value)
{
    ProfileMap configuration;
    Profile::Clear();
    Profile::LoadFile(configuration, "/boot/config.uEnv");
    configuration.Set(keyvalue, value);
}

    Und die Funktion müsste dann so aufgerufen werden, oder?

    LoadConfig("rotation", 1);
    0

  • Fast - du mußt noch die richtigen Typen bei den Funktionsparametern angeben (C++ ist typsicher).
    Und & ist nur bei Referenzparametern anzugeben.

    Und ich würde die Funktion ' ChangeConfig' nennen und noch den 'Profile::SaveFile(...)'-Aufruf hinzufügen (sonst macht die Funktion bisher nichts wirklich sinnvolles).

    PS: Wenn du verschiedene Typen für value benötigst (und nicht nur int), dann brauchst du hier schon fortgeschrittenere C++-Funktionalität: template.

    0
  • B

    Vielen Dank für deine geduldige Erklärung!!

    Ich habe die Funktion soweit angepasst:

    static void ChangeConfig(const sting keyvalue, const sting value)
{
    ProfileMap configuration;
    Profile::Clear();
    Profile::ChangeConfig(configuration, "/boot/config.uEnv");
    configuration.Set(keyvalue, value);
    Profile::SaveFile(configuration, "/boot/config.uEnv");
}

    Der Aufruf erfolgt mit:

    ChangeConfig("rotation", 1);

    Der Teil müsste so nun funktionieren, oder?

    Tatsächlich wird es auch einen Aufruf in der Art geben müssen:

    ChangeConfig("LANG", "de_DE.UTF-8");

    Das heißt, wenn ich dich richtig verstehe, dass das so noch nicht geht?

    0
  • B

    Kompilieren bringt noch Fehler:

    | src/OV/OpenVarioMenu.cpp:98:16: error: aggregate 'ProfileMap configuration' has incomplete type and cannot be defined
|    98 |     ProfileMap configuration;
|       |                ^~~~~~~~~~~~~
| src/OV/OpenVarioMenu.cpp:99:14: error: 'Clear' is not a member of 'Profile'
|    99 |     Profile::Clear();
|       |              ^~~~~
| src/OV/OpenVarioMenu.cpp:100:14: error: 'ChangeConfig' is not a member of 'Profile'; did you mean 'ChangeConfig'?
|   100 |     Profile::ChangeConfig(configuration, "/boot/config.uEnv");
|       |              ^~~~~~~~~~~~
| src/OV/OpenVarioMenu.cpp:96:13: note: 'ChangeConfig' declared here
|    96 | static void ChangeConfig(const string keyvalue, const string value)
|       |             ^~~~~~~~~~~~
| make: *** [build/compile.mk:107: output/UNIX/opt/src/OV/OpenVarioMenu.o] Error 1
    0

  • Sorry, aber so wird das nichts - du mußt schon die Grundlagen (Syntax und Semantik) von C++ verstehen.

    Am besten (wenn du englisch kannst) mittels LearnCpp.
    Es gibt auch noch (für den Einstieg) Wiki: C++ Programmierung.
    Sonst s.a. noch Linkliste.

    Ich gebe dir jetzt (einmalig) mal den Code:

    #include "Map.hpp" // für ProfileMap

static void ChangeConfig(const string &keyvalue, int value)
{
    const char * const ConfigPath = "/boot/config.uEnv"; // um nur eine Codestelle dafür zu haben

    ProfileMap configuration;
    // Profile::Clear(); // <- diese Funktion gibt es wohl nicht bei XCSoar
    Profile::LoadFile(configuration, ConfigPath); // Funktionsname wieder zurück geändert
    configuration.Set(keyvalue, value);
    Profile::SaveFile(configuration, ConfigPath);
}

    Für andere Aufrufe (z.B. string) bräuchtest du, wie schon geschrieben, ein Template:

    template<typename T>
static void ChangeConfig(const string &keyvalue, T value)
{
  // der Rest bleibt gleich
}
    0
  • B

    Danke!! Ich schaue mir deine Empfehlungen an. So ganz allmählich blicke ich es aber schon. Allerdings führt deine Code noch zu Fehlern:

    | src/OV/OpenVarioMenu.cpp: In function 'void ChangeConfigInt(const string&, int)':
| src/OV/OpenVarioMenu.cpp:102:38: error: could not convert 'ConfigPath' from 'const char* const' to 'Path'
|   102 |     Profile::LoadFile(configuration, ConfigPath);
|       |                                      ^~~~~~~~~~
|       |                                      |
|       |                                      const char* const
| src/OV/OpenVarioMenu.cpp:103:22: error: no matching function for call to 'ProfileMap::Set(const string&, int&)'
|   103 |     configuration.Set(keyvalue, value);
|       |     ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
| In file included from src/OV/OpenVarioMenu.cpp:42:
    0

  • Ich habe mir die genaue API nicht angeschaut, sondern mich nur an deinem Code (+ den Fehlermeldungen) orientiert.
    Du mußt anscheinend ein Objekt der Klasse Path erzeugen und übergeben:

    const Path ConfigPath("/boot/config.uEnv"); // evtl. für Unicode: L"/boot/config.uEnv"

    Und statt string muß wohl direkt const char * als Typ für keyvalue verwendet werden (oder aber du benutzt beim Aufruf configuration.Set(keyvalue.c_str(), value).

    0
  • B

    Es läuft noch nicht ganz. Ich habe heute viel gelesen, um das Problem möglichst selbst zu finden, und denke, ich bin grundsätzlich auf der richtigen Spur. Folgender Fehler kommt noch:

    |   LINK    output/UNIX/bin/OpenVarioMenu
| src/OV/OpenVarioMenu.cpp:176: error: undefined reference to 'Profile::LoadFile(ProfileMap&, Path)'
| src/OV/OpenVarioMenu.cpp:177: error: undefined reference to 'ProfileMap::Set(char const*, int)'
| src/OV/OpenVarioMenu.cpp:178: error: undefined reference to 'Profile::SaveFile(ProfileMap const&, Path)'
| collect2: error: ld returned 1 exit status

    Das Problem ist wohl, das Profile nicht public ist. Soweit ich das recherchieren konnte, muss die Funktion ChangeConfigInt in Zeile 171 und folgende eingebunden werden. Damit sollte sie eigentlich public sein. Ich nehme aber an, dass da noch eine Kleinigkeit fehlt.

    Hier noch mal der Link zum Skript, damit es besser nachvollziehbar ist: https://github.com/freevariode/XCSoar/blob/master/src/OV/OpenVarioMenu.cpp

    0

  • Mit public/protected/private hat das nichts zu tun.
    Das Problem ist dass die drei Funktionen einfach nirgends definiert sind.

    0

  • Siehst du beim Kompilieren eine Liste der Dateien?
    Dabei sollten dann sowohl src/Profile/File.cpp als auch src/Profile/Map.cpp auftauchen (eigenartigerweise ist dort aber nur ProfileMap::Set(const char *key, const char *value) definiert und nicht die anderen Set-Überladungen, auch nicht in der Headerdatei ProfileMap.hpp).

    Nach Suche habe ich aber dann herausgefunden, daß die ProfileMap-Funktionen über diverse andere Source-Dateien (u.a. NumericValue.cpp) verteilt sind (very strange).
    Es sollten also alle Source-Dateien im Profile-Ordner mitkompiliert werden, so daß es keine Linkerfehler mehr gibt.

    0
  • B

    also in der Liste der Dateien beim Kompilieren kommen unter Anderem diese Dateien vor:

    |   CXX     output/UNIX/opt/src/Profile/Profile.o
|   CXX     output/UNIX/opt/src/Profile/File.o
|   CXX     output/UNIX/opt/src/Profile/Map.o

    wie in dieser Datei (https://github.com/XCSoar/XCSoar/blob/master/src/Input/InputEventsSettings.cpp) habe ich auch versuchsweise folgendes eingetragen:

    #include "Profile/Profile.hpp"
#include "Profile/ProfileKeys.hpp"
#include "Profile/Settings.hpp"
#include "Profile/Current.hpp"

    Aber das verändert nichts. Ich verstehe nicht, was in dieser Datei anders gemacht wird als in meiner.

    0
Anmelden zum Antworten