Optimierung?

[------------]

Hi,

wie kann ich diese Funktion optimieren?

bool istSchaltjahr(int jahr)
{
   bool schaltjahr=false;
   if(jahr%4==0)
   {
      schaltjahr=true;
      if(jahr%100==0)
      {
         schaltjahr=false;
         if(jahr%400==0)
         {
            schaltjahr=true;
         };
      };
   };
   return schaltjahr;
};

[dali]

Du brauchst keine bool variable sondern kannst gleich returnen:

if((year%4 == 0 && year % 100 != 0) || year % 400 == 0)
return true;
return false

[finix]

dali schrieb:

Du brauchst keine bool variable sondern kannst gleich returnen:

if((year%4 == 0 && year % 100 != 0) || year % 400 == 0)
return true;
return false

Wieso machst du's dann nicht?

[dali]

Stimmt hab nicht nachgedacht ;):

return ((year%4 == 0 && year % 100 != 0) || year % 400 == 0);

[dali]

Ich brauchte zwar auch keine aber Verzweigung kannst du natürlich kicken

[Helium]

edit: ---