faire un return d'un tableau

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : faire un return d'un tableau

bjs

Bonjour,

je fais une conversion entier vers binaire (l'entier doit avoir valeur entre 0 et 255) et je veux récuperer ça dans une autre fonction pour l'envoyer au lcd. je voudrais récuperer le resultat de la conversion dans un tableau de char.
voici le code ça marche sauf pour conversion et envoi au lcd (ç'est incomplet)

Code :

#include <conio.h>
#include<stdlib.h>
#include<stdio.h>
#include<Windows.h>
#define PA 0// on met en entree à 1 (PA pour Data)
#define PB 1//en sortie à 0 inutilisé
#define PC 2//pcl en sortie et pch en sortie ? (PC1 pour R/W) et (PC2 pour RS)
void _stdcall set8255cw(unsigned char cw_byte);
unsigned char _stdcall read8255(unsigned char PortNumber);
void _stdcall write8255(unsigned char PortNumber,unsigned char Value);
void _stdcall init_lcd();
unsigned char _stdcall check_busy_byte();
void menus();
void commandeLCD(int booleen,unsigned char adresse);
void envoichaine();
void decalage();
char* affBin(int decimal);//fonction pour conversion decimal vers binaire
//PC0 pour E *** PC1 pour R/W *** PC2 pour RS *** PA pour Data
//cycle d'écriture: RS à 0 instruction/1data, R/W à 0 ecriture(R/W à 1 lecture), mettre E à 1, écrire la data, mettre E à 0
void main()
{
set8255cw(0x8A); //1000 1010
printf("\nInitialisation du LCD : Appuyez sur une touche enter" );
init_lcd();//appuyer sur enter pour sortir de la fonction
commandeLCD(0,0x01);//efface l'ecran 1er paramètre =0 car instruction et deuxième paramètre ==1 car envoi du byte 1 dans cours sur PA
//est ce necessaire après initlcd de faire effacer ecran ??? si on met sur la ligne 1 colonne 1 alor ça efface l'ecran ????commandeLCD(0,0x80);
menus();
}
void menus()
{
char Menu;
short choix=1;
do
{
printf("____________________________________\n" );
printf("1) Afficher Chaine (1er ligne)\n" );
printf("2) Afficher Chaine (2eme ligne)\n" );
printf("3) Decalage \n" );
printf("4) Conversion decimal vers binaire\n" );
printf("5) Quitter" );
printf("____________________________________\n" );
fflush(stdin);
scanf_s("%d",&Menu);
switch(Menu)
{
case 1 :
commandeLCD(0,0x80);
envoichaine();
break;
case 2 :
commandeLCD(0,0xC0);
envoichaine();
break;
case 3 :
decalage();
break;
case 4:
break;
case 5 :
choix=0;
break;
}
}while(choix!=0);
}
// void commandeLCD >>>> choix de instruction ou data via booleen en paramètre et ensuite cycle d'ecriture
void commandeLCD(int booleen,unsigned char adresse)//booleen == 0 alors instruc sinon data alors 1 dans booleen ***** adresse pour 1 effacer lcd ou 0x80 pour première ligne ou deuxième ligne
{
//cycle d'écriture: RS à 0/1, R/W à 0, mettre E à 1, écrire la data, mettre E à 0
unsigned char busy_byte;
unsigned char PC2Byte;
do{
busy_byte=check_busy_byte();
busy_byte&=0x80;
}while(busy_byte!=0);
if( booleen ==0)// selon qu'on fait une instruction ou data donc si ==0 alors instruction
{
PC2Byte=read8255(PC);
PC2Byte=PC2Byte & 0xFB;//mise à 0 de RS (PC2) donc instruction 0xFB 11111011
write8255(PC, PC2Byte);
}
else
{
if(booleen == 1)// selon qu'on fait une instruction ou data
{
PC2Byte=read8255(PC);
PC2Byte=PC2Byte | 0x04;//mise à 1 de RS (PC2) ça veut dire envoi d'une donnée et non pas une instruction
write8255(PC, PC2Byte);
}
}
PC2Byte=read8255(PC);
PC2Byte = PC2Byte & 0xFD; //Mise à 0 de R/W (PC1) donc ecriture
write8255(PC, PC2Byte);
PC2Byte=read8255(PC);
PC2Byte = PC2Byte | 0x01; //Mise à 1 de E (PC0) sans changer les autres bits>> demarre l'ecriture
write8255(PC, PC2Byte);
//écriture de la donnée
write8255(PA, adresse);//adresse==1 OU 0x80 OU deuxième ligne OU *chaine pour envoi de caractères
Sleep(1);
PC2Byte=read8255(PC);
PC2Byte = PC2Byte & 0xFE; //Mise à 0 de E (PC0) et les autres bits ne sont pas changés>>> envoi de l'ecriture
write8255(PC, PC2Byte);
}
//write8255(PA,1);//après avoir mis rs à 0 voici l'instru pour effacer le lcd mettre à 1 le bit 1 de PA voir cours
//write8255(PC, 0)//enable à 0 la donnee part (l'ecran est efface)
// void envoichaine()>>>>>>> saisie d'une chaine de caractères et envoi sur l'ecran lcd
//via l'appel de fonction commandeLCD qui a comme paramètres 1 (pour dire que ç'est une instruction) et chaine[i] envoi caract par caract
void envoichaine()
{
char chaine[15];
short i=0;
short taille;
printf("Saisir chaine : " );
fflush(stdin);
fgets(chaine,15,stdin);
taille=strlen(chaine);
chaine[taille+1]='\0';
while (chaine[i]!=chaine[taille])
{
commandeLCD(1,chaine[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
i++;
}
puts(chaine);
}
// void decalage() >>>>> choix de decalage via un menu par l'user de décalage à gauche(a) ou droite(b) puis saisie du nombre de decalage
//à effectuer et puis appel à la fonction commandeLCD qui a comme paramètres 1(car ç'est data et pas instruction)
void decalage()
{
char Menu;
short choix=1;
short nbldecalage;
do
{
printf("____________________________________\n" );
printf("a) Ecrire a pour Decalage droite\n" );
printf("b) Ecrire b pour Decalage gauche\n" );
printf("c) Quitter" );
printf("____________________________________\n" );
fflush(stdin);
scanf("%s",&Menu);
switch(Menu)
{
case 'a' ://droite decalage
printf("\nEntrez le nombre de decalages : " );
scanf("%d", &nbldecalage);
for( int i=0; i<nbldecalage; i++ )
{
commandeLCD(1,0x1C); //00011R/L00 bit R/L à 1 si decalage 1 à droite(decalage du texte DDRAM) donc 00011100
}
break;
case 'b'://gauche decalage
printf("\nEntrez le nombre de decalages : " );
scanf("%d", &nbldecalage);
for( int i=0; i<nbldecalage; i++ )
{
commandeLCD(1, 0x18); //00011R/L00 bit R/L à 0 si decalage 1 à gauche(decalage du texte DDRAM) donc 00011000
}
break;
case 'c' :
choix=0;
break;
}
}while(choix!=0);
}
/*convertir()
// diviser le nombre decimal par 2 successivement et le reste de la division represente le nombre binaire
division entière donc si y a un reste le bit à 1 sinon le bit à 0 puis inverser pour le lire le nombre binaire
44/2 >> division sans reste donc bit à 0
22/2 >> division sans reste donc bit à 0
11/2>> division avec reste donc bit à 1
5/2..... bit à 1
2/2 bit à 0
1/2 avec reste donc bit à 1
lecture du nombre binaire à l'envers 00101100 ce qui vaut 44 en decimal*/
char* affBin(int decimal)
{
char*chainebinaire=NULL;
chainebinaire=(char*)malloc(8*sizeof*chainebinaire);
int nbrdecimal;
int i=0;
puts("Entrez un nombre positif" );
scanf("%d", &nbrdecimal);
if (decimal != 0)//condition de sortie de la boucle fonction recursive
{
if (decimal > 1)//2ème condition de sortie de fonction recursive
{
affBin(decimal / 2);//verifie si divisible par 2 et si toujours supèrieur à 1 et
// les valeurs successives de decimal sont mises sur la pile
//printf("%d",decimal);
}
//printf("%d\n",decimal);
printf("%d", decimal % 2);
int i=0;
while(i<8)
{
chainebinaire[i]=(decimal%2);
sprintf(chainebinaire,((char*)(decimal%2)));
//chainebinaire[i]=itoa(chainebinaire,decimal%2,2);
i++;
}
//la division modulo d'un nombre vaut 0(chiffre pair) si pas de reste et 1(chiffre impaire) si y a un reste
} // on va afficher directement le premier bit comme ça ç'est bien à l'envers et
//ainsi de suite(on affiche les modulo des valeurs de decimal qui ont été mises sur la pile)
else
{
printf("0" );
for(int i=0;i<8;i++)
{
chainebinaire[i]='0';
}
}
return &chainebinaire;
}
void envoichaine2()
{
short i=0;
short taille;
int size;
char decim[25];
int decim2;
char chaine1[100]={"Entrez un nombre: "};
char chaine2[100]={"decimal: "};
char conversionbinaire[200];
taille=strlen(chaine1);
chaine1[taille+1]='\0';
commandeLCD(0,0x80);//on se positionne sur la première ligne du lcd pour envoyer entrez un nombre
while (chaine1[i]!=chaine1[taille])
{
commandeLCD(1,chaine1[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
i++;
}
commandeLCD(0,0xC0);//on se place sur la deuxième ligne pour envoyer la phrase decimal:
taille=strlen(chaine2);
chaine2[taille+1]='\0';
while (chaine2[i]!=chaine2[taille])
{
commandeLCD(1,chaine2[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
i++;
}
//saisie du nombre decim à convertir et envoi sur l'ecran lcd
fflush(stdin);
gets(decim);
size=strlen(decim);
//commandeLCD(0,0xC0);//je suis déjà sur la deuxième ligne ?? ça va ecrire à la suite de decimal:????
i=0;
do
{
commandeLCD(1,decim[i]);
i++;
}while (decim[i] !=decim[size]);
strcpy( conversionbinaire,affBin(decim));
/*size=strlen(conversionbinaire);
i=0;
do
{
commandeLCD(1,conversionbinaire[i]);
i++;
}while (conversionbinaire[i] !=conversionbinaire[size]);*/
}

Message édité par bjs le 13-03-2017 à 22:12:18

Publicité

rat de combat

attention rongeur méchant!

J'ai pas essayé de comprendre le code, c'est pas très digeste et je vois des choses douteuses genre sprintf(chainebinaire,((char*)(decimal%2)));.

De manière générale: Impossible de retourner un tableau depuis une fonction. On peut retourner un pointeur issu de malloc comme tu le fais mais ce n'est pas forcément très propre car il faut penser à libérer la mémoire ailleurs (ce que tu ne fais pas, en tout cas je ne vois pas de free()).
Tu peux passer un pointeur vers un espace mémoire déjà réservé(!!) à ta fonction.

Après si il faut absolument retourner des tableaux on peut les mettre dans un struct mais c'est pas forcément très élégant... (et inutile içi).

edit: Tu devrais activer (et prendre en compte) les warnings de ton compilateur!
> "gcc" "t.c" -Wall -O3 -c -std=c99
t.c:20:6: warning: return type of 'main' is not 'int' [-Wmain]
t.c: In function 'menus':
t.c:43:9: warning: implicit declaration of function 'scanf_s' [-Wimplicit-function-declaration]
t.c: In function 'decalage':
t.c:148:13: warning: format '%d' expects argument of type 'int *', but argument 2 has type 'short int *' [-Wformat]
t.c:156:13: warning: format '%d' expects argument of type 'int *', but argument 2 has type 'short int *' [-Wformat]
t.c: In function 'affBin':
t.c:216:5: warning: return from incompatible pointer type [enabled by default]
t.c:216:5: warning: function returns address of local variable [enabled by default]
t.c:184:9: warning: unused variable 'i' [-Wunused-variable]
t.c: In function 'envoichaine2':
t.c:257:5: warning: passing argument 1 of 'affBin' makes integer from pointer without a cast [enabled by default]
t.c:179:7: note: expected 'int' but argument is of type 'char *'
t.c:224:9: warning: unused variable 'decim2' [-Wunused-variable]

Message édité par rat de combat le 13-03-2017 à 22:35:18

rat de combat

attention rongeur méchant!

Désolé, je viens de regarder le début de ton code et pour moi c'est disons pas au point...

P.ex.

Code :

void envoichaine()
{
char chaine[15];
short i=0;
short taille;
printf("Saisir chaine : " );
fflush(stdin);
fgets(chaine,15,stdin);
taille=strlen(chaine);
chaine[taille+1]='\0';
while (chaine[i]!=chaine[taille])
{
commandeLCD(1,chaine[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
i++;
}
puts(chaine);
}

Ceci
taille=strlen(chaine);
chaine[taille+1]='\0';
c'est dangereux: Si l'utilisateur entre 14 ou plus de caractères strlen() retourne 14 et donc taille+1=15 et donc out of bound...

Tu veux enlèver un éventuell '\n' c'est ça?
Alors

Code :

char chaine[15];
short taille;
printf("Saisir chaine : " );
fgets(chaine,15,stdin);
taille=strlen(chaine);
if(taille!=0 && chaine[taille-1]=='\n')
chaine[taille-1]='\0';
printf("-%s-\n", chaine);

fgets() termine toujours avec un '\0', pas de soucis de ce côté.

Ensuite
commandeLCD(1,chaine[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
La fonction est définie ainsi:
void commandeLCD(int booleen,unsigned char adresse)
Donc tu passe un caractère (imprimable) comme adresse? Ou c'est plutôt une donnée et le nom est mal choisi? Sans parler du problème de signedness (char vs unsigned char)... Petite remarque: Le mot c'est boolean.

La condition de la boucle n'est pas fausse mais pas très lisible, plutôt un classique while(chaine[i]!='\0') ou même while(chaine[i]).

Ceci
fflush(stdin);
a un résultat indéfini (source K&R 2nd ed. page 242).

Vu les warnings il y a forcément d'autres problèmes...

EDIT: Aie!
char Menu; scanf("%s",&Menu); (dans void decalage())
Refléchis à ce qui va se passer si l'utilisateur entre iosfiposdfiodsfimdspfioidsof p.ex...

Comme je pense que c'est un projet universitaire je ne vais pas corriger d'avantage.

Message édité par rat de combat le 13-03-2017 à 23:09:38

bjs

void envoichaine2()

{

short i=0;
short taille;
int size;
char decim[10];
int decim2=0;
char chaine1[100]={"Entrez un nombre: "};
char chaine2[100]={"decimal: "};
char conversionbinaire[10];
char conversionhexa[10];

taille=strlen(chaine1);

commandeLCD(0,0x80);//on se positionne sur la première ligne du lcd pour envoyer entrez un nombre

while (chaine1[i]!=chaine1[taille])
{
commandeLCD(1,chaine1[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
i++;

}
commandeLCD(0,0xC0);//on se place sur la deuxième ligne pour envoyer la phrase decimal:
taille=strlen(chaine2);

while (chaine2[i]!=chaine2[taille])
{
commandeLCD(1,chaine2[i]);//on envoie caractère tant qu'on n'est pas arrivé au caractère 0 de fin de chaine
i++;

}
//saisie du nombre decim à convertir et envoi sur l'ecran lcd
fflush(stdin);
fgets(decim,8,stdin);
size=strlen(decim);
//commandeLCD(0,0xC0);//je suis déjà sur la deuxième ligne ?? ça va ecrire à la suite de decimal:????

i=0;
do
{
commandeLCD(1,decim[i]);
i++;
}while (decim[i] !=decim[size]);

decim2=atoi(decim);//conversion du char en int car itoa attend un int
itoa (decim2,decim,10);//conversion d'un entier en char en base 10
printf ("decimal: %s\n",decim);
itoa (decim2,conversionhexa,16);//conversion d'un entier en char en base 16
printf ("hexadecimal: %s\n",conversionhexa);
itoa (decim2,conversionbinaire,2);// conversion d'un entier en char en base 16
printf ("binaire: %s\n",conversionbinaire);
commandeLCD(0,0x01);//effacer l'ecran avant d'afficher les conversions et se place sur la première ligne du lcd
int size2=strlen(conversionbinaire);
i=0;
do
{
commandeLCD(1,conversionbinaire[i]);
i++;
}while (conversionbinaire[i] !=conversionbinaire[size2]);

commandeLCD(0,0xC0);// se placer sur la deuxième ligne de l'ecran lcd
int size3=strlen(conversionhexa);
i=0;
do
{
commandeLCD(1,conversionbinaire[i]);
i++;
}while (conversionbinaire[i] !=conversionbinaire[size3]);
}

Message édité par bjs le 13-03-2017 à 23:16:00

bjs

j'ai supprimé la fonction affBin qui me cassait la tete et j'ai tout fait à la suite dans une seule fonction
pour le caractère de fin de chaine '0' ça va pas sur le lcd car il le connait pas et affiche un caractère inconnu .... j'ai esssayé sur le lcd en labo
pour adresse j'avais mal choisi le nom
ç'est juste un petit labo pas un projet on a 5 labos à faire ...

Message édité par bjs le 13-03-2017 à 23:24:25

rat de combat

attention rongeur méchant!

Citation :

pour le caractère de fin de chaine '0' ça va pas sur le lcd car il le connait pas et affiche un caractère inconnu

Déjà je suppose que c'est une faute de frappe de ta part, il y a une grosse différence entre '0' et '\0'! Bien sûr faut pas envoyer '\0' au LCD mais un code bien fait (while(string[i]!='\0')) ne le fera pas!?! :??:

Citation :

ç'est juste un petit labo pas un projet on a 5 labos à faire ...

Mais il y a bien une note au but? Alors ça revient au même.

C'est quoi comme études que tu fais et en quelle année?

bjs

je suis en haute école. je suis en deuxième et j'ai 2 cours de première qui me restent.
ce labo fait partie d'un cours de première
je pense que ç'est mieux la deuxième façon car je m'y retrouve mieux et elle est plus simple et je pense qu'il y a pas d'erreurs dans la fonction envoichaine2()??

Message cité 1 fois

bjs

ç'est pas coté y a juste le dernier labo qui est plus dur qui est coté sur le robot voiture
ç'est pas noté(ah en français pas en belge) y a juste le dernier labo qui est noté (robot)

Message édité par bjs le 13-03-2017 à 23:34:02

rat de combat

attention rongeur méchant!

bjs a écrit :

je suis en haute école.

Ah d'accord, c'est en Belgique, je ne connais pas leur système.

Citation :

je pense qu'il y a pas d'erreurs dans la fonction envoichaine2()??

Il n'y a rien qui va faire planter le PC à première vue mais c'est loin d'être propre!

Déjà c'est quoi ces machins, tu veux faire quoi??

Code :

char chaine1[100]= {"Entrez un nombre: "};
taille=strlen(chaine1);
chaine1[taille+1]='\0';

C'est parfaitement inutile. chaine1 contient déjà un '\0' au bon endroit et si il y en avait pas il ne faudrait surtout pas utiliser strlen()! Les {} ne sont pas nécessaires comme l'indication d'une taille à moins de vouloir modifier le contenu plus tard et d'avoir besoin de plus de place.

Ensuite tu mélanges les while() et les do while() et il n'y a pas de gestion d'erreurs, que se passe-t-il si l'utilisateur entre autre chose qu'un nombre?
En plus je répète: fflush(stdin) est indéfini.
Aussi tu utilises des constantes un peu partout, c'est mieux (plus lisible et/ou plus sûr) d'utiliser des #define (ou des const, aucune idée pourquoi le #define est si répandu pour ce genre de trucs...) ou selon la situation un truc genre typedef enum.

Message édité par rat de combat le 13-03-2017 à 23:49:32

bjs

ok merci pour tes conseils, j'ai enlevé les do while et j'ai remplacé par un while.
j'ai enlevé les accolades pour l'initialisation de la chaine à envoyer au lcd(ça doit être comme l'exemple de la capture de l'ecran lcd du cahier de labo)
ç'est un petit labo et les autres cours de première y a bcp de blindage des saisies donc il le demande pas ici et on suppose que l'user est intelligent

Message édité par bjs le 14-03-2017 à 00:00:42

FORUM HardWare.fr

Programmation

faire un return d'un tableau

Sujets relatifs
Regex Split tableau	Tableau javascript et liste déroulante?
Plusieurs variables ou un tableau? (Performances)	Tableau de couleur php
Affichage bouton dans un tableau !	[VBA] Problème avec tableau
Interface Swing à partir d'un tableau 2D (jeu)	C : Envoi et retour d'un tableau dans des fonctions.
Eviter les quotes quand j'insère du texte dans un tableau	renvoyer 2 tableaux avec return
Plus de sujets relatifs à : faire un return d'un tableau

Page générée en 0.167 secondes