Request - generate words starting with letter

[sitedev3]

Would love to see an option that will generate random words starting with a specific letter (parameter a-z).

Something like

let generateRandomWordStartingWith = wordList.filter(function (word) {
  return word[0].toLowerCase() === 'a';
});

I would like that aswell.

[BoDonkey]

How to work on this issue

1. Fork this repo into your own account
2. Make changes to the code - since it is in your account, you could work directly on main, but wouldn't you rather get some Git practice by working on a branch, and then merging to your own main?
3. Write one or more tests to make sure your feature works
4. Make sure your code changes don't break any existing tests.
5. Update the CHANGELOG.md file - since other people might contribute during the same sprint cycle, add your change log message under 'UNRELEASED', not a specific version. We will add the version number when everything is approved and published on npm.
6. When you are done, return to this repository and create a PR to pull code from your fork. Read more about this here. Make sure to fill out the PR template as best as you can.
7. Navigate to our Discord server here if you have already joined, or here if you need an invitation and post a message in the open-source contribution channel that you need a PR review.
8. After (hopefully) a short amount of time, one of our team engineers will review your PR and potentially advise you about things they want to see changed.
9. If you need to make changes, go back to your local fork, make changes, and contribute those back to the main repo. This will update the PR.
10. When you are done with changes and want a re-review, ask in Discord once again.
11. After your PR is accepted, celebrate!!! 🎉

Code suggestions for this issue

If you look at the generateWordWithMaxLength() function, this suggests a way to address this issue. In that function, a random word is generated and then checked against the size the user wants. If that size isn't correct, the function generates another. Not the most elegant, maybe, but very straightforward.

One edge case you need to worry about is if there aren't any words starting with that letter or if the user wants ten words starting with z and there are only four. Right now, there aren't any guardrails in place because the other options can almost always be fulfilled. Suggesting how you would take care of this either here on the issue or in the Discord would be great to get feedback and make sure you are on a good track.

Finally, please make use of the Discord to ask questions. Try to answer the questions yourself using internet resources, but don't be afraid to ask questions on the Discord about anything. We are here to help!

[ldd]

I can do a pull request, but the gist of my change would be:

  function generateRandomWordStartingWith(startingLetter){
    const filteredList = wordList.filter(word => word.startsWith(startingLetter));
    return filteredList[randInt(filteredList.length)];
  }

Notes on code suggestions:

• unlike generateWordWithMaxLength, we can quickly figure out the start of each word in our wordList. and whether they would be a valid pick or not.
• the case where there aren't any words starting with that letter (e.g: ñ) is to be considered. The easiest solution is to return an empty list, but other alternatives do exist.
• the library already outputs the same word more than once occasionally
  (words({exactly: 3}) can already give as an output ["might", "might", "rubbed"] which has repeated words)
  so having only 4 words that start with z while the user wants 10 will repeat words, as asking for 3000 words regularly would (there are 1952 words in the wordList)

Let me know if it's better if we talk about this on discord or on an actual PR. Thanks for working on this awesome library btw
:D

[BoDonkey]

Hi @ldd
"Let me know if it's better if we talk about this on discord or on an actual PR. " Why can't we do both!?😀
So a PR for this would be great. Overall, it seems like a good approach - how are you going to trigger the function?
What do you suggest for handling something like ñ?
I think the repeated words will work just fine.
Cheers,
Bob

[Suyash699]

Is this issue still open? Can I attempt it?

[BoDonkey]

I haven't heard anything from @idd since the last message, so I guess they are not going to work on it. Read the notes and give it a try! Let me know in our Discord or here when you need a code review.

[Suyash699]

Are the users supposed to specify the amount of words to be generated along with the starting letter? Or should I print all the letters starting with the specified letter?

[BoDonkey]

This would be an add-on option. So a user should be able to specify that they want 5 words all starting with 'a', for example. But, they should also be able to leave off how many need to be returned if they only want one.
I don't know the current library coverage, so you will also have to deal with there not being any words that fit the criteria in the library.

[Suyash699]

Hi @BoDonkey , here's the link to index.js file of my attempt at the issue: https://github.com/Suyash699/random-words/blob/main/index.js

[BoDonkey]

Hi @Suyash699, Thanks for trying this! At first glance, your function to filter the words looks reasonable. Although I would say that charAt(0) is likely more efficient. The only thing is, what if the user wants 3 words, each 6 letters, that start with 'c'? Your code needs to take this into account. Right now it looks like you are returning every word that starts with that letter in the dictionary.
The other thing that I'm not sure about is this line: if(options[0].toLowerCase>= "a" && options[0].toLowerCase<= "z") What are you trying to test here? What do you expect options[0] to return?
Cheers,
Bob

[boutell]

One thing I can see is that "toLowerCase" is a method, but you are not invoking it, so you are comparing functions themselves, not strings.

I suggest that you open a PR with your current attempt and mark it as draft if it's not ready, so it's easier to give feedback.

[Suyash699]

Hi @BoDonkey, will this work? :

function generateRandomWordStartingWith(startingLetter){
     const filteredList = startingLetter.charAt(0).toLowerCase().charCodeAt(0) >= 97 && startingLetter.charAt(0).toLowerCase().charCodeAt(0) <= 122 ? 
  wordList.filter(word => word.charAt(0).toLowerCase() === startingLetter.charAt(0).toLowerCase())
  : "error";

return filteredList.length === 0 ? null : filteredList[randInt(filteredList.length)];
 }

So basically, added the previous function in a ternary operator that checks if the given letter falls in the unicode range of a-z or not, and if yes, then checks whether it exists in the filteredList or not. I am still a bit confused as to how the "options" parameters work, so still haven't been able to solve the "how many words will you print?" issue.

As for the "toLowerCase" problem, I have corrected that as well. I'm considering making a PR as well @boutell