ms-access简单的SQL代码

Question

这是好的代码吗？ 可以简化吗？

    SELECT u.id,u.title,u.title,u.first,u.last FROM 
  (((tblusers u LEFT JOIN tbluserstudentteacher 
     ON u.id = tbluserstudentteacher.student_teacher_user_id) 
     LEFT JOIN tblUsersSubjects ON u.id = tblUsersSubjects.user_id) 
     LEFT JOIN tblUserAvailability ON u.id=tblUserAvailability.user_id) 
     LEFT JOIN chavrusas ON u.id=chavrusas.luser_id 
       WHERE 1=1 AND (u.gender) LIKE 'm*' 
       AND (chavrusas.luser_type)='shliach' 
       AND (chavrusas.ruser_type)='shliach' AND (u.last LIKE 'd*') 
 GROUP BY u.id, u.title, u.title, u.first, u.last 
 ORDER BY last;

Answer 1

您这样做的方式是错误的。 查看已经提供的所有答案。

跳入代码。

根据需要进行更改，以仅在需要时才加载所需数据。

了解为什么不按要求使用分组bys会受伤，以及DISTINCTS如何提供帮助。

您如何简化查询？

Answer 2

看起来好像是从Access Query-Builder界面生成的。 它可以简化，但是我认为您应该先了解代码的使用位置以及在修改代码之前要实现的目标。

另外，除非它消耗大量资源，否则您真的不想过早地进行优化。

Answer 3

从我对您的其他问题的回答中 ，我也在哪里进行了清理。 请参阅该帖子以了解原因。

SELECT u.id, u.title, u.first, u.last
  FROM (tblusers u LEFT JOIN chavrusas c ON u.id = c.luser_id
  AND u.gender LIKE 'm*'
  AND u.last LIKE 'd*'
  AND c.luser_type = 'shliach'
  AND c.ruser_type = c.luser_type)
ORDER BY last;

Answer 4

Abrashka，您可以研究一些事情来优化上面的SQL代码。 如果可能的话，我建议对所涉及的表的表模式进行规范化处理，以帮助减少执行的连接数，并摆脱不必要的group by子句。 LIKE语句也影响性能，因此您可能还需要重新考虑使用它们。

Answer 5

对我来说还不错。 一些建议：

• 漂亮的格式，例如右对齐查询关键字以创建“管道”，并在新行上缩进ON子句。
• 在关联名称之前使用AS关键字。
• 使用DISTINCT关键字，而不是对整个SELECT子句进行分组（更好地传达了意图）。
• 将“过滤”谓词移出联接并移入WHERE子句。
• 拆下括号。
• 删除1=1谓词，并使用参数化的PROCEDURE而不是动态SQL;）

像这样：

SELECT DISTINCT u.id, u.title, u.first, u.last
  FROM tblusers AS u 
       LEFT OUTER JOIN chavrusas AS c 
          ON u.id = c.luser_id
             AND c.ruser_type = c.luser_type
 WHERE u.gender LIKE 'm*'
       AND u.last LIKE 'd*'
       AND c.luser_type = 'shliach'
 ORDER 
    BY last;

同时查看您的SQL DDL。 用户是否具有“性别”（男性，女性，中性等）或性别（ ISO 5218 ）？