char Array an Funktion übergeben

[basti (Gast)]

 char mona[11];                           // Array Monatsname

void get( uint8_t ss)
{ 
 uint8_t month ;
 
 if ( month==1)               // Januar
 { 
  strcpy( mona, "jan" );  // Kopiere Monatsname in mona-Array

  write_temp_timeDate(mona);   
 }
}


void writing_sd_temp(char *monath)
{
 uint8_t i = 0;
  
   if ( hour==0 && minute==30 && second==45 )
    {
   write_temp_timeDate(monath ); // Uhrzeit und Datum in Datei 
    }
}


unsigned char write_temp_timeDate(char *name ) 
//###################################################################################
{
//
  unsigned char by;
  unsigned int written;
  unsigned char  result;

  sprintf( Buffer, "%02d:%02d:%02d %02d.%02d.%02d",hour,minute,second,day,month,year);  //

  sprintf( datname, "%02d%02d%02d%02d",hour,minute,day,month);  //


  strcat( datname, ".txt" ); // .txt an datname  anhängen damits ein 8.3DOS-Name wird

  Chdir("temper");           // ORDNER temper  ÖFFNEN
  Chdir(name);             // ORDNER Monats-Name  ÖFFNEN




   if(Fopen(datname,'a')!=F_OK) return 0;
    {        
       written=Fwrite((unsigned char *)Buffer,strlen(Buffer));
    
        by='\r';                //Enter
        result=Fwrite(&by,1);  //Schreiben
          
    by='\n';               //Neue Zeile
        result=Fwrite(&by,1); //Schreiben
        
     
    }
      Fclose();

    Chdir("..");        //Ordner Monats-Name  Schliessen
    Chdir("..");        //Ordner  temper Schliessen            
return 1;//result;
}

[basti (Gast)]

oh da hab ich zu schnell gedrückt,

ich möchte gerne das Array mona an 2 Funktionen übergeben, leider klappt 
das nicht so.

in der Funktion void get( uint8_t ss) (wird von der Mainloop alle 
5sekunden aufgerufen) soll das array an die Funktion 
write_temp_timeDate(mona); weitergegeben werden ,in der ein Ordner 
geöffnet werden soll leider funktioniert das nicht so.

Wenn ich den Ordner so Chdir("jan" ); öffne geht es.

vielleicht kann mir einer von ihnen weiterhelfen was ich falsch mache 
mit dem array

 char mona[11];                           // Array Monatsname

void get(uint8_t month)
{ 
 uint8_t month ;
 
 if ( month==1)               // Januar
 { 
  strcpy( mona, "jan" );  // Kopiere Monatsname in mona-Array

  write_temp_timeDate(mona);   
 }
}


void writing_sd_temp(char *monath)
{
 uint8_t i = 0;
  
   if ( hour==0 && minute==30 && second==45 )
    {
   write_temp_timeDate(monath ); // Uhrzeit und Datum in Datei 
    }
}


unsigned char write_temp_timeDate(char *name ) 

{
//
  unsigned char by;
  unsigned int written;
  unsigned char  result;

  sprintf( Buffer, "%02d:%02d:%02d %02d.%02d.%02d",hour,minute,second,day,month,year);  //

  sprintf( datname, "%02d%02d%02d%02d",hour,minute,day,month);  //


  strcat( datname, ".txt" ); // .txt an datname  anhängen damits ein 8.3DOS-Name wird

  Chdir("temper");           // ORDNER temper  ÖFFNEN
  Chdir(name);             // ORDNER Monats-Name  ÖFFNEN




   if(Fopen(datname,'a')!=F_OK) return 0;
    {        
       written=Fwrite((unsigned char *)Buffer,strlen(Buffer));
    
        by='\r';                //Enter
        result=Fwrite(&by,1);  //Schreiben
          
    by='\n';               //Neue Zeile
        result=Fwrite(&by,1); //Schreiben
        
     
    }
      Fclose();

    Chdir("..");        //Ordner Monats-Name  Schliessen
    Chdir("..");        //Ordner  temper Schliessen            
return 1;//result;
}

[Klaus W. (mfgkw)]

Sorry, aber erstens verstehe ich nicht, was das Ganze soll, und zweitens 
sind da ein paar -ähm- Merkwürdigkeiten, die mich glauben lassen, daß es 
irgendwie auch nicht sinnvoll gehen kann, und drittens ist die 
Formatierung so grottig, daß ich mich als Compiler weigern würde, etwas 
davon zu übersetzen.

