Getting the top 3 keywords from a text content

Guys,

I run into some problems when I execute the following javascript at the line mentioned inline in the source code below,

<html>
<head>
<title>JavaScript Keyword Density Analyzer</title>
<script language="JavaScript">

//Global variable declarations. This variable will hold the cleaned up String (with stopwords removed from the textElement)
var newStr = "a";

//array to hold the top three keywords based on the top 3 density of occurence
topThreeArray = new Array(3);

//the top 3 density values
var oldDensity1 = 0;
var oldDensity2 = 0;
var oldDensity3 = 0;

//function to calculate the keyword density
function calculteKeywordDensity(textElement) {
	str=document.getElementById(textElement).value;

alert("The actual String is " + str);
	var words = str.replace(/[^a-zA-Z0-9]/g,' ').split(' ');
	var assoc = new Array();
    total_words=words.length;

stopwords = [ "a", "about", "above", "accordingly", "after",
	  "again", "against", "ah", "all", "also", "although", "always", "am", "among", "amongst", "an",
	  "and", "any", "anymore", "anyone", "are", "as", "at", "away", "be", "been",
	  "begin", "beginning", "beginnings", "begins", "begone", "begun", "being",
	  "below", "between", "but", "by", "ca", "can", "cannot", "come", "could",
	  "did", "do", "doing", "during", "each", "either", "else", "end", "et",
	  "etc", "even", "ever", "far", "ff", "following", "for", "from", "further", "furthermore",
	  "get", "go", "goes", "going", "got", "had", "has", "have", "he", "her",
	  "hers", "herself", "him", "himself", "his", "how", "i", "if", "in", "into",
	  "is", "it", "its", "itself", "last", "lastly", "less", "many", "may", "me",
	  "might", "more", "must", "my", "myself", "near", "nearly", "never", "new",
	  "next", "no", "not", "now", "o", "of", "off", "often", "oh", "on", "only",
	  "or", "other", "otherwise", "our", "ourselves", "out", "over", "perhaps",
	  "put", "puts", "quite", "s", "said", "saw", "say", "see", "seen", "shall",
	  "she", "should", "since", "so", "some", "such", "t", "than", "that", "the",
	  "their", "them", "themselves", "then", "there", "therefore", "these", "they",
	  "this", "those", "though", "throughout", "thus", "to", "too",
	  "toward", "unless", "until", "up", "upon", "us", "ve", "very", "was", "we",
	  "were", "what", "whatever", "when", "where", "which", "while", "who",
	  "whom", "whomever", "whose", "why", "with", "within", "without", "would",
      "yes", "your", "yours", "yourself", "yourselves", ",", ";", "?", ".", "$", " ", "example", "using"];

//iterate over the total words and remove the stopwords
    for(i=0; i<total_words; i++) {
	  isStopword(stopwords, words[i]);
    }

alert("The NEW STR...." + newStr);

//split the new words formed
    var newWords = newStr.replace(/[^a-zA-Z0-9]/g,' ').split(' ');

alert("The new words are " + newWords);

//getting the count in order to iterate over all the words
    var new_total_words=newWords.length;

for (i=0; i><new_total_words; i++) {
	  if (!newWords[i])
	     continue;
	  if (assoc[newWords[i]] == null)
	     assoc[newWords[i]]=0;
	  assoc[newWords[i]]++;
    }

output = new Array();

//populate the top 3 words
    for (word in assoc) {
	  output[output.length]=word + ": " + assoc[word] + ", Density: " + Math.round((assoc[word]/new_total_words)*100);

var newDensity = Math.round((assoc[word]/new_total_words)*100;

if(!isNaN(newDensity)) {
	    if(newDensity > oldDensity1 && newDensity > oldDensity2 && newDensity > oldDensity3) {
		   oldDensity1 = newDensity;
		   topThreeArray[0] = word;
	    }
	    else if(newDensity > oldDensity2 && newDensity > oldDensity3) {
		   oldDensity2 = newDensity;
		   topThreeArray[1] = word;
	    }
	    else if(newDensity > oldDensity3) {
		   oldDensity3 = newDensity;
		   topThreeArray[2] = word;
	    }
	 }

}

//removes the stopwords
function isStopword(stopWordArray, element) {
  var flag = 0;
  for (var i = 0; i < stopWordArray.length; i++) {
    if (stopWordArray[i].toLowerCase() == element.toLowerCase()) {
      flag = 1;
    }
    if (element == " " || element == "") {
      flag = 1;
    }
  }
  if (flag == 0) {
    newStr = newStr + " " + element;
  }
}

</script>
</head>
<body>
<h2>JavaScript Keyword Density Analyzer</h2>
<textarea name="txt_text" id="txtId" cols="50" rows="5"></textarea><br/><br/>
<input type="submit" name="Submit" value="Calculate Keyword Density" onclick="javascript: count('txtId');"> //Getting error here saying object Expected???
<br/><br/>
<div id="outputId"></div>
</body>
</html>

Can you please help me out?

Jothi Shankar Kumar wrote:I run into some problems when I execute the following javascript ...

Does your keyboard burst into flame?

(In other words, just telling us "you have a problem" doesn't do anything to tell us what it is.)

Yes, I have mentioned the problem as an inline comment in the code above. It gives me an Object Expected javascript error in the line

<input type="submit" name="Submit" value="Calculate Keyword Density" onclick="javascript: count('txtId');">

which I do not understand.

Actually, I'm trying to identify the top 3 words that are being used in the textarea. I'm filtering out the unwanted words which I cann stopwords and finding out the density and based on the density, populate the array of top 3 words. Any ideas on why I get an Object Expected javascript error?

Jothi Shankar Kumar wrote:Yes, I have mentioned the problem as an inline comment in the code above.

Not a great approach. the harder you make it for people to help you, the less help you are likely to receive.

onclick="javascript: count('txtId');">

What's with the completely unnecessary "javascript:" prefix?

And why should you be surprised that it doesn't know what "count" is when such a function is never defined?

Thanks for pointing that out. But even after giving the correct method name and removing the javascript:, I keep getting the same error.

It looks like this now,

<input type="submit" name="Submit" value="Calculate Keyword Density" onclick="calculateKeywordDensity('txtId');">

That's not defined either.

Yes, corrected the spelling mistake. Later I figured out that I was missing a closing brace in one of the lines inside the calculateKeywordDensity function.