Question

我在尝试连接总共三个表时遇到问题：

表用户：userid，cap（ADSL带宽）
表记帐：userid，sessiondate，used bandwidth
表格adhoc：用户ID，日期，购买金额

我希望有一个查询返回一组所有用户，他们的上限，本月的使用带宽以及本月的特殊购买次数：

< TABLE 1 ><TABLE2><TABLE3>
User   | Cap | Adhoc | Used
marius | 3   | 1     | 3.34
bob    | 1   | 2     | 1.15
(simplified)

以下是我正在处理的查询：

SELECT
        `msi_adsl`.`id`,
        `msi_adsl`.`username`,
        `msi_adsl`.`realm`,
        `msi_adsl`.`cap_size` AS cap,
        SUM(`adsl_adhoc`.`value`) AS adhoc,
        SUM(`radacct`.`AcctInputOctets` + `radacct`.`AcctOutputOctets`) AS used
FROM
        `msi_adsl`
INNER JOIN
        (`radacct`, `adsl_adhoc`)
ON
        (CONCAT(`msi_adsl`.`username`,'@',`msi_adsl`.`realm`) 
           = `radacct`.`UserName` AND `msi_adsl`.`id`=`adsl_adhoc`.`id`)

WHERE
        `canceled` = '0000-00-00'
AND
        `radacct`.`AcctStartTime`
BETWEEN
        '2010-11-01'
AND
        '2010-11-31'
AND
        `adsl_adhoc`.`time`
BETWEEN
        '2010-11-01 00:00:00'
AND
        '2010-11-31 00:00:00'
GROUP BY
        `radacct`.`UserName`, `adsl_adhoc`.`id` LIMIT 10

查询有效，但它会为adhoc和used返回错误的值;我的猜测是我的联接中的逻辑错误，但我看不到它。非常感谢任何帮助。

Answer 1

根据我的口味，您的查询布局太过分散。特别是，BETWEEN / AND条件应该分别在1行，而不是每行5行。我也删除了反引号，但你可能需要它们用于'时间'专栏。

由于您的表格布局与您的示例查询不匹配，因此会让生活变得非常困难。但是，表格布局都包含UserID（这是明智的），因此我编写了查询以使用UserID执行相关联接。正如我在评论中指出的那样，如果您的设计使得必须使用CONCAT操作来连接两个表，那么您就会遇到性能灾难。更新您的实际架构，以便可以通过UserID连接表，因为您的表布局建议应该是可行的。显然，您可以在连接中使用函数结果，但是（除非您的DBMS支持“功能索引”并且您创建了适当的索引），DBMS将无法使用评估函数的表上的索引来加速查询。对于一次性查询，这可能无关紧要;对于生产查询，它通常很重要。

这有可能完成你想要的工作。由于您要聚合两个表，因此需要FROM子句中的两个子查询。

SELECT u.UserID,
       u.username,
       u.realm,
       u.cap_size AS cap,
       h.AdHoc,
       a.OctetsUsed
  FROM msi_adsl AS u
  JOIN (SELECT UserID, SUM(AcctInputOctets + AcctOutputOctets) AS OctetsUsed
          FROM radact
         WHERE AcctStartTime BETWEEN '2010-11-01' AND '2010-11-31'
         GROUP BY UserID
       )    AS a ON a.UserID = u.UserID
  JOIN (SELECT UserID, SUM(Value) AS AdHoc
          FROM adsl_adhoc
         WHERE time BETWEEN '2010-11-01 00:00:00' AND '2010-11-31 00:00:00'
         GROUP BY UserId
       )    AS h ON h.UserID = u.UserID
 WHERE u.canceled = '0000-00-00'
 LIMIT 10

每个子查询计算指定时间段内每个用户的聚合值，生成UserID和聚合值作为输出列;然后，主查询只是从主用户表中提取正确的用户数据，并与聚合子查询连接。

Answer 2

我认为问题出在这里

FROM  `msi_adsl`
INNER JOIN
        (`radacct`, `adsl_adhoc`)
ON
        (CONCAT(`msi_adsl`.`username`,'@',`msi_adsl`.`realm`)
           = `radacct`.`UserName` AND `msi_adsl`.`id`=`adsl_adhoc`.`id`)

您正在使用笛卡尔积混合连接，这不是一个好主意，因为调试起来要困难得多。试试这个：

FROM  `msi_adsl`
INNER JOIN
        `radacct`
ON
      CONCAT(`msi_adsl`.`username`,'@',`msi_adsl`.`realm`) = `radacct`.`UserName`
JOIN  `adsl_adhoc` ON  `msi_adsl`.`id`=`adsl_adhoc`.`id`

使用SUM（）在3个表上INNER JOIN

2 个答案: