Create an 'if' statement based on value from array

Question

I'm trying to find out how ever a user has an event in it's calendar that they are not invited as guest on. The part I think I'm wrong in is this:

if (guestArray.indexOf(calendarId) == false) {

Code:

function myFunction() {      
    var calendarId = '[email protected]';
    var calendar = CalendarApp.getCalendarById(calendarId); 
    var events = calendar.getEvents(new Date('June 1, 2015 00:00:00 CST'), 
                                    new Date('June 5, 2015 23:59:59 CST'));
    for(var i = 0; i < events.length; i++) {
        var ev = events[i];
        var guestList  = ev.getGuestList();
        var guestArray = [];

        for (var n in guestList) {
            var guestEmail = (guestList[n].getEmail());
            guestArray.push(guestEmail);
        }

    if (guestArray.indexOf(calendarId) == false) {
        Logger.log("User not on guestlist!");
        Logger.log("TITLE: "  + ev.getTitle());
        Logger.log("DATE: "   + ev.getStartTime());  
        Logger.log("GUESTS :" + guestArray);
    }
}

Using the Date constructor to parse strings is not a good idea. — RobG
– RobG, Commented Jun 12, 2015 at 1:21
You can just do if (!guestArray.indexOf(claendarId)) instead of if (guestArray.indexOf(calendarId) == false)and if (GetValue()) instead of if (GetValue() == true) — AustinWBryan
– AustinWBryan, Commented May 18, 2018 at 0:46

jfriend00 · Accepted Answer · 2015-06-12 00:20:52Z

2

.indexOf() returns -1 if the value is not found and returns the index if it is found. It doesn't return true/false.

So, change this:

if (guestArray.indexOf(calendarId) == false) {

to this:

if (guestArray.indexOf(calendarId) === -1) {

answered Jun 12, 2015 at 0:20

jfriend00

711k104 gold badges1.1k silver badges1k bronze badges

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

1 Comment

Zig Mandel Over a year ago

personally i think its more readable with indexOf(calendarId) < 0

Collectives™ on Stack Overflow

Create an 'if' statement based on value from array

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