Assign value in for loop

Question

I am trying to get the actual month from a loop. When I step through it in developer tools and month[i] == 4 it doesn't assign actualMonth to checkMonth

Do I have to assign getMonth to month[] and then try and query the value?

var showCurrentMonth = function() {
    var getMonth = new Date().getMonth();

    var month = ["january", "february", "march", "april", "may", "june", "july", "august", "september", "october", "november", "december"];

    var actualMonth = "";
    for (var i = 0; i < month.length; i++) {
        var checkMonth = month[i];
        console.log(month[i]);
        if (getMonth == month[i]) {
            actualMonth = checkMonth;
        }
    }
    console.log(actualMonth);
}
window.addEventListener('DOMContentLoaded', showCurrentMonth, false);

getMonth() returns an integer: developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/… — rvandoni
– rvandoni, Commented May 8, 2015 at 9:46

SharpEdge · Accepted Answer · 2015-05-08 09:49:04Z

2

Too simple?

var showCurrentMonth = function() {
    var getMonth = new Date().getMonth();

    var month = ["january", "february", "march", "april", "may", "june", "july", "august", "september", "october", "november", "december"];

    var actualMonth = month[getMonth];
    
    console.log(actualMonth);
}
window.addEventListener('DOMContentLoaded', showCurrentMonth, false);

answered May 8, 2015 at 9:49

SharpEdge

1,7621 gold badge10 silver badges20 bronze badges

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

1 Comment

StudentRik Over a year ago

just what I wanted :)

Zee · Accepted Answer · 2015-05-08 09:50:19Z

2

Change your if to if (month[getMonth] == month[i]) {

Do it like this :

 var showCurrentMonth = function() {
    var getMonth = new Date().getMonth();

    var month = ["january", "february", "march", "april", "may", "june", "july", "august", "september", "october", "november", "december"];

    var actualMonth = "";
    for (var i = 0; i < month.length; i++) {
        var checkMonth = month[i];
        console.log(month[i]);
        if (month[getMonth] == month[i]) {  //Compare get month like this
            actualMonth = checkMonth;
        }
    }
    console.log(actualMonth);
}
window.addEventListener('DOMContentLoaded', showCurrentMonth, false);

answered May 8, 2015 at 9:50

Zee

8,4805 gold badges39 silver badges59 bronze badges

Comments

AlvaroAV · Accepted Answer · 2015-05-08 09:53:57Z

2

Your problem is that you are comparing an integer and string:

var getMonth = new Date().getMonth(); // This return number from 0 to 11

The code Date().getMonth() returns an integer, and your month list has strings on it

Your code should be:

var showCurrentMonth = function() {
    var getMonth = new Date().getMonth();

    var month = ["january", "february", "march", "april", "may", "june", "july", "august", "september", "october", "november", "december"];

    var actualMonth = month[getMonth];
    console.log(actualMonth);
}
window.addEventListener('DOMContentLoaded', showCurrentMonth, false);

To get the actual month you only need to access the month list with getMonth as index

edited May 8, 2015 at 9:53

answered May 8, 2015 at 9:45

AlvaroAV

10.6k12 gold badges65 silver badges92 bronze badges

2 Comments

rvandoni Over a year ago

actually getMonth() returns value between 0 and 11: developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/…

AlvaroAV Over a year ago

Yup, you're right I was changing the answer because the boucle is not needed there, he only needs to access the month list with the getMonth as index

Mohammad Faizan khan · Accepted Answer · 2015-05-08 09:53:16Z

1

slightly Modification required

var showCurrentMonth = function() {
        var getMonth = new Date().getMonth();
        console.log(getMonth);
        var month = ["january", "february", "march", "april", "may", "june", "july", "august", "september", "october", "november", "december"];

        var actualMonth = "";
        for (var i = 0; i < month.length; i++) {
            var checkMonth = month[i];

            if (getMonth == i) {
                actualMonth = checkMonth;
            }
        }
        console.log(actualMonth);
    };
    window.addEventListener('DOMContentLoaded', showCurrentMonth, false);

answered May 8, 2015 at 9:53

Mohammad Faizan khan

1,3453 gold badges17 silver badges34 bronze badges

1 Comment

AlvaroAV Over a year ago

The boucle is not needed, the only need there is to access the month list with the getMonth value as index, no need to enter in a boucle of month length

Collectives™ on Stack Overflow

Assign value in for loop

4 Answers 4

1 Comment

Comments

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related