Encrypting Script [Archive] - CodingForums.com

NanakiXIII

01-21-2004, 07:47 PM

I wrote this simple script which returns an error when I try it, it refers me to line 10, character 4, saying its expecting an object. I don't see any error, though. Any help is appreciated.

<html>
<head>
<script langueage="javascript">
function encrypt()
{
var consonants = " bcdfghjklmnpqrstvwxz";
var vowels = "aeiouy";
var message = document.encryptform.message.value;
var lowercase = toLowerCase(message);
var letter;
var encrypted;

for(i=0;i<lowercase.length;i++)
{
for(b=0;b<consonants.length;b++)
{
if(lowercase.charAt(i) == consonants.charAt(b))
{
letter = b;
if(letter == 0)
{
letter = 0;
}
else
{
letter += 5;
if(letter == 0)
{
letter += 1;
}
}
encrypted += consonants.charAt(letter);
}
else
{
for(b=0;b<vowels.length;b++)
{
if(lowercase.charAt(i) == vowels.charAt(b))
{
letter = b;
letter += 5;
if(letter == 0)
{
letter += 1;
}
}
encrypted += vowels.charAt(letter);
}
}
}
}

document.encryptform.encrypted.value = encrypted;
}
</script>
</head>
<body>
<form name="encryptform">
<textarea cols="50" rows="20" name="message"></textarea><br /><br />
<textarea cols="50" rows="20" name="encrypted"></textarea><br /><br />
<input type="button" name="submitbutton" value="Encrypt" onClick="encrypt()">
</form>
</body>
</html>

Roelf

01-21-2004, 08:05 PM

var lowercase = toLowerCase(message);
should be:
var lowercase = message.toLowerCase();

NanakiXIII

01-22-2004, 02:16 PM

Oops, thanks.

Could someone tell me why the script freezes when the form's button is pressed? I'm not getting any errors so I checked the entire script, yet can't find any flaws. I suppose the problem's in the loops, but they look fine to me.

Danne

01-22-2004, 05:16 PM

You are using the same variable for two nested loops..

for(b=0;b<consonants.length;b++)
...
for(b=0;b<vowels.length;b++)

NanakiXIII

01-22-2004, 05:27 PM

Thanks alot.