Alert message not displaying in javascript

This site is best viewed in a modern browser with JavaScript enabled.

Alert message not displaying in javascript

soupi

<?php
$expression = '';   

$input      = array( 'a' => 5,    

                     'b' => 10,
                     'c' => 15,
                     'd' => 20); 
$total = 0;
$sum   = '';
// Override default expression and input values
if(isset($_POST['Calculate']))
{
    $input      = $_POST['input'];			//input being passed via POST method
    $expression = $_POST['expression'];		//expression being passed via POST method 
}
// make sure the users expression is safe to use
if(preg_match('#^[a-z+-\s]+$#i', $expression))
{
    $sum = $expression;			//evaluates sum as expression
	$input_letters = array();	//puts letters into a array
	$input_operators = array(); //puts operators into a array


for ($i = 0; $i < strlen($expression) ; $i++) { //gets string length
if (($i % 2) == 0) {
$input_letters[] = $expression[$i]; //getting the character for the position i 
} else {
$input_operators[] = $expression[$i];
}
}
// 
for ($i = 0 ; $i < sizeof($input_letters); $i++) { //size of= numbers (4)
$value = $input[$input_letters[$i]]; //takes the value  from the letters 
if ($i == 0) { 
	$sum += $value; //adds the sum if = 0
} else {
if ($input_operators[$i-1] == '+') { //checks to see if the operator is +
$sum += $value;
} else if ($input_operators[$i-1] == '-')  { //checks to see if the operator is -
$sum -= $value;
}
}
print_r($value);
}
} else {    

echo "Error: expression not correct form"; //error message
}
?>
<form action=""  method="post">     <!--Form begins--> 
    Number A: <input type="text" name="input[a]" value="<?php echo $input['a']; ?>" size=3> <br/>
    Number B: <input type="text" name="input[b]" value="<?php echo $input['b']; ?>" size=3> <br> 
    Number C: <input type="text" name="input[c]" value="<?php echo $input['c']; ?>" size=3> <br> 
    Number D: <input type="text" name="input[d]" value="<?php echo $input['d']; ?>" size=3> <br>
    <br>
    Expression: <input type = "text" name = "expression" value="<?php echo $expression ?>">  = 
<?php echo $sum; ?><br>
    Sum: <?php echo $sum; ?><br>
    <input type="submit" name="Calculate" /> 
</form>

<script src="http://code.jquery.com/jquery-1.9.1.js"></script>
<script>
$( "form" ).submit(function( event ) {
var myExpression = $('[name="expression"]').val();
if $("$input_letters").validate({
myExpression: { maxlength: 4 
				}
        };
} else {
alert("Enter only four expressions");
}
});
</script>

in the code i provided, how would i add a javascript on the bottom check to see that user entered four letters, if they enter more then for expressions, then when submit is clicked, put a alert message saying you cant. it starts in the script section but Im not sure why it isnt working when i click submit after I run it.

Padonak

var val_is_good=($('input[name="expression"]').val().length==4)?true:false;
if(!val_is_good){alert('Enter four letters or die!');}

soupi

thanks, where would I put this coded?

soupi

Padonak;1293877 wrote:
var val_is_good=($('input[name="expression"]').val().length==4)?true:false;
if(!val_is_good){alert('Enter four letters or die!');}

thanks padonak, where would I put this code? do i erase anything else

soupi

would it be like this?

<script src="http://code.jquery.com/jquery-1.9.1.js"></script>
<script>

$( "form" ).submit(function( event ) {
var val_is_good=($('input[name="expression"]').val().length==4)?true:false;
if(!val_is_good){alert('Enter four letters or die!');}

});
</script>

soupi

When i put the script, it still runs but it gives me the alert message, is there a way to stop the forum?

Padonak

in case you want to check the length only try this:

<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8" />
<title>123</title>
<script src="http://code.jquery.com/jquery-latest.min.js"></script>
<script>
$(document).ready(function(){

$("#myform").submit(function(event){
var val_is_good=($('input[name="expression"]').val().length==4)?true:false;
if(!val_is_good){alert('Enter four letters or die!');return false;}
else{return true;}
});

});
</script>
</head>
<body>
<form id="myform" action="" method="post">     <!--Form begins--> 
Number A: <input type="text" name="input[a]" value="1" size=3 /><br/>
Number B: <input type="text" name="input[b]" value="2" size=3 /><br /> 
Number C: <input type="text" name="input[c]" value="3" size=3 /><br />
Number D: <input type="text" name="input[d]" value="4" size=3 /><br />
<br />
Expression: <input type="text" name="expression" value="11">  = sum goes here<br />
Sum: <br />
<input type="submit" name="Calculate" />
</form>
</form>
</body>
</html>

soupi

Ok, so i added your two lines and it gave me a error reguardless if I was posting four letters or not.
in the expression the user should be able to do this for example, a+b+c+d, but when they do a+b+c+d+d, thats when I want the error message to come up. but if they enter the four characters to it should be fine.

The code and forum works fine, but it still calculates when you do more then 4 characters, I just want the user to be able to input four $inputs. Im just confused how to write the javascript to do it. but i think yours was correct....

heres your javascript impleneted into mine. tell me if it works.

<?php
$expression = '';   

$input      = array( 'a' => 5,    

                     'b' => 10,
                     'c' => 15,
                     'd' => 20); 
$total = 0;
$sum   = '';
// Override default expression and input values
if(isset($_POST['Calculate']))
{
    $input      = $_POST['input'];			//input being passed via POST method
    $expression = $_POST['expression'];		//expression being passed via POST method 
}
// make sure the users expression is safe to use
if(preg_match('#^[a-z+-\s]+$#i', $expression))
{
    $sum = $expression;			//evaluates sum as expression
	$input_letters = array();	//puts letters into a array
	$input_operators = array(); //puts operators into a array


for ($i = 0; $i < strlen($expression) ; $i++) { //gets string length
if (($i % 2) == 0) {
$input_letters[] = $expression[$i]; //getting the character for the position i 
} else {
$input_operators[] = $expression[$i];
}
}
// 
for ($i = 0 ; $i < sizeof($input_letters); $i++) { //size of= numbers (4)
$value = $input[$input_letters[$i]]; //takes the value  from the letters 
if ($i == 0) { 
	$sum += $value; //adds the sum if = 0
} else {
if ($input_operators[$i-1] == '+') { //checks to see if the operator is +
$sum += $value;
} else if ($input_operators[$i-1] == '-')  { //checks to see if the operator is -
$sum -= $value;
}
}
print_r($value);
}
} else {    

echo "Error: expression not correct form"; //error message
}
?>
<form action=""  method="post">     <!--Form begins--> 
    Number A: <input type="text" name="input[a]" value="<?php echo $input['a']; ?>" size=3> <br/>
    Number B: <input type="text" name="input[b]" value="<?php echo $input['b']; ?>" size=3> <br> 
    Number C: <input type="text" name="input[c]" value="<?php echo $input['c']; ?>" size=3> <br> 
    Number D: <input type="text" name="input[d]" value="<?php echo $input['d']; ?>" size=3> <br>
    <br>
    Expression: <input type = "text" name = "expression" value="<?php echo $expression ?>">  = 
<?php echo $sum; ?><br>
    Sum: <?php echo $sum; ?><br>
    <input type="submit" name="Calculate" /> 
</form>

<script src="http://code.jquery.com/jquery-1.9.1.js"></script>
<script>

$( "form" ).submit(function( event ) {
var val_is_good=($('input[name="expression"]').val().length==4)?true:false;
if(!val_is_good){alert('Enter four letters or die!');}


});
</script>

thanks

Padonak

sometimes i'm not good with English and could not have understood your question

try this one

<?php
$expression = '';   

$input      = array( 'a' => 5,    

                     'b' => 10,
                     'c' => 15,
                     'd' => 20); 
