Brauch mal jemand der Ahnung hat. Funktionen auslagern

Warum soll jahr gleichzeitig 2018 und true darstelllen?

DocShoe

@BW82 sagte in Brauch mal jemand der Ahnung hat. Funktionen auslagern:

Es funktioniert alles ohne Probleme, dass ist ja das lustige…

Soso... ist 1900 ein Schaltjahr?

Edit: Eigene Dummheit

BW82

@manni66 sagte in Brauch mal jemand der Ahnung hat. Funktionen auslagern:

Warum soll jahr gleichzeitig 2018 und true darstelllen?

heißt also ich muss dem boolschen wert noch zu ordnen??

Mein Gott Leute, wenn ich nicht Hilfe bräuchte würde ich nicht fragen, also spart euch doch dumme Kommentare und gebt lieber konstruktive KRITIK , die einem hilft.
Danke!!!

@BW82 sagte in Brauch mal jemand der Ahnung hat. Funktionen auslagern:

@manni66 sagte in Brauch mal jemand der Ahnung hat. Funktionen auslagern:

Warum soll jahr gleichzeitig 2018 und true darstelllen?

heißt also ich muss dem boolschen wert noch zu ordnen??

Keine Ahnung wie du das meinst. Es sollten zwei Variablen sein.

DirkB

@BW82 sagte in Brauch mal jemand der Ahnung hat. Funktionen auslagern:

Mein Gott Leute, wenn ich nicht Hilfe bräuchte würde ich nicht fragen, also spart euch doch dumme Kommentare und gebt lieber konstruktive KRITIK , die einem hilft.

Mannis Kommentar war konstruktiv.
Du sollst darüber nachdenken ob das gut ist, wenn jahr den Wert false hat.
Den hat es, wenn if(jahr % 100== 0) ausgeführt wird.

BW82

@manni66 also eine variable für true und false, und eine separate für Jahr.
also sowas in der art?

if(jahr % 100== 0){
       if (jahr % 400 == 0)
        {
            annahme = true;
        }
    else {
       annahme = false;
    }
        
    }else {annahme = true;}

}else { annahme = false;}


return annahme;
}

BW82

So funktioniert es.
hier in main:

  bool annahme;
                //Eingabe
			cout << endl;
			cout << ("Jahreszahl ");
			cin >> jahr;
               annahme = istSchaltjahr(jahr);
                if (annahme == true){
                    cout << jahr << " ist ein Schaltjahr" << endl;}
                else{
                    cout << jahr << " ist kein Schaltjahr" << endl;}

bool istSchaltjahr(int jahr){
    bool annahme;

    if (jahr % 4 == 0)
    {
        annahme= false;
        
        if(jahr % 100== 0){
            
            if (jahr % 400 == 0)
            {
                annahme = true;
            }
        else {
            annahme = false;
        }
            
        }else {annahme = true;}
    
    }else { annahme = false;}
    

    return annahme;
    
}

*john 0

@BW82
Das ganze lässt sich auch noch kompakter schreiben

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

Weil man hier über die Abfolge der Prüfungen das gewünschte Ziel erreichen kann.

BW82

@john-0 Dank dir!

camper

@john-0 sagte in Brauch mal jemand der Ahnung hat. Funktionen auslagern:

@BW82
Das ganze lässt sich auch noch kompakter schreiben
bool check_leap_year (int year) {
    if ((year % 400) == 0) return true;
    if ((year % 100) == 0) return false;
    if ((year % 4)   == 0) return true;
    return false;
}
Weil man hier über die Abfolge der Prüfungen das gewünschte Ziel erreichen kann.

Man kann sich noch zusätzlich um Effizienz bemühen, die nötigen Division sind ja u.U. vergleichsweise aufwändig, also ist es wahrscheinlich günstig, die im Durchschnitt nötige Anzahl von Bedigungen zu minimieren. Die gezeigte Variante ist dabei am ungünstigsten, hier werden durchschnittlich 2.9875 if-Bedigungen pro Funktionsaufruf geprüft.

Besser ist es, die Sequenz herumzudrehen:

bool is_leap_year (int year) {
    // return (year % 4 == 0) && ((year % 100 != 0) || (year % 400 == 0));
    if ((year % 4) != 0) return false;
    if ((year % 100) != 0) return true;
    if ((year % 400) != 0) return false;
    return true;
}

mit nur durchschnittlich 1.26 zu prüfenden Bedigungen je Funktionsaufruf.