按 newid() 排序 - 读取器无法正确读取

Question

我编写了从表中随机选择电影并显示星星的代码，但由于某种原因，当我使用 newid（） 的 ORDER 时，我的星星可能会显示这是我的存储过程：

CREATE PROCEDURE [dbo].[select_forside]
AS
   SELECT TOP(1) * 
   FROM [film] 
   INNER JOIN billeder ON film.fk_bil_id = billeder.bill_id 
   ORDER BY newid()
   RETURN 0

和我的代码隐藏：

SqlConnection conn3 = new SqlConnection();
conn3.ConnectionString = System.Configuration.ConfigurationManager.ConnectionStrings["DatabaseConnectionString1"].ConnectionString;
SqlCommand cmd3 = new SqlCommand();
cmd3.CommandType = CommandType.StoredProcedure;
cmd3.CommandText = "select_forside";
cmd3.Connection = conn3;
//  cmd3.Parameters.Add("@login", SqlDbType.Decimal).Value = Session["login"];
conn3.Open();
SqlDataReader reader3 = cmd3.ExecuteReader();
if (reader3.Read())
{
    int stardisplay = Convert.ToInt32(reader3["film_rating"]);
    // display rating
    rating_panel_display.Visible = true;
    // hvis rating = 1
    if (stardisplay == 1)
    {
        star6.Visible = true;
    }
    // hvis rating = 2
    if (stardisplay == 2)
    {
        star6.Visible = true;
        star7.Visible = true;
    }
    // hvis rating = 3
    if (stardisplay == 3)
    {
        star6.Visible = true;
        star7.Visible = true;
        star8.Visible = true;
    }
    // hvis rating = 4
    if (stardisplay == 4)
    {
         star6.Visible = true;
         star7.Visible = true;
         star8.Visible = true;
         star9.Visible = true;
    }
    // hvis rating = 5
    if (stardisplay == 5)
    {
         star6.Visible = true;
         star7.Visible = true;
         star8.Visible = true;
         star9.Visible = true;
         star10.Visible = true;
    }
}
conn3.Close();

如果我说WHERE film_id = 2，那么它将显示电影 2 和正确数量的星星，它是关于order by newid()的，但我不知道我问过的人也不知道

Answer 1

一位朋友告诉我删除我的中继器并在阅读器中分配值，并为帮助工作 evryone 这里是解决方案

 SqlConnection conn3 = new SqlConnection();
        conn3.ConnectionString = System.Configuration.ConfigurationManager.ConnectionStrings["DatabaseConnectionString1"].ConnectionString;
        SqlCommand cmd3 = new SqlCommand();
        cmd3.CommandType = CommandType.StoredProcedure;
        cmd3.CommandText = "select_forside";
        cmd3.Connection = conn3;
       //  cmd3.Parameters.Add("@login", SqlDbType.Decimal).Value = Session["login"];
        conn3.Open();
        SqlDataReader reader3 = cmd3.ExecuteReader();
         // Lav en reader som reader alt ud, billede title og rating
        // og så knyt det til dine forskellige ellementer som Lables og Image
        // Det bliver selvfølgelig et problem hvis du skal have flere end en, for så skal du bruger en repeater
        // og jeg ved ikke hvordan man kan kode bagved til at checke stjerne antallet
        if (reader3.Read())
        {
            int stardisplay = (int)Math.Round(Convert.ToDecimal(reader3["film_rating"]), 0);
            // display rating
            rating_panel_display.Visible = true;
            forside_img.ImageUrl = "images/plakater/"+ reader3["bill_sti"] +"";
            forside_h2.Text = "" + reader3["film_navn"] + "";
            forside_p.Text = "" + reader3["film_beskr"] + "";
            film_rating.Text = "" + reader3["film_rating"] + "";

            // hvis rating = 1
            if (stardisplay == 1)
            {
                star6.Visible = true;
            }
            // hvis rating = 2
            if (stardisplay == 2)
            {
                star6.Visible = true;
                star7.Visible = true;
            }
            // hvis rating = 3
            if (stardisplay == 3)
            {
                star6.Visible = true;
                star7.Visible = true;
                star8.Visible = true;
            }
            // hvis rating = 4
            if (stardisplay == 4)
            {
                star6.Visible = true;
                star7.Visible = true;
                star8.Visible = true;
                star9.Visible = true;
            }
            // hvis rating = 5
            if (stardisplay == 5)
            {
                star6.Visible = true;
                star7.Visible = true;
                star8.Visible = true;
                star9.Visible = true;
                star10.Visible = true;
            }
        }
        conn3.Close();