$total = 0;
$sum   = '';
// Override default expression and input values
if(isset($_POST['Calculate']))
{
    $input      = $_POST['input'];			//input being passed via POST method
    $expression = $_POST['expression'];		//expression being passed via POST method 
}
// make sure the users expression is safe to use
if(preg_match('#^[a-z+-\s]+$#i', $expression))
{
    $sum = $expression;			//evaluates sum as expression
	$input_letters = array();	//puts letters into a array
	$input_operators = array(); //puts operators into a array


for ($i = 0; $i < strlen($expression) ; $i++) { //gets string length
if (($i % 2) == 0) {
$input_letters[] = $expression[$i]; //getting the character for the position i 
} else {
$input_operators[] = $expression[$i];
}
}
// 
for ($i = 0 ; $i < sizeof($input_letters); $i++) { //size of= numbers (4)
$value = $input[$input_letters[$i]]; //takes the value  from the letters 
if ($i == 0) { 
	$sum += $value; //adds the sum if = 0
} else {
if ($input_operators[$i-1] == '+') { //checks to see if the operator is +
$sum += $value;
} else if ($input_operators[$i-1] == '-')  { //checks to see if the operator is -
$sum -= $value;
}
}
print_r($value);
}
} else {    

echo "Error: expression not correct form"; //error message
}
?>
<script src="http://code.jquery.com/jquery-1.9.1.js"></script>
<style>
input{text-align:center;}
input[type=submit]{cursor:pointer;}
#err{position:absolute;top:70px;left:150px;display:none;font-weight:bold;}
#err span{color:Crimson;}
</style>
<form action=""  method="post">     <!--Form begins--> 
    Number A: <input type="text" name="input[a]" value="<?php echo $input['a']; ?>" size="3" /><br />
    Number B: <input type="text" name="input[b]" value="<?php echo $input['b']; ?>" size="3" /><br />
    Number C: <input type="text" name="input[c]" value="<?php echo $input['c']; ?>" size="3" /><br />
    Number D: <input type="text" name="input[d]" value="<?php echo $input['d']; ?>" size="3" /><br />
    <br>
    Expression: <input type = "text" name = "expression" value="<?php echo $expression ?>">  = <?php echo $sum; ?><br />
    Sum: <?php echo $sum; ?><br />
	<input type="submit" name="Calculate" />
	<div id="err"></div>
</form>
<script>
$("form").submit(function(event){
var re=/^([A-Za-z]{1})+([\+\-]{1})+([A-Za-z]{1})+([\+\-]{1})+([A-Za-z]{1})+([\+\-]{1})+([A-Za-z]{1})$/,
val_is_good=re.test($('input[name="expression"]').val()),
err_msg='<span>Enter four letters divided by math operators</span> "+" <span>or</span> "-" <span>!</span>',
err=$('#err');
if(!val_is_good){
err.html(err_msg).fadeIn(800).delay(1000).fadeOut(500,function(){$(this).empty();$('input[name="expression"]').val('').focus();});
}
return val_is_good;
});
</script>

soupi

thanks, what does this line do?
var re=/^{([A-Za-z]{1})+([+-]{1})+([A-Za-z]{1})+([+-]{1})+([A-Za-z]{1})+([+-]{1})+([A-Za-z]{1})$/,}

soupi

can you write some comments in the script so I know what is going on, I want to rewrite it on my own and am going to try to follow your good comments on the lines, thank you.

Padonak

<script>
$("form").submit(function(event){
/* re is a regular expression which matches (i hope lol) a 
string like "letter math.operator letter math.operator letter math.operator letter" (without spaces, and math.operator can be "+" or "-") */
var re=/^([A-Za-z]{1})+([\+\-]{1})+([A-Za-z]{1})+([\+\-]{1})+([A-Za-z]{1})+([\+\-]{1})+([A-Za-z]{1})$/,
/* regularExpression.test(somestring) returns true if it matches and false if not matches the string 
so we can save the result in the variable val_is_good for futher use*/
val_is_good=re.test($('input[name="expression"]').val()),
/* this is just a message */
err_msg='<span>Enter four letters divided by math operators</span> "+" <span>or</span> "-" <span>!</span>',
/* this is a shorthand for $('#err') element which is <div id="err"></div> */
err=$('#err');
/* actions if not matches */
if(!val_is_good){
/* insert message.appear.timeout.disappear and a callback function when disappearing done*/
err.html(err_msg).fadeIn(800).delay(1000).fadeOut(500,function(){$(this).empty();$('input[name="expression"]').val('').focus();});
}
/* this submits the form if format correct (val_is_good=true) or prevent submitting (val_is_good=false) */
return val_is_good;
});
</script>

soupi

thanks padonak, you explained it very good. I learned alot .

Padonak

you're welcome )) sorry for lame english

soupi

no problem, reguardless of your english you were a BIG help.