2014-12-07 121 views
0

我有一个程序,显示每周的成员的每周生日。此代码正常运行,直到本月底。每当我们到达月底并且当前月份还剩下不到7天时,代码不再列出生日。每月从数据库获取每周生日的问题

我知道这是因为本周现在跨越两个月,但我已经尝试了一切,只是无法弄清楚我必须做什么才能解决这个问题。这可能是一个简单的问题。任何帮助将不胜感激。这里是我的代码:

public void CheckDayOfWeekAndReturnBirthdaysInDateRange() 
    { 
     //Check the current day of the week and make calculations according to find out which week to use. 
     if (DateTime.Now.DayOfWeek.ToString() == "Sunday") 
     { 
      DateTime weekStart = DateTime.Now.Date; 
      DateTime weekEnd = DateTime.Now.AddDays(7); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
     else if (DateTime.Now.DayOfWeek.ToString() == "Monday") 
     { 
      DateTime weekStart = DateTime.Now.AddDays(-1); 
      DateTime weekEnd = DateTime.Now.AddDays(6); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
     else if (DateTime.Now.DayOfWeek.ToString() == "Tuesday") 
     { 
      DateTime weekStart = DateTime.Now.AddDays(-2); 
      DateTime weekEnd = DateTime.Now.AddDays(5); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
     else if (DateTime.Now.DayOfWeek.ToString() == "Wednesday") 
     { 
      DateTime weekStart = DateTime.Now.AddDays(-3); 
      DateTime weekEnd = DateTime.Now.AddDays(4); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
     else if (DateTime.Now.DayOfWeek.ToString() == "Thursday") 
     { 
      DateTime weekStart = DateTime.Now.AddDays(-4); 
      DateTime weekEnd = DateTime.Now.AddDays(3); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
     else if (DateTime.Now.DayOfWeek.ToString() == "Friday") 
     { 
      DateTime weekStart = DateTime.Now.AddDays(-5); 
      DateTime weekEnd = DateTime.Now.AddDays(2); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
     else if (DateTime.Now.DayOfWeek.ToString() == "Saturday") 
     { 
      DateTime weekStart = DateTime.Now.AddDays(-6); 
      DateTime weekEnd = DateTime.Now.AddDays(1); 

      Birthdays = eventsService.ReturnBirthdaysForCurrentWeek(weekStart, weekEnd); 
     } 
    } 

    public void LoadDatagridComponents(DataGridView dataBirthdays) 
    { 
     dataBirthdays.AutoGenerateColumns = false; 
     dataBirthdays.Columns[0].Visible = true; 
     dataBirthdays.Columns[1].Visible = true; 
     dataBirthdays.Columns[2].Visible = true; 

     dataBirthdays.Columns[0].HeaderText = "Name"; 
     dataBirthdays.Columns[1].HeaderText = "Surname"; 
     dataBirthdays.Columns[2].HeaderText = "Birthday"; 

     //Loop through each birthday member and change date of birth to month/day fromat. 
     int currentRow = 0; 
     foreach (DataGridViewRow row in dataBirthdays.Rows) 
     { 
      string birthday = Convert.ToDateTime(dataBirthdays.Rows[currentRow].Cells[2].Value).ToString("MMMM dd"); 
      dataBirthdays.Rows[currentRow].Cells[2].Value = birthday; 
      currentRow++; 
     } 
    } 

然后从那里到这里:

public List<Birthdays> ReturnBirthdaysForCurrentWeek(DateTime weekStart, DateTime weekEnd) 
    { 
     var birthdays = new List<Birthdays>(); 

     int month = DateTime.Now.Month; 
     int dayStartOfWeek = weekStart.Day; 
     int dayEndOfWeek = weekEnd.Day; 

     using (MySqlConnection Conn = new MySqlConnection(Connect.sConnStr)) 
     { 
      Conn.Open(); 
      string sSql = "SELECT name, surname, date_birth FROM members WHERE MONTH(date_birth) = " + month + " AND DAY(date_birth) BETWEEN " + dayStartOfWeek + " AND " + dayEndOfWeek + ";"; 

      MySqlDataReader reader = Connect.getDataCommand(sSql, Conn).ExecuteReader(); 

      while (reader.Read()) 
      { 
       var birthday = new Birthdays() 
       { 
        MemberName = reader["name"].ToString(), 
        MemberSurname = reader["surname"].ToString(), 
        MemberBirthday = reader["date_birth"].ToString() 
       }; 
       birthdays.Add(birthday); 
      } 

      Conn.Close(); 
      Conn.Dispose(); 
     } 
     return birthdays; 
    } 
+0

我相信你会得到一些比较几个星期几个字符串,关注点分离的评论,所以我会跳过所有的。如果您包含一些示例数据以及您的方法返回的内容(例如,如何证明它不能正常工作),可能会有帮助。 – Jedidja 2014-12-07 17:21:01

+0

可能您可以在数据库级别执行此操作。看看这个问题:https://stackoverflow.com/questions/3769555/what-is-the-nicest-way-to-select-the-current-week-in-mysql – 2014-12-07 17:34:13

+0

检查这个答案:http:// stackoverflow的.com /一个/897326分之15433121。更多关于datepart:http://msdn.microsoft.com/en-us/library/ms174420.aspx – Neolisk 2014-12-07 21:51:48

回答

1

不是分手的日期比较逻辑到月和日比较(其中有您所发现的bug)为什么不直接比较日期?

例如,这可以通过SQL参数来实现:

string sSql = "SELECT name, surname, date_birth FROM members WHERE date_birth BETWEEN @param_val_1 AND @param_val_2"; 
MySqlCommand command = Connect.getDataCommand(sSql, Conn); 
command.Parameters.AddWithValue("@param_val_1", weekStart); 
command.Parameters.AddWithValue("@param_val_2", weekEnd); 

您还可以简化代码,以获得“当前”一周,以避免所有情况下的逻辑:

var today = DateTime.Today; 
// start with the numbers [0 .. 7) 
var startOfWeek = Enumerable.Range(0, count: 7) 
    // for each, subtract that many days from today 
    .Select(i => today.AddDays(-i)) 
    // find the first such date that is a Sunday 
    .First(dt => dt.DayOfWeek == DayOfWeek.Sunday); 
var endOfWeek = startOfWeek.AddDays(6); 

根据关于如何存储数据,您可能还必须小心时间组件。 DateTime.Today为您提供当天没有时间组件(例如当天午夜)。但是,如果数据库中的日期包含其他时间组件,则可能需要调整逻辑来解决这些问题。