Fill dataset hvis der er noget at læse

umiddelbart ved at teste på: ds.Tables("mintabel").Rows.Count > 0

nej, for min sql-sætning er et join mellem flere tabeller, hvor der i hver enkelt altid vil være værdier. Desuden skal jeg først fylde datasettet hvis testen er sand.

Jow - if minSqlDataReader.hasrows = true then

det kan jeg jo ikke, da jeg skal fylde datasettet etter at connection er closed, og jeg ikke kan læse med datareader når den er lukket.

har du prøvet således: ?

minConnection.Open();
minSqlDataReader = minCommand.ExecuteReader();
if minSqlDataReader.Hasrows = True {
    DataSet ds = new DataSet();

    while (minSqlDataReader.Read())
    {
        minSqlDataAdapter.SelectCommand = minCommand;               
    }
           
    minSqlDataReader.Close();           
    minConnection.Close();
    minSqlDataAdapter.Fill(ds);
    return ds;
}
else {
  Response.write "HEST - no data!";
}

BTW: om min C# syntax er OK er ikke sikkert da jeg kun koder VB.NET!

Gør jeg sådan, funderer det:

minConnection.Open();
minSqlDataReader = minCommand.ExecuteReader();
DataSet ds = new DataSet();
if (minSqlDataReader.HasRows == true)
{     
  while (minSqlDataReader.Read())
  {
    minSqlDataAdapter.SelectCommand = minCommand;   
  }
   
  minSqlDataReader.Close();                   
  minConnection.Close();
  minSqlDataAdapter.Fill(ds);
  return ds;
  }

  else
  {
    minSqlDataReader.Close();                   
    minConnection.Close();
    minSqlDataAdapter.Fill(ds);
               
    return ds;
  }

Et par små syntaktiske ting, ellers er det ok.

Tak!

PS: Hvorfor indkaplser du egentlig ikke din dbconnection i en try, catch, finally block?

det ville nok være en god ide...

er den her kode ikke lidt... ja, underlig... unødvendig

  while (minSqlDataReader.Read())
  {
    minSqlDataAdapter.SelectCommand = minCommand;   
  }

du tildelder vel den samme command x antal gange = spild af tid

jeg ville gøre det sådan her istedet:

minConnection.Open();
minSqlDataReader = minCommand.ExecuteReader();
DataSet ds = new DataSet();

if (minSqlDataReader.Read()) {

  minSqlDataAdapter.SelectCommand = minCommand;
 
  minSqlDataReader.Close();                   
  minConnection.Close();
  minSqlDataAdapter.Fill(ds);
  return ds;

} else {

  minSqlDataReader.Close();                   
  minConnection.Close();
  minSqlDataAdapter.Fill(ds);
               
  return ds;
}


Eller endnu bedre: drop readeren

DataSet ds = new DataSet();
minSqlDataAdapter.Fill(ds);

if (ds.Rows.Count > 0) {
  minSqlDataAdapter.SelectCommand = minCommand;
}

return ds;