3

関数の操作の数を 1 つのクエリだけに減らそうとしましたが、結果を取得できません。 

function retrive_avaible_operator($Hostname, $Username, $Password, $DatabaseName, $SupportUserPerDepaTable, $SupportUserTable,$dep){
    $query = "SELECT b.id  
              FROM ".$SupportUserTable." b
              INNER JOIN ".$SupportUserPerDepaTable." a
                  ON b.id=a.user_id
              WHERE a.department_id=? AND b.holiday='0' AND a.user_id!=".$_SESSION['id']." 
              ORDER BY b.assigned_tickets ASC LIMIT 1";

    $mysqli = new mysqli($Hostname, $Username, $Password, $DatabaseName);
    $stmt = $mysqli->stmt_init();
    $prepared = $stmt->prepare($query);

    if($prepared){
        if($stmt->bind_param('i', $dep)){
            if($stmt->execute()){
                $stmt->store_result();
                $result = $stmt->bind_result($camaro);
                if($stmt->num_rows>0){
                    while (mysqli_stmt_fetch($stmt))
                        $selopid=$camaro;
                    return $selopid;
                }
                else
                    return 'No Operator Available';
            }
            else
                return mysqli_stmt_error($stmt);
        }
        else
            return mysqli_stmt_error($stmt);
    }
    else
        return mysqli_stmt_error($stmt);
}

これを試す前の操作は次のとおりです (これらは 2 つの異なるクエリです)。

  1. $SupportUserPerDepaTableどこからすべてのIDを選択しますa.department_id=?
  2. $SupportUserTableid が前の結果内にある場所からすべての id を選択し、holiday='0'

うまくいきましたが、今は返すだけですNo Operator Available

また、これらはテーブルです：

