Domo'Duino, la domotique par Arduino (NEWS: Modules 2.4ghz )

Ouais c'est une vrai galère cet ENC28J60 ....

Je pense mettre un terme a mon projet domotique, pour le moment je me vois mal finaliser quelque chose de cohérent ... je n'est pas, seul, les compétence nécessaire a un tel projet ! Je suit en pleine reflexion car ce projet me tenais a coeur mais je n'arrive même pas a passer de 2 à 4 boutons sur mon interface ... et je parle même pas du capteur de température =(

chicotore: (...) j'ai un bon retour d'information sur le serial monitor mais je coince c'est pour l'afficher sur ma page. En effet sur un LCD, il y a la fonction très intuitive qu'est "lcd.print(le nom de la variable);" mais avec l'ethernet-shield comment faire ? (...)

Je n'ai pas l'habitude des shields ETH mais cela me fait penser à un soucis similaire lors d'un affichage via interface RS232; la fonction ?.print(variable) ne fonctionne pas. Il faut passer par une transformation de la variable en chaine afin de pouvoir l'envoyer par le print(). Un peu casse-bonbons et consomme de la ressource, mais à priori pas d'autres solutions...

Bonjour, j'ai suivi un peux ce sujet parce qu'il m’intéresse et là j'ai l'impression que tu met la charrue avant les bœufs A ta place j'aurais suivis les recommandations de ojal

ojal: Je me permets de revenir une fois de plus dans cette discussion... Je ne comprends pas du tout comment ce projet avance... Vous en êtes à essayer un problème très ponctuel d'affichage d'une image alors qu'aucune base n'a été définie pour ce projet de domotique... Une fois de plus, je préconise d'établir un cahier des charges, définir des orientations d'architecture etc etc...

Qu'en pensez-vous?

chicotore: Mon cahier des charges se définie sur le tas suivant ce que j'arrive a mettre en place et ce que je n'arrive pas a faire !

Le plus gros du problème ce situe là et reprendre le code (de plus complexe et imbuvable) de quelqu'un d'autre est une mauvaise idée quand on ne maitrise pas un peux le sujet,limite ça peut juste servir d'exemple pour mieux comprendre. Je sais que c'est pénible et qu'on a envie de tout avoir tout de suite avec un minimum d’investissement mais au final ça ne marchera pas, la preuve en est le résultat.

Prend ton temps, commence par maitriser un peux mieux le langage c (ou tout autre langage procédurale), les algorithmes de base et après tout te paraitra plus simple et clair, tu te demanderas même comment tu a pus faire certaine erreurs sur des chose pourtant très basiques et banales.

Je préféré être franc, pour l'instant tu perd ton temps :~ . Il est préférable de perdre 1 mois à ton aise (sur le site du zéro par ex) avec les bases plutôt que d'avancé et galérer 6 mois pour un résultat qui marchera peut être mais qui sera bancal, sans possibilité d'évolution (dépendances), bugger et non optimal.

Entre () j'ai aussi un projet domotique mais je pencherais plutôt sur une solution pages web hébergées délocalisée,ajax (XMLHttpRequest) ,serveur et contrôleur arduino + protocole maison genre x10 en hexa pour la communication et commandes entre le serveur arduino et le client (web). Ajax sera remplacé à terme par les websocket (standardisé) histoire de pouvoir notifié au client (l'explorateur) d'un changement d'état d'un capteur ou autre (impossible sans rafraichissement de la page par ex pour ton problème avec le lm35). J'ai déjà développé une application modulaire (plcbus, 1wire, ...) serveur (socket) de domotique en java donc ça ne devrait pas être trop compliqué d'appliqué les même principe à l'arduino.

Désolé si je t'ai découragé ou autre mais ce ne serait pas te rendre service que de t'aider à "débugger" ton code.

ekaki: Je n'ai pas l'habitude des shields ETH mais cela me fait penser à un soucis similaire lors d'un affichage via interface RS232; la fonction ?.print(variable) ne fonctionne pas. Il faut passer par une transformation de la variable en chaine afin de pouvoir l'envoyer par le print(). Un peu casse-bonbons et consomme de la ressource, mais à priori pas d'autres solutions...

