c# find item in list returned by LINQ query and compare its value with another item in list
up vote
3
down vote
favorite
I have a list of events returned from a LINQ query on a certain day (d).
var findJSE = from a in db.DailyGPSTables
where (a.EventType == "JE" || a.EventType == "JS") &&
a.EventDateTime.Value.Day == d.Day select a;
I want to compare the times of each matching set of JS and JE to see if they are in the right order eg. JS (Job start) is before JE (Job end). I am able to accomplish this if there is only one JS and One JE by using this.
foreach (DailyGPSTable e in itCompareDay)
{
if (e.EventType == "JE" && e.EventDateTime.Value.Day == d.Day)
{
var findJS = from a in db.DailyGPSTables where a.EventType == "JS" && a.EventDateTime.Value.Day == d.Day select a;
var time = findJS.FirstOrDefault();
js = time.EventDateTime.Value;
if (js > e.EventDateTime.Value)
{
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
However I am having a bit of trouble if there are more than one JS in a day which is allowed. I was provided an excellent potential fix by @NetMage but ran out of time trying to correctly implement it. My latest attempt is here:
//Response.Write("<br>3rd Condition number of JE and JS on " + e.EventDateTime.Value.Day + " match now we see if there is more than one js");
if (findJS.Count() > 1)
{
//Response.Write("<br>3.1 Condition there is more than one JS on " + e.EventDateTime.Value.Day + " get time of first js tag find the time of the first je tag");
var jsTime = findJS.ToList();
var jeTime = findJE.ToList();
for (int j = 0; j <= jsTime.Count - 1; j++)
{
//Response.Write("<br><br>The " + j + " " + jsTime[j].EventType + " Starts on " + jsTime[j].EventDateTime + "<br>");
var jsTimefor = jsTime[j].EventDateTime;
for (int k = 0; k <= jeTime.Count - 1; k++)
{
//Response.Write("<br><br>The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + "<br>");
var jeTimefor = jeTime[k].EventDateTime;
if (jeTime[k].EventDateTime < jsTime[j].EventDateTime)
{
//Response.Write(" The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + " and < " + jsTime[j].EventDateTime + " which is " + jsTime[j].EventType + "<br>");
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
}
This attempt highlights all JE rows for the day and creates an error condition where there is none.
UPDATE:
I have been able to create a list containing the event types and times.
How would I iterate through that list and make sure the first event is an JS and it is the earliest and the next event is a JE and it is next in order of time. if not report the error.
I am trying to consolidate and loop through days. This reports only one error on the 21st but there are errors on the 23rd and 27th.
for (DateTime d = startDate.Date; d <= endDate.Date; d = d.AddDays(1))
{
Response.Write("<br>" + d + "<br>");
var itCompareDay = (from h in db.DailyGPSTables
where h.EmplID == EmpID
&& (h.EventDateTime.Value.Day == d.Day)
&& (h.EventType == "SS" || h.EventType == "JS" || h.EventType == "LS" || h.EventType == "LE" || h.EventType == "JE" || h.EventType == "SE")
orderby h.EventDateTime
select h).ToList();
string EOOmessage="";
var lastEventType = itCompareDay.Any(x => x.EventType == "JS")? "JE" : "JS";
foreach (DailyGPSTable e in itCompareDay)
{
Response.Write("<br>Event Type " + e.EventType + " Count " + dc + " for the " + d.Day + "<br>");
if (e.EventDateTime.Value.Day == d.Day)
{
if (e.EventType == lastEventType)
{
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
Response.Write("<br>OUT OF SEQUENCE JE" + " ON " + e.EventDateTime.Value.ToShortDateString());
}
lastEventType = e.EventType;
}
dc = dc + 1;
}
rowNumber = rowNumber + dc;
}
Errors incorrectly reported.
c# asp.net linq
add a comment |
up vote
3
down vote
favorite
I have a list of events returned from a LINQ query on a certain day (d).
var findJSE = from a in db.DailyGPSTables
where (a.EventType == "JE" || a.EventType == "JS") &&
a.EventDateTime.Value.Day == d.Day select a;
I want to compare the times of each matching set of JS and JE to see if they are in the right order eg. JS (Job start) is before JE (Job end). I am able to accomplish this if there is only one JS and One JE by using this.
foreach (DailyGPSTable e in itCompareDay)
{
if (e.EventType == "JE" && e.EventDateTime.Value.Day == d.Day)
{
var findJS = from a in db.DailyGPSTables where a.EventType == "JS" && a.EventDateTime.Value.Day == d.Day select a;
var time = findJS.FirstOrDefault();
js = time.EventDateTime.Value;
if (js > e.EventDateTime.Value)
{
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
However I am having a bit of trouble if there are more than one JS in a day which is allowed. I was provided an excellent potential fix by @NetMage but ran out of time trying to correctly implement it. My latest attempt is here:
//Response.Write("<br>3rd Condition number of JE and JS on " + e.EventDateTime.Value.Day + " match now we see if there is more than one js");
if (findJS.Count() > 1)
{
//Response.Write("<br>3.1 Condition there is more than one JS on " + e.EventDateTime.Value.Day + " get time of first js tag find the time of the first je tag");
var jsTime = findJS.ToList();
var jeTime = findJE.ToList();
for (int j = 0; j <= jsTime.Count - 1; j++)
{
//Response.Write("<br><br>The " + j + " " + jsTime[j].EventType + " Starts on " + jsTime[j].EventDateTime + "<br>");
var jsTimefor = jsTime[j].EventDateTime;
for (int k = 0; k <= jeTime.Count - 1; k++)
{
//Response.Write("<br><br>The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + "<br>");
var jeTimefor = jeTime[k].EventDateTime;
if (jeTime[k].EventDateTime < jsTime[j].EventDateTime)
{
//Response.Write(" The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + " and < " + jsTime[j].EventDateTime + " which is " + jsTime[j].EventType + "<br>");
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
}
This attempt highlights all JE rows for the day and creates an error condition where there is none.
UPDATE:
I have been able to create a list containing the event types and times.
How would I iterate through that list and make sure the first event is an JS and it is the earliest and the next event is a JE and it is next in order of time. if not report the error.
I am trying to consolidate and loop through days. This reports only one error on the 21st but there are errors on the 23rd and 27th.
for (DateTime d = startDate.Date; d <= endDate.Date; d = d.AddDays(1))
{
Response.Write("<br>" + d + "<br>");
var itCompareDay = (from h in db.DailyGPSTables
where h.EmplID == EmpID
&& (h.EventDateTime.Value.Day == d.Day)
&& (h.EventType == "SS" || h.EventType == "JS" || h.EventType == "LS" || h.EventType == "LE" || h.EventType == "JE" || h.EventType == "SE")
orderby h.EventDateTime
select h).ToList();
string EOOmessage="";
var lastEventType = itCompareDay.Any(x => x.EventType == "JS")? "JE" : "JS";
foreach (DailyGPSTable e in itCompareDay)
{
Response.Write("<br>Event Type " + e.EventType + " Count " + dc + " for the " + d.Day + "<br>");
if (e.EventDateTime.Value.Day == d.Day)
{
if (e.EventType == lastEventType)
{
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
Response.Write("<br>OUT OF SEQUENCE JE" + " ON " + e.EventDateTime.Value.ToShortDateString());
}
lastEventType = e.EventType;
}
dc = dc + 1;
}
rowNumber = rowNumber + dc;
}
Errors incorrectly reported.
c# asp.net linq
I am having a bit of trouble if there are more than one JS in a day which is allowed - if you have more than one JobStart in a day, can't you just take the latest one and compare it to the JobEnd?
– stuartd
Nov 12 at 18:22
@stuartd not sure I follow you, can you perhaps put your question into code so I can elaborate?
– Doug Farrell
Nov 12 at 18:54
Perhaps think about how you would do this by hand, with a list of event cards, some marked JS/JE. Make only one traversal through the deck. Your (one pair) code fails if there are no JS events. You don't need to keep repeating the test forday
that was in the base query.
– NetMage
Nov 12 at 19:56
@NetMageplease see update.
– Doug Farrell
Nov 12 at 20:07
add a comment |
up vote
3
down vote
favorite
up vote
3
down vote
favorite
I have a list of events returned from a LINQ query on a certain day (d).
var findJSE = from a in db.DailyGPSTables
where (a.EventType == "JE" || a.EventType == "JS") &&
a.EventDateTime.Value.Day == d.Day select a;
I want to compare the times of each matching set of JS and JE to see if they are in the right order eg. JS (Job start) is before JE (Job end). I am able to accomplish this if there is only one JS and One JE by using this.
foreach (DailyGPSTable e in itCompareDay)
{
if (e.EventType == "JE" && e.EventDateTime.Value.Day == d.Day)
{
var findJS = from a in db.DailyGPSTables where a.EventType == "JS" && a.EventDateTime.Value.Day == d.Day select a;
var time = findJS.FirstOrDefault();
js = time.EventDateTime.Value;
if (js > e.EventDateTime.Value)
{
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
However I am having a bit of trouble if there are more than one JS in a day which is allowed. I was provided an excellent potential fix by @NetMage but ran out of time trying to correctly implement it. My latest attempt is here:
//Response.Write("<br>3rd Condition number of JE and JS on " + e.EventDateTime.Value.Day + " match now we see if there is more than one js");
if (findJS.Count() > 1)
{
//Response.Write("<br>3.1 Condition there is more than one JS on " + e.EventDateTime.Value.Day + " get time of first js tag find the time of the first je tag");
var jsTime = findJS.ToList();
var jeTime = findJE.ToList();
for (int j = 0; j <= jsTime.Count - 1; j++)
{
//Response.Write("<br><br>The " + j + " " + jsTime[j].EventType + " Starts on " + jsTime[j].EventDateTime + "<br>");
var jsTimefor = jsTime[j].EventDateTime;
for (int k = 0; k <= jeTime.Count - 1; k++)
{
//Response.Write("<br><br>The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + "<br>");
var jeTimefor = jeTime[k].EventDateTime;
if (jeTime[k].EventDateTime < jsTime[j].EventDateTime)
{
//Response.Write(" The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + " and < " + jsTime[j].EventDateTime + " which is " + jsTime[j].EventType + "<br>");
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
}
This attempt highlights all JE rows for the day and creates an error condition where there is none.
UPDATE:
I have been able to create a list containing the event types and times.
How would I iterate through that list and make sure the first event is an JS and it is the earliest and the next event is a JE and it is next in order of time. if not report the error.
I am trying to consolidate and loop through days. This reports only one error on the 21st but there are errors on the 23rd and 27th.
for (DateTime d = startDate.Date; d <= endDate.Date; d = d.AddDays(1))
{
Response.Write("<br>" + d + "<br>");
var itCompareDay = (from h in db.DailyGPSTables
where h.EmplID == EmpID
&& (h.EventDateTime.Value.Day == d.Day)
&& (h.EventType == "SS" || h.EventType == "JS" || h.EventType == "LS" || h.EventType == "LE" || h.EventType == "JE" || h.EventType == "SE")
orderby h.EventDateTime
select h).ToList();
string EOOmessage="";
var lastEventType = itCompareDay.Any(x => x.EventType == "JS")? "JE" : "JS";
foreach (DailyGPSTable e in itCompareDay)
{
Response.Write("<br>Event Type " + e.EventType + " Count " + dc + " for the " + d.Day + "<br>");
if (e.EventDateTime.Value.Day == d.Day)
{
if (e.EventType == lastEventType)
{
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
Response.Write("<br>OUT OF SEQUENCE JE" + " ON " + e.EventDateTime.Value.ToShortDateString());
}
lastEventType = e.EventType;
}
dc = dc + 1;
}
rowNumber = rowNumber + dc;
}
Errors incorrectly reported.
c# asp.net linq
I have a list of events returned from a LINQ query on a certain day (d).
var findJSE = from a in db.DailyGPSTables
where (a.EventType == "JE" || a.EventType == "JS") &&
a.EventDateTime.Value.Day == d.Day select a;
I want to compare the times of each matching set of JS and JE to see if they are in the right order eg. JS (Job start) is before JE (Job end). I am able to accomplish this if there is only one JS and One JE by using this.
foreach (DailyGPSTable e in itCompareDay)
{
if (e.EventType == "JE" && e.EventDateTime.Value.Day == d.Day)
{
var findJS = from a in db.DailyGPSTables where a.EventType == "JS" && a.EventDateTime.Value.Day == d.Day select a;
var time = findJS.FirstOrDefault();
js = time.EventDateTime.Value;
if (js > e.EventDateTime.Value)
{
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
However I am having a bit of trouble if there are more than one JS in a day which is allowed. I was provided an excellent potential fix by @NetMage but ran out of time trying to correctly implement it. My latest attempt is here:
//Response.Write("<br>3rd Condition number of JE and JS on " + e.EventDateTime.Value.Day + " match now we see if there is more than one js");
if (findJS.Count() > 1)
{
//Response.Write("<br>3.1 Condition there is more than one JS on " + e.EventDateTime.Value.Day + " get time of first js tag find the time of the first je tag");
var jsTime = findJS.ToList();
var jeTime = findJE.ToList();
for (int j = 0; j <= jsTime.Count - 1; j++)
{
//Response.Write("<br><br>The " + j + " " + jsTime[j].EventType + " Starts on " + jsTime[j].EventDateTime + "<br>");
var jsTimefor = jsTime[j].EventDateTime;
for (int k = 0; k <= jeTime.Count - 1; k++)
{
//Response.Write("<br><br>The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + "<br>");
var jeTimefor = jeTime[k].EventDateTime;
if (jeTime[k].EventDateTime < jsTime[j].EventDateTime)
{
//Response.Write(" The " + k + " " + jeTime[k].EventType + " Starts on " + jeTime[k].EventDateTime + " and < " + jsTime[j].EventDateTime + " which is " + jsTime[j].EventType + "<br>");
EOOmessage = " On " + e.EventDateTime.Value.ToShortDateString() + " JE before JS";
errorList.Add(EOOmessage);
errorListRow.Add(dc);
}
}
}
}
This attempt highlights all JE rows for the day and creates an error condition where there is none.
UPDATE:
I have been able to create a list containing the event types and times.
How would I iterate through that list and make sure the first event is an JS and it is the earliest and the next event is a JE and it is next in order of time. if not report the error.
I am trying to consolidate and loop through days. This reports only one error on the 21st but there are errors on the 23rd and 27th.
for (DateTime d = startDate.Date; d <= endDate.Date; d = d.AddDays(1))
{
Response.Write("<br>" + d + "<br>");
var itCompareDay = (from h in db.DailyGPSTables
where h.EmplID == EmpID
&& (h.EventDateTime.Value.Day == d.Day)
&& (h.EventType == "SS" || h.EventType == "JS" || h.EventType == "LS" || h.EventType == "LE" || h.EventType == "JE" || h.EventType == "SE")
orderby h.EventDateTime
select h).ToList();
string EOOmessage="";
var lastEventType = itCompareDay.Any(x => x.EventType == "JS")? "JE" : "JS";
foreach (DailyGPSTable e in itCompareDay)
{
Response.Write("<br>Event Type " + e.EventType + " Count " + dc + " for the " + d.Day + "<br>");
if (e.EventDateTime.Value.Day == d.Day)
{
if (e.EventType == lastEventType)
{
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
Response.Write("<br>OUT OF SEQUENCE JE" + " ON " + e.EventDateTime.Value.ToShortDateString());
}
lastEventType = e.EventType;
}
dc = dc + 1;
}
rowNumber = rowNumber + dc;
}
Errors incorrectly reported.
c# asp.net linq
c# asp.net linq
edited Nov 13 at 19:55
asked Nov 12 at 18:18
Doug Farrell
176
176
I am having a bit of trouble if there are more than one JS in a day which is allowed - if you have more than one JobStart in a day, can't you just take the latest one and compare it to the JobEnd?
– stuartd
Nov 12 at 18:22
@stuartd not sure I follow you, can you perhaps put your question into code so I can elaborate?
– Doug Farrell
Nov 12 at 18:54
Perhaps think about how you would do this by hand, with a list of event cards, some marked JS/JE. Make only one traversal through the deck. Your (one pair) code fails if there are no JS events. You don't need to keep repeating the test forday
that was in the base query.
– NetMage
Nov 12 at 19:56
@NetMageplease see update.
– Doug Farrell
Nov 12 at 20:07
add a comment |
I am having a bit of trouble if there are more than one JS in a day which is allowed - if you have more than one JobStart in a day, can't you just take the latest one and compare it to the JobEnd?
– stuartd
Nov 12 at 18:22
@stuartd not sure I follow you, can you perhaps put your question into code so I can elaborate?
– Doug Farrell
Nov 12 at 18:54
Perhaps think about how you would do this by hand, with a list of event cards, some marked JS/JE. Make only one traversal through the deck. Your (one pair) code fails if there are no JS events. You don't need to keep repeating the test forday
that was in the base query.
– NetMage
Nov 12 at 19:56
@NetMageplease see update.
– Doug Farrell
Nov 12 at 20:07
I am having a bit of trouble if there are more than one JS in a day which is allowed - if you have more than one JobStart in a day, can't you just take the latest one and compare it to the JobEnd?
– stuartd
Nov 12 at 18:22
I am having a bit of trouble if there are more than one JS in a day which is allowed - if you have more than one JobStart in a day, can't you just take the latest one and compare it to the JobEnd?
– stuartd
Nov 12 at 18:22
@stuartd not sure I follow you, can you perhaps put your question into code so I can elaborate?
– Doug Farrell
Nov 12 at 18:54
@stuartd not sure I follow you, can you perhaps put your question into code so I can elaborate?
– Doug Farrell
Nov 12 at 18:54
Perhaps think about how you would do this by hand, with a list of event cards, some marked JS/JE. Make only one traversal through the deck. Your (one pair) code fails if there are no JS events. You don't need to keep repeating the test for
day
that was in the base query.– NetMage
Nov 12 at 19:56
Perhaps think about how you would do this by hand, with a list of event cards, some marked JS/JE. Make only one traversal through the deck. Your (one pair) code fails if there are no JS events. You don't need to keep repeating the test for
day
that was in the base query.– NetMage
Nov 12 at 19:56
@NetMageplease see update.
– Doug Farrell
Nov 12 at 20:07
@NetMageplease see update.
– Doug Farrell
Nov 12 at 20:07
add a comment |
1 Answer
1
active
oldest
votes
up vote
0
down vote
I'm not sure I completely understand the design here, and it seems to me you'd be better off storing your events in an Event
table, where each row has a Start
and End
column.
But one way you can validate a list of related items as you've described is to order the list by EventDateTime
, and then keep track of the lastEventType
. This way you can check if the current event type is equal to the lastEventType
then it's an error.
For example:
public class Event
{
public string EventType { get; set; }
public DateTime? EventDateTime { get; set; }
public override string ToString()
{
return $"{EventDateTime}: {EventType}";
}
}
public class Program
{
static void Main(string args)
{
// Events for the day ordered by EventDateTime
var events = new List<Event>
{
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 12:23 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:29 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:40 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 2:15 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 3:12 AM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 12:12 PM")},
}.OrderBy(e => e.EventDateTime);
// Initialize lastEventType to the opposite of the type of event we
// expect to see first (which is "JS" unless there are no "JS" events)
var lastEventType = events.Any(e => e.EventType == "JS")
? "JE" : "JS";
foreach (var e in events)
{
var errorMessage = string.Empty;
// Check for error
if (e.EventType == lastEventType)
{
Console.ForegroundColor = ConsoleColor.Red;
// Create an error message
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
var dateStr = e.EventDateTime?.ToShortDateString() ?? "[null date]";
errorMessage = $" - On {dateStr} this error occurred: [{error}]";
// Your code would include:
// errorList.Add(errorMessage);
// errorListRow.Add(dc);
}
Console.WriteLine(e + errorMessage);
Console.ResetColor();
// Update the lastEventType with this event type
lastEventType = e.EventType;
}
GetKeyFromUser("nDone! Press any key to exit...");
}
}
Output
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort byEventDateTime
and then validate that the items are of alternatingEventType
, starting withJS
?
– Rufus L
Nov 12 at 21:47
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
This code is just too show the logic. You could replace the code that sets the console color withif (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
|
show 1 more comment
Your Answer
StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");
StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});
function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});
}
});
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53267928%2fc-sharp-find-item-in-list-returned-by-linq-query-and-compare-its-value-with-anot%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
1 Answer
1
active
oldest
votes
1 Answer
1
active
oldest
votes
active
oldest
votes
active
oldest
votes
up vote
0
down vote
I'm not sure I completely understand the design here, and it seems to me you'd be better off storing your events in an Event
table, where each row has a Start
and End
column.
But one way you can validate a list of related items as you've described is to order the list by EventDateTime
, and then keep track of the lastEventType
. This way you can check if the current event type is equal to the lastEventType
then it's an error.
For example:
public class Event
{
public string EventType { get; set; }
public DateTime? EventDateTime { get; set; }
public override string ToString()
{
return $"{EventDateTime}: {EventType}";
}
}
public class Program
{
static void Main(string args)
{
// Events for the day ordered by EventDateTime
var events = new List<Event>
{
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 12:23 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:29 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:40 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 2:15 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 3:12 AM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 12:12 PM")},
}.OrderBy(e => e.EventDateTime);
// Initialize lastEventType to the opposite of the type of event we
// expect to see first (which is "JS" unless there are no "JS" events)
var lastEventType = events.Any(e => e.EventType == "JS")
? "JE" : "JS";
foreach (var e in events)
{
var errorMessage = string.Empty;
// Check for error
if (e.EventType == lastEventType)
{
Console.ForegroundColor = ConsoleColor.Red;
// Create an error message
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
var dateStr = e.EventDateTime?.ToShortDateString() ?? "[null date]";
errorMessage = $" - On {dateStr} this error occurred: [{error}]";
// Your code would include:
// errorList.Add(errorMessage);
// errorListRow.Add(dc);
}
Console.WriteLine(e + errorMessage);
Console.ResetColor();
// Update the lastEventType with this event type
lastEventType = e.EventType;
}
GetKeyFromUser("nDone! Press any key to exit...");
}
}
Output
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort byEventDateTime
and then validate that the items are of alternatingEventType
, starting withJS
?
– Rufus L
Nov 12 at 21:47
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
This code is just too show the logic. You could replace the code that sets the console color withif (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
|
show 1 more comment
up vote
0
down vote
I'm not sure I completely understand the design here, and it seems to me you'd be better off storing your events in an Event
table, where each row has a Start
and End
column.
But one way you can validate a list of related items as you've described is to order the list by EventDateTime
, and then keep track of the lastEventType
. This way you can check if the current event type is equal to the lastEventType
then it's an error.
For example:
public class Event
{
public string EventType { get; set; }
public DateTime? EventDateTime { get; set; }
public override string ToString()
{
return $"{EventDateTime}: {EventType}";
}
}
public class Program
{
static void Main(string args)
{
// Events for the day ordered by EventDateTime
var events = new List<Event>
{
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 12:23 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:29 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:40 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 2:15 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 3:12 AM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 12:12 PM")},
}.OrderBy(e => e.EventDateTime);
// Initialize lastEventType to the opposite of the type of event we
// expect to see first (which is "JS" unless there are no "JS" events)
var lastEventType = events.Any(e => e.EventType == "JS")
? "JE" : "JS";
foreach (var e in events)
{
var errorMessage = string.Empty;
// Check for error
if (e.EventType == lastEventType)
{
Console.ForegroundColor = ConsoleColor.Red;
// Create an error message
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
var dateStr = e.EventDateTime?.ToShortDateString() ?? "[null date]";
errorMessage = $" - On {dateStr} this error occurred: [{error}]";
// Your code would include:
// errorList.Add(errorMessage);
// errorListRow.Add(dc);
}
Console.WriteLine(e + errorMessage);
Console.ResetColor();
// Update the lastEventType with this event type
lastEventType = e.EventType;
}
GetKeyFromUser("nDone! Press any key to exit...");
}
}
Output
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort byEventDateTime
and then validate that the items are of alternatingEventType
, starting withJS
?
– Rufus L
Nov 12 at 21:47
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
This code is just too show the logic. You could replace the code that sets the console color withif (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
|
show 1 more comment
up vote
0
down vote
up vote
0
down vote
I'm not sure I completely understand the design here, and it seems to me you'd be better off storing your events in an Event
table, where each row has a Start
and End
column.
But one way you can validate a list of related items as you've described is to order the list by EventDateTime
, and then keep track of the lastEventType
. This way you can check if the current event type is equal to the lastEventType
then it's an error.
For example:
public class Event
{
public string EventType { get; set; }
public DateTime? EventDateTime { get; set; }
public override string ToString()
{
return $"{EventDateTime}: {EventType}";
}
}
public class Program
{
static void Main(string args)
{
// Events for the day ordered by EventDateTime
var events = new List<Event>
{
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 12:23 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:29 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:40 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 2:15 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 3:12 AM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 12:12 PM")},
}.OrderBy(e => e.EventDateTime);
// Initialize lastEventType to the opposite of the type of event we
// expect to see first (which is "JS" unless there are no "JS" events)
var lastEventType = events.Any(e => e.EventType == "JS")
? "JE" : "JS";
foreach (var e in events)
{
var errorMessage = string.Empty;
// Check for error
if (e.EventType == lastEventType)
{
Console.ForegroundColor = ConsoleColor.Red;
// Create an error message
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
var dateStr = e.EventDateTime?.ToShortDateString() ?? "[null date]";
errorMessage = $" - On {dateStr} this error occurred: [{error}]";
// Your code would include:
// errorList.Add(errorMessage);
// errorListRow.Add(dc);
}
Console.WriteLine(e + errorMessage);
Console.ResetColor();
// Update the lastEventType with this event type
lastEventType = e.EventType;
}
GetKeyFromUser("nDone! Press any key to exit...");
}
}
Output
I'm not sure I completely understand the design here, and it seems to me you'd be better off storing your events in an Event
table, where each row has a Start
and End
column.
But one way you can validate a list of related items as you've described is to order the list by EventDateTime
, and then keep track of the lastEventType
. This way you can check if the current event type is equal to the lastEventType
then it's an error.
For example:
public class Event
{
public string EventType { get; set; }
public DateTime? EventDateTime { get; set; }
public override string ToString()
{
return $"{EventDateTime}: {EventType}";
}
}
public class Program
{
static void Main(string args)
{
// Events for the day ordered by EventDateTime
var events = new List<Event>
{
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 12:23 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:29 PM")},
new Event {EventType = "JE",
EventDateTime = DateTime.Parse("10/21/2018 2:40 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 2:15 PM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 3:12 AM")},
new Event {EventType = "JS",
EventDateTime = DateTime.Parse("10/21/2018 12:12 PM")},
}.OrderBy(e => e.EventDateTime);
// Initialize lastEventType to the opposite of the type of event we
// expect to see first (which is "JS" unless there are no "JS" events)
var lastEventType = events.Any(e => e.EventType == "JS")
? "JE" : "JS";
foreach (var e in events)
{
var errorMessage = string.Empty;
// Check for error
if (e.EventType == lastEventType)
{
Console.ForegroundColor = ConsoleColor.Red;
// Create an error message
var error = e.EventType == "JS"
? "JS before JE"
: "JE before JS";
var dateStr = e.EventDateTime?.ToShortDateString() ?? "[null date]";
errorMessage = $" - On {dateStr} this error occurred: [{error}]";
// Your code would include:
// errorList.Add(errorMessage);
// errorListRow.Add(dc);
}
Console.WriteLine(e + errorMessage);
Console.ResetColor();
// Update the lastEventType with this event type
lastEventType = e.EventType;
}
GetKeyFromUser("nDone! Press any key to exit...");
}
}
Output
edited Nov 13 at 18:13
answered Nov 12 at 21:41
Rufus L
16.9k31530
16.9k31530
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort byEventDateTime
and then validate that the items are of alternatingEventType
, starting withJS
?
– Rufus L
Nov 12 at 21:47
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
This code is just too show the logic. You could replace the code that sets the console color withif (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
|
show 1 more comment
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort byEventDateTime
and then validate that the items are of alternatingEventType
, starting withJS
?
– Rufus L
Nov 12 at 21:47
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
This code is just too show the logic. You could replace the code that sets the console color withif (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Thanks for taking a look Rufus. The list of events have to be ordered by time entered and then checked to make sure they are in the correct order.
– Doug Farrell
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort by
EventDateTime
and then validate that the items are of alternating EventType
, starting with JS
?– Rufus L
Nov 12 at 21:47
Ok, I see, there may be errors in the data. So you want to sort by
EventDateTime
and then validate that the items are of alternating EventType
, starting with JS
?– Rufus L
Nov 12 at 21:47
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
I've updated the answer with what I think you're asking for...can you check?
– Rufus L
Nov 12 at 21:57
This code is just too show the logic. You could replace the code that sets the console color with
if (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
This code is just too show the logic. You could replace the code that sets the console color with
if (e.EventType == lastEventType) { // error logging code here }
– Rufus L
Nov 13 at 16:28
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
I updated the example to show better how you might implement that
– Rufus L
Nov 13 at 18:13
|
show 1 more comment
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Some of your past answers have not been well-received, and you're in danger of being blocked from answering.
Please pay close attention to the following guidance:
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53267928%2fc-sharp-find-item-in-list-returned-by-linq-query-and-compare-its-value-with-anot%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
I am having a bit of trouble if there are more than one JS in a day which is allowed - if you have more than one JobStart in a day, can't you just take the latest one and compare it to the JobEnd?
– stuartd
Nov 12 at 18:22
@stuartd not sure I follow you, can you perhaps put your question into code so I can elaborate?
– Doug Farrell
Nov 12 at 18:54
Perhaps think about how you would do this by hand, with a list of event cards, some marked JS/JE. Make only one traversal through the deck. Your (one pair) code fails if there are no JS events. You don't need to keep repeating the test for
day
that was in the base query.– NetMage
Nov 12 at 19:56
@NetMageplease see update.
– Doug Farrell
Nov 12 at 20:07