Array terminieren NULL leeren

[Andreas F. (codecasa)]

Hallo

Wie kann man eine Array z.B der Classe Menu also Menu plapla[20] leeren
zurücksetzen so das mann Sie wieder befüllen kann ?

hier mal der Code:

    Menu    mnuProg[20];
    uint8_t usedMnu = 0;
            
Menu* allocateMenu()
{
  if( usedMnu >= 20 ) return NULL;        
  return &mnuProg[usedMnu++];
}


void mnuAddCMD(char *n)
{
  Menu* newMenu = allocateMenu();

  if( newMenu == NULL )  
  {
    // da ist wohl was schief gelaufen
    Serial.println("newMenu = NULL");
    return;
  }
  
  newMenu->name = n;
  

  Item11.addChild(*newMenu);
}

Ich will praktich wieder neue Menüs erstellen Ohne noch mehr Speicher zu 
verbrauchen

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:

> Menu* allocateMenu()
> {
>   if( usedMnu >= 20 ) return NULL;
>   return &mnuProg[usedMnu++];
> }

Das hier rückt genau ein Array Element nach dem nächsten raus.
Wenn du also ein Element zurückgeben willst, dann muss es das letzte 
sein. Mehr geht mit dieser Verwaltung erst mal nicht. (dafür ist sie 
allerdings einfach)
   usedMnu--;


Wenn du natürlich alle zurückgeben willst, dann reicht ein
   usedMnd = 0;

und das Array wird komplett recycled.

[Andreas F. (codecasa)]

Karl Heinz Buchegger schrieb:
> Andreas Frauenstein schrieb:
>
>> Menu* allocateMenu()
>> {
>>   if( usedMnu >= 20 ) return NULL;
>>   return &mnuProg[usedMnu++];
>> }
>
> Das hier rückt genau ein Array Element nach dem nächsten raus.
> Wenn du also ein Element zurückgeben willst, dann muss es das letzte
> sein. Mehr geht mit dieser Verwaltung erst mal nicht. (dafür ist sie
> allerdings einfach)
>
> Wenn du natürlich alle zurückgeben willst, dann reicht ein
>    usedMnd = 0;
>
> und das Array wird komplett recycled.

Ja Dachte ich mir auch
usedMnd = 0;