normal non ?

http://arduino.cc/en/Serial/Print

print() Description

Prints data to the serial port as human-readable ASCII text ...

Serial.print fonctionne mais pas client.print ;)

Fin du projet

Moi je dis repose toi :sleeping:, étudie un peut tout ça ]:D et plus tard repart sur de bonne base :%.

Sur ebay j'ai prit le moins chère, 10€ ethernet-shield chinois

Oui mais vrai casse tête chinois pour programmation petit scarabé !

Regarder lien là : http://www.ordinoscope.net/index.php/Electronique/Hardware/Communication/ENC28J60

A+ petit scarabé Maître chabot.

Je me permet d’insister lourdement sur l’existence d’une nouvelle librairie pour Ethershield
développée par Turicas (GitHub - turicas/Ethernet_ENC28J60: [NOT MAINTAINED, NOT COMPLETED] Implementation of an Arduino-compatible socket layer library that uses Microchip ENC28J60 Ethernet controller.).
Elle n’est pas finalisée mais déjà opérationnelle.
C’est une Librairie Ethernet pour Ethershield !
On utilise une Ethershield comme si c’était un shield (officiel) à base de W5100… avec le confort de codage qui va avec.
Un coup d’oeil à l’exemple (un des 7 exemples fournis avec) ci dessous montre à mon avis que le résultat est déjà intéressant. On croirait voir un programme ecrit pour la carte 'officielle’.
Les ‘client.print’ n’y manquent pas… on y voit également l’affichage de valeurs prises sur des entrées analogiques (pourquoi pas un LM35 ?)

Avant de jeter éventuellement l’éponge ton projet mérite sans doute d’aller faire un tour de piste avec cette librairie. (Il peut etre nécessaire de modifier la ligne 39 du fichier “enc28j60.c” pour tenir compte du câblage de la broche CS sur la carte utilisée)

WebServerSimpleLED.pde (2.23 KB)

J’ai essayer la nouvelle librairie dès que tu l’as posté ici mais chez moi ça ne fonctionne pas.

#include <Ethernet.h>
#define LED 8

byte mac[] = { 0x54, 0x55, 0x58, 0x10, 0x00, 0x24 };
byte ip[] = {192,168,1,15};
char link[30], LEDStatus[4];

Server server(80);

void setup() {
  pinMode(LED, OUTPUT);
  Ethernet.begin(mac, ip);
  server.begin();
}

