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 循环外相同。

$default 数组如下所示：

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

2 回答 2

3

您的代码有几个问题。

首先，该函数将$exclude_id其作为参数，但忽略该参数并将其分配给：

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

其次，您说这应该是多选，但是您的<SELECT>标签没有该MULTIPLE属性。

第三，您正在$row['group_id']比较$group_id. 这只允许一个默认值，而不是多个，你甚至没有设置变量（设置它的代码被注释掉，但它只会将它设置为$default数组的最后一个元素。DCoder 的评论似乎是正确的解决这个问题。

于 2012-09-19T05:53:32.470 回答
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 在他的答案中写的注释，并使用它htmlspecialchars来转义任何动态文本，例如$s_group_name.

此评论与您的​​具体问题无关，但您仍然应该考虑它。

请停止使用古老的 mysql_* 函数编写新代码。它们不再被维护并且社区已经开始了弃用过程。相反，您应该了解准备好的语句并使用PDO或MySQLi。如果你想学习，这里有一个相当不错的 PDO 相关教程。

于 2012-09-19T12:24:54.207 回答