[PIC]erreurs dans un programme

**ABN84** · 27/04/2008, 15h28

bonjour,
mplab m'indique des erreurs dans les lignes 5,6, et 7 de ce programme:

Code:

#include <p18f4520.h>
#include <delays.h>

uChar_Bits sens;
#define sens(sens.value)
#define s1(sens.b0)
#define s2(sens.b1)

int pos1,pos2;//
int posr1,posr2;

TRISA=0b000000;
TRISB=0b11111111;
TRISC=0b10001111;
TRISD=0b00000000;
TRISEbits.TRISE0=0;
PORTD=0b11111111;

void revolt1(s1){//tourner le moteur 1 dans le sens s1
	PORTDbits.RD1=s1;
	while(1){//Step
		PORTDbits.RD2 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD2 = 0;
		Delay10KTCYx(0);
		posr1=posr1+1;
	}
}

void revolt2(s2){//tourner le moteur 2 dans le sens s2
	PORTDbits.RD5=s2;
	while(1){//Step
		PORTDbits.RD6 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD6 = 0;
		Delay10KTCYx(0);
		posr2=posr2+1;
	}
}
		

void init(){//initialisation
	if(RC2){
		while(RC0!=1){
			revolt1(1);
		}
		while(RC1!=1){
			revolt2(1);
		}
	}
	posr1=0;//initialiser la position du moteur 1
	posr2=0;//initialiser la position du moteur 2
}

void track(){//suivi
	while(1){
		if(posr1<pos1){
			revolt1(1);
		}
		else if(posr1>pos1){
			revolt1(0);
		}
		if(posr2<pos2){
			revolt2(1);
		}
		else if(posr2>pos2){
			revolt2(0);
		}
	}
}
	
	

void main(){
	while(1){
		init();
		track();
	}
}

qqun saurait-il comment corriger svp?
merci

**Seb.26** · 27/04/2008, 15h51

Il te manque la définition de la structure uChar_Bits ( d'ailleur ça me rappelle quelque chose ...

)

Code:

#define uChar unsigned char
typedef union _uChar_b {
  struct {
    unsigned b0:1;
    unsigned b1:1;
    unsigned b2:1;
    unsigned b3:1;
    unsigned b4:1;
    unsigned b5:1;
    unsigned b6:1;
    unsigned b7:1;
  };
  uChar value;
};
#define uChar_Bits union _uChar_b

Mets ça dans un .h genre "myTypes.h" et ajoute l'include en tête des fichier .c dans lesquels tu l'utilise ...

[Edit] T'as aussi un souci : rajoute un espace entre "sens" et "(sens.value)" dans ton #define ... idem pour s1 et s2 ... sinon le pre-pro ne comprend pas ce que tu attends de lui ...

**Seb.26** · 27/04/2008, 15h55

Autre souci ... AMA ton #define est illégal car il boucle sur lui même ...

remplace :

Code:

uChar_Bits sens;
#define sens(sens.value)
#define s1(sens.b0)
#define s2(sens.b1)

par :

Code:

uChar_Bits _sens;
#define sens (_sens.value)
#define s1 (_sens.b0)
#define s2 (_sens.b1)

[Edit] d'ailleur, je pense que c'est ça le souci, et pas les espaces en fait ...

**ABN84** · 27/04/2008, 16h25

( d'ailleur ça me rappelle quelque chose ...

)

non, je vois pas

c'est corrigé:

Code:

#include <p18f4520.h>
#include <delays.h>
#include "uchar.h"


uChar_Bits sens;
#define sens (_sens.value)
#define s1 (_sens.b0)
#define s2 (_sens.b1)

à present l'erreur est dans les TRIS

A voir en vidéo sur Futura · Aujourd'hui

**Seb.26** · 27/04/2008, 16h31

ça devrait toujours pas marcher ...

...

Code:

#include <p18f4520.h>
#include <delays.h>
#include "uchar.h"

uChar_Bits sens;
#define sens (_sens.value)
#define s1 (_sens.b0)
#define s2 (_sens.b1)

il faut :

Code:

uChar_Bits _sens;

**Seb.26** · 27/04/2008, 16h33

Pour les TRIS, ton premier est sur 6 bits, ça ne pose sans doute pas de problèmes ( il met des 0) mais c'est pas très "beau" ...

C'est quoi ton compilo ? il accepte "0b11111111" ?

**ABN84** · 27/04/2008, 16h50

