read user input N times from console node js

Question

I'm trying to get an input from a user to a console in node js a finite number of times, and run a function on each input. while or for loop doesn't work.

any help?

here is my code (litle simplified):

function foo(num)
{
 console.log(num)
}

function ReadUseInput()
{
 const readline = require('readline').createInterface({
     input: process.stdin,
    output: process.stdout
   }); 
    readline.question('Enter number...', num => 
  {
         foo(num)
       readline.close();
     });
}

//for (var i = 0; i < 10; i++)// this line isnt working - shows warning: MaxListenersExceededWarning: Possible EventEmitter memory leak detected
ReadUseInput()

The default limit for Event Emitter is 10. so you hit the limit by looping ten times — flaxon
– flaxon, Commented Aug 25, 2021 at 21:27
its not a limit, you can go way beyond that. its a warning that you do something fishy — Raphael PICCOLO
– Raphael PICCOLO, Commented Aug 25, 2021 at 21:28
await is the solution stackoverflow.com/questions/43638105/… — Raphael PICCOLO
– Raphael PICCOLO, Commented Aug 25, 2021 at 21:30

theusaf · Accepted Answer · 2021-08-25 22:11:29Z

1

One solution could be to make the ReadUseInput() function take a number and decrement it to repeat a certain number of times:

function foo(num) {
 console.log(num)
}

function ReadUseInput(timesLeft) {
  // exit condition
  if(timesLeft <= 0) {
    return;
  }
  const readline = require('readline').createInterface({
    input: process.stdin,
    output: process.stdout
  }); 
  readline.question('Enter number...', num => {
    foo(num)
    readline.close();
    ReadUseInput(--timesLeft);
  });
}

ReadUseInput(10);

edited Aug 25, 2021 at 22:11

answered Aug 25, 2021 at 21:30

theusaf

1,8121 gold badge11 silver badges19 bronze badges

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

6 Comments

Alex K Over a year ago

the solution you offered @theusaf seems logical.. it doesnt throw warning anymore, but it still seem to run infinite times ( at least a lot more then 10) from my tests . the exit condition doesn't seem to work

theusaf Over a year ago

Oh, my bad. I made a small mistake. I'll fix it quickly

Alex K Over a year ago

it resolved it, thank you! I am a little curious to know what u changed that stopped it from looping infinitely, because I don't see a difference honestly @theusaf

theusaf Over a year ago

Previously, I had done timesLeft--, which passes in the current value before decrementing it, which is why it went on forever. I changed it to --timesLeft to decrement, then pass in the value.

Alex K Over a year ago

while on the topic, why recursion work in this case but for loop doesn't? what am i missing? what topic should i reference? (should i post this as a question?)

|

Collectives™ on Stack Overflow

read user input N times from console node js

1 Answer 1

6 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Linked

Related