C++ zeigt in der Ausgabe nichts an

[Stephan (Gast)]

Nabend,
ich möchte ein Programm schreiben, in dem ich ermittle, wie oft der 
Montag der 1.,2.,3.,.... Dienstag der1.,2.,... usw. war.

#include <stdio.h>
#include <time.h>


int main(void) {
    char *wday[] = { "Sonntag", "Montag", "Dienstag", "Mittwoch", "Donnerstag", "Freitag", "Samstag", "???" };
    struct tm ts;
    //int auswertung[7][31];
    int year, month, day;
 
  for(year=2000; year<=2015; year++){
    for(month=1; month<=12; month++){
      for(day=1; day<=31; day++){
        ts.tm_year = year - 1900;
        ts.tm_mon  = month - 1;
          ts.tm_mday = day;
          if ( mktime(&ts) == -1 )
            ts.tm_wday = 7;
            if(wday[ts.tm_wday]!="???"){
              printf("%s, %d, %d\n",wday[ts.tm_wday],ts.tm_wday,day);
          }
          
      }
    }
  }
  return 0;
}

bisher lasse ich mir die Tage und Datums in der Konsole ausgeben.
Um mein eigentliches Vorhaben zu machen, wollte ich eine Matrix 
schreiben (7x32)- Damit habe ich jede kombination gegeben und kann in 
jedem Feld den wert erhöhen.

das Problem ist, sobald ich die Zeile

int auswertung[7][31];

auskommentiere, wird mir in der Konsole nichts mehr angezeigt.
Woran liegt das? Das macht für mich keinen sinn.

Stephan

[Tom (Gast)]

blubb.c:19:32: warning: comparison with string literal results in 
unspecified behavior [-Waddress]

[Peter II (Gast)]

Stephan schrieb:
> Woran liegt das?

kann ich nicht sagen, aber

