如果你的代码被删除,我做了一些小改动。 我的建议是将职能清单格式细分为较小的职能。 例如,可以将Scanners移至自己的功能中。 因此,你不以结果为由放弃这一职能,而是将每个分任务分成一个较小的职能。 您也询问了您的当选。 如果你只需要两个或三个领域,这也是一个绩效问题。 你也可以回答你提出的问题。

public function ListRoomTotals($room, $date, $dateTo = null) {
    // If dateTo hasn t been set, make it now
    if(!isset($dateTo) || $dateTo == "") { //<-- Replaced or with ||
        $dateTo = $date;
    }

    // Return an array with each bundle number 以及 the quantity for each day
    $scanner = $this->GetScannerNumber($room);
    $sql = "SELECT * FROM `scanners` WHERE `Scanner` IN (".implode( , , $scanner);

    // replace foreach loop with implode
    //foreach($scanner as $x) {
    //  $sql .= $x . ",";
    //}

    $sql .= "0)
    AND `Date` BETWEEN  " . $date . "  AND  " . $dateTo . " 
    GROUP BY `KordNo`, `BundleNumber`;";

    $result = mysql_query($sql);

    while($row = mysql_fetch_array($result)) {
        $sql = "SELECT `BundleReference`, `QtyIssued`, `WorksOrder`,
        FROM `ArchiveBundle`
        WHERE `KordNo` =  " . $row[ KordNo ] . " 
        AND `BundleNumber` =  " . $row[ BundleNumber ] . " ;";

        $result2 = mysql_query($sql);

        while($row = mysql_fetch_array($result2)) {     
            $sql = "SELECT `ArchiveBundle`.`QtyIssued`, `ArchiveBundle`.`WorksOrder`,
            `ArchiveBundle`.`Colour`, `ArchiveBundle`.`Size`
            FROM `ArchiveBundle`
            WHERE `ArchiveBundle`.`KordNo` =  " . $x[ KordNo ] . "  AND
            `ArchiveBundle`.`BundleNumber` =  " . $x[ BundleNumber ] . " ;";

            $result3 = mysql_query($sql);
            $row     = mysql_fetch_row($result3);

            // Now we need to query what the product group is
            $sql = "SELECT `Stock`.`ProductGroup`
            FROM `Stock`, `TWOrder`
            WHERE `TWOrder`.`WorksOrderNumber` =  " . $row[1] . "  AND
            `TWOrder`.`Colour` =  " . $row[2] . "  AND
            `TWOrder`.`Size` =  " . $row[3] . "  AND
            `Stock`.`ProductCode` = `TWOrder`.`Product`;";

            $result4 = mysql_query($sql);
            $row2    = mysql_fetch_row($result4);

            if(in_array($row2[0], array(50, 100, 150, 300 , 350 , 925 ,930, 940))) {
                while($row = mysql_fetch_array($result2)) {
                    if($row[0] != "") {
                        $final[] = $row;
                    } else {
                        $final[] = array("Can t find bundle number", "N/A");
                    }
                }
            }
            return $final;
        }
    }
}

以及

$result = $scanner->ListRoomTotals($_GET[ room ], $_GET[ date ]);
    $total = 0;

    foreach($result as $x) {
        $content .= "<tr>
            <td>" . $x[0] . "</td>
            <td>" . $x[1] . "</td>
        </tr>";
        $total += $x[1];
    }
    $weeklyTotal += $total;
    $content .= "<tr><td>Total Pairage:</td><td>".$total."                    </td></tr>
            <tr><td>Total Dozens:</td><td>".number_format($total/12,2)."</td></tr></table>
</td>"; //<-- adding the closing double quote

最新资料:

无法确定:

$x[ KordNo ]
$x[ BundleNumber ]

页: 1

$row[ KordNo ]
$row[ BundleNumber ]

您的法典的编排格式使得人们难以看到什么进展。

清理后,似乎与你一样,进行了一系列的KQ查询,每个查询都取决于以前的询问。 利用联合网络进行单一查询,获取你想要的数据,将更有意义。

此外,请你提出两个问题。

 $sql = "SELECT `BundleReference`, `QtyIssued`, `WorksOrder`,
         FROM `ArchiveBundle`
         WHERE `KordNo` =  " . $row[ KordNo ] . " 
           AND `BundleNumber` =  " . $row[ BundleNumber ] . " ;";

以及

 $sql = "SELECT `ArchiveBundle`.`QtyIssued`, `ArchiveBundle`.`WorksOrder`,
                `ArchiveBundle`.`Colour`, `ArchiveBundle`.`Size`
         FROM `ArchiveBundle`
         WHERE `ArchiveBundle`.`KordNo` =  " . $x[ KordNo ] . "  
           AND `ArchiveBundle`.`BundleNumber` =  " . $x[ BundleNumber ] . " ;";

这似乎多余。

另外,你在此提及<条码>x,但<条码>x只是一个指数变量,用以生成一个 com式扫描仪清单;我怀疑这是你意思的。

My advice is this: figure out what information you re trying to pull from the DB, work out the correct SQL for this data request 以及 then go from there.

根据您的法典,我think。 请问:

   SELECT `scanners`.*, ArchiveBundle`.`QtyIssued`, `Stock`.`ProductGroup`
    FROM `Stock` INNER JOIN `TWOrder` ON 
                    `Stock`.`ProductCode` = `TWOrder`.`Product`
                 INNER JOIN `ArchiveBundle` ON
                    `TWOrder`.`WorksOrderNumber` = `ArchiveBundle`.`WorksOrder` AND
                    `TWOrder`.`Colour` = `ArchiveBundle`.`Colour` AND
                    `TWOrder`.`Size` = `ArchiveBundle`.`Size`
                 INNER JOIN `scanners` ON
                    `ArchiveBundle`.`KordNo` = `scanners`.`KordNo` AND 
                    `ArchiveBundle`.`BundleNumber` = `scanners`.`BundleNumber`
    WHERE `scanners`.`Scanners` IN (?...?)";