Arduino wifi

Hello, i’m a french student and for my studies i need to control the position of a servomotor. So i’ve done tutorials to learn how works the wifi shield. I’ve found a code in a tutorial on internet and i’ve complete it to control the servo on internet, but it doesn’t work. Can somebody help me this my code :

#include <Servo.h>

#include <SPI.h>
#include <WiFi.h>
#include <WiFiClient.h>
#include <WiFiServer.h>

Servo servo1; // servo control object

char ssid = “myssid”; // your network SSID (name)
char pass = “mypassword”; // your network password
int keyIndex = 0; // your network key Index number (needed only for WEP)

int status = WL_IDLE_STATUS;
WiFiServer server(80);

void setup() {
Serial.begin(9600); // initialize serial communication
servo1.attach(9);

// check for the presence of the shield:
if (WiFi.status() == WL_NO_SHIELD) {
Serial.println(“WiFi shield not present”);
while(true); // don’t continue
}

// attempt to connect to Wifi network:
while ( status != WL_CONNECTED) {
Serial.print("Attempting to connect to Network named: ");
Serial.println(ssid); // print the network name (SSID);

// Connect to WPA/WPA2 network. Change this line if using open or WEP network:
status = WiFi.begin(ssid, pass);
// wait 10 seconds for connection:
delay(10000);
}
server.begin(); // start the web server on port 80
printWifiStatus(); // you’re connected now, so print out the status
}

