2012-09-19 102 views
3

我有一个多场这样的:多重选择字段 - 选择=“选择”多次

<select name="excluded_groups[]"> 
    <?php echo $foo->multi_group_select_options($group_ids, $excluded_id); ?> 
</select> 

我管理通过此功能可以从数据库中获取结果,并把它们放在<select>,但我无法管理选定的选定值。

第一个参数应该将selected="selected"添加到提交之前标记的字段中,然后提交,第二个参数可以防止显示group_id(第二个参数按原样工作)。

下面是函数...

/** 
* group_options 
* Get group names in the dropdown list 
*/ 
public function multi_group_select_options($default = false, $exclude_id = '') 
{ 
    global $user; 

    $exclude_id = (isset($this->config['default_group'])) ? $this->config['default_group'] : 5; 

    $sql_where = ($user->data['user_type'] == USER_FOUNDER) ? '' : 'WHERE group_founder_manage = 0'; 
    $sql_where_and = (!empty($sql_where)) ? ", AND group_id <> $exclude_id" : "WHERE group_id <> $exclude_id"; 
    $sql = 'SELECT group_id, group_name, group_type 
      FROM ' . GROUPS_TABLE . " 
      $sql_where 
      $sql_where_and 
      ORDER BY group_name"; 
    $result = mysql_query($sql); 

    $s_group_options = ''; 
    while ($row = mysql_fetch_assoc($result)) 
    { 
     /*if (is_array($default)) 
     { 
      break; 
      $group_id = ''; 
      foreach ($default as $key => $group_id) 
      { 
       $group_id = $group_id; 
      } 
     } 

     print_r($default);*/ 

     $selected = ($row['group_id'] == $group_id) ? ' selected="selected"' : ''; 
     $s_group_name = ($row['group_type'] == GROUP_SPECIAL) ? $user->lang['G_' . $row['group_name']] : $row['group_name']; 
     $s_group_options .= '<option' . (($row['group_type'] == GROUP_SPECIAL) ? ' class="sep"' : '') . ' value="' . $row['group_id'] . '"' . $selected . '>' . $s_group_name . '</option>'; 
    } 
    $db->sql_freeresult($result); 

    return $s_group_options; 
} 

我把作为第一个参数数组是完全有效的。它只是一个带有键和值的正常数组,其中的值是组ID。

尝试在while中使用foreach - 没有工作,与while循环外部相同。

的$默认的阵列是这样的:

Array 
(
    [0] => 1 
    [1] => 7 
    [2] => 2 
    [3] => 3 
) 
+1

你没有设置'$ group_id'。试试'$ selected =(is_array($ default)&& in_array($ row ['group_id'],$ default))? 'selected =“选择”':'';'而是。 – DCoder

+0

您的解决方案奏效!谢谢。你能否发表你的评论作为答案,以便我可以接受它? – Aborted

回答

1

你没有设置$group_id下面的代码将使用默认值的数组,以及单个值工作:

if(is_array($default)) { 
    $selected = in_array($row['group_id'], $default); 
} else { 
    $selected = !strcasecmp($row['group_id'], $default); 
} 
$selected = $selected ? ' selected="selected"' : ''; 

除此之外,请看笔记@Barmar在his answer中写道,并使用htmlspecialchars来转义任何动态文本,如$s_group_name


此评论与您的​​具体问题无关,但您仍应考虑此问题。

请停止使用古代mysql_ *函数编写新代码。他们不再维护和社区已经开始deprecation process。相反,您应该了解准备好的声明并使用PDOMySQLi。如果你在意学习,here is a quite good PDO-related tutorial

+0

感谢您对mysql_函数的评论。我会记下你的话。 – Aborted

3

有几个问题与您的代码。

首先,函数将$exclude_id作为一个参数,但忽略了论证,并给它分配:

$exclude_id = (isset($this->config['default_group'])) ? $this->config['default_group'] : 5; 

其次,你说,这应该是一个多选,但你<SELECT>标签不具有MULTIPLE属性。

三,你在比较$row['group_id']$group_id。这只允许一个默认值,而不是多个,而且你甚至不设置该变量(设置它的代码已被注释掉,但它只会将其设置为$default数组的最后一个元素.DCoder的注释看起来像。正确地解决了这个