低效的 SQL

Question

我不是 MySQL 专家，但到目前为止我已经设法将一些可行的东西组合在一起。不幸的是，我最近的失败尝试导致服务器崩溃，所以显然我正在做一些效率极低的事情。谁能告诉我问题出在哪里以及如何在不每次都导致整个网站瘫痪的情况下获得相同的结果？

$sqlbest = "SELECT  
        wp_postmeta.meta_value 
      , wp_posts.post_title 
      , wp_posts.ID
      , (TO_DAYS(CURDATE())- TO_DAYS(wp_posts.post_date))+1 AS days 
    FROM  `wp_postmeta` ,  `wp_posts` 
WHERE  `wp_postmeta`.`post_id` =  `wp_posts`.`ID` 
  AND  `wp_posts`.`post_date` >= DATE_SUB( CURDATE( ) , INTERVAL 1 WEEK) 
  AND  `wp_postmeta`.`meta_key` =  'views' 
  AND  `wp_posts`.`post_status` =  'publish' 
  AND wp_posts.ID != '".$currentPostID."'
GROUP BY  `wp_postmeta`.`post_id` 
ORDER BY (CAST(  `wp_postmeta`.`meta_value` AS UNSIGNED ) / days) DESC 
LIMIT 0 , 4";

$results = $wpdb->get_results($sqlbest);

它使用帖子浏览次数来计算最近发布的帖子的浏览量/天，然后按该数字对它们进行排序，并获取前 4 个。

我想我发现它效率很低，因为它必须每次都计算浏览量/天几千个帖子，但我不知道如何做得更好。

提前致谢。

Answer 1

您可以消除每次调用这些日期函数的需要，方法是将它们静态地传递到 PHP 服务器（可能不与数据库同步）的查询中，或者您可以编写一个存储过程并保存这些日期函数的结果到随后将在查询中使用的变量。

Answer 2

SELECT  
  wp_postmeta.meta_value 
  , wp_posts.post_title 
  , wp_posts.ID
  , DATEDIFF(CURDATE(),wp_posts.post_date)+1 AS days <<--1: DATEDIFF
FROM  wp_postmeta
INNER JOIN wp_posts ON (wp_postmeta.post_id =  wp_posts.ID) <<--2: explicit join
WHERE wp_posts.post_date >= DATE_SUB( CURDATE( ) , INTERVAL 1 WEEK) 
  AND wp_postmeta.meta_key = 'views' 
  AND wp_posts.post_status = 'publish' 
  AND wp_posts.ID != '".$currentPostID."'
  AND wp_postmeta.meta_value > 1   <<-- 3: extra filter
/*GROUP BY wp_postmeta.post_id */  <<-- 4: group by not needed
ORDER BY (CAST( wp_postmeta.meta_value AS UNSIGNED ) / days) DESC 
LIMIT 0 , 4;

我尝试做一些改变。

1. 将对 TO_DAYS 的两次调用替换为对 DATEDIFF 的一次调用。
2. 用显式的inner join替换了丑陋的隐式where join，这不会做任何事情，只是让事情变得更清晰。它显示的一件事是，如果 wp_postmeta.post_id 是唯一的，那么您不需要分组依据，因为内部联接只会为每个 wp_postmeta 提供一行。 post_id。
3. 添加了一个额外的过滤器来过滤掉浏览量较低的帖子，这限制了 MySQL 必须排序的行数。
4. 消除了group by只有当wp_postmeta.post_id唯一时这才是正确的！