Im so close but, help! ethernet control

Using Arduino Project Guidance
4

zoomkat:

I want the arduino to be the server etc , i dont want to have it interfacing with this that and the next thing.

Something to try.

//zoomkat 12-8-11

//simple button GET with iframe code
//for use with IDE 1.0
//open serial monitor to see what the arduino receives
//use the \ slash to escape the " in the html
//address will look like http://192.168.1.102:84 when submited
//for use with W5100 based ethernet shields

#include <SPI.h>
#include <Ethernet.h>

byte mac[] = { 0xDE, 0xAD, 0xBE, 0xEF, 0xFE, 0xED }; //physical mac address
byte ip[] = { 192, 168, 1, 102 }; // ip in lan
byte gateway[] = { 192, 168, 1, 1 }; // internet access via router
byte subnet[] = { 255, 255, 255, 0 }; //subnet mask
EthernetServer server(84); //server port

String readString;

//////////////////////

void setup(){

pinMode(4, OUTPUT); //pin selected to control
  //start Ethernet
  Ethernet.begin(mac, ip, gateway, subnet);
  server.begin();

//enable serial data print
  Serial.begin(9600);
  Serial.println("server LED test 1.0"); // so I can keep track of what is loaded
}

void loop(){
  // Create a client connection
  EthernetClient client = server.available();
  if (client) {
    while (client.connected()) {
      if (client.available()) {
        char c = client.read();

//read char by char HTTP request
        if (readString.length() < 100) {

//store characters to string
          readString += c;
          //Serial.print(c);
        }

//if HTTP request has ended
        if (c == '\n') {

///////////////
          Serial.println(readString); //print to serial monitor for debuging

//now output HTML data header
             if(readString.indexOf('?') >=0) { //don't send new page
               client.println("HTTP/1.1 204 Zoomkat");
               client.println();
               client.println(); 
             }
             else {
          client.println("HTTP/1.1 200 OK"); //send new page
          client.println("Content-Type: text/html");
          client.println();

client.println("");
          client.println("");
          client.println("Arduino GET test page");
          client.println("");
          client.println("");

client.println("

Zoomkat's simple Arduino button

");
         
          client.println("<a href="/?on" target="inlineframe">ON");
          client.println("<a href="/?off" target="inlineframe">OFF");

//client.println("<IFRAME name=inlineframe src="res://D:/WINDOWS/dnserror.htm" width=1 height=1">");
          client.println("<IFRAME name=inlineframe style="display:none" >");         
          client.println("");

client.println("");
          client.println("");
             }

delay(1);
          //stopping client
          client.stop();

///////////////////// control arduino pin
          if(readString.indexOf("on") >0)//checks for on
          {
            digitalWrite(4, HIGH);    // set pin 4 high
            Serial.println("Led On");
          }
          if(readString.indexOf("off") >0)//checks for off
          {
            digitalWrite(4, LOW);    // set pin 4 low
            Serial.println("Led Off");
          }
          //clearing string for next read
          readString="";

}
      }
    }
  }
}

Thanks heaps , got that working ,
How do i change it so they are buttons?, ones that say "on" and "off" on them

Thanks, corey