Populating a multidimensional array with user input

Question

I am trying to fill a multidimensional array with user input. I have this function:

var buildIt = function(row, col){
 var a = []
 var input
  for(var i = 0; i < row; i++){
    a[i] = []
    input = prompt('Input: ')
    for(var j = 0; j < col; j++){
      a[i][j] = input.split('')
    }
  }
return a
}

window.buildIt(3,3)

I want to make it so that the user is prompted the number of times there are rows. If user creates a matrix with three rows, the user should be prompted for input three times. I want to store this input in each row. For example, user enters foo bar baz array a should like this:

a = [
  ['f','o','o'],
  ['b','a','r],
  ['b','a','z]
 ]

When I call the function:

var board = buildIt(3,3)
console.log(board[0][0])

It logs all the elements in the first row instead of the element at point [0][0] which should be f if user entered foo.

3x3 is 9. You are looping more than you expect

mplungjan
– mplungjan

2016-08-19 15:31:33 +00:00
Commented Aug 19, 2016 at 15:31 — mplungjan
– mplungjan, Commented Aug 19, 2016 at 15:31

Aman Rawat · Accepted Answer · 2016-08-19 15:30:33Z

1

You are using prompt inside the two loops so if you have a metrix of 3x3 then it will prompt you 9 times to insert the data so i removed the extra loop. Try this

    var buildIt = function(row, col){
    var a = []
    var input
    for(var i = 0; i < row; i++){
    	input = prompt('Input: ')
    	a[i] = input.split('')
    }
    return a
    }
    
    console.log(window.buildIt(3,3));

answered Aug 19, 2016 at 15:30

Aman Rawat

2,6251 gold badge27 silver badges40 bronze badges

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

Comments

Josh · Accepted Answer · 2016-08-19 15:30:26Z

1

The code you posted works and the output is given below (I substituted a random string for the input). However, your array should actually look like this afterward, as you are one level too shallow in your example (remember that a[i][j] is assigned the result of .split(''), which returns an array of characters in this case, so there will exist an a[i][j][0] which will be a character.

> buildIt(3,3)
[ [ [ 'G', 'C', 'T', 'J', 'O' ],
    [ 'S', 'Q', 'S', 'B', 'G' ],
    [ 'D', 'E', 'U', 'P', 'Y' ] ],
  [ [ 'D', 'B', 'I', 'Z', 'Q' ],
    [ 'G', 'M', 'U', 'N', 'R' ],
    [ 'D', 'T', 'I', 'A', 'V' ] ],
  [ [ 'J', 'L', 'O', 'H', 'V' ],
    [ 'U', 'C', 'D', 'B', 'H' ],
    [ 'U', 'C', 'X', 'E', 'W' ] ] ]

answered Aug 19, 2016 at 15:30

Josh

4,8162 gold badges22 silver badges33 bronze badges

Comments

Abdullah Razzaki · Accepted Answer · 2016-08-19 15:34:49Z

1

You don't need to pass the no of columns, just the rows and remove the inner loop because you only need to execute the code for the number of rows. The updated code

    var buildIt = function(row){
    var a = []
    var input
    for(var i = 0; i < row; i++){
       input = prompt('Input: ');
       a[i] = input.split('');
    }
    return a;
    }
    buildIt(3);

answered Aug 19, 2016 at 15:34

Abdullah Razzaki

1,00210 silver badges16 bronze badges

Collectives™ on Stack Overflow

Populating a multidimensional array with user input

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related