Asynchronous Loops in Node.js

Question

async function bald(fileName) {
    try {
        let aPayload = '';
        let aStream = fs.createReadStream(`./tmp/${fileName}`, 'utf8');
        aStream
            .on('data', (chunk) => {
                aPayload += chunk;
            })
            .on('end', async () => {
                let aJson = JSON.parse(aPayload);
                for await (let a of aJson) {
                    console.log(a.id);
                    console.log("Tick");
                    await dbItems.findOne({id: a.id}, (err, result) => {
                        console.log("Something Should Happen!");
                    });
                }
            });
    } catch (e) {
        throw e;
    }
}

In this answer is content I wanted to try: https://stackoverflow.com/a/50874507/12640130

I found out that forEach won't work at all so I changed it to for ... of, but it doesn't make a difference for me. Can somebody find out what's my problem?

Current output:

ID
Tick
ID
Tick
ID
Tick
// ... and so on.
Something Should Happen!
Something Should Happen!
Something Should Happen!
// ... and so on.

Expected output:

ID
Tick
Something Should Happen!
ID
Tick
Something Should Happen!
ID
Tick
Something Should Happen!
// ... and so on.

dor272 · Accepted Answer · 2020-05-16 21:35:52Z

2

You can't use await and callbacks at the same time. it should be one of the following:

this will not wait for result before iterating.

                    dbItems.findOne({id: a.id}, (err, result) => {
                        console.log("Something Should Happen!");
                    });

supposed to be what you're looking for.

                    const result = await dbItems.findOne({id: a.id})
                    console.log("Something Should Happen!");

answered May 16, 2020 at 21:35

dor272

7782 gold badges9 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

user12640130 Over a year ago

"You can't use await and callbacks at the same time." you literally nailed it. Now I realize why understanding promises, async & await has been so ridiculously "hard".

slebetman Over a year ago

@FooBar3098 Actually, there's no hard rule that says that it's not possible to use callbacks and promises (async/await) together. It's just that the API designer for the findOne() function didn't think of it. He probably thought: "Oh, you're passing me a callback? You probably don't want a promise then". Though to be fair I'd never allow both myself. It would be confusing. Which do I execute? The callback? The promise's .then() callback? Both? If both then which do I execute first etc.

user12640130 Over a year ago

True. Good example: Axios works nicely with .then but MongoDB methods don't. Not sure which ones or all but at least findOne .then is an undefined method.

Collectives™ on Stack Overflow

Asynchronous Loops in Node.js

1 Answer 1

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related