Arduino : Le navigateur aux commandes de l’Arduino - 2

Code :

1
2
3
4
5
6
7
8
9
10
11
 while (client.available())
      {
        char carLu = client.read();
        if (carLu != 10)
        {
          reception += carLu;
        }
        else
        {
          break;
        }

il vaut mieux éviter les chiffres magique et remplacer 10 directement par '\n' (et mettre un petit commentaire // a-t-on reçu la fin de ligne.)


Cette réception n'est pas correcte en général. Elle fonctionne car vous avez un peu de chance sur le timing.
Il se pourrait que "client.available()" soit vide mais qu'on n'ait pas encore reçu toute la requête si la connexion ethernet est très lente ou le client pas rapide et donc que vous n'ayez pas encore reçu tout le "GET / HTTP/1.1" dans votre chaîne.

Normalement il faut lire jusqu'à une ligne vide (fin de l'en-tête) ou un timeout éventuellement et pas dépendre de la rapidité de vidage du buffer client par rapport à la rapidité de remplissage.

La https://www.arduino.cc/en/Tutorial/WebServer
Vous pourriez prendre la même approche pour lire jusqu'au '\n'.

Cela dit et en référence à votre premier tuto où vous dites --> A noter que la bibliothèque considère qu'un client est toujours connecté même si on a envoyé close tant qu'on n'a pas vidé son buffer (available() n'est pas nul). Le client n'est pas libéré, j'ai jamais regardé en détail mais il y a peut-être une fuite mémoire si on ne vide pas ce buffer.




Quand vous dites:ce n'est pas la faute du développeur. ça vient du fait que les lecteurs n'ont pas pour la plupart le mécanisme de détection d'éjection. il l'a https://github.com/greiman/SdFat/issues/38.




Quand vous faites votre fonction accusé de réception

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
String reception;
boolean flag = true;
while (client.available())
{
  char carLu = client.read();
  if (flag)
  {  
    if (carLu != 10)
    {
      reception += carLu;
    }
    else flag = false;
  }
}

Tout à fait exact : je vais envisager une mise à jour.

Dans la troisième partie, je donne une version qui se passe de la classe String et qui utilise donc les tableaux de caractères. On gagne à peu près 2 Kio de mémoire flash mais on y perd un peu en simplicité. De plus, la mémoire dynamique est impactée, car pour ne pas compliquer encore plus le code avec des réallocations de mémoire, je déclare un tableau de taille fixe, et donc pour être tranquille, supérieur à mes besoins réels. Cela dit, en production, on peut ajuster précisément cette taille.

Bien sûr .

Merci pour vos commentaires,
amicalement,
naute

Code :

 while (client.available()) {

ce n'est pas non plus tout à fait correct. Ce qu'il se passe c'est que pendant que vous videz le buffer en appelant client.read(); il se remplit de l'autre côté potentiellement par interruptions. Si vous veniez à vider le buffer plus vite qu'il ne se remplit la condition de votre while devient fausse et vous sortez sans avoir lu le '\n' potentiellement. En pratique ce sera rare mais ça reste un risque, donc le plus simple serait d'itérer sur "tant que je n'ai pas reçu le '\n' alors si un truc est dispo je le lis"

Oui la classe String apporte quelques simplifications mais si on est habitué au fonctions standard, on retrouve assez facilement ses petits et comme vous le dites on gagne pas mal de place Flash.

Merci encore pour le travail !

1
2
3
void envoiFichier(EthernetClient& nomClient,  const char* fichierEnCours)
{
  if (SD.exists(fichierEnCours)) {....

1
2
3
4
5
6
7
8
9
void envoiFichier(WiFiClient& nomClient,  String fichierEnCours)
{
 
	/*char tableau[fichierEnCours.length() + 1];
	fichierEnCours.toCharArray(tableau, fichierEnCours.length() + 1);*/
	if (SD.exists(fichierEnCours/*tableau*/))
	{
		File fichier = SD.open(fichierEnCours/*tableau*/, FILE_READ);
		while (fichier.available())

Code :

void envoiFichier(EthernetClient& nomClient,  const char* fichierEnCours)

et sans le remettre en String

1
2
3
4
5
6
 
      // Methods to determine if the requested file path exists.
      boolean exists(const char *filepath);
      boolean exists(const String &filepath) {
        return exists(filepath.c_str());
      }