void loop() {
  Client client = server.available();
  if (client) {
    char request[10];
    int i = 0;
    boolean current_line_is_blank = true;
    request[9] = '\0';
    while (client.connected()) {
      if (client.available()) {
        char c = client.read();
        if (i < 9) {
          request[i] = c;
          i++;
        }
        if (c == '\n' && current_line_is_blank) {
          client.println("HTTP/1.1 200 OK");
          client.println("Content-Type: text/html");
          client.println();
          
          client.println("<html><head><title>Arduino etherShield</title></head><body>");
          client.println("<h1>Arduino etherShield Example</h1>");
          client.println("<table>");
          client.print("<tr><td>Analog inputs:</td><td><b>");
          for (int x = 0; x < 6; x++) {
            client.print(x);
            client.print(" = ");
            client.print(5000 * (analogRead(x) / 1024.0));
            client.print(" mV
");
          }
          client.println("</b></td></tr><tr><td>uptime:</td> <td><b>");
          client.print(millis() / 1000.0);
          client.println(" seconds</b></td></tr>");
          if (strncmp("GET /off", request, 8) == 0) {
            digitalWrite(LED, LOW);
            sprintf(LEDStatus, "OFF");
            sprintf(link, "<a href=\"/on\">Turn on</a>");
          }
          else {
            sprintf(LEDStatus, "ON");
            digitalWrite(LED, HIGH);
            sprintf(link, "<a href=\"/off\">Turn off</a>");
          }
          client.print("<tr><td>LED status:</td> <td><b>");
          client.print(LEDStatus);
          client.println("</b></td></tr><tr><td colspan=\"2\" align=\"center\">");
          client.println(link);
          client.println("</td></tr></table></body></html>");
          break;
        }
        if (c == '\n') {
          current_line_is_blank = true;
        } else if (c != '\r') {
          current_line_is_blank = false;
        }
      }
    }
    delay(1);
    client.stop();
  }
}

Impossible d’accéder au serveur …

(Il peut etre nécessaire de modifier la ligne 39 du fichier “enc28j60.c” pour tenir compte du câblage de la broche CS sur la carte utilisée)

Comment connaitre cette broche ? si cette librairie est faite pour le shield en enc28j60 la broche devrais être la bonne non ?

cette librairie reprend le câble normal d'une carte Ethershield non modifiée = CS en pin10 (pin 53 pour carte Mega). Je n'avais pas eu de probleme avec un 28J60 monté sur plaque d'essai. Dès que possible ( en soirée) je regarde à nouveau de près. Le déplacement du CS commence à intervenir quand il y a conflit avec d'autres composants SPI (carte SD...)

Je suis vraiment désolé pour toi que tu bloques totalement... personnellement je suis aussi sur des petits projets similaire pour contrôler à distance mon arduino. mais en vus que tu utilises une version chinoise de l'ethernet shield il est difficile pour moi d'aider. Et si je peux me permettre, oui la version officiel coûte plus cher mais tu aurais certainement eu moins de trouble et peut-être plus de facilité à avoir de l'aide :/ Je reste aussi sur le faite que l'arduino n'est vraiment pas adapter pour générer des pages html.

Je comprend vraiment pas ta façon de fonctionné, tu marche en aveugle là et au risque de me répéter tu met la charrue avant le bœufs tu n’es même pas certain que ton serveur marche que tu y ajoute la gestion de tes leds qui pourrait parasiter le code serveur …
Commence par des chose simple

teste si une connexion est possible d’abord, tu pourras déjà savoir si c’est un problème niveau configuration, matériel ou autre …

#include <Ethernet.h>

byte mac[] = { 0x54, 0x55, 0x58, 0x10, 0x00, 0x24 };
byte ip[] = {192,168,1,15};

Server server(80);

void setup() 
{
	Ethernet.begin(mac, ip);
	server.begin();
}

void loop() 
{
	Client client = server.available();
	if (client) 
	{
		Serial.print("client");
                
		delay(1);
		client.stop();
	}
}

Si c’est ok teste l’envoi de donnée au client …

#include <Ethernet.h>

byte mac[] = { 0xDE, 0xAD, 0xBE, 0xEF, 0xFE, 0xED };
byte ip[] = { 192, 168, 1, 177 };

Server server(80);

void setup() 
{
	Ethernet.begin(mac, ip);
	server.begin();
}

void loop() 
{
	Client client = server.available();
	if (client) 
	{
		boolean current_line_is_blank = true;
		while (client.connected()) 
		{
			if (client.available()) 
			{
				char c = client.read();
				if (c == '\n' && current_line_is_blank) 
				{
					client.println("HTTP/1.1 200 OK");
					client.println("Content-Type: text/html");
					client.println();
					
					client.print("hello world");
					break;
				}
				if (c == '\n') 
				{
					current_line_is_blank = true;
				} 
				else if (c != '\r') 
				{
					current_line_is_blank = false;
				}
			}
		}
		delay(1);
		client.stop();
	}
}

ensuite teste la réception de données …

#include <Ethernet.h>

byte mac[] = { 0x54, 0x55, 0x58, 0x10, 0x00, 0x24 };
byte ip[] = {192,168,1,15};

Server server(80);

void setup() 
{
	Ethernet.begin(mac, ip);
	server.begin();
}

void loop() 
{
	Client client = server.available();
	if (client) 
	{
		while(client.connected()) 
		{
			if(client.available()) 
			{
				Serial.print(client.read());
			}
		}
		delay(1);
		client.stop();
	}
}

etc, etc, tec, …

Edit: je viens de voire qu’il y a un exemple en mode debug … commence par là, tu pourras voire dans ta console ce qui marche ou pas …

C'est vrai qu'il aurait peut-etre été mieux de réaliser un site complet sur un serveur en ligne comme les FTP free et d'envoyer des ordres a l'ethernet-shield comme je faisait au début par le biais de processing ...

Enfin bref j'abandonne ;) bientôt mon matos sera en vente sur ebay ou ici ( duemilanove, ethernetshield, relais, lcd ... )

Si tu es un peu patient (une journée) j'ai fait un tutoriel, qui est correction orthographique :p qui explique comment communiquer arduino grâce à un site internet. je fournis les explications et les fichiers dans le tutoriel. C'est quand même dommage de stopper au points de revendre ton matériel :/

@chicotore
Comme indiqué je viens de réinstaller la librairie de Turicas (qui je le rappelle est en chantier)
Les exemples WebServer et WebserverSimple sont OK du premier coup sans retouches :).
L’exemple WebServerSimpleLED ne marche pas tel quel :frowning:
Pour le faire marcher (photo jointe) j’ai :
-augmenté un peu le buffer défini dans /utility/socket.#define BUFFER_SIZE   650
-raccourci un peu l’html envoyé en jouant soit sur le titre, sur le H1…ou autre chose de non essentiel
(il n’est pas sûr osaka que le mode debug soit opérationnel)

