Question

public string getString()
{

  con.ConnectionString = ConnString;
  con.Open();
  string sp = "select top 3 hotelid from hotel order by NEWID()";
  SqlCommand cmd = new SqlCommand(sp, con);

  SqlDataAdapter sa = new SqlDataAdapter(cmd);
  cmd.ExecuteNonQuery();
  reader = cmd.ExecuteReader();
  while (reader.Read()) //Call Read to move to next record returned by SQL //OR call --While(reader.Read()) 
  {
    det  = reader[0].ToString();

  }
  reader.Close();
  con.Close();
  return det;
}

When I’m executing this code I’m able to retrieve only a single item of data?,
but when I am executing the SQL query I am able to retrieve randomly 3 items of data.

Answer 1

You are overwriting the det variable on each while loop.

You either need to create a collection and add to it, or concatenate the string (note the +=)…

det += reader[0].ToString();

UPDATE

As suggested above, another option is to create a collection, something like…

public List<string> getString()
{
  ...
  List<string> ret = new List<string>;
  while (reader.Read())
    ret.Add(reader[0].ToString());
  ...
  return ret;
}