Trying to count words in string of unspecified length - javascript

[tobben]

Hey forum, i am currently trying to take some input consisting of an unspecified amount of words,

then count them and return how many times each word is used.. but i have been running into

some issues... 

 

This is my return, it's kindof correct. Just redundant... i want to prevent it from counting and returning

the same word several times. Desired output would be "er:3 det:2 da:1"

 

This is where i am getting my values and inserting the text in html. 

 

 

Here is my loop. I make an array based on the input text. As you can hopefully see. My loop recounts,

and returns the same word all over again whenever it encounters it ....  i haven't been able to prevent

that and i'm wondering if you have any suggestions... among various solutions i tried another nested

loop based on x doing txtarray.splice(x, txtarray) to remove a word after i have counted it once. But

that didn't work at all... it just ran the loop without doing anything.

 

i also tried setting a variable in the first loop "var check = txtarray" then doing txtarray.splice(x, check),

but again.. nothing happened. 

[Techdude154]

When you go to append the text to the HTML document, first check if count is greater than 1, if it is, don't append it.

 

I'm not that good with javascript, so I don't know how you would code that.

[tobben]

When you go to append the text to the HTML document, first check if count is greater than 1, if it is, don't append it.

 

I'm not that good with javascript, so I don't know how you would code that.

that isn't going to help though, count is checking how many times the loop encounters a specific word 

in the array. If it doesn't return a count grater than 1, that means whenever it counts a word twice or more

there is no output for that word.

 

I still need to count all the words, and return x amount it encounters said word. But i want to prevent it from 

returning the same word with the same count whenever it sees it, just return the word and how many times

it was encountered once.

[Techdude154]

That was my bad, I looked through the code some more and I would suggest using the indexOf function

http://www.w3schools.com/jsref/jsref_indexof.asp

 

you would basically check using the indexof function if the word exists already

 

Basically you would create a temporary string that you would append to after you check. When you go to add to the string, use the following code

if(str.indexOf(txtarray === -1)

{

str = str + check + ": " + count;

}

 

Then just do:

settxt.innerHTML = str;

 

Hopefully that makes sense, I'm not good at explaining things

[tobben]

Yes, that does indeed seem very sensible. Thanks, i will give it a try 

i'm still very curious as to how i should to solve my nested loop pitfall though,

to be able to handle them better, it's an issue i have encountered more than

once. without using indexof that is.

[tobben]

@Techdude154 And hallelulja! it's alive! it works! thanks for good advice 

 

 

 

[Techdude154]

No problem, glad to see you got it working.

[tobben]

Btw, if anyone are interested. My initial solution would have worked, it's just that i was using splice wrong,

hence why it didn't do anything. 

it's splice(start/value, how many)  so array.splice(i, 1)