2010-05-01 43 views
0
/* Errors exist, have user correct them */ 
    if($form->num_errors > 0) 
    { 
     return 1; //Errors with form 
    } 
    /* No errors, add the new account to the */ 
    else if($database->addLeagueInformation($subname, $subformat, $subgame, $subseason, $subwindow, $subadmin, $subchampion, $subtype)) 
    { 
     $database->addLeagueTable(); 
     $_SESSION['players'] == $subplayers; 
     $comp_name == '$format_$game_$name_$season'; 
     $_SESSION['comp_name'] == $comp_name; 
     return 0; //New user added succesfully 
    } 
    else 
    { 
     return 2; //Registration attempt failed 
    } 

这是没有做这些事情的时刻:构建一个如果,如果别的,else语句

$database->addLeagueTable(); 
    $_SESSION['players'] == $subplayers; 
    $comp_name == '$format_$game_$name_$season'; 
    $_SESSION['comp_name'] == $comp_name; 

有没有更好的方式来做到这一点?

编辑!

$comp_name = "$subformat_$subgame_$subname_$subseason"; 
     $_SESSION['comp_name'] = $comp_name; 

这段代码只会在$ subseason中产生什么? 这是否有明显的原因?

进一步编辑!

else if($database->addLeagueInformation($subname, $subformat, $subgame, $subseason, $subwindow, $subadmin, $subchampion, $subtype)) 
    { 
     $_SESSION['players'] = $subplayers; 
     $comp_name = "$subformat_$subgame_$subname_$subseason"; 
     $_SESSION['comp_name'] = $comp_name; 
     $database->addLeagueTable(); 
     return 0; //New user added succesfully 
    } 

和函数addLeagueTable()

function addLeagueTable() { 
    $q = "CREATE TABLE `$_SESSION[comp_name]` (
    `user` VARCHAR(30) NOT NULL , 
     `team` VARCHAR(40) NOT NULL , 
     `home_games_played` INT(3) NULL DEFAULT '0', 
     `home_wins` INT(3) NULL DEFAULT '0', 
     `home_draws` INT(3) NULL DEFAULT '0', 
     `home_losses` INT(3) NULL DEFAULT '0', 
     `home_points` INT(3) NULL DEFAULT '0', 
     `home_goals_for` INT(3) NULL DEFAULT '0', 
     `home_goals_against` INT(3) NULL DEFAULT '0', 
     `away_games_played` INT(3) NULL DEFAULT '0', 
     `away_wins` INT(3) NULL DEFAULT '0', 
     `away_draws` INT(3) NULL DEFAULT '0', 
     `away_losses` INT(3) NULL DEFAULT '0', 
     `away_points` INT(3) NULL DEFAULT '0', 
     `away_goals_for` INT(3) NULL DEFAULT '0', 
     `away_goals_against` INT(3) NULL DEFAULT '0' 
     )"; 
    return mysql_query($q, $this->connection); 
} 

什么想法?

我是怎么过..

 $retval = $session->createLeague($_POST['name'], $_POST['players'], $_POST['format'], $_POST['game'], $_POST['season'], $_POST['window'], $_POST['admin'], $_POST['champion'], $_POST['type']); 

这将它们发送给addLeagueInformation发送到功能!

回答

5

==是比较运算符,但是例如,在$_SESSION['players'] == $subplayers;中,您希望赋值运算符=
也许您还有一个问题$comp_name == '$format_$game_$name_$season';。在单引号字符串中,php不会替代变量。

+0

以下几行同样如此 – 2010-05-01 20:18:50

1

有没有更好的方法来做到这一点?

其实执行任务?

$_SESSION['players'] = $subplayers; 
// ------------------^ one '=' instead of two '==' (which is for comparison). 
$comp_name = "{$format}_{$game}_{$name}_{$season}"; 
// double quotes to allow substitution, 
// {...} to avoid interpreting the variable name as `$format_`. 
$_SESSION['comp_name'] = $comp_name; 

并确保addLeagueInformation真的在您的测试返回非错误值,如果你希望他们被执行。

+0

这有帮助,但我不认为$ comp_name ='$ format_ $ game_ $ name_ $ season'; 正在工作吗? – sark9012 2010-05-01 20:25:31

+0

@Luke:你需要双引号。 – kennytm 2010-05-01 20:27:04

+0

它使表格$ format_ $ game_ $ name_ $季节名称。 – sark9012 2010-05-01 20:28:04