CREATE TABLE IF NOT EXISTS `razorphyn_support_user_departments` (
`id`                BIGINT(11)  UNSIGNED    NOT NULL    AUTO_INCREMENT,
`department_id`     BIGINT(11)  UNSIGNED    NOT NULL,
`department_name`   VARCHAR(70)             NOT NULL,
`user_id`           BIGINT(11)  UNSIGNED    NOT NULL,
`holiday`           ENUM('0','1')           NOT NULL    DEFAULT '0',
PRIMARY KEY (`id`),
UNIQUE KEY(`department_name`,`user_id`),
INDEX(`department_id`,`department_name`,`user_id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=0;

と

CREATE TABLE IF NOT EXISTS `razorphyn_support_users` (
`id`                BIGINT(15)      UNSIGNED        NOT NULL AUTO_INCREMENT,
`name`              VARCHAR(50)                     NOT NULL,
`mail`              VARCHAR(50)                     NOT NULL,
`password`          VARCHAR(200)                    NOT NULL,
`reg_key`           VARCHAR(260)                    ,
`tmp_password`      VARCHAR(87)                     ,
`ip_address`        VARCHAR(50)                     NOT NULL,
`status`            ENUM('0','1','2','3','4')       NOT NULL    DEFAULT '3',
`holiday`           ENUM('0','1')                   NOT NULL    DEFAULT '0',
`mail_alert`        ENUM('no','yes')                NOT NULL    DEFAULT 'yes',
`assigned_tickets`  INT(5)          UNSIGNED        NOT NULL    DEFAULT 0,
`solved_tickets`    BIGINT(11)      UNSIGNED        NOT NULL    DEFAULT 0,
`number_rating`     BIGINT(6)       UNSIGNED        NOT NULL    DEFAULT 0,
`rating`            DECIMAL(4,2)    UNSIGNED        NOT NULL    DEFAULT 0,
PRIMARY KEY (`id`),
UNIQUE KEY(`mail`),
INDEX (`name`,`mail`,`status`,`holiday`,`assigned_tickets`,`solved_tickets`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=55;

編集
私は見つけました（残念ながら、beacuseは非常に醜くて面倒です）以前のコード（すべてのコントロールを削除しましたが、動作します）：

function retrive_avaible_operator($Hostname, $Username, $Password, $DatabaseName, $SupportUserPerDepaTable, $SupportUserTable,$dep){

$query = "SELECT `user_id`  FROM ".$SupportUserPerDepaTable." WHERE `department_id`=? AND `user_id`!=".$_SESSION['id'] ;
$mysqli = new mysqli($Hostname, $Username, $Password, $DatabaseName);
$stmt = $mysqli->stmt_init();
$prepared = $stmt->prepare($query);
    $stmt->bind_param('i', $dep)
        $stmt->execute()
            $stmt->store_result();
            $operator=array();
            $result = $stmt->bind_result($camaro);
            if($stmt->num_rows>0){
                while (mysqli_stmt_fetch($stmt))
                    $operator[]=$camaro;
                $operator=join(',',$operator);
                $query = "SELECT `id`  FROM ".$SupportUserTable." WHERE `id` IN (".$operator.") AND `holiday`='0' ORDER BY `assigned_tickets` ASC LIMIT 1" ;
                $prepared = $stmt->prepare($query);
                $prepared)
                    $stmt->execute()
                        $stmt->store_result();
                        $result = $stmt->bind_result($camaro);
                        if($stmt->num_rows>0){
                            while (mysqli_stmt_fetch($stmt))
                                $selopid=$camaro;
                            return $selopid;
                        }
                        else{
                            $query = "SELECT `id` FROM ".$SupportUserTable." WHERE `status`=2 AND `holiday`=0 AND `id`!=".$_SESSION['id']." ORDER BY `assigned_tickets` ASC, `solved_tickets` ASC LIMIT 1" ;
                            $prepared = $stmt->prepare($query);
                            $prepared
                                $stmt->execute()
                                    $stmt->store_result();
                                    $result = $stmt->bind_result($camaro);
                                    if($stmt->num_rows>0){
                                        while (mysqli_stmt_fetch($stmt))
                                            $selopid=$camaro;
                                        return $selopid;
                                    }
                                    else
                                        return 'No Operator Available';
                        }
            }
            else{
                $query = "SELECT `id` FROM ".$SupportUserTable." WHERE `status`='2' AND `holiday`='0' AND `id`!=".$_SESSION['id']." ORDER BY `assigned_tickets` ASC, `solved_tickets` ASC LIMIT 1" ;
                $prepared = $stmt->prepare($query);
                    $stmt->execute()
                        $stmt->store_result();
                        $result = $stmt->bind_result($camaro);
                        if($stmt->num_rows>0){
                            while (mysqli_stmt_fetch($stmt))
                                $selopid=$camaro;
                            return $selopid;
                        }
                        else
                            return 'No Operator Available';
            }
$mysqli->close();
}
4

1 に答える 1

0

これでかなり凝縮できるはずです。最初に$SupportUserPerDepaTable、user_id を取得するためだけにクエリを実行し、それらを 2 番目のクエリのカンマ区切りの文字列に結合するだけです。これをサブクエリに変えることでこれを排除できるはずです。3 番目と 4 番目のクエリは冗長であり、前の if/else ケースの製品であるため、上位の if 条件を削除したので、それらを組み合わせることができます。一番下の「醜い」コードを使用して、次のように要約しました。

$mysqli = new mysqli($Hostname, $Username, $Password, $DatabaseName);
$stmt = $mysqli->stmt_init();
$query = "
SELECT      `id`  
FROM        ".$SupportUserTable." 
WHERE       `id` IN (
    SELECT      `user_id`  
    FROM        ".$SupportUserPerDepaTable." 
    WHERE       `department_id`= ? 
    AND         `user_id`!=".$_SESSION['id']."
)
AND         `holiday`='0'
ORDER BY    `assigned_tickets` ASC 
LIMIT       1" ;

$prepared = $stmt->prepare($query);
$stmt->bind_param('i', $dep);
$stmt->execute();
$stmt->store_result();
$operator=array();
$result = $stmt->bind_result($camaro);
if($stmt->num_rows>0){
    while (mysqli_stmt_fetch($stmt))
        $selopid=$camaro;
    return $selopid;
}
else{
    $query = "
    SELECT      `id` 
    FROM        ".$SupportUserTable." 
    WHERE       `status`= 2 
    AND         `holiday`=0 
    AND         `id`!=".$_SESSION['id']." 
    ORDER BY    `assigned_tickets` ASC
                ,`solved_tickets`  ASC 
    LIMIT       1" ;
    $prepared = $stmt->prepare($query);
    $stmt->execute();
    $stmt->store_result();
    $result = $stmt->bind_result($camaro);
    if($stmt->num_rows>0){
        while (mysqli_stmt_fetch($stmt))
            $selopid=$camaro;
        return $selopid;
    }
    else{
        return 'No Operator Available';
    }
}
$mysqli->close();

次のように、合計制限が 1 の UNION を使用して、2 つのクエリを 1 つにさらに圧縮できる場合があります。

(
    SELECT      `id`  
    FROM        ".$SupportUserTable." 
    WHERE       `id` IN (
        SELECT      `user_id`  
        FROM        ".$SupportUserPerDepaTable." 
        WHERE       `department_id`= ? 
        AND         `user_id`!=".$_SESSION['id']."
    )
    AND         `holiday`='0'
    ORDER BY    `assigned_tickets` ASC 
    UNION
    SELECT      `id` 
        FROM        ".$SupportUserTable." 
        WHERE       `status`= 2 
        AND         `holiday`=0 
        AND         `id`!=".$_SESSION['id']." 
        ORDER BY    `assigned_tickets` ASC
                    ,`solved_tickets`  ASC 
)
LIMIT       1;

またはOR、dept テーブル内の従業員のステータス コードに応じて、次のような条件を使用することもできます。

SELECT      `id`  
FROM        ".$SupportUserTable." 
WHERE       `holiday`='0'
AND (       
        `id`    IN (
            SELECT      `user_id`  
            FROM        ".$SupportUserPerDepaTable." 
            WHERE       `department_id`= ? 
            AND         `user_id`!=".$_SESSION['id']."
        )
        AND     `status`= ('some code other than 2?') 
    )
    OR  (
                `status`= 2 
        AND     `id`! = ".$_SESSION['id']."
)
ORDER BY    status (asc or desc?)
            ,`assigned_tickets` ASC
            ,`solved_tickets`  ASC 
LIMIT       1;

データベーステーブルのセットアップ、サンプルデータの入力、およびすべてのテストに時間がかかりすぎるため、これをテストしていません。小さなエラーがあれば許してください。ただし、これは正しい方向を示しているはずです. これが機能せず、サンプル データをhttp://sqlfiddle.com/に投稿し、予想される出力の例をいくつか示したい場合は、クエリを再試行して何ができるかを確認します。

于 2013-08-02T11:20:34.867 に答える