char mona[11];                           // Array Monatsname

mona wird nicht initialisiert?

void get(uint8_t month)

"get" ist schnell geaschrieben, aber als Name nicht besonders 
aussagekräftig.

ok, es gibt einen Parameter month...

{
  uint8_t month ;

... und jetzt gleich noch eine lokale Variable mit demselben Namen? Wozu 
dann der Parameter? Welchen Wert bekommt die lokale Variable month?

  if ( month==1)               // Januar
  {
    strcpy( mona, "jan" );  // Kopiere Monatsname in mona-Array

    write_temp_timeDate(mona);
  }

1. ist der Januar wirklich 1 und nicht 0?
2. Was ist, wenn month nicht 1 ist? Dann darf das Programm mit dem 
zufälligen Inhalt von mona weiterlaufen?

}


void writing_sd_temp(char *monath)

Nach mona und month wieder eine Variable, die so ähnlich heißt, aber 
irgendwie anders ist. Man könnte mal sinnvolle Namen nehmen, damit man 
den Dingern ansieht, was wozu ist.
naja egal, die Funktion wird eh nicht verwendet.
Aber wozu steht sie dann hier? Um mich zu verwirren?
Wenn sie Teil des Problems sein kann, fehlt ein Aufruf.
Wenn sie nicht nötig ist, kann sie weg.

{
  uint8_t i = 0;

  if ( hour==0 && minute==30 && second==45 )

wo kommen diese drei jetzt her? Haben die sinnvolle Werte?

  {
    write_temp_timeDate(monath ); // Uhrzeit und Datum in Datei

Wenn du konsequent alle Rückgabewerte ignorierst, wundert es niemanden, 
wenn du nicht weisst, was in deinem Programm passiert.

  }
}


unsigned char write_temp_timeDate(char *name )

{
  //
  unsigned char by;
  unsigned int written;
  unsigned char  result;

Was machen die drei? Außer der letzten sieht man dem Namen nicht viel 
an, initalisiert sind sie natürlich auch wieder nicht, und verwendet 
werden sie auch nicht alle.

  sprintf( Buffer, "%02d:%02d:%02d %02d.%02d.%02d",hour,minute,second,day,month,year);  //

Wo kommt jetzt auf einmal Buffer her? Ist der groß genug? Wieso eine 
globale Variable?

  sprintf( datname, "%02d%02d%02d%02d",hour,minute,day,month);  //


  strcat( datname, ".txt" ); // .txt an datname  anhängen damits ein 8.3DOS-Name wird

Könnte man auch gleich vom sprintf mitmachen lassen, aber egal.

  Chdir("temper");           // ORDNER temper  ÖFFNEN
  Chdir(name);             // ORDNER Monats-Name  ÖFFNEN

Gibt es das Verzeichnis überhaupt? Was, wenn nicht?

  if(Fopen(datname,'a')!=F_OK) return 0;

Bekommt dein Fopen wirklich ein char als zweiten Parameter?

  {
    written=Fwrite((unsigned char *)Buffer,strlen(Buffer));

Was, wenn das Schreiben nicht klappt? Keine Reaktion darauf?

    by='\r';                //Enter
    result=Fwrite(&by,1);  //Schreiben

    by='\n';               //Neue Zeile
    result=Fwrite(&by,1); //Schreiben

Wieso nicht gleich beide auf einmal (Fwrite("\r\n",2))?
Was, wenn das Schreiben nicht klappt? Keine Reaktion darauf?

  }
  Fclose();

  Chdir("..");        //Ordner Monats-Name  Schliessen
  Chdir("..");        //Ordner  temper Schliessen
  return 1;//result;

Falscher Kommentar! Eben nicht result.

}

basti schrieb:
> vielleicht kann mir einer von ihnen weiterhelfen was ich falsch mache
> mit dem array

Ich frage mich eher, was an dem Programm überhaupt funktionieren soll; 
abgesehen davon, daß es nicht vollständig ist.

Sehr planvoll sieht es nicht aus, ehrlich gesagt.

basti schrieb:
> ich möchte gerne das Array mona an 2 Funktionen übergeben, leider klappt
> das nicht so.