wday[ts.tm_wday]!="???"){

ein vergleich mit char* kann man so nicht machen, prüfe lieber vorher ob 
ts.tm_wday zwischen 0 und 6 liegt.

[Stephan (Gast)]

Wieso kann ich das nicht machne? ich vergleiche doch nur zwei strings 
miteinander, oder nicht?
Habe es trotzdem geändert. Dennoch bleibt das Problem das selbe.

[Peter II (Gast)]

Stephan schrieb:
> Wieso kann ich das nicht machne? ich vergleiche doch nur zwei strings
> miteinander, oder nicht?

char* ist aber kein String sondern ein Array und Arrays kann man (so) 
nicht vergleichen.

[Peter II (Gast)]

Peter II schrieb:
> char* ist aber kein String sondern ein Array und Arrays kann man (so)
> nicht vergleichen.

genauer gesagt ist es ein pointer (nicht das sich jemand an Array stört)

[g457 (Gast)]

> char* ist aber kein String sondern ein Array und Arrays kann man (so)
> nicht vergleichen.

Nö, char* ist ein Zeiger, und der Compiler warnt zu Recht davor, 
selbigen Zeiger mit der ∗Adresse∗ eines String-Literals zu vergleichen.

[Yalu X. (yalu) (Moderator)]

Stephan schrieb:
> Habe es trotzdem geändert. Dennoch bleibt das Problem das selbe.

Auch den Elemente tm_hour, tm_min und tm_sec in ts solltest du sinnvolle
Werte zuweisen (im einfachsten Fall 0). Lediglich tm_wday, tm_yday und
tm_isdst kannst du uninitialisiert lassen, da sie von mktime berechnet
werden.

Edit:

tm_isdst musst du ebenfalls initialisieren (s. Man-Page von mktime).

[Stephan (Gast)]

Schreibe es jetzt in eine textdatei. das funktioniert auch soweit. Das 
Problem was ich jetzt habe, dass ich nicht weiss, wie ich überprüfen 
kann, ob der Tag überhaupt existiert hat. Momentan spuckt er mir aus, 
dass der 29.Februar2014 ein Samstag war und zählt diesen mit. 
(30.Februar2014->Sonntag, 31.Februar2014->Montag).

kann ich auch irgendwie kontrollieren, ob dieser Tag überhaupt existiert 
hat?

#include <fstream>
#include <iostream>
#include <string>
#include <stdio.h>
#include <time.h>

using namespace std;

#define numwdays 7
#define numdays 31

int main(void) {
    char *wday[] = { "Sonntag", "Montag", "Dienstag", "Mittwoch", "Donnerstag", "Freitag", "Samstag"};
    int x=0;
    struct tm ts;
    int auswertung[numdays][numwdays];
    for(int k=0; k<numdays; k++){
      for(int l=0; l<numwdays; l++){
        auswertung[k][l] = 0;
    }
  }
  ts.tm_hour = 0;
    ts.tm_min  = 0;
    ts.tm_sec  = 1;
    ts.tm_isdst = -1;
    
  for(int year=2014; year<2015; year++){
    for(int month=2; month<=2; month++){
      for(int day=1; day<=31; day++){
        ts.tm_year = year-1900;
        ts.tm_mon  = month-1;
          ts.tm_mday = day;
          
          if ( mktime(&ts) == -1 ){
            ts.tm_wday = 7;
        }
        printf("Dieser Tag ist ein %s\n", wday[ts.tm_wday]);    
        
          if(ts.tm_wday>=0 && ts.tm_wday<=6){
            auswertung[ts.tm_mday-1][ts.tm_wday] = auswertung[ts.tm_mday-1][ts.tm_wday] + 1;
            x = x + 1;
          }
        }
    }
  }
  
  ofstream f("text.txt");
  for(int m=0;  m<numdays; m++){
    for(int n=0; n<numwdays; n++){
      //printf("%-9d", auswertung[m][n]);
      f <<  auswertung[m][n];
      f << "   ";
    }
    f << ("\n");
  }
  f << x;
  f.close();
  return 0;
}

[Dirk B. (dirkb2)]

Stephan schrieb:
> kann ich auch irgendwie kontrollieren, ob dieser Tag überhaupt existiert
> hat?

Vergleiche nach dem mktime,  ob die Werte in ts dieselben sind, die du 
vorher rein geschrieben hast.

Zu deinem ersten Problem.
Es gibt in C eine feste Adresse, die man für ungültige Adressen 
verwendet: NULL

    char *wday[] = { "Sonntag", "Montag", "Dienstag", "Mittwoch", "Donnerstag", "Freitag", "Samstag", NULL};
...
            if(wday[ts.tm_wday] != NULL){

Allerdings ist in diesem Fall die Version mit dem Vergleich von 
ts.tm_wday < 7 besser.

Und das

printf("Dieser Tag ist ein %s\n", wday[ts.tm_wday]);

 muss mit in das nachfolgende if rein.

[Stephan (Gast)]

mhh,
habe es jetzt so gemacht, dass ich mir die Tage in einem Monat aus einer 
Funktion geben lasse und die for-Schleife für die Tage nur bis dahin 
laufen lass.

#include <fstream>
#include <iostream>
#include <string>
#include <stdio.h>
#include <time.h>

using namespace std;

#define numwdays 7
#define numdays 31


int getDaysInMonth(int month, int year);



int main(void) {
    char *wday[] = { "Sonntag", "Montag", "Dienstag", "Mittwoch", "Donnerstag", "Freitag", "Samstag", NULL};
    int x=0;
    int daysInMonth=0;
    struct tm ts;
    int auswertung[numdays][numwdays];
    for(int k=0; k<numdays; k++){
      for(int l=0; l<numwdays; l++){
        auswertung[k][l] = 0;
    }
  }
  ts.tm_hour = 0;
    ts.tm_min  = 0;
    ts.tm_sec  = 1;
    ts.tm_isdst = -1;
    
  for(int year=2015; year<=2015; year++){
    for(int month=2; month<=2; month++){
      daysInMonth = getDaysInMonth(month, year);
      for(int day=1; day<=daysInMonth; day++){
        ts.tm_year = year-1900;
        ts.tm_mon  = month-1;
          ts.tm_mday = day;
          
          if (mktime(&ts) == -1 ){  
            ts.tm_wday = 7;
        }  
          if(ts.tm_wday>=0 && ts.tm_wday<=6){
            printf("Dieser Tag ist ein %s\n", wday[ts.tm_wday]);  
            auswertung[ts.tm_mday-1][ts.tm_wday] = auswertung[ts.tm_mday-1][ts.tm_wday] + 1;
            x = x + 1;
          }
        }
    }
  }  
  ofstream f("text.txt");
  for(int m=0;  m<numdays; m++){
    for(int n=0; n<numwdays; n++){
      f <<  auswertung[m][n];
      f << "   ";
    }
    f << ("\n");
  }
  printf("%d",x);
  f << x;
  f.close();
  return 0;
}


bool leapyear(int year){
  if ((year % 400) == 0)
    return true;
  else if ((year % 100) == 0)
    return false;
  else if ((year % 4) == 0)
    return true;
  return false;
}  

int getDaysInMonth(int month, int year){
  //                     ungült,Jan,Feb,Mrz,Apr,Mai,Jun,Jul,Aug,Sep,Okt,Nov,Dez
  int arrDaysInMonth[13] = {  0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 };

  if (month == 2){
    if (leapyear(year)) return 29;
    else return 28;
  }
  else return arrDaysInMonth[month];
}

das mit dem vergleichen hört sich aber besser an, habe ich aber nicht 
hinbekommen. Da muss ich erst noch mal ein bisschen probieren.

aber danke schon mal für eure Hilfe.

[Jonas B. (jibi)]

>kann ich auch irgendwie kontrollieren, ob dieser Tag überhaupt existiert
> hat?

Ja, in dem du ermittelst ob das Jahr ein Schaltjahr war (wie man das 
programmiertechnisch macht, das kannst du googlen), in dem Falle gibt es 
halt den 29. Februar - ansonsten nicht.

Gruß J

[Dirk B. (dirkb2)]

Ich würde nur für den ersten Tag mktime nehmen. den Rest dann selber aus 
der Tabelle, bzw. mit Schleifen errechnen.


Meist macht man  arrDaysInMonth ein 2D-Array:

int arrDaysInMonth[][13] = { {  0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 },
                             {  0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 }
                           };

Und das Post/Preincrement geht nicht nur in Schleifen.
Dann ist das

auswertung[ts.tm_mday-1][ts.tm_wday]++;

 auch besser lesbar.

auswertung[ts.tm_mday-1][ts.tm_wday] += 1;

 geht auch.


Und wenn du Arrays initialisierst, werden die nicht aufgeführten 
Elemente mit 0 besetzt

int auswertung[numdays][numwdays] = {0}; // Alles mit 0 besetzt

Sonst ist memset dein Freund.

[Dirk B. (dirkb2)]

Dirk B. schrieb:
> Ich würde nur für den ersten Tag mktime nehmen.

Mit mktime kommst du (bei 32-Bit time_t) aber nur von ~1902 bis ~2038.

Dann kannst du Freitag, den 15. Oktober 1582 als Startpunkt nehmen. 
Davor war Donnerstag, der 4. Oktober.

[Jay (Gast)]

Was soll den jetzt dieses Gemisch aus C und C++? Das macht es auch nicht 
besser. Ebensowenig irgendwelche besch... Code dranzukleben.

Oben wurde es schon mal erwähnt: HANDBUCH LESEN!


> The mktime() function modifies the fields of the tm structure as follows:
> tm_wday and tm_yday are set to values determined from the contents of the
> other fields; if structure members are outside their valid interval, they
> will be normalized (so that, for example, 40 October is changed into 9
> November); tm_isdst is set (regardless of its initial value) to a
> positive value or to 0, respectively, to indicate whether DST is or is
> not in effect at the specified time. Calling mktime() also sets the
> external variable tzname with information about the current timezone.

D.h. hat mktime() den Tag verändert, so gibt es das Datum nicht -> 
Ergebnis wegschmeißen.

Einfachster Test, wenn es nur darum geht den 29. (Schaltjahre), 30. und 
31. eines Monats in den Griff zu bekommen:

...
ts.tm_mday = day;
if((mktime(&ts) == -1) || (ts.tm_mday != day)){  
    // Den Tag gibt es nicht
}

[Eric B. (beric)]

Jay schrieb:

> Oben wurde es schon mal erwähnt: HANDBUCH LESEN!
>
>> The mktime() function modifies the fields of the tm structure as follows:
>> tm_wday and tm_yday are set to values determined from the contents of the
>> other fields; if structure members are outside their valid interval, they
>> will be normalized (so that, for example, 40 October is changed into 9
>> November);
>
> D.h. hat mktime() den Tag verändert, so gibt es das Datum nicht ->
> Ergebnis wegschmeißen.

Nein, das heisst, eine einfache Schleife genügt:

heute = 1. Jan. 2000
WHILE heute != 1. Jan 2016
  wochentag = wochentag von heute
  tag = monatstag von heute
  trage (tag, wochentag) in resultat ein
  heute = heute + 1
  benutze mktime um heute zu normalisieren
END WHILE

zun schluss resultattabelle ausgeben

und damit sind alle Vorrichtungen um Schaltjahre und 28-, 29-, 30 oder 
31-Tagige Monate abzufangen hinfällig, weil mktime das alles 
übernimmt.