Execution time is more how to overcome

below my code check I have 10000 record on that user that time 10 to 15 min time is required for Execution is too much how to overcome

List list = new List();
List<string> listadd = new List<string>();
string area = null;
List DateLocation = new List();
int mon = 0;
int year = DateTime.Now.Year;
if(12<=mon)
{
mon = 01;
}
else
{
mon = DateTime.Now.Month;
}
string dk = DateTime.Now.Year + "-" + mon + "-" + 21;
var ccheckdata = (from chk in _context.MobileGPSLocation
where chk.UserId == (UserId)
select chk).ToList().Where(a=>a.CreatedDate.Date>=Convert.ToDateTime(dk).Date.AddDays(-30));
List <string> datelist = new List<string>();
foreach (var datelist11 in ccheckdata.Where(a=>a.CreatedDate.Date>Convert.ToDateTime(dk).Date.AddDays(-30)))
{
datelist.Add((datelist11.CreatedDate).Date.ToString("dd/MM/yyyy"));
}
List<string> l = new List<string>();
l = datelist.Distinct().ToList();
var CallsCount =( from text in _context.ImageTextAdd
where text.UserId ==( UserId)
select text).ToList();
var PunchTime = (from punchda in _context.InOutAdd
where punchda.UserId == UserId
select punchda).ToList();
//new BussinessLogic.MobileGPSLocation.MobileGPSLocation(new MobileGPSLocationRepository(new CRMContext())).getAllInOut().Where(a=> a.CreatedDate.Date > Convert.ToDateTime(dk).Date.AddDays(-30));
var datewisedata = (from chk in _context.MobileGPSLocation
where chk.UserId == (UserId)
select chk).ToList().Where(a=>a.CreatedDate.Date>=Convert.ToDateTime(dk).Date.AddDays(-30));
var vechinfo = (from vehi in _context.VehicalInfo
select vehi).ToList();
#region
foreach (var data in l)
{
string dddd="";
ViewModel.MobileGPSLocation.DateWLocation dlocatonmodel = new ViewModel.MobileGPSLocation.DateWLocation();
dlocatonmodel.CreatedDate = data;
var checkInOut = PunchTime.Where(a => a.CreatedDate.Date.ToString("dd/MM/yyyy") == (data) && a.UserId == UserId).FirstOrDefault();
if(checkInOut!=null)
{
dlocatonmodel.PunchTime = checkInOut.CreatedDate.TimeOfDay.ToString();
}
var checkcalls = CallsCount.Where(a => a.CreatedDate.Date.ToString("dd/MM/yyyy") == (data) && a.UserId==UserId).ToList();
List callList = new List();
foreach(var call in checkcalls)
{
ViewModel.MobileGPSLocation.DoctorCalls doct = new ViewModel.MobileGPSLocation.DoctorCalls();
doct.calls = call.PO;
doct.DAtet = call.CreatedDate.Date.ToString("dd-MM-yyyy");
callList.Add(doct);
}
dlocatonmodel.noOCalls = callList;
var datavar = datewisedata.Where(a => a.CreatedDate.Date.ToString("dd/MM/yyyy") == data && a.UserId == UserId).ToList();
double lat1 = 0;
double lat2 = 0;
double long1 = 0;
double long2 = 0;
List listgps = new List();
#region
foreach (var veh in vechinfo)
{
ViewModel.User.KMClass kmmodel = new ViewModel.User.KMClass();
foreach (var s in datavar.Where(a => a.VehicalType == veh.Id.ToString()))
{
ViewModel.MobileGPSLocation.MobileGPSLocation ss = new ViewModel.MobileGPSLocation.MobileGPSLocation();
if (area == null)
{
area = s.Address;
area = s.Address;
ss.Lat = s.Lat;
ss.Long = s.Long;
ss.Id = s.Id;
ss.Address = s.Address;
ss.VehicalType = s.VehicalType;
listgps.Add(ss);
}
else if (area == s.Address)
{
area = s.Address;
}
else
{
ss.Lat = s.Lat.Trim();
ss.Long = s.Long.Trim();
ss.Id = s.Id;
ss.Address = s.Address.Trim();
ss.VehicalType = s.VehicalType;
listgps.Add(ss);
area = s.Address;
if (lat1.Equals(0))
{
lat1 = Convert.ToDouble(s.Lat);
long1 = Convert.ToDouble(s.Long);
}
else if (lat2.Equals(0))
{
lat2 = Convert.ToDouble(s.Lat);
long2 = Convert.ToDouble(s.Long);
}
else if (lat1 != 0 && lat2 != 0)
{
string newFormat = GetCorrectDistance(lat1, long1, lat2, long2);
double che = Convert.ToDouble(newFormat);
sum += che;
var sCoord = new GeoCoordinate(lat1, long1);
var eCoord = new GeoCoordinate(lat2, long2);
var d = sCoord.DistanceTo(eCoord);
lat1 = lat2;
long1 = long2;
lat2 = Convert.ToDouble(s.Lat);
long2 = Convert.ToDouble(s.Long);
}
}
}
if (sum != 0)
{
ViewModel.MobileGPSLocation.Location locationmodel = new ViewModel.MobileGPSLocation.Location();
locationmodel.Distance = Convert.ToDouble(sum);
locationmodel.VehicalType = veh.VehicalName;
dddd = sum.ToString("F", CultureInfo.InvariantCulture) + veh.VehicalName;
}
kmmodel.DistKM = sum;
kmmodel.vehical = veh.VehicalName;
sum = 0;
}
#endregion
dlocatonmodel.Dist = dddd;
// dlocatonmodel.DLocation = loctList;
DateLocation.Add(dlocatonmodel);
}
#endregion

