I'm curious if there's a shorter way to code what I have here with the same outcome. Any pointers would be greatly appreciated.

Question

I'm using the .includes method in JavaScript, and my code is already a bit bulky, especially seeing as I plan to scale it. I tried 'or' and ||, but they didn't function properly. Any tips?

https://code.sololearn.com/W9SW79XTzct0/?ref=app

Answer 1

You're absolutely right. Thanks John!

Answer 2

Thanks, Mert. I'll give that a try.

Answer 3

Good suggestion, Maneren. I actually just implemented that approach, but I want to be able to make a long list of potential inputs to which the Bot would have the same response without having to have multiple 'if' statements. Any thoughts?

Answer 4

Oh, that's a good idea, Maneren! I'll give that a whirl a little later. I've got to get back to work soon. I'll let you know how that goes.

Answer 5

I tried to use the ||, joel. But it didn't function properly. I'm going to try to give a switch statement a go, see how that works for me.

Answer 6

Using regex is probably the best solution
if((/hi|hello/gi).test(human))
alert("Hello")

else if((/bye/gi).test(human))
alert("Goodbye")

Answer 7

Try getting of letters ceses by .toLowerCase() method to have just one statement for both "Hello" and "hello".

Answer 8

Maybe try switch statement

Answer 9

Since goodbye includes bye, you don't need to test it. You should add something to cover finding no matches such as "I'm sorry, but I haven't been trained to handle that as yet."

Answer 10

if (human.includes('hi') || ('hello')){
alert('Hi, can I help you pick a destination?');

Answer 11

use || for next goodbye statement