void loop() {

WiFiClient client = server.available(); // listen for incoming clients

if (client) { // if you get a client,
Serial.println(“new client”); // print a message out the serial port
String currentLine = “”; // make a String to hold incoming data from the client
while (client.connected()) { // loop while the client’s connected
if (client.available()) { // if there’s bytes to read from the client,
char c = client.read(); // read a byte, then
Serial.write(c); // print it out the serial monitor
if (c == ‘\n’) { // if the byte is a newline character

// if the current line is blank, you got two newline characters in a row.
// that’s the end of the client HTTP request, so send a response:
if (currentLine.length() == 0) {
// HTTP headers always start with a response code (e.g. HTTP/1.1 200 OK)
// and a content-type so the client knows what’s coming, then a blank line:
client.println(“HTTP/1.1 200 OK”);
client.println(“Content-type:text/html”);
client.println();

// the content of the HTTP response follows the header:
client.print(“Click <a href=”/H">here verrouiller
“);
client.print(“Click <a href=”/L”>here deverouiller
");

// The HTTP response ends with another blank line:
client.println();
// break out of the while loop:
break;
}
else { // if you got a newline, then clear currentLine:
currentLine = “”;
}
}
else if (c != ‘\r’) { // if you got anything else but a carriage return character,
currentLine += c; // add it to the end of the currentLine
}

// Check to see if the client request was “GET /H” or “GET /L”:
if (currentLine.endsWith(“GET /H”)) {
servo1.write(0); // GET /H pour ordonner au servo d’aller à la position 0
}
if (currentLine.endsWith(“GET /L”)) {
servo1.write(180); // GET /L pour ordonner au servo d’aller à la position 180
}
}
}
delay(1);
// close the connection:
client.stop();
Serial.println(“client disonnected”);
}
}

void printWifiStatus() {
// print the SSID of the network you’re attached to:
Serial.print("SSID: ");
Serial.println(WiFi.SSID());

// print your WiFi shield’s IP address:
IPAddress ip = WiFi.localIP();
Serial.print("IP Address: ");
Serial.println(ip);

// print the received signal strength:
long rssi = WiFi.RSSI();
Serial.print(“signal strength (RSSI):”);
Serial.print(rssi);
Serial.println(" dBm");
// print where to go in a browser:
Serial.print(“To see this page in action, open a browser to http://”);
Serial.println(ip);
}

Thanks !
(PS : Before this code i’ve done other tutorials and they worked: i’ve post an analog value on my website, so the problem is not the connexion. Also this programm worked once, not the first time but after some test, but now it doesn’t work anymore)

Please define "does not work." Did it move the servo but to the wrong position? Did it not move at all? Did it resurrect the ghost of Napoleon? What did it do or not do and how did that compare to what you expected?

I notice that you are reading char at a time into a String called currentLine. But this string is local to the loop function, so it is being destroyed and recreated each time the loop goes back around. It might work better if that String was global.

On a side note, you really shouldn't be using the String class on a microcontroller. String wastes resources and memory which the micro has very little of to begin with.

Learn to use c-strings. Null terminated char arrays.

He is not moving at all ? I understand why i should'nt use the function String, but what can i use if i don't use the function String ? Thanks for your answer i'm going to try when the String is global.

Valobal06: I understand why i should'nt use the function String, but what can i use if i don't use the function String ?

I told you, c-strings. Null terminated char arrays.

Thanks ! I've try once again : the servomotor turn once but after it won't work ^^

Post the new code. This time, use the little icon up above the smileys that has the <> symbol on it.

That will put the code in code tags and make it a lot easier to read and work with.

This is the new code that i’ve used

#include <Servo.h>
#include <SPI.h>
#include <WiFi.h>
#include <WiFiClient.h>
#include <WiFiServer.h>

String currentLine = "";      // make a String to hold incoming data from the client

Servo servo1; // servo control object

char ssid[] = "myssid";      //  your network SSID (name) 
char pass[] = "mypassword";   // your network password
int keyIndex = 0;                 // your network key Index number (needed only for WEP)

int status = WL_IDLE_STATUS;
WiFiServer server(80);

void setup() {
  Serial.begin(9600);      // initialize serial communication
 servo1.attach(9);     

  // check for the presence of the shield:
  if (WiFi.status() == WL_NO_SHIELD) {
  Serial.println("WiFi shield not present"); 
   while(true);        // don't continue
   } 

   // attempt to connect to Wifi network:
  while ( status != WL_CONNECTED) { 
    Serial.print("Attempting to connect to Network named: ");
    Serial.println(ssid);                   // print the network name (SSID);

    // Connect to WPA/WPA2 network. Change this line if using open or WEP network:    
     status = WiFi.begin(ssid, pass);
     // wait 10 seconds for connection:
     delay(10000);
   } 
  server.begin();                           // start the web server on port 80
  printWifiStatus();        // you're connected now, so print out the        status
}


void loop() {


  WiFiClient client = server.available();   // listen for incoming clients

  if (client) {                             // if you get a client,
    Serial.println("new client");           // print a message out the serial port
   
    while (client.connected()) {            // loop while the client's connected
     if (client.available()) {             // if there's bytes to read from the client,
     char c = client.read();             // read a byte, then
     Serial.write(c);                    // print it out the serial monitor
     if (c == '\n') {                    // if the byte is a newline character

      // if the current line is blank, you got two newline characters in a row.
      // that's the end of the client HTTP request, so send a response:
      if (currentLine.length() == 0) {  
        // HTTP headers always start with a response code (e.g. HTTP/1.1 200 OK)
        // and a content-type so the client knows what's coming, then a blank line:    
        client.println("HTTP/1.1 200 OK");
        client.println("Content-type:text/html");
        client.println();

        // the content of the HTTP response follows the header:
        client.print("Click <a href=\"/H\">here</a> verrouiller 
");
        client.print("Click <a href=\"/L\">here</a> deverouiller
");

        // The HTTP response ends with another blank line:
        client.println();
        // break out of the while loop:
        break;         
       } 
       else {      // if you got a newline, then clear currentLine:
         currentLine = "";
       }
     }     
     else if (c != '\r') {  // if you got anything else but a carriage return character,
      currentLine += c;      // add it to the end of the currentLine
    }

    // Check to see if the client request was "GET /H" or "GET /L":
    if (currentLine.endsWith("GET /H")) {
      servo1.write(0);               // GET /H pour ordonner au servo d'aller à la position 0
    }
    if (currentLine.endsWith("GET /L")) {
      servo1.write(180);                // GET /L pour ordonner au servo d'aller à la position 180
     }
   }
 }
 delay(1);
// close the connection:
client.stop();
Serial.println("client disonnected");
  }
}

void printWifiStatus() {
  // print the SSID of the network you're attached to:
  Serial.print("SSID: ");
 Serial.println(WiFi.SSID());

 // print your WiFi shield's IP address:
 IPAddress ip = WiFi.localIP();
 Serial.print("IP Address: ");
 Serial.println(ip);

 // print the received signal strength:
 long rssi = WiFi.RSSI();
 Serial.print("signal strength (RSSI):");
 Serial.print(rssi);
 Serial.println(" dBm");
// print where to go in a browser:
Serial.print("To see this page in action, open a browser to http://");
Serial.println(ip);
 }

Thanks for your help

This is the new code that i've used

String currentLine = "";

So, clearly, you've ignored the advice to quit using Strings. Well, good luck.

I don't understand what is "c-string. Null terminated char arrays"

I don't understand what is "c-string. Null terminated char arrays"

That's a real shame. Time to put the computer away. Go to a bookstore and buy a C programming book. People have been writing robust text processing applications in C for decades.