return makes integer from pointer without a cast

Version imprimable

Voir 40 message(s) de cette discussion en une page

bonjour tout le monde
je debute en C (apres du PHP et du python, dur dur) et je doit faire un pti programme adn to arn

Voila mon code je ne comprend pas pourquoi j ai ce message d'erreur à la compilation:
return makes integer from pointer without a cast
(je ne dois pas utiliser de pointeurs)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 #include <stdio.h> #include <stdlib.h> char adn2arn(char adn[],char arn[],int lSeq) { int i; for (i=0;i<lSeq;i++) { if(adn[i]=='T') { arn[i]='U'; } if(adn[i]=='A'||adn[i]=='G'||adn[i]=='C') { arn[i]=adn[i]; } } return (arn); } int main(void) { int lSeq=20; char seq[20]="GAATGCGTGATGCGTGAATT", arn[20]; printf("%s\n%s\n", seq, adn2arn(seq, arn, lSeq)); }
si quelqu'un peut m'aider, merci d'avance!

Salut,
le prototype de ta fonction adn2arn est de type "char" et à la fin, tu renvoies un (char *) [car arn est un tableau et donc un pointeur en C).

Tu modifies donc:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
#include <stdio.h>
#include <stdlib.h>
char * adn2arn(char adn[],char arn[],int lSeq)
{
     int i;
     for (i=0;i<lSeq;i++)
     {
>-------  if(adn[i]=='T')
>-------  {
>-------       arn[i]='U';
>-------  }    
>-------  if(adn[i]=='A'||adn[i]=='G'||adn[i]=='C')
>-------  {
>-------       arn[i]=adn[i];
>-------  }    
     }
     return (arn);
}    
int main(void)
{
     int lSeq=20;
     char seq[20]="GAATGCGTGATGCGTGAATT", arn[20];
     printf("%s\n%s\n", seq, adn2arn(seq, arn, lSeq));
return 0;
}

03/10/2006, 21h16
Corentor

Il y a un problème avec ta fonction.
Son type de retour est char et pourtant tu renvoies arn qui est déclaré comme un tableau de char.

Tu devrais essayer de déclarer ta fonction par :

Code:

void adn2arn(char adn[],char arn[],int lSeq)

ne pas renvoyer de valeur et l'appeler avant le printf :

Code:

printf("%s\n%s\n", seq, arn);

Sauf erreur de ma part, il y a un autre bogue subtil avec la taille de ton tableau et la longueur de ta chaîne d'ADN, mais je te laisse chercher un peu !

PS. Bulki, je n'avais pu lu ta réponse, nous avons dû poster en même temps !
03/10/2006, 21h21
nanocode

merci pour ta reponse bulki
j ai changé comme tu m'as dis mais j ai une autre erreur:
ligne 23:warning: cast from pointer to integer of different size
et apres compilation j ai toujour une erreur de segmentation..
03/10/2006, 21h24
nanocode

merci pour ta reponse Corentor
nous avons posté en meme temps..
en fait j ai besoin de la valeur de arn
le print du main est juste la pour faire marcher ma fonction
03/10/2006, 21h30
Corentor

Je ne comprends pas pourquoi tu as besoin de la valeur de arn puisque tu la connais lors de l'appel (arn est déclarée par main).

Avec la fonction

Code:

char * adn2arn(char adn[],char arn[],int lSeq)

tu ne fais que renvoyer l'adresse d'arn que tu as fournie en entrée.
03/10/2006, 21h34
nanocode

en fait c'est un enoncé, je doit declarer arn vide dans le main:

Créez une fonction nommée adn2arn() qui accèpte trois arguments : deux chaînes de
caractères et un entier (leur longueur).
Mettez dans le corps de cette fonction les instructions qui génèrent la séquence d'arn à partir
de la séquence d'adn.
Dans le programme principal, remplacez les instructions de génération de la chaîne d'arn par
l'appel à la fonction adn2arn().

merci d'essayer de me filer un coup de main ;)
03/10/2006, 21h40
Corentor

Citation:

Envoyé par nanocode

Créez une fonction nommée adn2arn() qui accèpte trois arguments : deux chaînes de
caractères et un entier (leur longueur).

Le type de retour de adn2arn() t'est-il imposé ? Je ne le vois pas ici.

Citation:

Envoyé par nanocode

Dans le programme principal, remplacez les instructions de génération de la chaîne d'arn par
l'appel à la fonction adn2arn().

Si je comprends bien, il y a un programme principal fourni avec l'énoncé, qu'il faut simplifier par l'appel d'une fonction adn2arn. C'est bien cela ? Si oui, pourrais-tu nous montrer ce programme ?

programme principal=main
ce sont des exos destinés à faire comprendre au biologiste les fonctions en c
j avais ce qui suit et je doit faire la meme chose avec une fonction de generation d'arn

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
#include <stdio.h>
#include <stdlib.h>
int main(void)
{
	int lSeq=20, i;
	float rapp=0.0;
	char seq[20]="GAATGCGTGATGCGTGAATT", arn[20];
	for (i=0;i<20;i++)
	{
		if(seq[i]=='T')
		{
			arn[i]='U';
		}
		if(seq[i]=='A'||seq[i]=='G'||seq[i]=='C')
		{
			arn[i]=seq[i];
		}
	}
	printf("%s\n%s\n", seq, arn);
}

j'aimerai bien savoir ce qui fait bugger le programme corrigé par bulki:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
 
