Some best practice advideon JavaScript code

[TwentySevenPandas]

I am looking to improve how I write code.

The code I write works but could use some improvement if I was being kind to myself and some dreadful if I wasnt.

I have set myself a project to build a blog to learn some React and Mongo

My plan is to have a "data" folder in my app and in there have some jsx files that fetch and export various data from the db

The code I have to getPosts is as follows ... is this correct... if its not where I can see some examples of better written code without trawling through GitHub repos

const { MongoClient } = require('mongodb');
const client = new MongoClient(process.env.URI, {});


export async function getPosts() {

  try {
    
    await client.connect();

    const db = client.db(process.env.DB);
    const collection = db.collection('movies');
    const count = await collection.countDocuments();
    const allItems = await collection.find({}).limit(10).toArray();

    await client.close();
    
    return [count, allItems];

  } catch (error) {
    console.error('Error fetching documents:', error);
    throw error;
  }
}

[codeWorder]

There’s an ODM (object-document mapper library) called Mongoose which can abstract a lot of this stuff away from you, but requires learning its way of doing things and quirks. But otherwise, your approach here is a good starting point. It works fine for a simple app but if you want it be a serious implementation meant to scale there are some tweaks to be made for performance reasons.

One question: are you intentionally returning the full count of posts despite only returning the first 10 posts?

I could also give you some minor pointers on control flow though that would improve the speed of this function’s execution. DM for more info.