hardware : mega328 et enc28j60 tous les deux en 3,3V (pas d’adaptation de tension, mais câblage identique à l’EtherShield)

webled.jpg

Non tu n'arrètes pas! Le développement c'est ça : toujours dans les emmerdes, mais quel plaisir et quelle satisfaction quand tu trouves la solution !

Comme beaucoup te l'ont dit reprends ton projet à zéro, mets tout sur la table et tries. Choisis des objectifs simples et atteignables, il est plus facile de monter un escalier de 20 marches de 10 cm qu'un escalier de 5 marches de 40 cm ! Je sais ça parrait être du temps perdu que de se limiter à de tout petits tests alors qu'on pourrait tout faire en une seule fois sauf que la deuxièmme solution mêne souvant à l'échec.

Il faut aussi que tu détermines si des applications auront besoin de matériels spécifiques (électronique et/ou mécanique), cette détermination définira un budjet.

Ton projet est ambitieux, peut être faut-il le mener à plusieurs ? Mais si veut trouver des volontaires pour te suivre il faut avoir une feuille de route et non pas partir au grès du vent.

Bon courage

Par curiosité est ce que ça marche, en adaptant l’ip et la baseurl évidement ?

#include "etherShield.h"


// please modify the following two lines. mac and ip have to be unique
// in your local area network. You can not have the same numbers in
// two devices:
static uint8_t mymac[6] = {0x54,0x55,0x58,0x10,0x00,0x24}; 
static uint8_t myip[4] = {1,1,1,1};
static char baseurl[]="http://xxxxxxx.dyndns-home.com/";
static uint16_t mywwwport =80; // listen port for tcp/www (max range 1-254)



#define BUFFER_SIZE 500
static uint8_t buf[BUFFER_SIZE+1];
#define STR_BUFFER_SIZE 22
static char strbuf[STR_BUFFER_SIZE+1];

EtherShield es=EtherShield();

// prepare the webpage by writing the data to the tcp send buffer
uint16_t print_webpage(uint8_t *buf, byte on_off, byte on_off2);
int8_t analyse_cmd(char *str);

// LED cathode connects the Pin4, anode to 5V through 1K resistor
#define LED_PIN  4
#define LED_PIN2  5


