[HaBo]

Ev0luti0n · 28.03.11, 21:34

Hey Leute. Ich bin's mal wieder.
Dieses Mal gibts ein kleines Code-Problem.

Code:

int search_letter(char letter,char *text)
                    {
                        while(*text)
                        {
                                   if(*text=letter)
                                                    return 1;
                                   else    
                                          *text++;
                        }
                      return 0;
                    }

Das ist meine Funktion.
Wenn ich diese jetzt benutz' , dann erhalte ich von Dev C++ immer die Warnung "Return makes integer from pointer without cast". Und ich verstehs einfach nich warum diese Meldung kommt.
Eine Erklärung wäre ganz nice

(Kein "Ja weil das Return einen Integer vom Zeiger ohne Cast macht"

)
Lg ev0luti0n

odigo · 28.03.11, 21:44

Code:

if(*text=letter)

= ist eine Zuweisung. Bei einem if muss true oder false rauskommen, sprich du brauchst einen Vergleichsoperator, z.B. ==. Ein klassischer Fehler

Der Vergleich von einem Pointer mit einem Char macht aber imho auch wenig Sinn, aber für eine genaue Erklärung ist mein Zeit mit C zu lang her

Anzeige

- Anzeige -

**bitmuncher** · 28.03.11, 21:48

Welche GCC-Version verwendest du? Ich kann den Fehler mit GCC 4.2 nicht nachvollziehen.

Code:

bitmuncher@alpha:~> gcc --version
i686-apple-darwin10-gcc-4.2.1 (GCC) 4.2.1 (Apple Inc. build 5666) (dot 3)
Copyright (C) 2007 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
bitmuncher@alpha:~> gcc -c -Wall test.c
test.c: In function 'search_letter':
test.c:3: warning: suggest parentheses around assignment used as truth value
test.c:6: warning: value computed is not used

Und diese Warnung kommt von der von odigo angesprochenen Zuweisung im if-Statement.

Ev0luti0n · 28.03.11, 21:49

Hey Danke für die Anmerkung

. Grr. Der scheiß mit dem Vergleichsoperator passiert andauernd aber daran liegts nich. Gleiche Warnung ("passing arg 1 of 'search letter' makes integer ......").
hmmm. Es liegt also schon an der Parameterübergabe. Aber an was genau*grübel*

Ev0luti0n · 28.03.11, 21:52

Zitat:

Welche GCC-Version verwendest du? Ich kann den Fehler mit GCC 4.2 nicht nachvollziehen.

Also ich verwende "Dev C++ 4.9.9.2". ( Drecks Freeware

)

**bitmuncher** · 28.03.11, 21:56

Zitat:

Zitat von Ev0luti0n

Gleiche Warnung ("passing arg 1 of 'search letter' makes integer ......").

Das ist offenbar nicht die gleiche Warnung. Beim ersten Mal bekommst du die Warnung aufgrund des Return-Werts, während nun explizit der Parameter angemangelt wird und das ist eigentlich klar, denn char ist nunmal kein int und ein Vergleich der Art 'if(wert1 == wert2)' ist immer ein Vergleich von Integer-Werten. Ohne -Wall würdest du diese Warnung vermutlich nichtmal bekommen, da char's im Normalfall bei Vergleichen automatisch gecastet werden. Es müsste also heissen:

Code:

if((int)*text == (int)letter)...

Da aber der Parameter bemängelt wird, würde ich eher darauf tippen, dass beim Aufruf der Funktion irgendwo ein Fehler ist. Wie rufst du die Funktion denn auf und welche Werte/Datentypen werden dabei übergeben?

Ev0luti0n · 28.03.11, 22:10

Also mit den Casts geht es auch nicht... Die Warnings sind die selben , jedoch hab ich oben n Teil weggelassen. Sorry

.
Also ich verwende sie im Code bei

Zitat:

