I’m doing an exercise which requires me to remove every other repeated word from an array of words.
Basically i’ve gor an array with 188 words called betterWords and another array called overusedWords which contains the ones i’ve got to remove every other time they appear in the betterWords array.
Could someone give me a hint on how to do it??
Here’s the code: (It’s wrong but I can’t figure out how to do it)
Basically I thought I would keep the count of the overusedWords and then remove those in an odd index position from the betterWords array, but obviously that does not work.
let story = 'Last weekend, I took literally the most beautiful bike ride of my life. The route is called "The 9W to Nyack" and it actually stretches all the way from Riverside Park in Manhattan to South Nyack, New Jersey. It\'s really an adventure from beginning to end! It is a 48 mile loop and it basically took me an entire day. I stopped at Riverbank State Park to take some extremely artsy photos. It was a short stop, though, because I had a really long way left to go. After a quick photo op at the very popular Little Red Lighthouse, I began my trek across the George Washington Bridge into New Jersey. The GW is actually very long - 4,760 feet! I was already very tired by the time I got to the other side. An hour later, I reached Greenbrook Nature Sanctuary, an extremely beautiful park along the coast of the Hudson. Something that was very surprising to me was that near the end of the route you actually cross back into New York! At this point, you are very close to the end.';
let overusedWords = ['really', 'very', 'basically'];
let unnecessaryWords = ['extremely', 'literally', 'actually' ];
const storyWords = story.split(' ');
//console.log(storyWords.length);
const betterWords = storyWords.filter(word => {
if(unnecessaryWords.includes(word) === false) {
return word;
}
});
//console.log(betterWords);
let reallyCount = 0;
let veryCount = 0;
let basicallyCount = 0;
const removeEveryOtherWord = betterWords.forEach(word => {
if(word === overusedWords[0]) {
reallyCount++;
} else if (word === overusedWords[1]) {
veryCount++;
} else if (word === overusedWords[2]) {
basicallyCount++;
}
for(i = 0; i < reallyCount.length; i++) {
if(reallyCount[i] % 2 != 0) {
betterWords.splice(reallyCount[i]);
} else if (veryCount[i] % 2 != 0) {
betterWords.splice(veryCount[i]);
} else if (basicallyCount[i] % 2 != 0) {
betterWords.splice(basicallyCount[i]);
}
}
});
the overused words should be removed every other time in general, or for each specific word?
ex. “I’m really exited for this, basically I was waiting my whole life.”
the two words are to be considered separatelly or, there are two overused words and the second one should be removed?
The last function (removeEveryOtherWord) is supposed to do that, but obviously it is wrong… Basically it has to return the betterWords array (which is the story array without the unnecessary words), deleting every other of each of the overusedWords.
I was trying to keep the count of each overusedWord and then delete every other one for EACH one of the overusedWords…
Someone suggested me that it would have a better result if I use .filter() on betterWords and then create an object which stores the count of each of the overusedWords and then, using the object, returning true or false, choose the words i want to delete…
I haven’t worked with objects a lot yet as I only started JS a month ago. I’m googling on how to do it but it’s being a bit difficult to understand the logic and how then I’d use the object to pick the words I want to delete and returning the final filtered array.
DRY principles are a thing, I mean, if it works is a first awesome step, but then, we want also to make it better (DRY is Don’t Reapeat Yourself)
so, you could use indexOf(), in this way you can also remove the first use of includes, you can do let index = ...; if (index < 0) {return true;} else { /* logic using the new index variable */ }
in this way you have one single block of code that does the thing for the three words, without having to repeat it for each of them
you could use an object or an array to keep count, in both ways you can use the same logic for everything
let index = ...; if (index < 0) {return true;} else { /* logic using the new index variable */ }
let index = indexOf(word)???
The first IF statement (index < 0) will always equal to false but the last word of the array right?
Of what and how do i have to keep count?
How would I use an object to keep count of the overusedWords and then delete the ones i want? How do i insert information from the .filter() loop in the object??