void setup(){
  
   /*initialize enc28j60*/
	 es.ES_enc28j60Init(mymac);
   es.ES_enc28j60clkout(2); // change clkout from 6.25MHz to 12.5MHz
   delay(10);
        
	/* Magjack leds configuration, see enc28j60 datasheet, page 11 */
	// LEDA=greed LEDB=yellow
	//
	// 0x880 is PHLCON LEDB=on, LEDA=on
	// enc28j60PhyWrite(PHLCON,0b0000 1000 1000 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x880);
	delay(500);
	//
	// 0x990 is PHLCON LEDB=off, LEDA=off
	// enc28j60PhyWrite(PHLCON,0b0000 1001 1001 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x990);
	delay(500);
	//
	// 0x880 is PHLCON LEDB=on, LEDA=on
	// enc28j60PhyWrite(PHLCON,0b0000 1000 1000 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x880);
	delay(500);
	//
	// 0x990 is PHLCON LEDB=off, LEDA=off
	// enc28j60PhyWrite(PHLCON,0b0000 1001 1001 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x990);
	delay(500);
	//
  // 0x476 is PHLCON LEDA=links status, LEDB=receive/transmit
  // enc28j60PhyWrite(PHLCON,0b0000 0100 0111 01 10);
  es.ES_enc28j60PhyWrite(PHLCON,0x476);
	delay(100);
        
  //init the ethernet/ip layer:
  es.ES_init_ip_arp_udp_tcp(mymac,myip,80);
  
 	pinMode(LED_PIN, OUTPUT); 
 	digitalWrite(LED_PIN, LOW);  // switch on LED
         pinMode(LED_PIN2, OUTPUT); 
 	digitalWrite(LED_PIN2, LOW); 
}

void loop(){
  uint16_t plen, dat_p;
  int8_t cmd;
  byte on_off = 1;
  byte on_off2 = 1;

  plen = es.ES_enc28j60PacketReceive(BUFFER_SIZE, buf);

	/*plen will ne unequal to zero if there is a valid packet (without crc error) */
  if(plen!=0){
	           
    // arp is broadcast if unknown but a host may also verify the mac address by sending it to a unicast address.
    if(es.ES_eth_type_is_arp_and_my_ip(buf,plen)){
      es.ES_make_arp_answer_from_request(buf);
      return;
    }

    // check if ip packets are for us:
    if(es.ES_eth_type_is_ip_and_my_ip(buf,plen)==0){
      return;
    }
    
    if(buf[IP_PROTO_P]==IP_PROTO_ICMP_V && buf[ICMP_TYPE_P]==ICMP_TYPE_ECHOREQUEST_V){
      es.ES_make_echo_reply_from_request(buf,plen);
      return;
    }
    
    // tcp port www start, compare only the lower byte
    if (buf[IP_PROTO_P]==IP_PROTO_TCP_V&&buf[TCP_DST_PORT_H_P]==0&&buf[TCP_DST_PORT_L_P]==mywwwport){
      if (buf[TCP_FLAGS_P] & TCP_FLAGS_SYN_V){
         es.ES_make_tcp_synack_from_syn(buf); // make_tcp_synack_from_syn does already send the syn,ack
         return;     
      }
      if (buf[TCP_FLAGS_P] & TCP_FLAGS_ACK_V){
        es.ES_init_len_info(buf); // init some data structures
        dat_p=es.ES_get_tcp_data_pointer();
        if (dat_p==0){ // we can possibly have no data, just ack:
          if (buf[TCP_FLAGS_P] & TCP_FLAGS_FIN_V){
            es.ES_make_tcp_ack_from_any(buf);
          }
          return;
        }
        if (strncmp("GET ",(char *)&(buf[dat_p]),4)!=0){
          	// head, post and other methods for possible status codes see:
            // http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
            plen=es.ES_fill_tcp_data_p(buf,0,PSTR("HTTP/1.0 200 OK\r\nContent-Type: text/html\r\n\r\n<h1>200 OK</h1>"));
            goto SENDTCP;
        }
 	if (strncmp("/ ",(char *)&(buf[dat_p+4]),2)==0){
                plen=print_webpage(buf, on_off, on_off2);
            goto SENDTCP;
         }
        cmd=analyse_cmd((char *)&(buf[dat_p+5]));
        
        if (cmd==2){
                on_off=1;
        	digitalWrite(LED_PIN, LOW);  // switch on LED
        }
        else if (cmd==3){
                on_off=0;
        	digitalWrite(LED_PIN, HIGH);  // switch off LED
        }
        else if (cmd==4){
                on_off2=1;
        	digitalWrite(LED_PIN2, LOW);  // switch on LED
        }
        else if (cmd==5){
                on_off2=0;
        	digitalWrite(LED_PIN2, HIGH);  // switch off LED
        }
        
        plen=print_webpage(buf, on_off, on_off2);
        	
//        	   plen=print_webpage(buf, on_off);
SENDTCP:  es.ES_make_tcp_ack_from_any(buf); // send ack for http get
           es.ES_make_tcp_ack_with_data(buf,plen); // send data       
      }
    }
  }
        
}
// The returned value is stored in the global var strbuf
uint8_t find_key_val(char *str,char *key)
{
        uint8_t found=0;
        uint8_t i=0;
        char *kp;
        kp=key;
        while(*str &&  *str!=' ' && found==0){
                if (*str == *kp){
                        kp++;
                        if (*kp == '\0'){
                                str++;
                                kp=key;
                                if (*str == '='){
                                        found=1;
                                }
                        }
                }else{
                        kp=key;
                }
                str++;
        }
        if (found==1){
                // copy the value to a buffer and terminate it with '\0'
                while(*str &&  *str!=' ' && *str!='&' && i<STR_BUFFER_SIZE){
                        strbuf[i]=*str;
                        i++;
                        str++;
                }
                strbuf[i]='\0';
        }
        return(found);
}

