I am trying to compare a list AAA that contain Date with a range of dates. I want to see if any of the range of date is present in the list or not. If the date is present I copy the list items to another list BBB else I add empty values to the list BBB.
The problem I am having is that with my actual code, is I don`t know how not pass through the false statement of the while loop, till it reaches the end of the comparison.
With the code below, it is passing both the true and false in the while loop, which is falsifying the required result. The result I am obtaining is for every time that is present, I am having the same time as false. In short, lets say the list contains the date 6/5/2010, and the range of date is 4/5/2010 to 7/5/2010. so I will have an item created in the true part and AN ITEM CREATED INTHE FALSE PART, which is wrong. The date present can either be in true or false part. Not both, such that I have two items bing created!
How can I achieve the right result? Any other method or suggetsion please.
My code is as follows:
DateTime StartDate;
DateTime EndDate;
Datetime tempDate = StartDate;
List<DateTime> dateToEvaluate;
bool TimeIsPresent = false;
foreach (var tempItem in TaskList)
{
while (EndDate.AddDays(1) != tempDate)
{
if (tempItem.Date[0] == tempDate)
{
TimeIsPresent = True;
break;
}
else
{
if (TimeIsPresent == False)
{
if (!(tempDate.DayOfWeek == DayOfWeek.Sunday)
{
dateToEvaluate = new List<DateTime>();
dateToEvaluate.Add(tempDate);
tempTask.Add(new GroupedTask { ID = null,
TaskID = null,
Date = dateToEvaluate });
}
}
}
tempDate = tempDate.AddDays(1);
}
if (TimeIsPresent == True)
{
tempTask.Add(new GroupedTask { ID = tempItem.ID,
TaskID = tempItem.TaskID,
Date = tempItem.Date });
TimeIsPresent = false;
}
}
let me give you an example. My range of date is as follows: Mon 8 Aug - Sunday 14 Aug.
Now my tasklist is as follows: item1: Date 9Aug, item2: Date 11Aug.
So my tempTask must be as follows:
item1: Date 8 Aug, taskID: null, ID: null,
item2: Date 9 Aug, taskID: 678, ID: 7,
item3: Date 10Aug, taskID: null, ID: null,
item4: Date11 Aug, taskID:890, ID: 34,
item5: Date 12 Aug, taskID: null, ID: null,
item6: Date 13 Aug, taskID: null, ID: null
Second example:
My range of date is as follows: Mon 8 Aug - Sunday 14 Aug.
Now my tasklist is as follows: item1: Date 9Aug, item2: Date 11Aug, item3: Date 14Aug
So my tempTask must be as follows:
item1: Date 8 Aug, taskID: null, ID: null,
item2: Date 9 Aug, taskID: 678, ID: 7,
item3: Date 10Aug, taskID: null, ID: null,
item4: Date11 Aug, taskID:890, ID: 34,
item5: Date 12 Aug, taskID: null, ID: null,
item6: Date 13 Aug, taskID: null, ID: null,
item4: Date14 Aug, taskID:894, ID: 74,
I think you're making things more difficult than they really are. As I understand it, you're taking each item in TaskList and seeing if the date falls in a certain range. If it does, you add it to another List and go to the next item, otherwise you add a blank entry to the other list and keep checking.
If my understanding is correct, try this:
EDITED based on OP's comment
The code now goes through the entire range for each item in TaskList, and adds either an empty object with the date or the corresponding task for the date.
No need to use a bool to determine if the date is present in this scenario.
// Note that you'll have to assign values to StartDate and EndDate, otherwise you'll get
// a Null Reference Exception
DateTime StartDate;
DateTime EndDate;
Datetime tempDate = StartDate;
List<DateTime> dateToEvaluate;
foreach (var tempItem in TaskList)
{
// Reset tempDate to the starting date before each loop
tempDate = StartDate;
while (EndDate.AddDays(1) != tempDate)
{
if (tempDate.DayOfWeek != DayOfWeek.Sunday)
{
if (tempItem.Date[0] == tempDate)
{
tempTask.Add(new GroupedTask { ID = tempItem.ID,
TaskID = tempItem.TaskID,
Date = tempItem.Date });
}
else
{
dateToEvaluate = new List<DateTime>();
dateToEvaluate.Add(tempDate);
tempTask.Add(new GroupedTask { ID = null,
TaskID = null,
Date = dateToEvaluate });
}
}
tempDate = tempDate.AddDays(1);
}
}
EDITED to add
Assume a 2 week range, with 7/1 starting on a Monday going through 7/14. Assume two tasks - task 1 with a date of 7/3 and task 2 with a date of 7/12. I would expect the following in tempTask:
26 elements (13 dates for each of the two task items), with all elements having a null ID except for one each for the two tasks.
Are you actually wanting a consolidated list with no repeats? I.e., with my example, there would be 13 elements, and 2 would have non-null IDs? What happens if two or more tasks have the same date?
I did find one error, in that I wasn't resetting the tempDate to the start before each loop.
EDIT Based on new understanding
Ok, so you're attempting to get a second list that has all the dates in a given range, and the GroupedTask object will either be an existing GroupedTask for that date, or a null GroupedTask for that date, if there is no match.
I suggest you take a look at Enigmativity's answer, as that may be a more elegant solution (I haven't looked at it in detail), but here's another approach. The biggest change is that I flipped the while loop and foreach loops.
// Note that you'll have to assign values to StartDate and EndDate, otherwise you'll get
// a Null Reference Exception
DateTime StartDate;
DateTime EndDate;
// Declare an instance of GroupedTask for use in the while loop
GroupedTask newTask;
Datetime tempDate = StartDate;
// Loop through the entire range of dates
while (EndDate.AddDays(1) != tempDate)
{
// You included Sundays in your example, but had earlier indicated they
// weren't needed. If you do want Sundays, you can remove this outer if
// block
if (tempDate.DayOfWeek != DayOfWeek.Sunday)
{
// Create a "null" GroupedTask object
// The Date property in GroupedTask appears to be a List<DateTime>,
// so I chose to initialize it along with the other properties.
newTask = new GroupedTask() { ID = null,
TaskID = null,
Date = new List<DateTime>() { tempDate }};
// For each date in the range, check to see if there are any tasks in the TaskList
foreach (var tempItem in TaskList)
{
// If the current item's date matches the current date in the range,
// update the newTask object with the current item's values.
// NOTE: If more than one item has the current date, the last one in
// will win as this code is written.
if (tempItem.Date[0] == tempDate)
{
newTask.ID = tempItem.ID;
newTask.TaskID = tempItem.TaskID;
newTask.Date = tempItem.Date;
}
}
// Add the newTask object to the second list
tempTask.Add(newTask);
}
}
I'm not sure what is EndDate, tempDate and some other things in your example. But if you are trying to loop through a DateRange and checking the existence of a particular date, then you could consider the following example:
static void yourFunction()
{
//
//Some Stuffs
//
foreach (var tempItem in TaskList)
{
if (DateRange.Contains(tempItem.Date[0]))
{
//Do Task
}
else
{
//Do Task
}
}
//
//Some Stuffs
//
}
public static IEnumerable<DateTime> DateRange
{
get
{
for (DateTime day = startDate; day < EndDate; day = day.AddDays(1))
{
yield return day;
}
}
}
Encapsulating a range of dates on a property is the idea of Jon Skeet, I learned it from his book C# in Depth
I found your code a little confusing, but if I understand your intent then I have a solution for you using LINQ. My approach might be a bit confusing to start with, but I'm happy to help you work through it.
My understanding is that you have a range of dates that you want to create a matching list of GroupedTask objects where you will take the object(s) from an existing TaskList for each matching date in the range or create a "dummy" instance if there isn't a match.
I assume that you have defined a StartDate variable along with the EndDate variable you used in your question.
My solution (which I have tested) looks like this:
var query =
from d in dates
from t in getTasksForDate(d)
where (t.ID != null) || (d.DayOfWeek != DayOfWeek.Sunday)
select t;
tempTask.AddRange(query);
Ignoring for the moment the two parts that need to be defined (dates & getTasksForDate) the query works by running through each date from the start to the end and selecting the tasks for that date (either from the TaskList or a "dummy" task if none exist for the date) and then filtering out any "dummy" tasks that fall on a Sunday. The tasks are then added to the tempTask list.
Now for the missing parts.
To get the dates list, just do this:
var days = EndDate.Date.Subtract(StartDate.Date).Days + 1;
var dates =
Enumerable
.Range(1 - days, days)
.Select(d => EndDate.AddDays(d));
As long as StartDate is on or before EndDate you will now have a list of dates that starts on StartDate and ends on EndDate.
The getTasksForDate is the trickier part.
You need to first turn the TaskList list into a lookup function that turns any date into a list of GroupedTask objects for that date. With LINQ it's easy:
var lookup = TaskList.ToLookup(x => x.Date[0].Date, x => new GroupedTask()
{
ID = x.ID,
TaskID = x.TaskID,
Date = x.Date,
});
Next you need to create the getTasksForDate function that will take a date and return either the list of GroupedTask from the lookup for the date or a single "dummy" GroupedTask object if there were no tasks for the date.
Func<DateTime, IEnumerable<GroupedTask>> getTasksForDate = d =>
{
return lookup[d].DefaultIfEmpty(new GroupedTask()
{
ID = null,
TaskID = null,
Date = new List<DateTime>() { d, },
});
};
That's it.
If you want to define StartDate and/or EndDate based on actual values from TaskList you can use this code:
var StartDate = TaskList.Select(t => t.Date[0].Date).Min();
var EndDate = TaskList.Select(t => t.Date[0].Date).Max();
I've used .Date after most of the DateTime references to ensure that there is no time component to the date.
Yell out if you'd like any further explanation.
If I understand what you are trying to do, I would change the way you are doing it in this way. First I would find all the dates in your range that have one or mor associated tasks (and put themn in a Dictionary in order to be able to get them knowing the date), then I would create tempTask. Something like this:
DateTime StartDate;
DateTime EndDate;
DateTime tempDate = StartDate;
List<DateTime> dateToEvaluate;
Dictionary<DateTime, List<Task>> dateTaskDict = new Dictionary<DateTime, List<Task>>();
bool TimeIsPresent = false;
foreach (Task tempItem in TaskList)
{
while (EndDate.AddDays(1) != tempDate)
{
if (tempItem.Date[0] == tempDate)
{
List<Task> tasksForDate;
if (!dateTaskDict.TryGetValue(tempDate, out tasksForDate))
{
tasksForDate = new List<Task>();
dateTaskDict[tempDate] = tasksForDate;
}
tasksForDate.Add(tempItem);
break;
}
tempDate = tempDate.AddDays(1);
}
}
tempDate = StartDate;
while (EndDate.AddDays(1) != tempDate)
{
List<Task> tasks;
if (dateTaskDict.TryGetValue(tempDate, out tasks))
{
foreach (Task aTask in tasks)
tempTask.Add(new GroupedTask { ID = aTask.ID,
TaskID = aTask.TaskID,
Date = tempDate });
}
else
{
if (tempDate.DayOfWeek != DayOfWeek.Sunday)
{
tempTask.Add(new GroupedTask { ID = null
TaskID = null,
Date = tempDate });
}
}
}
Related
My first time working with DateTime in C#. We print out delivery tickets and because our system runs on UTC, I need to convert the date on the ticket back to local time so that tickets with an Estimated Arrival after 6:00PM dont show up as the next day.
I've gotten the timezone from the well and now I am struggling with how to inject the new timezone into my new view model for the ticket.
The error I get is:
After reading, I understand that my truck.EstimatedArrival may be nullable so i tried to insert an .HasValue but so far I can't get the syntax to work for me. Any suggestions on how I need to lay this out?
public async Task<DispatchTruckTicketViewModel> GetDeliveryTruckTicket(string truckId)
{
var truck = await _dispatchTruckRepo.GetDispatchTruckForTicket(truckId);
var dispatchId = await _dispatchRepo.GetAllQueryable()
.Include(did => did.SourceWell.Id)
.SingleOrDefaultAsync(did => truck.DispatchId == did.Id);
var well = await _wellRepo.GetAllQueryable()
.SingleOrDefaultAsync(w => w.Id == dispatchId.SourceWellId);
TimeZoneInfo tz;
tz = TimeZoneInfo.FindSystemTimeZoneById(well.TimeZoneName);
if (truck.EstimatedArrival.HasValue){
TimeZoneInfo.ConvertTimeFromUtc(truck.EstimatedArrival, tz);
} -- still not working, same error as above.
if (truck == null)
{
throw new ServiceException("Truck does not exist");
}
var model = new DispatchTruckTicketViewModel
{
Id = truck.Id,
Type = "Delivery",
TicketNumber = truck.TicketNumber,
OrderedBy = await _accountService.GetNameFromUserId(truck.Dispatch.Header.CreateUserId),
// DateShipped = truck.EstimatedArrival?.ToLocalTime().ToShortDateString() ?? "",
DateShipped = TimeZoneInfo.ConvertTimeFromUtc(truck.EstimatedArrival, tz).ToShortDateString() ?? "",
Items = _mapper.Map<IEnumerable<DispatchItem>, IEnumerable<DispatchItemViewModel>>(truck.Items),
DriverName = truck.DriverName,
SwamperName = truck.SwamperName,
};
AddWellInformation(model, truck.Dispatch.DestinationWell);
return model;
}
The correct way is:
TimeZoneInfo tz;
tz = TimeZoneInfo.FindSystemTimeZoneById(well.TimeZoneName);
if (truck.EstimatedArrival.HasValue){
TimeZoneInfo.ConvertTimeFromUtc(truck.EstimatedArrival.Value, tz);
}
else
{
// Handle failure of getting estimated time
}
HasValue of a nullable value type returns true when the element has a value (is not null). To get the value, you need to call its Value property. As a shortcut, you might use the GetValueOrDefault() method.
I am trying to retrieve dates from a SharePoint List with C#. The dates are created with a SharePoint datepicker. My problem is that SharePoint adds 1 day. I have tried .ToLocalTime() and .ToUniversalTime() right before and I save the time to a list. I have searched different solutions, none of which have been successful. Does anybody know how to convert the date?
My code:
public List < DateTime > GetHolidays(ClientContext context) {
List < DateTime > holidayDates = new List < DateTime > ();
var items = LoadListHoliday(context);
if (context != null) {
foreach(ListItem listItem in items) {
DateTime date = (DateTime) listItem["u12q"];
holidayDates.Add(date.Date.ToLocalTime());
}
}
}
return holidayDates;
}
LoadListHoliday(context);
is a method that retrives the list.
solved it like this:
foreach (ListItem listItem in items)
{
DateTime date = System.TimeZone.CurrentTimeZone.ToLocalTime((DateTime)listItem["u12q"]);
holidayDates.Add(date.Date);
}
I have used entity framework with code first approach.
when I am trying to pass record one by one Fromdate to Todate, 1st time its work, after it gives error like: "The property 'ID' is part of the object's key information and cannot be modified."
var fd = todaycooked.CookDate; // 2016-07-01
var td = todaycooked.ToCookDate; //2016-11-01
for (var date = fd; date <= td; date = date.AddDays(1))
{
var product = db.Products.Find(todaycooked.ProductID);
product.Qty = product.Qty + todaycooked.QTY;
todaycooked.Product = product;
todaycooked.CookDate = date;
db.TodayCookeds.Add(todaycooked);
db.SaveChanges();
}
Thanks in Advance.
You are setting the Product and CookDate once per day, so I assume you want one record per day - which means you mean one object per day. I suspect you actually want something like:
var fd = todaycooked.CookDate; // 2016-07-01
var td = todaycooked.ToCookDate; //2016-11-01
// this doesn't change per day, so only fetch it once
var product = db.Products.Find(todaycooked.ProductID);
for (var date = fd; date <= td; date = date.AddDays(1))
{
var toAdd = todaycooked.Clone(); // TODO: add a suitable clone method
toAdd.Product = product;
toAdd.CookDate = date;
db.TodayCookeds.Add(toAdd);
product.Qty = product.Qty + todaycooked.QTY;
db.SaveChanges();
}
However, you can probably also get away with moving the db.SaveChanges() to outside of the loop, which would make the whole thing atomic (rather than risking getting the first 4 of 8 days saved, then an error):
...
for (var date = fd; date <= td; date = date.AddDays(1))
{
...
}
db.SaveChanges();
I have a list in the form-
FieldName- DataType-
Date DateTime
DateString String
Unit double
Price double
I want to perform an operation such that if weakday on Date is not Monday then update DateString with empty string otherwise keep the value of DateString as it is.
Data is present in List.
UPDATE
I have applied aggregate function on DataTable dtGas as follows-
var qGas = from x in dtGas.AsEnumerable()
group x by new
{
Date = x.Field<DateTime>("Date"),
DateString = x.Field<string>("DateString")
} into egroup
select new
{
Date = egroup.Key.Date,
DateString = egroup.Key.DateString,
Unit = egroup.Sum(r => r.Field<double>("Unit")),
Price = egroup.Sum(r => r.Field<double>("Price"))
};
Now I need to show this result into Chart. Due to large amount of data values are overlapping on X-axis.
That is why I need to remove some values from DateString and show only few of them.
You can simply use conditional operator like this:-
group x by new
{
Date = x.Field<DateTime>("Date"),
DateString = x.Field<string>("DateString")
} into egroup
let isMonday = egroup.Key.Date.DayOfWeek.ToString() == "Monday"
select new
{
Date = egroup.Key.Date,
DateString = isMonday ? egroup.Key.DateString : "",
..other properties
See i have this following code below, at line no.9 i'm calculating some dates to move forward, so im using AddDays method but in return i'm getting all dates same for all rows.
If i do AddDays(6) like this then it is returning correctly by moving all dates by 6 days.
How should i do it for adding days according to my logic at this point.
[DataContract]
public class JQGridRow
{
[DataMember]
public long id;
[DataMember]
public object[] cell;
}
var sortedItems = invBatch.ListOfItems.OrderBy(i => i.RunDateIndex);//This will return IEnumerable<Class> List
DateTime startDate = DateTime.Parse(lblStartDate.Text);
JQGrid.JQGridRow[] rowData = (
from i in sortedItems
select new JQGrid.JQGridRow() {
id = i.ID,
cell = new string[] {
i.ID.ToString(),
i.Status.ToString(),
i.StatusTitle,
i.RunDate.AddDays((startDate.Subtract(i.RunDate)).Days+1).ToString(Utility.DATE_FORMAT),
//Here in above line the array returning same values for all columns of this row
i.StartTimeString,
i.EndTimeString,
i.EndTime.ToString(),
}}).ToArray();
The dates are all same and equal the next day after startDate because you calculate them as such.
i.RunDate.AddDays((startDate.Subtract(i.RunDate)).Days+1)
Rounding to whole day, you compute RunDate+(startDate-RunDate+1) = startDate + 1. i.RunDate does not matter.