if(monat==1||monat==3||monat==5||.....)

wie z.B.

if(monat==1,3,5,7,8,10,12)

Was ja leider nicht funktioniert?

Gruß

bool IsLeapyear(int year);

int DaysPerMonth(int month, int year)
{
	switch(month)
	{
		case 1:
		case 3:
		case 5:
		case 7:
		case 8:
		case 10:
		case 12:
		    return(31);
		case 4:
		case 6:
		case 9:
		case 11:
		    return(30);
		case 2:
		    return(28+IsLeapyear(year));
	}
    // hier ein assert und 0 zurückgeben, Ausnahme werfen oder Ähnliches
}

if(monat==1||monat==3||monat==5||.....)

Gruß

switch(month)
    {
        case 1:case 3:case 5:case 7:case 8:case 10:case 12:  return(31);
        case 4:case 6:case 9:case 11: return(30);
        case 2: return(28+IsLeapyear(year));
    }

:xmas2: :xmas1: :xmas2:

#define MASK(X)	((X)&6)
if (monat == 2)
  return 28;
if (MASK(monat) == 1 || MASK(monat) == 8)
  return 31;
if (MASK(monat) == 0 || MASK(monat) == 12)
  return 30;
/* respektive: */
if (monat == 2)
  return 28;
switch (MASK(monat))
{
case 1:
case 8:
  return 31;
case 0:
case 12:
  return 30;
}
#undef MASK

Wichtig ist dabei den Februar vorher auszuwerten, weil du sonst 30 Tage erhältst.

31 Tage ungerade
30 Tage gerade

Ich komm hier gerade auf:

return monat==2 ? 28 : 30 + ((monat ^ monat>>3)&1);

Total fehlerhaft
Edit, jetzt müsste es passen!

:p

"\x1f\x1c\x1f\x1e\x1f\x1e\x1f\x1f\x1e\x1f\x1e\x1f"[month-1]

PS: Ich vermisse den WPC.

return "OLONONOONONO"[month-1]-48+(month==2&&!(year%4)&&year%400);

Hübsch, aber klappt noch nicht ganz.

"OLONONOONONO"[month-1]-48+(month==2&&(!(year%4)&&year%100||!(year%400)))

bool IsLeapyear(int year)
{
	if(year<1)
		return false;
	if(year%4) return false;
	if(year<=1582) return true;
	if(year%100) return true;
	if(year%400) return false;
	return true;
}
int DaysPerMonth(int month, int year)
{
	switch(month)
	{
		case 1:
		case 3:
		case 5:
		case 7:
		case 8:
		case 10:
		case 12:
			return 31;
		case 4:
		case 6:
		case 9:
		case 11:
		    return 30;
		case 2:
			return(28+IsLeapyear(year));
		default:
			return 0;
	}
}

OLONONOONONO schrieb:

return "OLONONOONONO"[month-1]-48+(month==2&&!(year%4)&&year%400);

Hübsch, aber klappt noch nicht ganz.

"OLONONOONONO"[month-1]-48+(month==2&&(!(year%4)&&year%100||!(year%400)))

<krümelkack>
Das ist nicht portabel, da der C++ Standard kein ASCII erzwingt. Ich glaube, die einzige Garantie, die da gemacht wird, ist, dass '0'+x mit 0<=x<=9 die richtige Dezimalziffer gibt, also dass '0', '1', '2', ..., '9' aufeinanderfolgende char-Werte sind.
</krümelkack>

Ich würd's wahrscheinlich mit switch machen. Das ist einfach am leserlichsten. Ich möchte trotzdem eine Lösung zeigen, die wir hier noch nicht gesehen haben:

template<class T, usnigned N, class U>
inline bool binary_search(const T(&oa)[N], U x)
{
  return std::binary_search(oa+0,oa+N,x);
}

...
if (month==2) return 28+isLeapYear(year);
static const int m30[] = {4,6,9,11};
return binary_search(m30,month) ? 30 : 31;

@Vicious Falcon: Du weißt, dass return keine Funktion ist und die Klammern um den Ausdruck überflüssig sind, ja?

int wieviele_tage(int monat)
{
   return ((0xEEFBB3 >> ((monat - 1)*3)) & 0x3) + 28;
}

auch nett! das *3 sollte aber wohl ein *2 sein.
...fehlt nur noch das Schaltjahr...

int wieviele_tage(int monat, bool schaltjahr)
{
   return (((0xEEFBB3+(schaltjahr<<2)) >> ((monat - 1)<<1)) & 0x3) + 28;
}