cout<<"Geben Sie eine Jahreszahl ein:";
cin>>i; {
if (i%4>0) << ergebnis = 0<<endl;
else if (i%4%100>0)<< ergebnis = 0<<endl;
else if (i%4%400>0)<< ergebnis = 0<<endl;
else << ergebnis = 1<<endl;
}
if ergebnis = 0 cout<<"Das ist kein Schaltjahr"<<endl;
else cout<<"Das ist ein Schaltjahr"<<endl;

char p[50];
cin.getline(p,50);
cin.getline(p,50);
}

könnte es sein, dass du so was wolltest:

#include <iostream.h> 
int main() { 
int i,ergebnis; 

cout<<"Geben Sie eine Jahreszahl ein:"; 
cin>>i; 
if (i%4>0) ergebnis = 0; 
else if (i%4%100>0)ergebnis = 0; 
else if (i%4%400>0)ergebnis = 0; 
else ergebnis = 1; 
if (ergebnis == 0 )cout<<"Das ist kein Schaltjahr"<<endl; 
else cout<<"Das ist ein Schaltjahr"<<endl; 

char p[50]; 
cin.getline(p,50); 
cin.getline(p,50); 
return 0;
}

Hab jetz meinen Code nicht kontrolliert auf Sinn, aber zumindest mal die Syntaxfehler raus, die mir aufgefallen sind.

#include <iostream.h>  //veraltet, statt dessen

#include <iostream>

void main()  //macht man auch nicht

int main() //is da schon angebrachter

und was der Rest sollte...

vielleicht arbeitest du lieber mal ein paar Tutorials durch

so sollte es funktionieren:

#include <iostream>
using namespace std;

int main() 
{
int i;

        cout<<"Geben Sie eine Jahreszahl ein:";
        cin>>i;  //was sollte die Klammer hier?
        if (i%4>0||i%4%100>0||i%4%400>0) 
                cout<<"Das ist kein Schaltjahr\n";
        else 
                cout<<"Das ist ein Schaltjahr\n";

   cin.clear();
   cin.ignore(std::cin.rdbuf()->in_avail());
   cin.get(); 

return 1;
}

mit den besten Wünschen

theduke

es geht ja: wenn %4=0(Schaltjahr) außer %100=0(kein Schaltjahr) außer %400=0(dann doch Schaltjahr)

funktionieren würds dann wohl so:

if(j%4==0)
    ergebnis=IS_LEAPYEAR;  //enum halt
else if(j%100==0)
    ergebnis=NO_LEAPYEAR;
else if(j%400==0)
    ergebnis=IS_LEAPYEAR;
else 
    ergebnis=NO_LEAPYEAR;

theduke

So?

edit: ops... sollte nun stimmen so

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

So?

Nein. Korrekt ist so:

bool is_leap_year(size_t year)
{
   if( ( !(year % 4) && year % 100 ) || !(year % 400) )
      return true;
   return false;
}

Bei deiner Funktion wird das Jahr 2004 z.B. nicht als Schaltjahr erkannt...

Caipi

Nein. Korrekt ist so:

bool is_leap_year(size_t year)
{
   if( ( !(year % 4) && year % 100 ) || !(year % 400) )
      return true;
   return false;
}

Bei deiner Funktion wird das Jahr 2004 z.B. nicht als Schaltjahr erkannt...
Caipi

da hab ich aber den nicht so korrekten code lieber. den kann ich wenigstens verstehen und gegebenefalls reparieren. deinen versteh ich nicht.

Caipi schrieb:

Nein. Korrekt ist so:

bool is_leap_year(size_t year)
{
   if( ( !(year % 4) && year % 100 ) || !(year % 400) )
      return true;
   return false;
}

Bei deiner Funktion wird das Jahr 2004 z.B. nicht als Schaltjahr erkannt...
Caipi

da hab ich aber den nicht so korrekten code lieber. den kann ich wenigstens verstehen und gegebenefalls reparieren. deinen versteh ich nicht.

Sorry, aber ich verstehe nicht worauf du hinaus willst (Falls es Ironie ist, diese zu erkennen ist nicht unbedingt meine Stärke).

Klär mich auf. Bitte

Caipi

Klär mich auf. Bitte

ich kann einfach

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

viel besser lesen als deinen code.
den les ich da so von oben nach unten.
wenn das jahr durch 400 teilbar ist, dann ist es ein schaltjahr.
ansonsten, wenn das jahr durch 100 teilbar ist, dann ist es kein schaltjahr.
ansonsten, wenn das jahr durch 4 teilbar ist, dann ist es ein schaltjahr.
ansonsten ist es kein schaltjar.
das kann ich irgendwie ganz leicht nachvolziehen. das direkte rausreturnen (bin ja eh fan von singe-entry/multiple-exit) erlaubt mir, gedankengänge früb abzuschließen und unabhängig vom rest zu beurteilen.
aber ich muss heut zu müde sein für sowas. ich seh grad den 2004-fehler nicht, den du gezeigt hast.

//edit zu langsam

theduke

ich seh grad den 2004-fehler nicht, den du gezeigt hast.

Die Bedingung für ein Schaltjahr (nach dem Gregorianischen) Kalender heißt: Ein Jahr ist dann ein Schaltjahr (und nur dann) wenn es durch 4 teilbar ist aber dann *nicht* durch 100. Außerdem ist es ein Schaltjahr wenn es durch 400 teilbar ist.

Caipi

Das jahr 2004:

Da greift dann dein 2. if: 2004 % 100 = 4 > 0 --> return false.

// edit ebenfalls zu langsam

Dast ist reine Gewohnheit. Ich kann den Einzeiler da schneller lesen als die vielen Zeilen die du benötigst, die noch dazu nicht korrekt sind:

ah, klar. da muß natürlich auch == stehen. sonst wär der code auch nicht so hübsch, wie er jetzt ist.

dann vermute ich, du würdest diesen wust dadurch heilen, daß du ein paar leerzeichen einfügst?

bool datumIstGueltig(int tag,int monat,int jahr){
   return monat>0&&monat<13&&tag>0&&tag<32&&(tag<31¦¦(monat>7!=monat%2==1))
   &&(monat!=2¦¦(tag<30&&(((jahr%4==0&&jahr%100!=0)¦¦jahr%400==0)¦¦tag<29)));
}

mfg f.-th.