2 Replies

Know the answer? Post it — somebody with the same question will find it here.

Kalyani Shevale7y ago
below my code check I have 10000 record on that user that time 10 to 15 min time is required for Execution is too much how to overcome
            List list = new List();

            List<string> listadd = new List<string>();
            string area = null;
            List DateLocation = new List();


            int mon = 0;
            int year = DateTime.Now.Year;
            if(12<=mon)
            {
                mon = 01;
            }
            else
            {
                mon = DateTime.Now.Month;
            }

            string dk = DateTime.Now.Year + "-" + mon + "-" + 21;
            var ccheckdata = (from chk in _context.MobileGPSLocation
                             where chk.UserId == (UserId)

                             select chk).ToList().Where(a=>a.CreatedDate.Date>=Convert.ToDateTime(dk).Date.AddDays(-30));


List <string> datelist = new List<string>();

            foreach (var datelist11 in ccheckdata.Where(a=>a.CreatedDate.Date>Convert.ToDateTime(dk).Date.AddDays(-30)))
            {
                datelist.Add((datelist11.CreatedDate).Date.ToString("dd/MM/yyyy"));
            }
            List<string> l = new List<string>();
            l = datelist.Distinct().ToList();

            var CallsCount =( from text in _context.ImageTextAdd
                             where text.UserId ==( UserId)

                             select text).ToList();

            var PunchTime = (from punchda in _context.InOutAdd
                            where punchda.UserId == UserId
                            select punchda).ToList();
                //new BussinessLogic.MobileGPSLocation.MobileGPSLocation(new MobileGPSLocationRepository(new CRMContext())).getAllInOut().Where(a=> a.CreatedDate.Date > Convert.ToDateTime(dk).Date.AddDays(-30));
            var datewisedata = (from chk in _context.MobileGPSLocation
                               where chk.UserId == (UserId)

                               select chk).ToList().Where(a=>a.CreatedDate.Date>=Convert.ToDateTime(dk).Date.AddDays(-30));

            var vechinfo = (from vehi in _context.VehicalInfo
                            select vehi).ToList();
            #region
            foreach (var data in l)
            {
                string dddd="";

                ViewModel.MobileGPSLocation.DateWLocation dlocatonmodel = new ViewModel.MobileGPSLocation.DateWLocation();



                dlocatonmodel.CreatedDate = data;
                var checkInOut = PunchTime.Where(a => a.CreatedDate.Date.ToString("dd/MM/yyyy") == (data) && a.UserId == UserId).FirstOrDefault();
                if(checkInOut!=null)
                {
                    dlocatonmodel.PunchTime = checkInOut.CreatedDate.TimeOfDay.ToString();
                }
                var checkcalls = CallsCount.Where(a => a.CreatedDate.Date.ToString("dd/MM/yyyy") == (data) && a.UserId==UserId).ToList();
                List callList = new List();
               foreach(var call in checkcalls)
                {
                    ViewModel.MobileGPSLocation.DoctorCalls doct = new ViewModel.MobileGPSLocation.DoctorCalls();
                    doct.calls = call.PO;
                    doct.DAtet = call.CreatedDate.Date.ToString("dd-MM-yyyy");
                    callList.Add(doct);
                }
                dlocatonmodel.noOCalls = callList;
                var datavar = datewisedata.Where(a => a.CreatedDate.Date.ToString("dd/MM/yyyy") == data && a.UserId == UserId).ToList();
                double lat1 = 0;
                double lat2 = 0;
                double long1 = 0;
                double long2 = 0;
                List listgps = new List();
                #region
                foreach (var veh in vechinfo)
                {
                    ViewModel.User.KMClass kmmodel = new ViewModel.User.KMClass();
                    foreach (var s in datavar.Where(a => a.VehicalType == veh.Id.ToString()))
                    {
                        ViewModel.MobileGPSLocation.MobileGPSLocation ss = new ViewModel.MobileGPSLocation.MobileGPSLocation();
                        if (area == null)
                        {
                            area = s.Address;
                            area = s.Address;
                            ss.Lat = s.Lat;
                            ss.Long = s.Long;
                            ss.Id = s.Id;
                            ss.Address = s.Address;
                            ss.VehicalType = s.VehicalType;
                            listgps.Add(ss);
                        }
                        else if (area == s.Address)
                        {
                            area = s.Address;
                        }
                        else
                        {
                            ss.Lat = s.Lat.Trim();
                            ss.Long = s.Long.Trim();
                            ss.Id = s.Id;
                            ss.Address = s.Address.Trim();
                            ss.VehicalType = s.VehicalType;
                            listgps.Add(ss);
                            area = s.Address;
                            if (lat1.Equals(0))
                            {

                                lat1 = Convert.ToDouble(s.Lat);
                                long1 = Convert.ToDouble(s.Long);
                            }
                            else if (lat2.Equals(0))
                            {
                                lat2 = Convert.ToDouble(s.Lat);
                                long2 = Convert.ToDouble(s.Long);
                            }
                            else if (lat1 != 0 && lat2 != 0)
                            {
                                string newFormat = GetCorrectDistance(lat1, long1, lat2, long2);
                                double che = Convert.ToDouble(newFormat);
                                sum += che;
                                var sCoord = new GeoCoordinate(lat1, long1);
                                var eCoord = new GeoCoordinate(lat2, long2);
                                var d = sCoord.DistanceTo(eCoord);
                                lat1 = lat2;
                                long1 = long2;
                                lat2 = Convert.ToDouble(s.Lat);
                                long2 = Convert.ToDouble(s.Long);
                            }
                        }

                    }
                    if (sum != 0)
                    {
                        ViewModel.MobileGPSLocation.Location locationmodel = new ViewModel.MobileGPSLocation.Location();
                        locationmodel.Distance = Convert.ToDouble(sum);
                        locationmodel.VehicalType = veh.VehicalName;
                        dddd = sum.ToString("F", CultureInfo.InvariantCulture) + veh.VehicalName;

                    }
                    kmmodel.DistKM = sum;
                    kmmodel.vehical = veh.VehicalName;

                    sum = 0;
                }
                #endregion


                dlocatonmodel.Dist = dddd;
               // dlocatonmodel.DLocation = loctList;
               DateLocation.Add(dlocatonmodel);
            }
            #endregion
0
Vishal Patel7y ago
Please mention detail about 'What you are trying to do inside method?'

Meanwhile, you can also check some optimization tips : http://www.dotnetperls.com/optimization
+1