#include <stdio.h>
#include <stdlib.h>
char*  adn2arn(char adn[],char arn[],int lSeq)
{
	int i;
	for (i=0;i<lSeq;i++)
	{
		if(adn[i]=='T')
		{
		arn[i]='U';
		}
		if(adn[i]=='A'||adn[i]=='G'||adn[i]=='C')
		{
		arn[i]=adn[i];
		}
	}
	return (arn);
}
int main(void)
{
	int lSeq=20;
	char seq[20]="GAATGCGTGATGCGTGAATT", arn[20];
	printf("%s\n%s\n", seq, adn2arn(seq, arn, lSeq));
return 0;
}

ps: j ai pas vu le deuxieme pti bug :)

03/10/2006, 21h54
Corentor

Tu remarqueras que dans la solution que je te proposais, la ligne du printf est identique à l'énoncé original.

Si je lis bien ton énoncé, il ne t'était pas demandé de toucher à ce printf.

Pour la solution de Bulki, je vais regarder (s'il ne trouve pas d'ici là ;) )

De plus, comme je l'avais signalé, il me semble y avoir un bogue à la ligne suivante :

Code:

char seq[20]="GAATGCGTGATGCGTGAATT", arn[20];
03/10/2006, 22h00
nanocode

merci corentor de suivre mon pb
c est une suite d'exos et pour la suite j ai besoin de cette fonction retournant ma chaine d'arn et non pas de l'afficher
je me serais bien contenté d'un printf! ;)

Citation:

De plus, comme je l'avais signalé, il me semble y avoir un bogue à la ligne suivante :

aie, je compte 20 bases pour un tableau a 20 entrées (0-G,1-A...19-T)... suis nul ou quoi?!
03/10/2006, 22h03
diogene

Citation:

j'aimerai bien savoir ce qui fait bugger le programme corrigé par bulki

La chaîne construite par adn2arn(...) n'est pas terminée correctement par la valeur 0. Du coup, le printf dépasse la zone mémoire attribuée à arn[].
Retourner arn n'est pas forcément une mauvaise idée puisqu'elle permet ce type d'écriture à un coût minime

Code:

printf("%s\n%s\n", seq, adn2arn(seq, arn, lSeq));
03/10/2006, 22h09
nanocode

merci diogene
il faut que je rajoute une entrée avec 0 à arn?
de quelle maniere je m'y prend?
03/10/2006, 22h09
diogene

Citation:

ligne 23:warning: cast from pointer to integer of different size

Je ne sais pas où intervient ce warning dans le code, mais il est la signature d'un code mal foutu : Il ne doit y avoir aucun cast ou conversion entre entier et pointeur (sauf à la rigueur si l'entier est 0) même si la chose est tolérée (hélas) par le langage
03/10/2006, 22h10
Corentor

Aucune des deux solutions (avec type de retour à void ou à char *) ne limite l'utilisation à l'affichage. Ce que j'essaie de t'expliquer est que le type de retour est ici redondant, puisque tu ne gères pas les erreurs (la valeur de retour sera toujours égal à la valeur d'une de tes entrées, que tu connais déjà).

Citation:

Envoyé par nanocode

aie, je compte 20 bases pour un tableau a 20 entrées (0-G,1-A...19-T)... suis nul ou quoi?!

Tu as bien compté, et c'est bien là le problème.;).
03/10/2006, 22h15
nanocode

Citation:

Envoyé par diogene

Je ne sais pas où intervient ce warning dans le code, mais il est la signature d'un code mal foutu : Il ne doit y avoir aucun cast ou conversion entre entier et pointeur (sauf à la rigueur si l'entier est 0) même si la chose est tolérée (hélas) par le langage

a l'appel de ma fonction adn2arn, je n ai pas essayé de faire ce genre de chose sciemment!
03/10/2006, 22h17
Corentor

Citation:

Envoyé par diogene

Retourner arn n'est pas forcément une mauvaise idée puisqu'elle permet ce type d'écriture à un coût minime

Code :
printf("%s\n%s\n", seq, adn2arn(seq, arn, lSeq));

Je suis d'accord sur ce point.
La valeur de retour a un côté pratique pour l'appel depuis des fonctions.
Par contre, on peut utiliser les valeurs de arn avec une fonction retournant void ou même un code d'erreur, or d'après les réponses de nanocode, je ne suis pas sûr qu'il en soit convaincu.
03/10/2006, 22h17
nanocode

Citation:

Envoyé par Corentor

Ce que j'essaie de t'expliquer est que le type de retour est ici redondant, puisque tu ne gères pas les erreurs (la valeur de retour sera toujours égal à la valeur d'une de tes entrées, que tu connais déjà).

donc ma fonction me retourne arn[] vide comme je lui ai entré?

Citation:

il faut que je rajoute une entrée avec 0 à arn?
de quelle maniere je m'y prend?

Avant de sortir de la fonction, lorsque tout ce qui est utile a été écrit dans arn, tu ajoutes derrière 0
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 char seq[21]="GAATGCGTGATGCGTGAATT", arn[21]; ..... for (i=0; seq[i] !=0;i++) { if(seq[i]=='T') { arn[i]='U'; } if(seq[i]=='A'||seq[i]=='G'||seq[i]=='C') { arn[i]=seq[i]; } } arn[i] =0;

03/10/2006, 22h19
nanocode

Citation:

Envoyé par Corentor

on peut utiliser les valeurs de arn avec une fonction retournant void ou même un code d'erreur, or d'après les réponses de nanocode, je ne suis pas sûr qu'il en soit convaincu.

effectivement :)
je suis largué, tout est beaucoup plus simple en PHP ou en python!

Voir 40 message(s) de cette discussion en une page