Array - how do I collect ALL items in only one list or array?

[divina11]

It's a very simple program, I don't know what I'm doing incorrect, all I want to do is to input items in the text box and then 'collect them in ONE list/array' and then when I press 'view' to display the product (append to existing list), My coding is shown below:

function letsShop() {
 
    var item=document.getElementById("item").value;
 
    var c= new Array(item);
    for(var i=0; i<c.length;i++);
    alert(c[i]);
    alert(c.length);
}

Ta

[gits]

hi ...

you always put the actual item in new array c ... so you always will get only ONE item to and from it ... you may use a global variable and then use the array push(item); method to extend the list everytime you call the letsShop-function ...

kind regards

ps: ok more explainations - i commented your code:

// function call - i assume you will add to an array here?
function letsShop() {
    // you declare and assign the value of an html-node here
    // that's ok
    var item=document.getElementById("item").value;
 
    // you declare the variable c here locally within the 
    // function only and always assign a new array with 
    // one item
    var c= new Array(item);
 
    // this is useless ;) nothing is in the loop 
    // first: you always loop through 1 element only
    // second: nothing is done with the elements ??
    for(var i=0; i<c.length;i++);
 
    // now you alert the last reached value (i) of the 
    // array c -> always c[0]
    alert(c[i]);
 
    // this is always 1
    alert(c.length);
}
 

[divina11]

Thank you that, making or declaring array global did the trick.

However, when i do a sort() i'm not getting a matching list/array - 'items' are not matching 'quantity', currently 'item' array & 'quantity' array are different. Where am i going wrong? see my code:

var c=[];                  //item array
var d=[];                 //quantity array
 
function letsShop() {
 
    var item=document.getElementById("item").value;
    var quantity=document.getElementById("quantity").value;
 
    c[c.length]=item;
    d[d.length]=quantity;
 
    Basket_display(item,quantity);
}
 
 
function Basket_display(item,quantity) {
    document.getElementById('display').innerHTML+=item + " X "+ quantity+'<br>';    
}
</script> 

I think i need to 'join' the two arrays together, not sure how to do this?

[gits]

hi ...

please use code-tags when posting code ...

to your problem ... when you need more complex data-structures, you should consider an js-object ... have a look at the following example:

// your list
var c = [];
 
// within your function letsShop() declare an object
// for an item
function letsShop() {
    // item object
    var item = {};
 
    // assign your item name
    item.name = 'foo';
 
    // assign your item quantity
    item.quantity = 1;
 
    // add the item(record) to your list
    c.push(item);
}
 

kind regards

[divina11]

thank you, one more final question of the day,

How & 'where' would I place the 'for loop', to print or sort etc. Would this be in another function or...

for(var=i, i<c.length, i++ )

Ta

[gits]

hi ...

for the code-tag issue have a look here ...

and please make yourself a little bit more clear. you have to put the loop there where you have to iterate to do something. for example: you should put it in a seperate function when you want to call it on different places in your code so you don't need to duplicate your code -> simply call the function.

kind regards