char tipp,wort[100];
[...]
if(search_letter(tipp,&wort))
{ printf("\n");
for(i=0;wort[i];i++)
{
if(wort[i]==tipp[i])
txt[i]=tipp[i];
//putchar(txt[i]);
}

Hoffe das hilft.
PS: Was ist -wall?(eine Compiler-Anweisung , damit dieser nicht automatisch castet?) und gibt es Möglichkeiten solche Bedingungen zu optimieren(Bin noch ein Newbie)?

odigo · 28.03.11, 22:35

Du solltest die Methode ändern wie du dir C beibringen willst. Ich seh in deinem Code auf den ersten Blick mindestens 5 grundsätzlich Fehler.
Such dir ein Buch/Tutorial was wirklich bei Hello World anfängt und arbeite dich schrittweise weiter. Punkt für Punkt.

**bitmuncher** · 28.03.11, 22:42

-Wall sorgt dafür, dass der Compiler sämtliche Warnungen auswirft, auch zu unsauberem Code u.ä.. Oft findet man so Fehler, auf die man sonst nicht aufmerksam wird, weil ihre Ursache in "fehlerhaften" Code davor zu finden ist. Beim Entwickeln sollte man diesen Parameter immer nutzen. Probier's mal mit diesem Parameter und poste die Warnungen + zugehörigen Code, denn momentan kann ich deinen Fehler nicht nachvollziehen:

Code:

bitmuncher@alpha:~> cat test.c
int search_letter(char letter,char *text) {
  while(*text) {			    
    if(*text==letter)
      return 1;
    else    
      *text++;
  }
  return 0;
}
bitmuncher@alpha:~> gcc -c -Wall test.c
test.c: In function 'search_letter':
test.c:6: warning: value computed is not used

Die Warnung "value computed is not used" ist allerdings bei Objektdateien normal, wenn man sie einzeln kompiliert, da die Werte ja tatsächlich innerhalb der Funktion nicht benutzt werden, und hat mit dem von dir beschriebenen Fehler nichts zu tun.

Ev0luti0n · 29.03.11, 07:32

Zitat:

Zitat von odigo

Du solltest die Methode ändern wie du dir C beibringen willst. Ich seh in deinem Code auf den ersten Blick mindestens 5 grundsätzlich Fehler.
Such dir ein Buch/Tutorial was wirklich bei Hello World anfängt und arbeite dich schrittweise weiter. Punkt für Punkt.

Nett dass du mich darauf hinweist , aber wenn dann würd ich diese "grundsätzlichen Fehler" auch gerne genannt haben. Nicht so "es ist schlecht. mach's besser"

.

benediktibk · 29.03.11, 21:11

Code:

*text++;

Die Zeile meinst du wahrscheinlich anders:

Code:

text++;

Damit erhöhst du den Pointer um eins auf das nächste Zeichen, nicht wie zuvor den (ASCII-)Wert des ersten Zeichens.
Und, kleiner Tipp am Rande, wenn du sowieso nur 0 oder 1 zurückgibst tuts ein bool als Rückgabewert auch. Ist leichter zum Lesen und verhindert somit Fehler.

Ich weiß nicht wo odigo die Unmengen an restlichen Fehlern findet, aber vielleicht bin ich einfach zu müde, als dass ich sie sehen würde.

Kannst du bitte sagen bei genau welcher Zeile der Compiler meckert? Ich schätze mal die Zeilenangabe steht bei der Fehlermeldung dabei.
mfg benediktibk

enkore · 29.03.11, 21:18

Zitat:

Zitat von Ev0luti0n

Also ich verwende "Dev C++ 4.9.9.2". ( Drecks Freeware

)

Besorg die mal das aktuelle Codeblocks-Release mit einem GCC von hier -> TDM-GCC

Ev0luti0n · 30.03.11, 13:52

Den Error gibt er in der Zeile mit dem Aufruf den Funktion...
Zu der Software:Habe mir gestern schon CodeBlocks geholt. Ist wirklich um einiges besser.

Alighieri · 01.04.11, 19:39

Zitat:

Damit erhöhst du den Pointer um eins auf das nächste Zeichen, nicht wie zuvor den (ASCII-)Wert des ersten Zeichens.

Das stimmt nicht , *pointer++ mit diesem Ausdruck zählt er nicht den ASCII Wert hinauf sondern übergibt ganz normal wie auch bei *pointer einen Wert. Postkrement wird dann die Adresse hinauf gezählt auf das nächste Objekt.

Bsp.

Code:

int  *pi,*pi2;
pi2=pi;
*pi++=5;
*pi=99;
printf("%i\n%i",*pi2,*pi);

Ausgabe: 5 und 99

mauralix · 01.04.11, 21:04

Viele Standardfunktionen gibt es ja in C nicht, aber die es gibt sollte man nutzen. z.B.: strchr um ein Zeichen zu suchen.

Anzeige

- Anzeige -

28.03.11, 21:34	#1 (permalink)
Ev0luti0n Registriert seit: 06.03.11 Likes: 0	C-"Return makes integer from pointer without cast"-Problem Anzeige Hey Leute. Ich bin's mal wieder. Dieses Mal gibts ein kleines Code-Problem. Code: int search_letter(char letter,char text) { while(text) { if(text=letter) return 1; else text++; } return 0; } Das ist meine Funktion. Wenn ich diese jetzt benutz' , dann erhalte ich von Dev C++ immer die Warnung "Return makes integer from pointer without cast". Und ich verstehs einfach nich warum diese Meldung kommt. Eine Erklärung wäre ganz nice (Kein "Ja weil das Return einen Integer vom Zeiger ohne Cast macht" ) Lg ev0luti0n

28.03.11, 21:44	#2 (permalink)
odigo Senior Member Registriert seit: 25.12.04 Likes: 54	Code: if(*text=letter) = ist eine Zuweisung. Bei einem if muss true oder false rauskommen, sprich du brauchst einen Vergleichsoperator, z.B. ==. Ein klassischer Fehler Der Vergleich von einem Pointer mit einem Char macht aber imho auch wenig Sinn, aber für eine genaue Erklärung ist mein Zeit mit C zu lang her

28.03.11, 22:42	#9 (permalink)
bitmuncher Moderator Registriert seit: 30.09.06 Likes: 441	-Wall sorgt dafür, dass der Compiler sämtliche Warnungen auswirft, auch zu unsauberem Code u.ä.. Oft findet man so Fehler, auf die man sonst nicht aufmerksam wird, weil ihre Ursache in "fehlerhaften" Code davor zu finden ist. Beim Entwickeln sollte man diesen Parameter immer nutzen. Probier's mal mit diesem Parameter und poste die Warnungen + zugehörigen Code, denn momentan kann ich deinen Fehler nicht nachvollziehen: Code: bitmuncher@alpha:~> cat test.c int search_letter(char letter,char text) { while(text) { if(text==letter) return 1; else text++; } return 0; } bitmuncher@alpha:~> gcc -c -Wall test.c test.c: In function 'search_letter': test.c:6: warning: value computed is not used Die Warnung "value computed is not used" ist allerdings bei Objektdateien normal, wenn man sie einzeln kompiliert, da die Werte ja tatsächlich innerhalb der Funktion nicht benutzt werden, und hat mit dem von dir beschriebenen Fehler nichts zu tun. __________________ Mein Blog - Mein Job - Diaspora Der Ring uns zu knechten besteht aus 12 Sternen auf blauem Grund. Neue Beiträge im Habo via Twitter - Das HaBo auf FB - Das HaBo bei G+

29.03.11, 21:11	#11 (permalink)
benediktibk Registriert seit: 03.05.07 Likes: 50	Code: *text++; Die Zeile meinst du wahrscheinlich anders: Code: text++; Damit erhöhst du den Pointer um eins auf das nächste Zeichen, nicht wie zuvor den (ASCII-)Wert des ersten Zeichens. Und, kleiner Tipp am Rande, wenn du sowieso nur 0 oder 1 zurückgibst tuts ein bool als Rückgabewert auch. Ist leichter zum Lesen und verhindert somit Fehler. Ich weiß nicht wo odigo die Unmengen an restlichen Fehlern findet, aber vielleicht bin ich einfach zu müde, als dass ich sie sehen würde. Kannst du bitte sagen bei genau welcher Zeile der Compiler meckert? Ich schätze mal die Zeilenangabe steht bei der Fehlermeldung dabei. mfg benediktibk

01.04.11, 21:04	#15 (permalink)
mauralix Registriert seit: 17.04.06 Likes: 3	Viele Standardfunktionen gibt es ja in C nicht, aber die es gibt sollte man nutzen. z.B.: strchr um ein Zeichen zu suchen. __________________ http://chm0815.blogspot.com

28.03.11, 21:49	#4 (permalink)
Ev0luti0n Themenstarter Registriert seit: 06.03.11 Likes: 0	Hey Danke für die Anmerkung . Grr. Der scheiß mit dem Vergleichsoperator passiert andauernd aber daran liegts nich. Gleiche Warnung ("passing arg 1 of 'search letter' makes integer ......"). hmmm. Es liegt also schon an der Parameterübergabe. Aber an was genaugrübel

28.03.11, 22:35	#8 (permalink)
odigo Senior Member Registriert seit: 25.12.04 Likes: 54	Du solltest die Methode ändern wie du dir C beibringen willst. Ich seh in deinem Code auf den ersten Blick mindestens 5 grundsätzlich Fehler. Such dir ein Buch/Tutorial was wirklich bei Hello World anfängt und arbeite dich schrittweise weiter. Punkt für Punkt.

30.03.11, 13:52	#13 (permalink)
Ev0luti0n Themenstarter Registriert seit: 06.03.11 Likes: 0	Den Error gibt er in der Zeile mit dem Aufruf den Funktion... Zu der Software:Habe mir gestern schon CodeBlocks geholt. Ist wirklich um einiges besser.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

[HaBo]

C-"Return makes integer from pointer without cast"-Problem