C Funktionen Anfänger Problem *HELP*

[Wissenslücke]

hallo liebe community,

ich weiss bei folgendem Problem nicht was ich falsch gemacht habe.

Problem statement
Your function is given a year greater than 1000 AD and less than 10000 AD. Return the string "Leap year" if the given year is a leap year and "Not a leap year" otherwise.

A year is considered a leap year if it is perfectly divisible by 4. Years that are perfectly divisible by 100 (for example, 2000, 2100, 2200) are exceptions. Such a year is a leap year only if it is also perfectly divisible by 400.

mein Vorschlag

Code: Alles auswählen

const char * isLeapYear(int year)
{
   char ly[]= "Leap year";
   char nly[]= "Not a leap year";
   
   
    if((year%4==0 && year%100!=0 )|| (year%100==0 && year%400==0)){
        
        return ly;
       
    }else {
        
        return nly;
       
    }  
    /* Complete this function */
}

Gruß The WL

[cloidnerux]

ich weiss bei folgendem Problem nicht was ich falsch gemacht habe.

Du solltest uns schon mitteilen, was du deiner Meinung nach nicht Funktioniert.
Ist es die Auswertung des Schaltjahres, ist es das zurückgeben der Strings?

Bitte liefer auch ein compilierbaren Code, dass man dein Problem nachvollziehen kann.

[Wissenslücke]

Die Aufgabe ist von einer Seite mit integriertem Compiler. Die Funktion war so ohne Inhalt und ohne main Fkt vorgegeben. Die Seite testet bei Absenden des Codes dann ob das outcome richtig ist. Bei mir kommt halt nur die Info dass das outcome falsch ist. Der Code ist nach meiner Meinung schick, aber ich bin auch ein Anfänger.

[cloidnerux]

Die Aufgabe ist von einer Seite mit integriertem Compiler. Die Funktion war so ohne Inhalt und ohne main Fkt vorgegeben. Die Seite testet bei Absenden des Codes dann ob das outcome richtig ist

Das Hilft einem doch schon weiter.

Der Code ist nach meiner Meinung schick, aber ich bin auch ein Anfänger.

Dein Code an sich Funktioniert auch.

Wenn man das ganze aber mal selber Compiliert:

Code: Alles auswählen

#include "stdio.h"

const char * isLeapYear(int year)
{
   char ly[]= "Leap year";
   char nly[]= "Not a leap year";
   
   
    if((year%4==0 && year%100!=0 )|| (year%100==0 && year%400==0)){
        
        return ly;
       
    }
    else 
    {
        return nly; 
    }  
}

int main()
{
	printf("%s", isLeapYear(2000));
	return 0;
}

Fallen zwei Dinge auf:
1. Der Compiler meckert das man lokale Variablen zurück gibt
2. Es kommt kein "leap year" oder "not leap year" raus, sondern kauderwelsch.
Woran liegt es: Du gibst einen Pointer auf eine Variable im Stack zurück, die nach beenden der Funktion für andere Dinge verwendet wird. Daher kann es keine sinnvolle Ausgabe geben.
Als Lösung gibt es zwei Optionen:
1. Die Variablen als "static" deklarieren.
2. Mit malloc speicher anfordern, den Text dort hinein kopieren und den Speicher zurück geben.

[mfro]

... und dann gibt's noch eine dritte Möglichkeit (und das ist m.E. die richtige): die Strings als Stringkonstanten im Textsegment anlegen, dann belegt das auch keinen extra Speicher:

Code: Alles auswählen

const char *ly = "Leap year";
const char *nly = "Not a leap year";
...

[cloidnerux]

und dann gibt's noch eine dritte Möglichkeit (und das ist m.E. die richtige): die Strings als Stringkonstanten im Textsegment anlegen, dann belegt das auch keinen extra Speicher:

Und auch darauf achten, dass man

Code: Alles auswählen

const char * ly

statt

Code: Alles auswählen

const char ly[]

schreibt. Letzteres wird bei mir mit gcc 4.8.1 unter Windows 7 nicht als Text, sondern als lokale Variable erkannt, weswegen ich es auch nicht als Lösung vorgeschlagen hatte.

[Wissenslücke]

wunderbar. Ich habs mit Stringkonstanten getestet und es hat funktioniert. Wie immer vielen Dank