Pour les TRIS, ton premier est sur 6 bits, ça ne pose sans doute pas de problèmes ( il met des 0) mais c'est pas très "beau"

RA6 et RA7 correspondent à OSC1 et OSC2 c'est pourquoi j'evite de les configurer

C'est quoi ton compilo ?

C18.

il accepte "0b11111111" ?

il devrait.

**Seb.26** · 27/04/2008, 16h53

Envoyé par einstein

RA6 et RA7 correspondent à OSC1 et OSC2 c'est pourquoi j'evite de les configurer

Mais tu écris un octet ... pas seulement les 6 bits que tu précise ... les autres sont à 0, mais ils sont écrits ...

invite03481543 · 27/04/2008, 16h53

Avec TRISA=0; ça devrait mieux se passer.

**Seb.26** · 27/04/2008, 16h59

PS: tu es sûr d'avoir besoin de "int" ???

C'est quelle taille un int ? ...

**ABN84** · 27/04/2008, 17h02

Avec TRISA=0; ça devrait mieux se passer

IDEM

PS: tu es sûr d'avoir besoin de "int" ???

non, un char suffit, mais je m'occuperais de la gestion de la memoire une fois que tout marchera.

invite03481543 · 27/04/2008, 17h03

Tu as quoi comme message d'erreur?

**Seb.26** · 27/04/2008, 17h03

Tu peux mettre l'erreur stp ... ça va être moins drôle certes, mais un poil plus facile ...

invite03481543 · 27/04/2008, 17h06

2 messages en même temps, je vais laisser seb.26 tranquille.

**Seb.26** · 27/04/2008, 17h08

Bah ... Y'a pas de mal ...

... mais ça fait 2 fois quand même ( 17h53 + 18h03 ) et dans le même post en plus !!!

**ABN84** · 27/04/2008, 17h15

Cliquez pour afficher

**Seb.26** · 27/04/2008, 17h21

Tu peux poster aussi les 20 première ligne de ton main.c ?!

**ABN84** · 27/04/2008, 17h43

les voici:

Code:

#include <p18f4520.h>
#include <delays.h>
#include "uchar.h"


uChar_Bits _sens;
#define sens (_sens.value)
#define s1 (_sens.b0)
#define s2 (_sens.b1)

int pos1,pos2;//
int posr1,posr2;

TRISA=0;
TRISB=0b11111111;
TRISC=0b10001111;
TRISD=0b00000000;
TRISEbits.TRISE0=0;
PORTD=0b11111111;

void revolt1(s1){//tourner le moteur 1 dans le sens s1
	PORTDbits.RD1=s1;
	while(1){//Step
		PORTDbits.RD2 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD2 = 0;
		Delay10KTCYx(0);
		posr1=posr1+1;
	}
}

void revolt2(s2){//tourner le moteur 2 dans le sens s2
	PORTDbits.RD5=s2;
	while(1){//Step
		PORTDbits.RD6 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD6 = 0;
		Delay10KTCYx(0);
		posr2=posr2+1;
	}
}
		

void init(){//initialisation
	if(RC2){
		while(RC0!=1){
			revolt1(1);
		}
		while(RC1!=1){
			revolt2(1);
		}
	}
	posr1=0;//initialiser la position du moteur 1
	posr2=0;//initialiser la position du moteur 1
}

void track(){//suivi
	while(1){
		if(posr1<pos1){
			revolt1(1);
		}
		else if(posr1>pos1){
			revolt1(0);
		}
		if(posr2<pos2){
			revolt2(1);
		}
		else if(posr2>pos2){
			revolt2(0);
		}
	}
}
	
	

void main(){
	while(1){
		init();
		track();
	}
}

**Seb.26** · 27/04/2008, 17h55

Oh purée, j'avais pas vu !!!

Tu as mis du code ( TRISA=0; ) en dehors du programme ...

Deplace :

Code:

TRISA=0;
TRISB=0b11111111;
TRISC=0b10001111;
TRISD=0b00000000;
TRISEbits.TRISE0=0;
PORTD=0b11111111;

Dans une fonction ( init() par exemple )

**ABN84** · 27/04/2008, 18h01

le probleme c'est que j'utilise ces differents ports dans differents programmes.

**RISC** · 27/04/2008, 18h08

Einstein,

Ou as-tu déclaré les bits de configuration de ton PIC ?
Je ne les vois nulle part dans ton programme...à moins que tu ais fait cela dans un fichier header.

