ASP.NET: Kode i aspx-fil

Hvorfor er det lige du ikke vil gøre bruge af din .cs fil
og så smide en Label i din aspx fil?

dennism du skal altid kode i .cs filen! :) Rigtig dårlig vane at bruge aspx filen til c# kode.. Gør at den er meget usikker og ikke kan optimeres på samme måde.

hvis det er fordi du ikke vil hente værdien ud fra det objekt på aspx siden, skal du tilknytte source i .cs filen og hente din værdi derfra

Tak for jeres svar.
Måske jeg lige skulle poste noget af min kode, for at synliggøre hvilken situation jeg er i.

http://dennismadsen.net/uploads/Default.aspx.txt
http://dennismadsen.net/uploads/Default.aspx.cs.txt

Der er problem på linie 46 i Default.aspx:
    <%# this.SetSensorValue(this.GetSensorValueFromService((int)DataBinder.Eval(Container.DataItem, "Id")))%>

Den melder fejlen "cannot convert type 'void' to 'object'", hvilket åbenbart er fordi at <%# gerne vil printe noget.

Hvordan kan jeg komme uden om dette?

protected void Page_Load(object sender, EventArgs e)
    {
        RepeaterTanks.DataSource = GetTanks();
        RepeaterTanks.DataBind();
    }

    private DataTable GetTanks()
    {
        DataTable dt = new DataTable();
        dt.Columns.Add("Id", typeof(int));
        dt.Columns.Add("Name", typeof(string));
        dt.Columns.Add("Sensors", typeof(DataTable));

        //Hent db service her
        var qTanks = from r in service.Tanks select r;

        foreach (var tank in qTanks)
        {
            DataRow dr = dt.NewRow();
            dr["Id"] = tank.ID;
            dr["Name"] = tank.Name;
            dr["Sensors"] = GetSensors(tank.ID);

            dt.Rows.Add(dr);
        }

        return dt;
    }

    private DataTable GetSensors(int tankID)
    {
        DataTable dt = new DataTable();
        dt.Columns.Add("Id", typeof(int));
        dt.Columns.Add("Name", typeof(string));
        dt.Columns.Add("WarningValue", typeof(string));
        dt.Columns.Add("cssClass ", typeof(string));

        //Hent db service her
        var qSensors = from r in service.Sensors where r.TankID == tankID select r;

        foreach (var sensor in qSensors)
        {
            DataRow dr = dt.NewRow();
            dr["Id"] = sensor.ID;
            dr["Name"] = sensor.Name;
            dr["WarningValue"] = sensor.WarningValue;

            dr["cssClass"] = "valueIconGreen";
            if (sensor.Value < sensor.WarningValue)
                dr["cssClass"] = "valueIconRed";
            if (sensor.Value - 5 < sensor.WarningValue)
                dr["cssClass"] = "valueIconYellow";

            dt.Rows.Add(dr);
        }

        return dt;
    }

Du kan sikkert også lave det med linq tables. Men jeg er først lige startet på linq, men håber du forstår meningen.

Ligeledes har jeg flyttet din kode fra aspx filen til cs filen, så kan du også debugge din kode rigtig.

Kan være du også kan lave det hele i linq a la;

        var q = from tank in service.Tanks
                select new
                {
                  Id = tank.Id,
                  Name = tank.Name,
                  Sensors = select new
                  {
                      Name = tank.Sensors.Name
                  }
                };

og så noget if else via:
boolStatement ? ifStatementIsTrue : ifStatementIsFalse

<%# %> til webcontroller, <%= %> uden for.

Dog kræver <%# %> at kontrollen/siden databindes.

/J

Du mener <% %> uden for

<%= %> svare til <%Response.Write( );%> hvilket han selvfølgelige også kan bruge

Dit forslag med min virker ikke med den select inden i en select. Kan ikke hente tank.Sensors.Name inden i den, da Sensors indeholder mange Sensor.

Rettelse:
Dit forslag med LINQ virker ikke med den select inden i en select. Kan ikke hente tank.Sensors.Name inden i den, da Sensors indeholder mange Sensor.

Ja det var sådan set også meningen, men jeg ved ikke hvordan det skal laves i linq.

Jeg har lavet om, så jeg nu har dette LINQ:

    RepeaterTanks.DataSource = from tank in service.GetTanks()
                              select new {
                                  Id = tank.Id,
                                  Name = tank.Name,
                                  Sensors = (from sensor in tank.Sensors
                                            where sensor.TankId == tank.Id
                                              let NewestValue = this.GetSensorValueFromService(sensor.Id)
                                            select new {
                                                TypeName = sensor.SensorType.Name,
                                                Value = NewestValue,
                                                WarningValue = sensor.WarningValue,
                                                CssClass = this.GetSensorValueCssClass((double)NewestValue,(double)sensor.WarningValue)
                                            })
                              };

Og følgende i aspx:
        <asp:Repeater ID="RepeaterSensors" runat="server" DataSource='<%# DataBinder.Eval(Container.DataItem, "Sensors")%>'>
            <ItemTemplate>
                <tr>
                    <td>
                    </td>
                    <td>
                        <%# DataBinder.Eval(Container.DataItem, "TypeName")%>
                    </td>
                    <td><%# DataBinder.Eval(Container.DataItem, "Value")%> / <%# DataBinder.Eval(Container.DataItem, "WarningValue")%></td>
                    <td><div class="<%# DataBinder.Eval(Container.DataItem, "CssClass")%>"></div></td>
                </tr>
            </ItemTemplate>
        </asp:Repeater>

Det virker :) Mange tak for hjælpen. Kan godt se, at koden så bliver langt pænere! Vil du smide et svar cars10?

Ja det var meget bedre. Så fik jeg også lært noget nyt Linq. Den eneste ulempe jeg kan se er at det er sværere at debugge Linq, til gengæld er det hurtigere end det kode jeg postede.