If value matches another value in array display items JavaScript

Question

I have a list of objects, in each object I have a date and time.

onClick of a td element (its a calendar) I check the time on the calendar, then check if it matches any times in my list of objects.

onSelect: function(date) {
    for (var i = 0; i < dateArray.length; i++) {
        for (var prop in dateArray) {
            if (date === dateArray[prop].date) {
                console.log(dateArray[prop].time);
            }
        }
    }
}

So I loop through my array, which is an array of objects. To be able to get anything out I need to do a for in loop, and in there I do a condional statement to say if the dates match. The console.log out the corect time.

However, the time gets console logged out 10 times since it's in a loop. But the only way I could get inside my object array was to loop through them.

How should I actually be doing this.

EDIT

dateArray structure: list of objects like [ object, object, object, object ] and once I loop through them, within each object it looks like :

Object {title: "The Title", date: "01/01/2017", time: "07:30pm", available: true,}

use indexOf function in javascript. If value is there it will return index or it will return -1. — Mani
– Mani, Commented Jun 22, 2016 at 6:42
does for (var prop in dateArray) actually give you an index number to use in dateArray[prop]? shouldn't you be using i? — domdomcodecode
– domdomcodecode, Commented Jun 22, 2016 at 6:47
@Coderchu I don't know if I should be looping twice. Date is coming from jQuery Date picker. It's basically a list of all the dates in the current month. — pourmesomecode
– pourmesomecode, Commented Jun 22, 2016 at 6:48
@FrancescoD'Alesio Sorry! Just updated it now with the sturcutre of dateArray — pourmesomecode
– pourmesomecode, Commented Jun 22, 2016 at 6:51

fradal83 · Accepted Answer · 2016-06-22 06:54:14Z

2

I think you are simply looping twice.

Given it is an array, you should loop it using for, like this

onSelect: function(date) {
    for (var i = 0; i < dateArray.length; i++) {
        if (date === dateArray[i].date) {
            console.log(dateArray[i].time);
        }
    }
}

answered Jun 22, 2016 at 6:54

fradal83

2,0221 gold badge12 silver badges9 bronze badges

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

1 Comment

pourmesomecode Over a year ago

Dude that was it! I should of been using i instead of doing another for...in loop. Sorry, amateur hour over here :) thanks man!

Collectives™ on Stack Overflow

If value matches another value in array display items JavaScript

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related