Si ce n'est pas le cas je te recommande de TOUJOURS programmer les bits de configuration dans ton programme.
De cette manière tu n'auras plus a te poser de question...

a+

**ABN84** · 27/04/2008, 18h09

ok ,j'ai tout mis dans la fct main:

Code:

#include <p18f4520.h>
#include <delays.h>
#include "uchar.h"


uChar_Bits _sens;
#define sens (_sens.value)
#define s1 (_sens.b0)
#define s2 (_sens.b1)

int pos1,pos2;//
int posr1,posr2;


void revolt1(s1){//tourner le moteur 1 dans le sens s1
	PORTDbits.RD1=s1;
	while(1){//Step
		PORTDbits.RD2 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD2 = 0;
		Delay10KTCYx(0);
		posr1=posr1+1;
	}
}

void revolt2(s2){//tourner le moteur 2 dans le sens s2
	PORTDbits.RD5=s2;
	while(1){//Step
		PORTDbits.RD6 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD6 = 0;
		Delay10KTCYx(0);
		posr2=posr2+1;
	}
}
		

void init(){//initialisation
	if(RC2){
		while(RC0!=1){
			revolt1(1);
		}
		while(RC1!=1){
			revolt2(1);
		}
	}
	posr1=0;//initialiser la position du moteur 1
	posr2=0;//initialiser la position du moteur 1
}

void track(){//suivi
	while(1){
		if(posr1<pos1){
			revolt1(1);
		}
		else if(posr1>pos1){
			revolt1(0);
		}
		if(posr2<pos2){
			revolt2(1);
		}
		else if(posr2>pos2){
			revolt2(0);
		}
	}
}
	
	

void main(){
	TRISA=0b00000000;
	TRISB=0b11111111;
	TRISC=0b10001111;
	TRISD=0b00000000;
	TRISEbits.TRISE0=0;
	PORTD=0b11111111;
	while(1){
		init();
		track();
	}
}

l'erreur se trouve à present à la fct revolt1():

Clean: Deleting intermediary and output files.
Clean: Done.
Executing: "C:\mcc18\bin\mcc18.exe" -p=18F2520 "main.c" -fo="main.o" /i"C:\mcc18\h" -Ou- -Ot- -Ob- -Op- -Or- -Od- -Opa-
MPLAB C18 v2.40 (demo)
Copyright 1999-2004 Microchip Technology Inc.
Days remaining until demo becomes feature limited: 12
c:\mpap\mpap\main.c:15:Error: syntax error
Halting build on first failure as requested.
BUILD FAILED: Sun Apr 27 19:08:51 2008

**RISC** · 27/04/2008, 18h15

Einstein,

Je viens de voir que tu utilises une super vieille version du compilateur C18 (v2.40 !!!!).

Télécharges de suite la version 3.16 (gratuite) ici : www.microchip.com/C18

Et par la même occasion la toute dernière version de MPLAB (8.10) toute chaude d'avant hier : www.microchip.com/MPLAB

Elle ajoute des fonctions super comme une barre d'icone pour les points d'arrêt et aussi la possibilité de gérer des répertoires dans les projets ;=)

Et cerise pour le gateau pour ceux qui ont un real-ice, des points d'arrêt logiciels pour les PIC18 et la possibilité de visualiser les variables globales pendant l'exécution !!!

Les nouvelles fonctionalités sont listées ici : http://forum.microchip.com/tm.aspx?m=333388

a+

YESSSSSSSS : Et de 500 posts pour ma pomme ;=)

invite03481543 · 27/04/2008, 18h26

