低效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这没有做任何事情，只是使事情更清晰。它显示的一件事，如果wp_postmeta.post_id是唯一的，那么你不需要通过组，因为内部连接只会给每一wp_postmeta.post_id一行。
3. 添加了一个额外的过滤器来过滤低查看次数的帖子，这限制了MySQL必须排序的行数。
4. 消除group by这是唯一的，如果wp_postmeta.post_id是唯一的！