int8_t analyse_cmd(char *str)
{
        int8_t r=-1;
     
        if (find_key_val(str,"cmd")){
                if (*strbuf < 0x3a && *strbuf > 0x2f){
                        // is a ASCII number, return it
                        r=(*strbuf-0x30);
                }
        }
        return r;
}


uint16_t print_webpage(uint8_t *buf, byte on_off, byte on_off2)
{

       int i=0;
    
        uint16_t plen;
        
        plen=es.ES_fill_tcp_data_p(buf,0,PSTR("HTTP/1.0 200 OK\r\nContent-Type: text/html\r\n\r\n"));
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<body bgcolor=\"#EFECCA\"> "));
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<center><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/chambr10.png\"  >"));
 
      	plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<center><a href=\""));
        plen=es.ES_fill_tcp_data(buf,plen,baseurl);
        
        if(on_off)
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=2\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/on11.png\"></a>"));
        else
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=3\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/off12.png\"></a>"));
        
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<center><a href=\""));
        plen=es.ES_fill_tcp_data(buf,plen,baseurl);
                
        if(on_off2)
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=4\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/on11.png\"></a>"));
        else
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=5\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/off12.png\"></a>"));
        
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("</body> "));
                
        return(plen);
}

Avec ton code j’obtient donc 2 boutons cliquables mais en fait même en cliquant dessus ils restent toujours à la valeur (cmd=2 et cmd=4) donc toujours ON et l’image qui va avec :roll_eyes:

merci a toi

Les variables on_off sont mal positionnée il me semble elle sont local à la fonction loop (j’avais repris ton dernier code fonctionnel) donc réinitialisé à chaque boucle à 1, faut les mettre en global en dehors de toute méthodes (avant setup).

byte on_off = 1;
byte on_off2 = 1;

version corrigé

#include "etherShield.h"


// please modify the following two lines. mac and ip have to be unique
// in your local area network. You can not have the same numbers in
// two devices:
static uint8_t mymac[6] = {0x54,0x55,0x58,0x10,0x00,0x24}; 
static uint8_t myip[4] = {xxx,xxx,x,xx};
static char baseurl[]="http://xxxxxxx.dyndns-home.com/";
static uint16_t mywwwport =80; // listen port for tcp/www (max range 1-254)



#define BUFFER_SIZE 500
static uint8_t buf[BUFFER_SIZE+1];
#define STR_BUFFER_SIZE 22
static char strbuf[STR_BUFFER_SIZE+1];

EtherShield es=EtherShield();

// prepare the webpage by writing the data to the tcp send buffer
uint16_t print_webpage(uint8_t *buf, byte on_off, byte on_off2);
int8_t analyse_cmd(char *str);

// LED cathode connects the Pin4, anode to 5V through 1K resistor
#define LED_PIN  4
#define LED_PIN2  5


byte on_off = 1;
byte on_off2 = 1;

