C- Warning Message

[harry3]

Hallo!

Ich habe da ein Programm(in C) für meinen Taschenrechner(TI 200) geschrieben, doch bei einer Funktion gibts jedesmal eine Warning(jedoch keinen Error!!!), obwohl alles perfekt funktioniert.

PHP-Code:

char *stab(double);//oben einmal die Fkt definieren

void _main(void)
{
.
.
.
printf("\n->%s",stab(hm)); //Aufruf der Fkt. in _main()
.
.
.
}

char* stab(double hm)  //die Funktion 
{ 

    char* erg="Fehler\0"; //wenn ich char erg[20] schreib gehts komischerweise garnicht(Compile Error)
    
    if(hm>0)
        erg="stabil";
    
    else if(hm==0)
        erg="indifferent";
    
    else if(hm<0)
        erg="labil";    

 return erg;

} 


Warning Message: Initialization discards qualifiers from pointer target type


Also vielleicht kann jemand helfen. Es läuft zwar alles prima, aber gerade deswegen kann ich nicht verstehen dass der Compiler eine Meldung ausspuckt!




Grüße,
Harri

[Who-T]

also ich hab das ganze mal über gcc mit -Wall laufen lassen, der sagt mir nix.

ich seh auch auf anhieb keinen fehler.
welche zeile beschwert er sich eigentlich?

[harry3]

Jedesmal wenn ich erg="..." schreibe!

PHP-Code:

        char* erg="Fehler\0"; //hier
    
    if(hm>0)
        erg="stabil"; //hier
    
    else if(hm==0)
        erg="indifferent"; //hier
    
    else if(hm<0)
        erg="labil"; //und hier 


Naja, Hauptsache es läuft.



Viele Grüße,
Harri

[_m3]

Hmmm, meine C Kentnisse sind ja etwas eingerostet, aber sollte man nicht wo mal Speicher fuer erg reservieren?
Ev. regt er sich auf, dass man dem Pointer einen statischen String zuweist?

[harry3]

Hmmm, meine C Kentnisse sind ja etwas eingerostet, aber sollte man nicht wo mal Speicher fuer erg reservieren?

Ich dachte dass man das nur bei großen Daten machen muss. Bei dem Mini String sollte das also eigentlich nicht nötig sein!

Ev. regt er sich auf, dass man dem Pointer einen statischen String zuweist?

Habs in Turbo C++ durchlaufen lassen, und dort gibts 0 Fehler/0 Warnungen!


Viele Grüße,
Harri

[Biri]

hi !

@m3_: stimmt, man muss für den string speicher reservieren.

wie groß der sting da ist, ist egal - ein "char" speichert genau 1 zeichen. also z.B. char buffer[20] verwenden

zuweisen kann man einen string (in C) nicht man muss dazu ein stringfunktion verwenden. z.B. strcpy(buffer,"stabil");

weoterer fehler: du vereinbarst in einer funktion einen char - dieser wird am stack angelegt. am ende der funktion gibts du einen zeiger auf eine stack variable zurück !
das führt früher oder später zu fehlern, da der stack nach dem verlassen der Fu. freigegeben wird, die variable also einfach zerstört wird.

Lösung: static verwenden, oder in der main funktion speicher für den string allokieren und die adresse dieses speichers an die aufgerufenen Fu. übergeben -
dort dann (wie auch jetzt) die zuweisung machen.

...ist in C++ oder C# alles einfacher, aber ich denke mal, das gibts am TI 200 nicht.

fg
-hannes

[_m3]

@biri: Danke fuer die Bestaetigung

Original geschrieben von harry3
Ich dachte dass man das nur bei großen Daten machen muss. Bei dem Mini String sollte das also eigentlich nicht nötig sein!

Denkfehler
Warum sollte das bei kleinen Strings nicht noetig sein?

Habs in Turbo C++ durchlaufen lassen, und dort gibts 0 Fehler/0 Warnungen!

Dann dreh mal den Warning-Level, ANSI-Compliance, etc. auf

[harry3]

OK, danke für die Antworten.
Also das Allokieren hätte ich mir sparen können, das muss anscheinend nicht sein(lt. Compiler).
Wenn man static weglässt, so beschwert sich der Compiler auch nicht. Aber ich schätze mal, dass das static schon wichtig ist!?
Die Fehlermeldungen sind verschwunden, als ich die direkte Zuweisung gegen ein strcpy ausgtauscht habe. Aha, interessant...da ist man ja doch etwas verwöhnt von C++. Wieso hats denn mit direkter Zuweisung trotzdem funktioniert?

Gibts irgendwo eine Seite wo kurz die Eigenarten von C gegenüber C++ erklärt werden? Oder gibts außer bei den Strings eh nicht viel Unterschiede?(die Objektorientierung, Streams etc... jetzt mal weggelassen?)



Viele Grüße,
Harri

PHP-Code:

char* stab(double hm)
{ 

    
    static char* erg;
    erg=(char*)malloc(15);
    
    strcpy(erg,"Fehler");
    
    if(hm>0)

        strcpy(erg,"stabil");
    
    else if(hm==0)
        strcpy(erg,"indifferent");
    
    else if(hm<0)
        strcpy(erg,"labil");    

 return erg;

} 


EDIT: Hab das malloc jetzt wieder weggetan, weil ich dann auch irgendwann free() ausführen müsste, und ich wüsste echt nicht wo ich das tun soll! Im _main() gibts die Variable nicht, und free() vor dem return ausführen wäre auch ein bisschen extrem sinnlos!

[harry3]

Original geschrieben von _m3
@biri: Danke fuer die Bestaetigung

Denkfehler
Warum sollte das bei kleinen Strings nicht noetig sein?

Hallo!

Ich meinte damit, dass man in C++ sich das Allokieren von kleinen Daten sparen kann, weils eh wurscht ist wenn sie unveränderbar groß sind. Denn ob jetzt 1byte nicht verwendet werden kann ist mir relativ wurscht. Bei großen Daten ist das wieder etwas heikler, v.a. am TI200.

...ist in C++ oder C# alles einfacher, aber ich denke mal, das gibts am TI 200 nicht.

Nö, leider
Aber ich bin eh schon froh dass es C fürn TI gibt, früher wurde fürn TI alles mit Assembler geschrieben!!! Da schreibt man mal 1 Seite nur um den String "Hallo Welt" auszugeben!


Viele Grüße,
Harri

[harry3]

Das malloc hab ich jetzt komplett weggetan.
Wenn ich static char* erg; schreibe, dann hängt sich der TI auf! (mit der Meldung Protectit memory violation verabschiedet er sich)
Wenn ich aber char erg[15]; schreibe, dann gibts keine Probleme.
Hat da jemand eine Erklärung dafür?


Grüße,
Harri