London class-10- Junita Lama-Node module-week 2-chat server react app

[Junitalama]

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

• Your Name:
• Your City:
• Your Slack Name:

Homework Details

• Module:
• Week:

Notes

• What did you find easy?

• What did you find hard?

• What do you still not understand?

• Any other notes?

[ShayanMahnam]

Great job, Junita! Your work is amazing. I'll provide some advice and feedback in further comments.

[ShayanMahnam]

You have your own front end now you dont need this anymore

[ShayanMahnam]

you can do error handling like this and generate id for msgs here:(generate id here its just a example in real world we dont use this we will use uuid)

app.post("/messages", function (request, response) {
  const newMessage = request.body;

  // Check if the 'from' and 'text' properties are present in the request body
  if (!newMessage.from || !newMessage.text) {
    return response.status(400).json({ error: "Both 'from' and 'text' are required." });
  } else {
    // Generate a new unique id for the message
    newMessage.id = messages.length;
    messages.push(newMessage);
    response.status(201).json(newMessage);
    console.log(newMessage);
  }
});

[ShayanMahnam]

you can validate once more to see if the msg with that id is there

app.get("/messages/:id", function (request, response) {
  const idToFind = Number(request.params.id);
  const message = messages.find((msg) => msg.id === idToFind);

  // Check if the message with the provided ID exists
  if (message) {
    response.json(message);
  } else {
    response.status(404).json({ error: "Message not found." });
  }
});

[ShayanMahnam]

same check check if the message exists

app.delete("/messages/:id", function (request, response) {
  const idToDelete = Number(request.params.id);
  const messageIndex = messages.findIndex((msg) => msg.id === idToDelete);

  // Check if the message with the provided ID exists
  if (messageIndex !== -1) {
    const deletedMessage = messages.splice(messageIndex, 1);
    response.json(deletedMessage[0]);
  } else {
    response.status(404).json({ error: "Message not found." });
  }
});

[ShayanMahnam]

app.get("/messages/latest", (req, res) => {
  const startIndex = Math.max(messages.length - 10, 0);
  const latestMessages = messages.slice(startIndex);
  res.json(latestMessages);
});

[ShayanMahnam]

app.get("/messages/search", (req, res) => {
  const searchText = req.query.text.toLowerCase();
  const result = messages.filter((msg) => msg.text.toLowerCase().includes(searchText));
  res.json(result);
});

[ShayanMahnam]

its good to have some msgs for app.linsten to make sure when you run it, its working

const port = 8080;
app.listen(port, () => {
  console.log(`Server is running on http://localhost:${port}`);
});

[Junitalama]

Great job, Junita! Your work is amazing. I'll provide some advice and feedback in further comments.

@ShayanMahnam Thank you so much for reviewing it.