"klappt nicht" ist keine wirklich sinnvolle Fehlerbeschreibung.

Vielleicht wird der Programmteil in get gar nicht durchlaufen, weil 
month nicht 1 ist?

[basti (Gast)]

hallo,
hhour,minute,second,day,month,year sind Globale Variablen.
char Buffer[50];// Array Ausgabe Buffer für alle möglichen lcd ausgaben 
ist ein globales ARRAY

void get(uint8_t month) wird vom Hauptprogramm aufgerufen und enthält 
den Monat 1 bis 12,
wenn zB. month==1 dann ist das Januar.

Meine Ordner sind auf der SD-Card vorhanden in TEMPER  sind die 
Moants-Ordner  von Januar bis Dezember.

Aber leider kann ich die Monats-Ordner so nicht öffnen wie ichs jetzt 
habe.
Wenn ich das jetzt so mache in write_temp_timeDate(char *name ),
  Chdir("temper");           // ORDNER temper  ÖFFNEN
  Chdir("jan");             // ORDNER Monats-Name  ÖFFNEN
dann gehts,aber ich wollte das gerne anders machen so wie unten.

 char mona[11];                           // Array Monatsname für die Manats-Ordner
 char datname[11];                         // Array Text-Datei-Name in den Manats-Ordner 8.3 DOSNAME 


void get(uint8_t month)
{ 

       if ( month==1)               // Januar
        { 
         strcpy( mona, "jan" );  // Kopiere Monatsname in mona-Array
         write_temp_timeDate(mona);   ("jan");    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if ( month==2)         //Februar
        { 
         strcpy( mona, "feb" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if ( month==3)         //März
        { 
         strcpy( mona, "mae" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if ( month==4)         //April
        { 
         strcpy( mona, "apr" );      // Kopiere Monatsname in mona-Array 
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (  month==5)         //Mai
        { 
         strcpy( mona, "mai" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (  month==6)         //Juni
        { 
         strcpy( mona, "jun" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (  month==7)         //Juli
        { 
         strcpy( mona, "jul" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (  month==8)         //August
        { 
         strcpy( mona, "aug" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (  month==9)         //September
        { 
         strcpy( mona, "sep" );      // Kopiere Monatsname in mona-Array   
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (  month==10)       //October
        { 
         strcpy( mona, "okt" );      // Kopiere Monatsname in mona-Array
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (   month==11)       //November
        { 
         strcpy( mona, "nov" );      // Kopiere Monatsname in mona-Array  
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }

       if (   month==12)       //Dezember
        { 
         strcpy( mona, "dez" );      // Kopiere Monatsname in mona-Array
         write_temp_timeDate(mona);   (mona);    // Vergleich Stunde zum Temperratur-Daten schreiben
        }
}


void writing_sd_temp(char *monath)
{
 uint8_t i = 0;
  
   if ( hour==0 && minute==30 && second==45 )
    {
     write_temp_timeDate(monath ); // Uhrzeit und Datum in Datei 
    
       for(i = 1; i < 19; i++)
        {
         write_temp(monath,i,TempWert[i],TempWertkomma[i],TempgWert[i]);
        }
    }
}


unsigned char write_temp_timeDate(char *name )
{         
  unsigned char by;
  unsigned int written;
  unsigned char  result;

  sprintf( Buffer, "%02d:%02d:%02d %02d.%02d.%02d",hour,minute,second,day,month,year);  //

  sprintf( datname, "%02d%02d%02d%02d",hour,minute,day,month);  //


  strcat( datname, ".txt" ); // .txt an datname  anhängen damits ein 8.3DOS-Name wird

  Chdir("temper");           // ORDNER temper  ÖFFNEN
  Chdir(name);             // ORDNER Monats-Name  ÖFFNEN




   if(Fopen(datname,'a')!=F_OK) return 0;
    {        
       written=Fwrite((unsigned char *)Buffer,strlen(Buffer));
    
        by='\r';                //Enter
        result=Fwrite(&by,1);  //Schreiben
          
        by='\n';               //Neue Zeile
        result=Fwrite(&by,1); //Schreiben
              
      Fclose();
    }

    Chdir("..");        //Ordner Monats-Name  Schliessen
    Chdir("..");        //Ordner  temper Schliessen            
return result;
}

[Troll (Gast)]

>[...Viele if...]
Schon mal was von switch und case gehört?