einstein, il vaut mieux que tu crées une fonction genre pic_init() dans laquelle tu déclares toutes les initialisations nécessaires (ports, enable d'interruptions, timer, etc).
C'est une bonne habitude à prendre et ce sera plus lisible quand tu feras des programmes plus importants.

Ton main() deviendra:

void main(){

pic_init();
.......
}

invite03481543 · 27/04/2008, 18h35

Autre chose, dans un programme on déclare les variables, les define, puis les prototypes des fonctions, les fonctions, et le main.

Le mieux est de te créer un gabarit type, qui sera ta feuille par défaut que tu chargeras à chaque nouveau projet.
Elle contiendra les sections nommées ci dessus du programme.

Dans la section projet j'ai posté il y a quelques temps un programme (strobo+dimmer), tu peux t'en inspirer si tu veux.
Un programme c'est avant tout de l'organisation.
@+

**Seb.26** · 27/04/2008, 18h42

Envoyé par einstein

l'erreur se trouve à present à la fct revolt1():

Normal

... Tu dois indiquer le type de tes paramètres dans les fonctions ...

Et en plus, tu as nommé tes paramètres comme tes #define ... moralité, le compilo tombe sur void revolt1( (_sens.b0) ) ... ce qui veut pas dire grand chose ...

**ABN84** · 27/04/2008, 18h50

Dans la section projet j'ai posté il y a quelques temps un programme (strobo+dimmer), tu peux t'en inspirer si tu veux.

ok, je vais y jetter un coup d'oeuil. merci

Et en plus, tu as nommé tes paramètres comme tes #define ... moralité, le compilo tombe sur void revolt1( (_sens.b0) ) ... ce qui veut pas dire grand chose ...

ce que je veux, c'est que la fct revolt1 recoive comme argument un bit et que la valeur de ce bit soir celle que j'affecte à PORTDbits.RD1

**ABN84** · 27/04/2008, 18h53

un autre petit truc:
j'ai mis les deux fct revolt en commentaire, pour voir s'il y a d'autres erreurs, et apparemment C18 n'accepte pas l'ecriture:

while(RC1!=1)

me disant que RC1 n'est pas declaré, comment ecrire cette boucle alors?
while(PORTCbits.RC1!=1) ?
merci

invite03481543 · 27/04/2008, 18h58

Code:

#include <p18f4520.h>
#include <delays.h>
#include "uchar.h"


uChar_Bits _sens;
#define sens (_sens.value)
#define s1 (_sens.b0)
#define s2 (_sens.b1)

int pos1,pos2;//
int posr1,posr2;


/* Prototype des fonctions */

void revolt1(uChar_Bits s1);
void revolt2(uChar_Bits s2);
void init(void);
void track(void);
void init_pic(void);
void init_var(void);

/* init du Pic */

void init_pic()
{
/* il faut donner ici une valeur aux différentes sorties */
TRISA=0b00000000;

/* idem */
TRISB=0b11111111;

/* idem */
TRISC=0b10001111;

/* idem */
TRISD=0b00000000;

TRISEbits.TRISE0=0;

PORTD=0b11111111;
/* à mettre avant TRISD
}

/* initialisation des variables globales */

void init_var(void)
{
/* ........ */
}

/* Fonction revolt1 */

void revolt1(s1){//tourner le moteur 1 dans le sens s1
	PORTDbits.RD1=s1;
	while(1){//Step
		PORTDbits.RD2 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD2 = 0;
		Delay10KTCYx(0);
		posr1=posr1+1;
	}
}

void revolt2(s2){//tourner le moteur 2 dans le sens s2
	PORTDbits.RD5=s2;
	while(1){//Step
		PORTDbits.RD6 = 1;
		Delay10KTCYx(0);
		PORTDbits.RD6 = 0;
		Delay10KTCYx(0);
		posr2=posr2+1;
	}
}
		

void init(){//initialisation
	if(RC2){
		while(RC0!=1){
			revolt1(1);
		}
		while(RC1!=1){
			revolt2(1);
		}
	}
	posr1=0;//initialiser la position du moteur 1
	posr2=0;//initialiser la position du moteur 1
}

void track(){//suivi
	while(1){
		if(posr1<pos1){
			revolt1(1);
		}
		else if(posr1>pos1){
			revolt1(0);
		}
		if(posr2<pos2){
			revolt2(1);
		}
		else if(posr2>pos2){
			revolt2(0);
		}
	}
}
	
	

void main(){
	
     init_pic();
     init_var();
	while(1){
	init();
	track();
	}
}

invite03481543 · 27/04/2008, 19h09

Envoyé par einstein

un autre petit truc:
j'ai mis les deux fct revolt en commentaire, pour voir s'il y a d'autres erreurs, et apparemment C18 n'accepte pas l'ecriture:

me disant que RC1 n'est pas declaré, comment ecrire cette boucle alors?
while(PORTCbits.RC1!=1) ?
merci

C'est là que tu devrais affecter des define pour nommer tes bits de port, ce serait plus digeste.

#define RC1 PORTCbits.RC1

ensuite tu utilises RC1 c'est tout.

[PIC]erreurs dans un programme

[PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Re : [PIC]erreurs dans un programme

Discussions similaires

Erreurs dans un sujet de maths ??

erreurs dans mon pc

Erreurs dans les éoliennes?

Erreurs Importants dans L'état de ma Connexion

Problème dans un programme de PIC