Hi All,

Basically i have the following code. When the foreach statement is finished and the method continues, at the end of the method it re-enters the foreach statement ( which i don't want to happen as i have all the data i need ) The error i am getting is 'Object reference not set to an instance of an object'. However, more importantly how do i stop the code stepping into the foreach statement for a second time? Thanks, mbge9pjb

public void btnSave_Click(object sender, System.EventArgs e)


{

if ( Page.IsValid)


{

bool newState = false;


foreach(DataGridItem dgi in EditCalendar.Items)

{
for ( int i =0; i <checkedMonthOfYear.Length; i++)
{
CheckBox chkEfaPeak = (CheckBox) dgi.FindControl(checkedMonthOfYear[i]);
newState = chkEfaPeak.Checked;
DataSet years = getDataSetFromSessionYears();
years.Tables[0].Rows[ dgi.DataSetIndex][months[i]]= (newState ? "Y": "N");
EditCalendar.EditItemIndex = -1;
}

}

//some nmore code here but ultimately i think the problem is with the above sample. Once the nested loop has finished and continues with the method, it returns back to the 'foreach' statement which i don't want to happen

At what point is it going back to the foreach?

-Peter

Hi Peter,

Here is the full code of the method. Basically it always hit's the catch statement i have thrown. It exits the foreach statement then goes upto "calendar.saveCalendar ....." At this point it should go to the finally statement but it returns to the foreach statement. Any ideas why this might be happening?

Cheers
mbge9pjb

public void btnSave_Click(object sender, System.EventArgs e)

{

try
{
   foreach(DataGridItem dgi in EditCalendar.Items)
    {

    for ( int i =0; i< checkedMonthOfYear.Length; i++)
    {
    CheckBox chkEfaPeak = (CheckBox) dgi.FindControl(checkedMonthOfYear[i]);
    bool newState = chkEfaPeak.Checked;
    DataSet years = getDataSetFromSessionYears();
    years.Tables[0].Rows[dgi.DataSetIndex][months[i]]= (newState ? "Y": "N");
    }

   }
Int32 idCalendar = Int32.Parse.lstCalendars.Items[ lstCalendars.SelectedIndex].Value);
Int32 idCalendarVersion = Int32.Parse( hiddenCalendarVersionId.Value);
Int16 retcode;
DataSet changes = getDataSetFromSessionYears().GetChanges( DataRowState.Modified);
Calendar calendar = new Calendar( Configuration.getConnStr());
calendar.saveCalendar idCalendar, idCalendarVersion, SecurityManager.getUserID(),changes, out retcode);
}

catch (NullReferenceException )
{
 // makes sure the method catches datagrid referencing null objects
 Response.Redirect("../configuration/Peak_Scaling.aspx");
}

  finally
  {
  setDataSetToSessionYears(null);
  }
}

sorry, mistake in the last description, it gets to the finally statement and then goes back to the foreach loop

The execution returning to the loop once it had entered finally?
Just check your program flow once again. I see a high possibility of
btnSave_Click getting called twice. What you r seeing as a
'return to foreach' may just be the second execution of the function.

Just put a break point on the name of the function or put a trace at
the beginning of the function before try block and debug it to see
if the function is getting called twice.

Regards
Ankur Verma