void setup(){
  
   /*initialize enc28j60*/
	 es.ES_enc28j60Init(mymac);
   es.ES_enc28j60clkout(2); // change clkout from 6.25MHz to 12.5MHz
   delay(10);
        
	/* Magjack leds configuration, see enc28j60 datasheet, page 11 */
	// LEDA=greed LEDB=yellow
	//
	// 0x880 is PHLCON LEDB=on, LEDA=on
	// enc28j60PhyWrite(PHLCON,0b0000 1000 1000 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x880);
	delay(500);
	//
	// 0x990 is PHLCON LEDB=off, LEDA=off
	// enc28j60PhyWrite(PHLCON,0b0000 1001 1001 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x990);
	delay(500);
	//
	// 0x880 is PHLCON LEDB=on, LEDA=on
	// enc28j60PhyWrite(PHLCON,0b0000 1000 1000 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x880);
	delay(500);
	//
	// 0x990 is PHLCON LEDB=off, LEDA=off
	// enc28j60PhyWrite(PHLCON,0b0000 1001 1001 00 00);
	es.ES_enc28j60PhyWrite(PHLCON,0x990);
	delay(500);
	//
  // 0x476 is PHLCON LEDA=links status, LEDB=receive/transmit
  // enc28j60PhyWrite(PHLCON,0b0000 0100 0111 01 10);
  es.ES_enc28j60PhyWrite(PHLCON,0x476);
	delay(100);
        
  //init the ethernet/ip layer:
  es.ES_init_ip_arp_udp_tcp(mymac,myip,80);
  
 	pinMode(LED_PIN, OUTPUT); 
 	digitalWrite(LED_PIN, LOW);  // switch on LED
         pinMode(LED_PIN2, OUTPUT); 
 	digitalWrite(LED_PIN2, LOW); 
}

void loop(){
  uint16_t plen, dat_p;
  int8_t cmd;

  plen = es.ES_enc28j60PacketReceive(BUFFER_SIZE, buf);

	/*plen will ne unequal to zero if there is a valid packet (without crc error) */
  if(plen!=0){
	           
    // arp is broadcast if unknown but a host may also verify the mac address by sending it to a unicast address.
    if(es.ES_eth_type_is_arp_and_my_ip(buf,plen)){
      es.ES_make_arp_answer_from_request(buf);
      return;
    }

    // check if ip packets are for us:
    if(es.ES_eth_type_is_ip_and_my_ip(buf,plen)==0){
      return;
    }
    
    if(buf[IP_PROTO_P]==IP_PROTO_ICMP_V && buf[ICMP_TYPE_P]==ICMP_TYPE_ECHOREQUEST_V){
      es.ES_make_echo_reply_from_request(buf,plen);
      return;
    }
    
    // tcp port www start, compare only the lower byte
    if (buf[IP_PROTO_P]==IP_PROTO_TCP_V&&buf[TCP_DST_PORT_H_P]==0&&buf[TCP_DST_PORT_L_P]==mywwwport){
      if (buf[TCP_FLAGS_P] & TCP_FLAGS_SYN_V){
         es.ES_make_tcp_synack_from_syn(buf); // make_tcp_synack_from_syn does already send the syn,ack
         return;     
      }
      if (buf[TCP_FLAGS_P] & TCP_FLAGS_ACK_V){
        es.ES_init_len_info(buf); // init some data structures
        dat_p=es.ES_get_tcp_data_pointer();
        if (dat_p==0){ // we can possibly have no data, just ack:
          if (buf[TCP_FLAGS_P] & TCP_FLAGS_FIN_V){
            es.ES_make_tcp_ack_from_any(buf);
          }
          return;
        }
        if (strncmp("GET ",(char *)&(buf[dat_p]),4)!=0){
          	// head, post and other methods for possible status codes see:
            // http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
            plen=es.ES_fill_tcp_data_p(buf,0,PSTR("HTTP/1.0 200 OK\r\nContent-Type: text/html\r\n\r\n<h1>200 OK</h1>"));
            goto SENDTCP;
        }
 	if (strncmp("/ ",(char *)&(buf[dat_p+4]),2)==0){
                plen=print_webpage(buf, on_off);
            goto SENDTCP;
         }
        cmd=analyse_cmd((char *)&(buf[dat_p+5]));
        
        if (cmd==2){
                on_off=1;
        	digitalWrite(LED_PIN, LOW);  // switch on LED
        }
        else if (cmd==3){
                on_off=0;
        	digitalWrite(LED_PIN, HIGH);  // switch off LED
        }
        else if (cmd==4){
                on_off=1;
        	digitalWrite(LED_PIN2, LOW);  // switch on LED
        }
        else if (cmd==5){
                on_off=0;
        	digitalWrite(LED_PIN2, HIGH);  // switch off LED
        }
        
        plen=print_webpage(buf, on_off);
        	
//        	   plen=print_webpage(buf, on_off);
SENDTCP:  es.ES_make_tcp_ack_from_any(buf); // send ack for http get
           es.ES_make_tcp_ack_with_data(buf,plen); // send data       
      }
    }
  }
        
}
// The returned value is stored in the global var strbuf
uint8_t find_key_val(char *str,char *key)
{
        uint8_t found=0;
        uint8_t i=0;
        char *kp;
        kp=key;
        while(*str &&  *str!=' ' && found==0){
                if (*str == *kp){
                        kp++;
                        if (*kp == '\0'){
                                str++;
                                kp=key;
                                if (*str == '='){
                                        found=1;
                                }
                        }
                }else{
                        kp=key;
                }
                str++;
        }
        if (found==1){
                // copy the value to a buffer and terminate it with '\0'
                while(*str &&  *str!=' ' && *str!='&' && i<STR_BUFFER_SIZE){
                        strbuf[i]=*str;
                        i++;
                        str++;
                }
                strbuf[i]='\0';
        }
        return(found);
}

