SQL查询根据日期和月份检索用户

Question

I have the following table:

User

• UserID
• CompanyID
• Name
• IsActive
• Date_Joined
• Date_Left

I'm trying to get a query to get the billable users, criteria being:

• A user that joined the month of billing, is not seen as a billable user.
• A user that left on the month of billing, is a billable user.

I have the following query:

SELECT * from user u WHERE CompanyID = 1205
AND (p.IsActive = 1 AND MONTH(p.Date_Joined) != MONTH(GETDATE())
OR (p.is_active = 0 AND MONTH(p.Date_Left) = MONTH(GETDATE())

The problem is that this query isn't working as it doesn't check the year of the user (users joined in september 2017 are not seen as billable, even though they should as they were joined in 2017 and not this year, the persons who are created in september 2018 should not be seen as billable).

How can I implement the year or optimize my query?

Answer 1

EOMONTH can be used to determine the last day of month. For example:

/* last day of prev month */ EOMONTH('2018-09-05', -1) = 2018-08-31
/* last day of curr month */ EOMONTH('2018-09-05')     = 2018-09-30

You can easily check if active user joined on previous month(s) or inactive user left on current month as follows:

DECLARE @billdate AS DATE = '2018-09-05';

SELECT * 
FROM user
WHERE CompanyID = 1205 AND (
    (
        IsActive = 1 AND 
        Date_Joined <= EOMONTH(@billdate, -1)
    )
    OR
    (
        IsActive = 0 AND 
        Date_Left >  EOMONTH(@billdate, -1) AND
        Date_Left <= EOMONTH(@billdate)
    )
)

Answer 2

• I think your parenthesis are off. Putting them around an AND doesn't do anything. You probably wanted to put them around the OR .
• There is no target table for the alias p . From the column names I think the columns prefixed with it are also from user , so change the alias to u .
• And to check the year you can use year() similar to how you used month() .

SELECT *
       FROM user u
       WHERE u.companyid = 1205
             AND (u.isactive = 1
                  AND month(p.date_joined) != month(getdate())
                  AND year(p.date_joined) = year(getdate())
                   OR u.is_active = 0
                      AND month(p.date_left) = month(getdate())
                      AND year(p.date_left) = year(getdate()));

Answer 3

I think you need check between current month start date and End date. Try this

DECLARE @D_FROM_DATE DATE,@D_TO_DATE DATE

SELECT @D_FROM_DATE=DATEADD(DAY,-DATEPART(DAY,GETDATE()-1),GETDATE()),
@D_TO_DATE=DATEADD(MONTH,1,DATEADD(DAY,-DATEPART(DAY,GETDATE()),GETDATE()))

    SELECT * FROM USER P
    WHERE P.COMPANYID = 1205
    AND 
    (
        CAST(P.DATE_JOINED AS DATE) BETWEEN  @D_FROM_DATE AND @D_TO_DATE
        OR 
        CAST(P.DATE_LEFT AS DATE) BETWEEN  @D_FROM_DATE AND @D_TO_DATE
    )

Answer 4

You can use like this

SELECT * from user u WHERE CompanyID = 1205
AND (p.IsActive = 1 AND CONCAT(YEAR(p.Date_Joined),'/',MONTH(p.Date_Joined)) != CONCAT(YEAR(GETDATE()),'/',MONTH(GETDATE())))
OR (p.is_active = 0 AND CONCAT(YEAR(p.Date_Left),'/',MONTH(p.Date_Left)) = CONCAT(YEAR(GETDATE()),'/',MONTH(GETDATE())))