Match array with strings from another array by word relevance

Question

var myArray = [
    ["good morning", "am", "morning"],
    ["good night", "night", "evening"],
    ["good day", "day is good", "it's a good day"]
];

function compare(entry, string) {
    for (var x in entry) {
        strings = string.split(' ');
        array1 = entry[x][0].split(' ');
        if (strings.every(s => array1.indexOf(s) !== -1)) {
            items = entry[x].slice(1);
            return items
        } else {
            //return 'not found'
        }
    }
}
$("#input").keydown(function(e) {
    keyword = $(this).val();
    if (e.which === 13) {
        $(this).val("");
        text = keyword.toLowerCase();
        result = '<div>' + compare(myArray, text) + '</div>'
        $('#results').append(result);
    }
});

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<input id="input" type="text" placeholder="Search..." autocomplete="off" />
<div id="results" class="reset"></div>

strings = Keywords from input box is converted into an array

array1 = Each first items from group of arrays are converted into an array

...compare them then return items from the same group where it found the matched keywords. I used .slice(1) so it wont return the first item of each group.

Using every() is the closest iv got after trying includes(), .some() etc...

I have two problems on the above code:

it will return undefined if the keyword is more than the words present in the array.

it should return true if keywords are:

good,

good morning dude,

good night folks,

good day people

if I put "else" on the condition, it wont function same as without "else"...

thanks for your help..

have you considered String.prototype.includes rather than parsing out individual words? developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/… — Randy Casburn
– Randy Casburn, Commented Oct 29, 2020 at 1:35
it will return undefined if the keyword is not present in the array. what should be the default return value in this case then? — web-sudo
– web-sudo, Commented Oct 29, 2020 at 1:40

Randy Casburn · Accepted Answer · 2020-10-29 02:13:23Z

1

Use .map() twice, see if the string is included in the inner array value, then flatten the array and filter out non-string values.

This code finds matching words in each string. The ternary either returns the string (if found) or returns null if not found. That gives an array of arrays. So they get flattened into a single array. Finally, the null values are filtered out.

here is one way to accomplish your goal:

var myArray = [
  ["good morning", "am", "morning"],
  ["good night", "night", "evening"],
  ["good day", "day is good", "it's a good day"]
];

function compare(entry, string) {
  let out = string
             .split(' ')
             .map(str => 
                   entry.map(e => 
                          e.map(f => (f.includes(str)) ? f : null))
              .flat()
              .filter(a => a !== null));
  return out.join(', ');
}

compare(myArray, 'good night dude');

$("#input").keydown(function(e) {
  keyword = $(this).val();
  if (e.which === 13) {
    $(this).val("");
    text = keyword.toLowerCase();
    result = '<div>' + compare(myArray, text) + '</div>'
    $('#results').append(result);
  }
});

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<input id="input" type="text" placeholder="Search..." autocomplete="off" />
<div id="results" class="reset"></div>

edited Oct 29, 2020 at 2:13

answered Oct 29, 2020 at 1:45

Randy Casburn

14.2k1 gold badge19 silver badges32 bronze badges

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

5 Comments

88willr Over a year ago

looks good but it did not solved problem #1, for example >> "good night dude" .

Randy Casburn Over a year ago

ah - see my edit. This will find any singular search term that matches any singular term in the string of the array, and return the entire matched string.

88willr Over a year ago

can we return the most relevant result on top? the original code does that.

Randy Casburn Over a year ago

I realize that in your mind your requirements are crystal clear. But, your question lacks clarity (for me). For instance, there is no definition of what you expect "the most relevant result" to be. What would it be for a search string such as "dude, tonight, this evening is a good night and I cannot wait until a good morning"

88willr Over a year ago

i also forgot to mention that >> I used .slice(1) so it wont return the first item of each group.

cabrillosa · Accepted Answer · 2020-10-29 01:40:13Z

Return some value that represents "not found" and check in keydown() function before appending the text to div

var myArray = [
    ["good morning", "am", "morning"],
    ["good night", "night", "evening"],
    ["good day", "day is good", "it's a good day"]
];

function compare(entry, string) {
    for (var x in entry) {
        strings = string.split(' ');
        array1 = entry[x][0].split(' ');
        if (strings.every(s => array1.indexOf(s) !== -1)) {
            items = entry[x].slice(1);
            return items
        } else {
            return -1;
        }
    }
}
$("#input").keydown(function(e) {
    keyword = $(this).val();
    if (e.which === 13) {
        $(this).val("");
        text = keyword.toLowerCase();
        var matched = compare(myArray, text);
        if(matched!=-1){
          result = '<div>' + matched + '</div>'
        } else {
          result = '<div>Not Found</div>'
        }
       
        $('#results').append(result);
    }
});

Collectives™ on Stack Overflow

Match array with strings from another array by word relevance

2 Answers 2

5 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related