Jump to content

Archived

This topic is now archived and is closed to further replies.

indalecio

Running If / Else argument problems

Recommended Posts

This DOES use phpBB, but thats not the concern.  I am trying to get this If / Else statement to read the $problem variable, but for some reason it just doesn't to, and it moves onto the item creation.  I have tried this current setup, isset(), and !isset(). 

Any help is appreciated.


[code]/////////////

// Weapons //

/////////////



if ( $_POST['equipment_type'] == 'weapon' ) {

if ( isset($_POST['submit']) ) {

if ( !isset($_POST['weapon_name']) ) {
$problem = '1';
$problem_statement = 'Your weapon did not have a name<br>';
}
if ( !isset($_POST['weapon_type']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'Your weapon did not have a type<br>';
}
if ( !isset($_POST['weapon_job1']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'Your weapon did not have a class<br>';
}
if ( !isset($_POST['weapon_job1']) && isset($_POST['weapon_job2']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'There was a problem with the classes you chose.<br>';
}
if ( !isset($_POST['weapon_job1']) && !isset($_POST['weapon_job2']) && isset($_POST['weapon_job3']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'There was a problem with the classes you chose.<br>';
}
if ( !isset($_POST['weapon_dmg_low']) && !isset($_POST['weapon_dmg_hi']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'Your weapon is missing a damage parameter.<br>';
}
if ( !isset($_POST['weapon_lvl']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'Your weapon is missing the level parameter.<br>';
}
if ( !isset($_POST['weapon_cost']) ) {
$problem = '1';
$problem_statement = $problem_statement . 'Your weapon is missing the cost parameter.<br>';
}
if ( !isset($_POST['weapon_weight']) ) {
$_POST['weapon_weight'] = '0';
}
if ( !isset($_POST['weapon_def']) ) {
$_POST['weapon_def'] = '0';
}

if ($problem == '1') {

if ( $_POST['weapon_hands'] == '1' ) {

$template->set_filenames(array(
'body' => 'admin/weapon_add_one.tpl')
);

$template->assign_vars(array(
'PROBLEM_STATEMENT' => $problem_statement )
        );

} elseif ( $_POST['weapon_hands'] == '2' ) {

$template->set_filenames(array(
'body' => 'admin/weapon_add_two.tpl')
);

$template->assign_vars(array(
'PROBLEM_STATEMENT' => $problem_statement )
        );
}

} else {


/****************************
* Insert weapon to database *
*****************************/

$update = DATABASE INFO


if (!$update) {

message_die(GENERAL_MESSAGE, 'Your weapon could not be added.' );

}


$success = TRUE;

}

} else {



/************************
* Go to Add weapon Page *
*************************/
if ( $_POST['weapon_hands'] == '1' ) {


$template->set_filenames(array(

'body' => 'admin/weapon_add_one.tpl' )
);

} elseif ( $_POST['weapon_hands'] == '2' ) {


$template->set_filenames(array(

'body' => 'admin/weapon_add_two.tpl' )

);
}



$template->assign_vars(array(

'FORM_ACTION' => append_sid("admin_equipment.$phpEx") )

        );

}


[/code]

Share this post


Link to post
Share on other sites
Try adding a semicolon after every if:

if ( !isset($_POST['weapon_def']) ) {
$_POST['weapon_def'] = '0';
}[color=red][b];[/b][/color]

Orio.

Share this post


Link to post
Share on other sites
if that fails still, try putting the "equipment_type" into a variable be4 running the if-- but i myab e wrong in that.

Share this post


Link to post
Share on other sites
Do you really need to quote your integers..., that could have something to do with your problem.

Share this post


Link to post
Share on other sites
First, the first answer about putting a semi-colon after the end curly bracket is incorrect.
Next, copying the value to a temporary variable will not make any difference.

How is your form set up? Are all of the posted fields you are testing text fields? If they are, then they will be set when your script is invoked when the form is submitted whether or not any data is entered. What you need to do is test to see if the field is blank.

I usually do the following test:
[code]<?php
if (strlen(trim(stripslashes($_POST['somefield']))) == 0) {
//
//  field is blank
//
}?>[/code]

Also, you can use:
[code]<? $problem_statement .= 'New message'; ?>[/code]
instead of
[code]<? $problem_statement = $problem_statement . 'New Message'; ?>[/code]

Another potential problem is that I don't see you initializing the $problem variable to '0'. I would initialize it to 'false' and then when you find an error set it to 'true', then your if statement would be:
[code]<? if ($problem) ?>[/code]

Ken

Share this post


Link to post
Share on other sites

×

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.