int8_t analyse_cmd(char *str)
{
        int8_t r=-1;
     
        if (find_key_val(str,"cmd")){
                if (*strbuf < 0x3a && *strbuf > 0x2f){
                        // is a ASCII number, return it
                        r=(*strbuf-0x30);
                }
        }
        return r;
}


uint16_t print_webpage(uint8_t *buf, byte on_off, byte on_off2)
{

       int i=0;
    
        uint16_t plen;
        
        plen=es.ES_fill_tcp_data_p(buf,0,PSTR("HTTP/1.0 200 OK\r\nContent-Type: text/html\r\n\r\n"));
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<body bgcolor=\"#EFECCA\"> "));
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<center><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/chambr10.png\"  >"));
 
      	plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("<center><a href=\"");
        plen=es.ES_fill_tcp_data(buf,plen,baseurl);
        
        if(on_off)
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=2\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/on11.png\"></a>");
            
        else
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=3\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/off12.png\"></a>");
                
        if(on_off2)
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=4\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/on11.png\"></a>");
            
        else
                plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("?cmd=5\"><img src=\"http://i45.servimg.com/u/f45/11/05/63/75/off12.png\"></a>");
        
        plen=es.ES_fill_tcp_data_p(buf,plen,PSTR("</body> "));
                
        return(plen);
}

petite explication:

Il faut voire la fonction loop comme une boucle infinie

while(true) //la condition est toujours vraie donc pas de sortie de boucle.
{
}

ici ta variable initialisée à 1 le sera à chaque entrée de boucle.

while(true)
{
    byte on_off = 1;
    on_off+=1;
}

Bon alors avec ton code cela ne fonctionne pas, voila l’erreur …

J’ai donc repris le premier code que tu m’avait donné mais ou l’image ne changeait pas et j’ai déplacer les “byte on_off et byte on_off2” comme cela

// LED cathode connects the Pin4, anode to 5V through 1K resistor
#define LED_PIN  4
#define LED_PIN2  5

byte on_off = 1;
byte on_off2 = 1;

La le compilateur ne détecte pas d’erreur (bizarre d’ailleurs car c’est la même chose que ton code non ? :astonished: j’ai pas trouver d’autres différences que le déplacement de ces deux choses ), mais ça ne fonctionne pas quand je veut accéder a l’arduino par internet … “serveur introuvable”