Establishing a repeated sequence for an alternative condition within a function

Question

Establishing a repeated sequence for an alternative condition within a function

Struggling to create a loop that includes an else statement. The if statement works fine, but when adding the else statement, it either shows nothing or creates multiple loops. It seems I may be placing the else statement in the wrong spot. Can someone explain where to correctly place the else statement and the nesting criteria for ending curly braces within a function? Here is my code that works well until I add the else statement. Thanks!

var sports = ["golf", "cricket", "tennis", "badminton", "squash"];

function checkSport(sportToCheck) {
    for (var i = 0; i <= sports.length; i++) {
        if (sportToCheck == sports[i]) {
            alert("yes we offer that sport");
        }
    }
}

checkSport("tennis")

javascript arrays function loops if-statement

Answer 1

Answer №1

It seems like in this scenario, you don't necessarily need to include an else statement based on the variable names used, but rather you want to terminate the loop or function as soon as the sport is found:

function checkSportAvailability(sport) {
  for (var x = 0; x <= sportsList.length; x++) {
    if (sport == sportsList[x]) {
      alert("Yes, that sport is available.");
      return; // End the function execution
    } else {
      console.log("There's no action required here, so the else statement can be omitted.");
    }
  }
  alert("Sorry, we do not offer that particular sport."); // Display message if no match is found after looping through all options
}

Answer 2

It seems like in this scenario, you don't necessarily need to include an else statement based on the variable names used, but rather you want to terminate the loop or function as soon as the sport is found:

function checkSportAvailability(sport) {
  for (var x = 0; x <= sportsList.length; x++) {
    if (sport == sportsList[x]) {
      alert("Yes, that sport is available.");
      return; // End the function execution
    } else {
      console.log("There's no action required here, so the else statement can be omitted.");
    }
  }
  alert("Sorry, we do not offer that particular sport."); // Display message if no match is found after looping through all options
}

Answer 3

Answer №2

To strictly adhere to the principle of Do One Thing, it is recommended that your function, named check(), should return either true or false.

While there are multiple approaches to resolving your problem, personally I advocate for using find().

var sports = ["golf", "cricket", "tennis", "badminton", "squash"];

function checkSport(sportToCheck) {
  return sportToCheck === sports.find((sport) => sport === sportToCheck);
}
   
console.log(checkSport("football")); // Expected outcome: false
console.log(checkSport("tennis"));   // Expected outcome: true

If you wish to enhance @Hangindev's solution, you can consider implementing the following:

for (sport of sports) {
  true === (sportToCheck === sport) && alert("yes we offer that sport");
}

Answer 4

To strictly adhere to the principle of Do One Thing, it is recommended that your function, named check(), should return either true or false.

While there are multiple approaches to resolving your problem, personally I advocate for using find().

var sports = ["golf", "cricket", "tennis", "badminton", "squash"];

function checkSport(sportToCheck) {
  return sportToCheck === sports.find((sport) => sport === sportToCheck);
}
   
console.log(checkSport("football")); // Expected outcome: false
console.log(checkSport("tennis"));   // Expected outcome: true

If you wish to enhance @Hangindev's solution, you can consider implementing the following:

for (sport of sports) {
  true === (sportToCheck === sport) && alert("yes we offer that sport");
}

Answer 5

Answer №3

A more efficient approach is to iterate through each value in the sports array without using unnecessary else statements. By utilizing the return statement, you can halt script execution within the function as soon as a match is found.

This function will return either true or false based on whether the given string is present in the sports array.

var sports = ["golf", "cricket", "tennis", "badminton", "squash"];

function checkSport(sportToCheck) {
    for (var i = 0; i <= sports.length; i++) {
        if (sportToCheck == sports[i]) {
            alert("yes we offer that sport");
            return true;
        }
        if (i == sports.length) {
            alert("sorry we do not offer this sport");
            return false;
        }
    }
}

checkSport("tennis")

This method can be used in scenarios like:

if (checkSport("tennis")) {
    // sport exists
} else {
    // sport does not exist
}

Answer 6

A more efficient approach is to iterate through each value in the sports array without using unnecessary else statements. By utilizing the return statement, you can halt script execution within the function as soon as a match is found.

This function will return either true or false based on whether the given string is present in the sports array.

var sports = ["golf", "cricket", "tennis", "badminton", "squash"];

function checkSport(sportToCheck) {
    for (var i = 0; i <= sports.length; i++) {
        if (sportToCheck == sports[i]) {
            alert("yes we offer that sport");
            return true;
        }
        if (i == sports.length) {
            alert("sorry we do not offer this sport");
            return false;
        }
    }
}

checkSport("tennis")

This method can be used in scenarios like:

if (checkSport("tennis")) {
    // sport exists
} else {
    // sport does not exist
}

Establishing a repeated sequence for an alternative condition within a function

Answer №1

Answer №2

Answer №3

Similar questions

Is the maxJsonLength property in the web.config referenced by JQuery's Ajax?

Establish the focus when the TinyMCE editor loads

Unexpected behavior observed in while loop with dual conditions

Stop the continuous AJAX loop or look for an alternative method to retrieve and compare a list of HTML pages

JS Issue with triggering .click() function

Exploring the use of Firebase for personal endeavors: What are the potential financial risks involved and how can they be mitigated?

I need to know the right way to send a file encoded in Windows-1255 using Express

How can I modify my array from using images to using string values for radio buttons?

Break down for me what Json is in a way that a five-year-old could understand

What is the best way to implement a dialog box that displays a website within it, all while keeping my SharePoint site running in the background?

Verify if the specified date (in string form) has already occurred or is upcoming using JavaScript

Creating a scrolling effect similar to the Nest Thermostat

Issues encountered when updating values in MaterialUI's TextField using Formik

Tips for accessing the information received from an AJAX call

Steps for implementing AJAX to display a success function and update database results in real-time

Identifying the presence of a property value within a collection of objects

Combining nested arrays, objects, and properties into one comprehensive flat table through a left join

Unsuccessful attempts to animate with Jquery in Google Chrome are persisting

a simple guide to setting a "lowest possible value" in javascript

There are encountered issues with the `POST` value not functioning properly on certain pages