Aber der Zeigt mir danach im Menü komische Zeichen an :(
Kopletter code im Anhang

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:


>> Wenn du natürlich alle zurückgeben willst, dann reicht ein
>>    usedMnd = 0;
>>
>> und das Array wird komplett recycled.
>
> Ja Dachte ich mir auch
> usedMnd = 0;
>
> Aber der Zeigt mir danach im Menü komische Zeichen an :(

Weil die hälfte deiner Pointer in den Menu Objekten nicht NULL sind.
(In der Allocate Funktion auf NULL setzen!)

[Andreas F. (codecasa)]

Karl Heinz Buchegger schrieb:
> Andreas Frauenstein schrieb:
>
>
>>> Wenn du natürlich alle zurückgeben willst, dann reicht ein
>>>    usedMnd = 0;
>>>
>>> und das Array wird komplett recycled.
>>
>> Ja Dachte ich mir auch
>> usedMnd = 0;
>>
>> Aber der Zeigt mir danach im Menü komische Zeichen an :(
>
> Weil die hälfte deiner Pointer in den Menu Objekten nicht NULL sind.
> (In der Allocate Funktion auf NULL setzen!)


etwa so ?

Menu* allocateMenu()
{
  if( usedMnu >= 20 ) return NULL;        
  return &mnuProg[usedMnu++];
  mnuProg[usedMnu - 1] = NULL;
}

[Karl H. (kbuchegg)]

Eigentlich sollte ich das ja nicht tun.
Denn eigentlich solltest du erst mal die Sprache lernen, die du benutzt 
und auch einfache Algorithmen.

Aber hier hast du einen gepimpten einfachen MenuManager (ich kann da 
nicht länger zuschauen)

Er wird so verwendet

#include "Menu.h"

MenuManager pool;

int main()
{
  Menu* m1 = pool.Allocate();    // einen allokieren
  Menu* m2 = pool.Allocate();    // noch einen allokieren

  pool.Free( m1 );               // der wird nicht mehr gebraucht: zurückgeben

  pool.Reset();                  // ach was solls: ich gebe alle zurück

  m1 = pool.Allocate();          // denn dann kann wieder neu allokiert werden
  m2 = pool.Allocate();          // so viele, bis keine mehr da sind.


  return 0;
}

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:

> etwa so ?
>

> Menu* allocateMenu()
> {
>   if( usedMnu >= 20 ) return NULL;
>   return &mnuProg[usedMnu++];
>   mnuProg[usedMnu - 1] = NULL;
> }
> 
>

denk nachmal ganz scharf nach.
Wird Code NACH einem return jemals ausgeführt?

Du solltest DRINGEND an deinem Coding Style arbeiten. Der ist 
unübersichtlich bis zum geht nicht mehr. Das du da nichts siehst wundert 
mich nicht.

[Andreas F. (codecasa)]

Ja da muss ich dir Recht geben ich will die Tage alles ein bischen 
Verlagern in Headerdateien

Ich probiere das jetzt gleich mal aus Danke vielmals und desweiteren 
werde ich schauen das ich nicht nur Kopiere

[Andreas F. (codecasa)]

Oh ich denke Ich bekomme noch mehr Probleme wenn ich statt nur die 
dynamichen auch die Statischen mit der neuen Menu classe initialisiere.

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:
> Oh ich denke Ich bekomme noch mehr Probleme wenn ich statt nur die
> dynamichen auch die Statischen mit der neuen Menu classe initialisiere.

Sagt kein Mensch, das du das musst.
Der MenuManager verwaltet einen Pool von 20 Menu Objekten. Auf 
Anforderung rückt er eines raus (wenn noch eines da ist) oder nimmt 
wieder eines unter seine Fittiche, wenn du es nicht mehr brauchst.

[Andreas F. (codecasa)]

Ich finde in der cpp überhaupt kein costruktor ?????

Muss ich dann noch die Original Menu.h Includieren ???

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:
> Ich finde in der cpp überhaupt kein costruktor ?????

Der ist inline im Header File

> Muss ich dann noch die Original Menu.h Includieren ???

Natürlich
Sieh dir das Codebeispiel weiter oben an.

[Andreas F. (codecasa)]

Was bedeutet inline ?

Dann überschreibt mann aber die Original Menu.h oder ?????

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:
> Was bedeutet inline ?

Kauf dir ein C++ Buch
Dringend!

> Dann überschreibt mann aber die Original Menu.h oder ?????

Es ist nach wie vor alles da, was in deiner originalen Menu.h auch war.

[Andreas F. (codecasa)]

Ja du inludierst in der Main-DATEI :)

Menu.h

damit ist aber nur deine Menu.h gemeint also kann ich die Original 
Menu.h
In ein Müll-Ordner packen ???

[Andreas F. (codecasa)]

Karl Heinz Buchegger schrieb:
> Andreas Frauenstein schrieb:
>> Was bedeutet inline ?
>
> Kauf dir ein C++ Buch
> Dringend!
>
>> Dann überschreibt mann aber die Original Menu.h oder ?????
>
> Es ist nach wie vor alles da, was in deiner originalen Menu.h auch war.


Achso na dann ist ja Super Tausend Dank

Ich hoffe nur das Ich das mit Array auch hinbekomme

[Andreas F. (codecasa)]

/usr/share/arduino/libraries/Menu/Menu.h:15:17: error: ISO C++ forbids 
declaration of ‘boolean’ with no type
/usr/share/arduino/libraries/Menu/Menu.h:15:17: error: typedef ‘boolean’ 
is initialized (use decltype instead)
/usr/share/arduino/libraries/Menu/Menu.h:15:19: error: ‘canFunction’ was 
not declared in this scope
/usr/share/arduino/libraries/Menu/Menu.h:22:24: error: ‘canFunction’ has 
not been declared
/usr/share/arduino/libraries/Menu/Menu.h:38:3: error: ‘canFunction’ does 
not name a type
/usr/share/arduino/libraries/Menu/Menu.h: In constructor 
‘Menu::Menu(char*, int)’:
/usr/share/arduino/libraries/Menu/Menu.h:27:5: error: class ‘Menu’ does 
not have any field named ‘canEnter’

[Karl H. (kbuchegg)]

Andreas Frauenstein schrieb:
> /usr/share/arduino/libraries/Menu/Menu.h:15:17: error: ISO C++ forbids
> declaration of ‘boolean’ with no type

Ach ich vergass.
In deiner originalen war ein #include für ein Header File, von dem ich 
nicht weiß wo es herkommt bzw. was da drinnen ist.
Da dürfte der typedef für boolean drinnen gewesen sein.

Ansonsten mach ein

typedef unsigned char boolean;

ins Header File oben rein.

[Karl H. (kbuchegg)]

Noch mal nachgesehen.
Das war der

#include "WProgram.h"

im originalen Menu.h

[Andreas F. (codecasa)]

schon okay ich hatte es erst gesehen aber keine beachtung geschenkt wie 
so oft

es war noch
#include "WProgramm"

[Andreas F. (codecasa)]

So läuft klasse aber leider nur Beim ersten aufruf des Menus Item11

wenn ich mit der Backtaste rausgehe un wieder rein Ist alles leer und 
Der AVR reargiert nicht mehr

[Andreas F. (codecasa)]

Ich denke Will die Item11 noch die &Childs von Iteam11.addChild hat

[Andreas F. (codecasa)]

Habs  :)

Menu.h ausschnitt

public:
  Menu(char *n = NULL, canFunction c = NULL )
  : parent( NULL ),
    child( NULL ),
    sibling( NULL ),
    name( n ),
    canEnter( c )
  {}

  void addChild(Menu &c);                //Adds the child c to the Menu.
                                         //If the menu already has a child, ask the child to add it as a sibling
  void lreset(); <<<<<<-------

Menu.cpp ausschnitt

void Menu::reset()
{
  parent   = NULL;
  child    = NULL;
  sibling  = NULL;
  name     = NULL;
  canEnter = NULL;
}

void Menu::lreset()
{
  child    = NULL;
  sibling  = NULL;
}
//*****************************

Und in der Haupdatei:

Item11.lreset(); // damit wird die Adresse von Child, Sibling und somit all anderen kinder entfernt