some problem in my code ?

hi everyone, so i looked it this code
and i made a connection with my Laptop
so the Arduino is the Server and the Laptop is the client
so everything works fine
BUT
when it goes here

    if ( thisChar == 't'){
      Serial.println("YOU ARE IN 1 ");
      client.println("YOU ARE IN two");
      while(1){
      digitalWrite(LED, HIGH); 
      delay(5000);
      digitalWrite(LED, LOW);
      delay(5000);
      Serial.println("YOU ARE IN");

the LED should blink every 5 second
but it’s only keep printing “YOU ARE IN”
i need some help at this point
thx,

 /*
 DHCP Chat  Server

 A simple server that distributes any incoming messages to all
 connected clients.  To use, telnet to your device's IP address and type.
 You can see the client's input in the serial monitor as well.
 Using an Arduino Wiznet Ethernet shield.

 THis version attempts to get an IP address using DHCP

 Circuit:
 * Ethernet shield attached to pins 10, 11, 12, 13

 created 21 May 2011
 modified 9 Apr 2012
 by Tom Igoe
 modified 02 Sept 2015
 by Arturo Guadalupi
 Based on ChatServer example by David A. Mellis

// Example 01 : Blinking LED 

const int LED = 13;
void setup () {
pinMode(LED, OUTPUT); }
void loop() {
digitalWrite(LED, HIGH); 
delay(5000);
digitalWrite(LED, LOW);
delay(5000)
; }
 */

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

// Enter a MAC address and IP address for your controller below.
// The IP address will be dependent on your local network.
// gateway and subnet are optional:
byte mac[] = {
  0x00, 0xAA, 0xBB, 0xCC, 0xDE, 0x02
};
IPAddress ip(192, 168, 1, 72);
IPAddress myDns(8, 8, 8, 8);
IPAddress gateway(192, 168, 100, 1);
IPAddress subnet(255, 255, 255, 0);

// telnet defaults to port 23
EthernetServer server(23);  
boolean gotAMessage = false; // whether or not you got a message from the client yet
const int LED = 13; // mine 
void setup() {
  pinMode(LED, OUTPUT); // mine
  // You can use Ethernet.init(pin) to configure the CS pin
  //Ethernet.init(10);  // Most Arduino shields
  //Ethernet.init(5);   // MKR ETH shield
  //Ethernet.init(0);   // Teensy 2.0
  //Ethernet.init(20);  // Teensy++ 2.0
  //Ethernet.init(15);  // ESP8266 with Adafruit Featherwing Ethernet
  //Ethernet.init(33);  // ESP32 with Adafruit Featherwing Ethernet

  // Open serial communications and wait for port to open:
  Serial.begin(9600);
  while (!Serial) {
    ; // wait for serial port to connect. Needed for native USB port only
  }

  // start the Ethernet connection:
  Serial.println("Trying to get an IP address using DHCP");
  if (Ethernet.begin(mac) == 0) {
    Serial.println("Failed to configure Ethernet using DHCP");
    // Check for Ethernet hardware present
    if (Ethernet.hardwareStatus() == EthernetNoHardware) {
      Serial.println("Ethernet shield was not found.  Sorry, can't run without hardware. :(");
      while (true) {
        delay(1); // do nothing, no point running without Ethernet hardware
      }
    }
    if (Ethernet.linkStatus() == LinkOFF) {
      Serial.println("Ethernet cable is not connected.");
    }
    // initialize the Ethernet device not using DHCP:
    Ethernet.begin(mac, ip, myDns, gateway, subnet);
  }
  // print your local IP address:
  Serial.print("My IP address: ");
  Serial.println(Ethernet.localIP());

  // start listening for clients
  server.begin();

  
}

void loop() {
  // wait for a new client:
  EthernetClient client = server.available();

  // when the client sends the first byte, say hello:
  if (client) {
    if (!gotAMessage) {
      Serial.println("We have a new client");
      client.println("Hello, client!");
      client.println("Enter ON or OF");
      gotAMessage = true;
    }

    // read the bytes incoming from the client:
    char thisChar = client.read();
    // echo the bytes back to the client:
//    server.write(thisChar);
    // echo the bytes to the server as well:
    Serial.print(thisChar);
    if ( thisChar == 't'){
      Serial.println("YOU ARE IN 1 ");
      client.println("YOU ARE IN two");
      while(1){
      digitalWrite(LED, HIGH); 
      delay(5000);
      digitalWrite(LED, LOW);
      delay(5000);
      Serial.println("YOU ARE IN");
      }
    }
    Ethernet.maintain();
    
  }
}

The functions delay() and delayMicroseconds() block the Arduino until they complete.
Have a look at how millis() is used to manage timing without blocking in Several Things at a Time.

And see Using millis() for timing. A beginners guide if you need more explanation.

Your 5000 millisecond delay()s will make the program very unresponsive.

...R

Robin2:
The functions delay() and delayMicroseconds() block the Arduino until they complete.
Have a look at how millis() is used to manage timing without blocking in Several Things at a Time.

And see Using millis() for timing. A beginners guide if you need more explanation.

Your 5000 millisecond delay()s will make the program very unresponsive.

...R

that doesn't solve my problem
the problem is when the code go inside the *while loop *
it doesn't make the LED blink i don't know why
i used telnet to make my computer act as Client to Arduino
when my computer Send 't'
i want the Arduino to make the LED blink

Which Ethernet interface are you using on what type of Arduino?

Strange that your Ethernet works without calling init().

Why do you want to hang the “client” after reception of a ‘t’?

khalidd7:
that doesn’t solve my problem

Then please post the latest version of your program and tell us exactly what it does and what you want it to do that is different.

…R