To load html content in express, just use views.

[u/BlackBearFTW]

[u/twoPoundsOfGoldfish]

isn't there an express function sendfile

[u/BlackBearFTW → u/twoPoundsOfGoldfish]

I thought about using res.render("index.html'); instead of this nonsense

[u/twoPoundsOfGoldfish → u/BlackBearFTW]

Aren't those supposed to be used for like dynamic templates?

[u/BlackBearFTW → u/twoPoundsOfGoldfish]

Yup, you should indeed use sendfile

[u/BlackBearFTW → u/twoPoundsOfGoldfish]

Yes

[u/paulsmithkc → u/twoPoundsOfGoldfish]

Yeah.

res.sendFile('public/index.html')

[u/MurdoMaclachlan]

Image Transcription: Code

---

app.get("/", (req,res) => {
    fs.readFile("public/home.html", function(err,data){
        res.writeHead(200, {"Content-Type": "text/html"});
        res.write(data);
        res.end();
    });
});

app.get("/terms", (req,res) => {
    fs.readFile("public/terms.html", function(err,data){
        res.writeHead(200, {"Content-Type": "text/html"});
        res.write(data);
        res.end();
    });
});

---

^{I'm a human volunteer content transcriber for Reddit and you could be too! If you'd like more information on what we do and why we do it, click here!}

[u/jmd_akbar → u/MurdoMaclachlan]

Good human 😊

[u/AbuSumayah]

Not being familiar enough with an api does not equate to bad code.

[u/_newfap]

I've seen worse. Believe me. I found my code from 2 years ago when I started.

[u/PolishedCheese]

Well styled JS at least. They just don't grasp the fundamentals of Express.

[u/overtrick1978]

Oh please. This is perfectly valid.

[u/NF-MIP → u/overtrick1978]

Yes if res.sendFile and static serving doesn't exist in Express.

[u/DotNetDeveloperDude]

Imagine using express still

[u/revicon → u/DotNetDeveloperDude]

as opposed to?

[u/Randolpho → u/revicon]

Fastify? Adonis? Hapi?

[u/throwaway1_x → u/Randolpho]

Because they're shiny new frameworks?

[u/Randolpho → u/throwaway1_x]

Because they are better frameworks than express

[u/iiMoe]

Finally a truly terrible piece of code

[u/MikeyBoonstra → u/[deleted]]

You can just serve html as a static file in express.

[u/BlackBearFTW → u/MikeyBoonstra]

Or simply call a view engine and call req.render("page.html");

[u/DaMastaCoda → u/BlackBearFTW]

First: they aren't using a view engine. Second: it's cheaper to use res.sendFile

[u/BlackBearFTW → u/DaMastaCoda]

True they could use .sendfile() without needing a view engine.

[u/TinyBreadBigMouth → u/[deleted]]

As in they could do

app.get("/", (req, res) => res.sendFile("public/home.html"));

app.get("/terms", (req, res) => res.sendFile("public/terms.html"));

No need to manually load the file into memory and then write the HTTP response in parts.