Køre subs fra onClick

hermmm...

du har ikke secficeret at img skal være en html control, gøres ved at tilføge runat=server" i dit img tag

secficeret = specificeret... ja, klokken er mange ;)

blah - hvad med om du læste lidt mere på din VB Script syntax ?

du kan ikke caste din variabel nr som du prøver på

bare fjern det, så virker det fint ( i IE og ikke andet )

hvis det er noget postback halløj, så kan du heller ikke lave det på den måde som du prøver !

Først skal du putte billedet ind som et element i en form ( input type=image ), og derefter bruge onServerClick

du skal dog være opmærksom på du ikke kan sende værdier med på den måde du prøver, men du kan derimod sætte dem i et andet input felt

skg>> og hvis du vidste bedre ville du vide at man ikke bruger VB Script i asp.net

hmm, jeg havde håbet på en let løsning :-( 

Og ja, jeg er godt klar over at det er en dejlig blanding mellem vb.net, asp.net, asp og vb-script, men man kan ikke blive andet end forvirret og fortvivlet, når man normalt bruger almindelig asp og så frygtesløst kaster sig ud i asp.net :-)

Nå, men det kan være at I kan hjælpe mig med at løse det på en anden måde. Her er hvad jeg skal lave:

Jeg er ifærd med at lave en indkøbskurv. Jeg danner en produktliste i en tabel ved hjælp af asp:repeater, og i hver række skal der være mulighed for at skrive et antal i en tekstboks og trykke på "bestil" (enten billede eller knap). Når man trykker på "bestil" skal der køres en sub, der sørger for at tilføje produktet til indkøbskurven. Jeg skal derfor både bruge produktnummer samt antal for det valgte produkt. Hvordan gør jeg det ?

Prøv at kigge lidt på <ASP:ImageButton> :-)

<asp:ImageButton Runat="server" CssClass="din_css_klasse" ImageUrl="dit_billede" OnClick="Bestil_click"></asp:ImageButton>

/askhoej

nå jo, men hvordan får jeg så i sub'en Bestil_clik fat i det antal der bliver skrevet i tekstboksen, samt produktnummeret på det produkt, der er i den række ?

Cyperfessor: der stod jo ikke lige noget om at det var serverside, derfor den første kommentar...

men...

hvis du kommer fra en ASP-baggrund, så kan du jo bare "misbruge" din viden derfra. Du kan stadig lave ASP'en som "i gamle dage".

så kunne du evt. skrive en form ud for hver "repeat" og så bare sætte dit produktID/Navn i et hidden felt, og badabing, så har du ligepludselig noget der ligner

  ProductID    = "14"
  ProductCount = "nogetmanindtaster"

Sendt tilbage til serveren....


Ja, der findes mere elegante metoder til det, men det her virker også.

skg>> kategorien er asp.net

Og hvordan skulle den form så se ud ?

For at bruge et billede som submit-knap, skal der vel på onclick være noget i retning af formX.submit(), men hvordan ved den hvilken for der skal submittes.

Cyberfessor -> Det lyder somom du ikke mener, at det er den rette måde at gøre det på. Hvordan skal jeg så gøre det ?

rbl>> hvis du nu satte dig lidt ind i tingene ville du hurtigt indse at du ikke bare kan kalde en sub der kører på serveren på den måde du gør det...
For det første skal du køre dit link i en <form runat="server"> tag.
For det andet skal du sørge for at dit link, billede eller hvad du nu vælger at bruge også er en server-control, altså også har runat="server" specificeret. Det kan enten være et <img> tag, eller måske et <asp:imagebutton> tag som også er blevet foreslået.
For det tredie kan du ikke kalde din sub med parametre, altså bliver det en ren onclick=Bestil_Click
For det fjerde er din sub skrevet forkert, det skal være sådan her

sub bestil_click(Src as Source, e as Eventargs)
end sub

mudderkast igen :-)

Ej, pis til side.

I ASP/VBScript ville det nok se sådan her ud:

Do While MitRS.EOF = False

  Response.Write("<form name=""Prod" & MitRS("ProductID") & """>")
  Response.Write("<input type=""hidden"" name=""ProductID"" value=""" & MitRS("ProductID") & """>")
  Response.Write("Navn:<b>" & MitRS("ProductName") & "</b><br>")
  Response.Write("Antal:<input type=""text"" name=""ProductCount"" value=""1""><br>")
  Response.Write("<input type=""image"" src=""DinKoepeKnap.gif"">")
  Response.Write("</form>")

  MitRS.MoveNext()

Loop


I .Net kan man sagtens lave det på samme måde, men ja, der findes så mange andre muligheder som er ret uoverskuelige uden at man har skabt et lille overblik over dem. Personligt synes jeg det er ret svært at holde styr på det hele uden at man sidder og fedter med sit Visual Studio .Net, men her er det dog også ret let.

(Jeg sidder ikke lige ved min Visual Studio nu ...)

skg -> Ja, jeg havde også selv tænkt på at gøre det på en lignende måde, men jeg vil gerne finde ud af den "rigtige" måde i asp.net

Cyberfessor -> Det er selvfølgelig godt nok, at du skriver at jeg har gjort det forkert, men jeg mangler stadig at finde ud af, hvordan jeg får fat i de to værdier fra min sub.

de er ikke testet, men det her burde virke

<%@ Page Language="C#" ContentType="text/html" ResponseEncoding="iso-8859-1" %>
<script language="C#" runat="server">
void Bestil_click(Object Src, EventArgs e) {
    myLabel.Text = "Du har trykket på nr"+ nr.value
}
</script>
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<form runat="server">
<input type="hidden" id="nr" value="1" runat="server" />
<asp:imagebutton ID="myImgButton" OnClick="Bestil_click" runat="server" />
</form>
<asp:label ID="myLabel" runat="server" />
</body>
</html>

der må kun være én serverside form på en side

rbl>> ja... og??

der skal jo være en textboks og bestilknap i hver række i tabellen

Du kan os lave det sådan her :)

<%@ Page Language="VB" runat=server %>
<script runat=server>
  Sub Click(Obj as Object, e as Imageclickeventargs)
    Label.Text = Obj.id.substring(3)
  End Sub
</script>
<html><body>
<form runat=server>
<asp:imagebutton runat="server" id="Img1" imageurl="src" onclick="click" />
<asp:imagebutton runat="server" id="Img2" imageurl="src" onclick="click" />
<asp:label id="Label" runat=server />
</form>
</body></html>

rbl>> der er en grund til at det hedder webforms, og det er at det hele kører på postback, og derfor er der ikke nogen der forhindrer en i at lade ens form dække over HELE siden, altid, altså

<html>
<body>
<form runat="server">
... din side...
</form>
</body>
</html>

ang. din tekstbox og bestilknap i hver række i tabellen kan du bruge en repeater eller datagrid til at få vist alle de poster du nu engang skal have vist på din side

cyberfessor > bare fordi man KAN betyder det ikke at man SKAL

Du får jo heller ikke buskevand hver dag, bare fordi det er muligt.


Hvis du flækker en form rundt om hele din side, kunne jeg forestille mg nogle temmeligt onde viewstates der blev kastet med rundt.

skg>> har jeg sagt at han skal? jeg skrev at der er ikke noget der forhindrer ham i det, og hvis man programmeringsmessigt kan finde ud af at styre sine viewstates, så er det problem også ude af verden.
Hans problem var åbenbart at han havde brugt to forms på en side... jeg pointere så at løsningen til det var at lave den ene form man må have på en side favne over lidt mere.

Lad os sige at du i starten har en <asp:linkbutton> og i bunden af siden har en <asp:button> i forbindelse med noget input. I sådan en tilfælde må man strække sin form fra starten af siden til slutningen for at kunne bruge postback.

man må altså ikke gøre sådan her

<%@ Page Language="C#" ContentType="text/html" ResponseEncoding="iso-8859-1" %>
<script language="C#" runat="server">
void myButton_Click(Object src, EventArgs e) {

}

void myLink_Click(Object src, EventArgs e) {

}

</script>
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<form runat="server">
<asp:linkbutton ID="myLink" runat="server" OnClick="myLink_Click" Text="myLink" />
</form>
... en masse tekst...
<form runat="server">
<asp:button ID="myButton" runat="server" OnClick="myButton_Click" Text="myButton" />
</form>
</body>
</html>

men skal gøre

<%@ Page Language="C#" ContentType="text/html" ResponseEncoding="iso-8859-1" %>
<script language="C#" runat="server">
void myButton_Click(Object src, EventArgs e) {

}

void myLink_Click(Object src, EventArgs e) {

}

</script>
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<form runat="server">
<asp:linkbutton ID="myLink" runat="server" OnClick="myLink_Click" Text="myLink" />

... en masse tekst ...

<asp:button ID="myButton" runat="server" OnClick="myButton_Click" Text="myButton" />
</form>
</body>
</html>

Man du har stadig ikke svaret på, hvordan jeg fortæller sub'en hvilken knap der blev trykket på og hvilket textfelt, der hører til knappen. Hvis der f.eks. er 20 rækker med varer, og man så skriver et antal i tekstfeltet og trykker på Bestilknappen ud for varen med produktnummer X. Alle de 20 knapper aktiverer den samme sub ved onClick, så jeg skal på en eller anden måde i sub'en finde ud af hvad produktnummeret er, og hvilket tal der er skrevet i tekstfeltet.

her skulle gerne stå noget grundlæggende om databinding... jeg kan anbefale det sidste ekspempel (8'eren), som gør det du søger

http://samples.gotdotnet.com/quickstart/aspplus/doc/webdatalist.aspx

nej, den leder bare efter alle checkbokse, der er checket. Og dog...

-> Jeg kiggede lige på det -psycho- foreslog. Hvis det virker, så er jeg godt på vej. Hvis jeg sætter id på imagebutton'en lig med produktnummeret, så har jeg produktnummeret på det produkt, der er trykket ud for.

Kan jeg så løbe alle tekstbokse igennem og checke på deres id, for at se om det passer til knappens id, på den måde som du foreslår cyberfessor ?

rbl : Værsgo her er i hvert fald en måde du kan lave det på..

<%@ Page Language="VB" runat=server %>
<script runat=server>
  Sub Click(Obj as Object, e as Imageclickeventargs)
    Label.Text = "Du har trykket på knap nr " & Obj.id.substring(6) & ". I tekstbox nr" & Obj.id.substring(6) & " har du skrevet '" & Request.Form("txtbox" & Obj.id.substring(6)) & "'"
  End Sub
</script>
<html><body>
<form runat=server>
<asp:textbox runat="server" id="Txtbox1" />
<asp:imagebutton runat="server" id="Imgbtn1" imageurl="src" onclick="click" /><br>
<asp:textbox runat="server" id="Txtbox2" />
<asp:imagebutton runat="server" id="Imgbtn2" imageurl="src" onclick="click" /><br>
<asp:label id="Label" runat=server />
</form>
</body></html>

rbl : Kan du bruge det ??

Hej igen, undskyld det tog så lang tid.

Jeg er stødt ind i et lille problem. Den vil ikke være med til at jeg sætter produktnummeret ind i "id" på knappen:

<asp:imagebutton runat='server' id='<%# DataBinder.Eval (Container.DataItem, "varenr") %>' imageurl='src' onclick='bestil_Click' />

Den kommer med denne fejl:
'<%# DataBinder.Eval (Container.DataItem, "varenr") %>' is not a valid identifier.

Id må ikke kun være et tal..

Du er nødtil at sætte et bogstav(er) foran sådan her

<asp:imagebutton runat='server' id='"ID" & <%# DataBinder.Eval (Container.DataItem, "varenr") %>' imageurl='src' onclick='bestil_Click' />

Hvis du bruger "ID" skal du for et få selve din id køre sådan her obj.id.substring(2)

så den fjerner "ID" fra din id og du kun har selve varenr tilbage

det er det substring gør.. hvis din id er "ID0001" og du bruger
substring(2) så henter den kun "0001", hvis det var substring(1) så ville den hente "D0001" eller hvis substring(3) "001"

Stadig ingen held:

'"ID" & <%# DataBinder.Eval (Container.DataItem, "varenr") %>' is not a valid identifier.

Hov min fejl undskyld det skal være
'<%# "ID" & DataBinder.Eval (Container.DataItem, "varenr") %>'

Den melder stadig samme fejl. Det er mærkeligt, at den ikke vil være med til det

Hmm, prøv bare for en test at smide en label på og så databind dens tekst til '<%# "ID" & DataBinder.Eval (Container.DataItem, "varenr") %>' bare for at se hvad det er den prøver at skrive i id.

jeg prøvede noget andet. Jeg erstattede id med name, så der kom til at stå:
name='<%# "ID" & DataBinder.Eval (Container.DataItem, "varenr") %>'

Da IE viste siden, valgte jeg "vis kildekode". Jeg fandt det sted, hvor knappen var, og det var skrevet fuldstændig rigtigt: name='ID0001'

Ærgeligt, at jeg ikke kan bruge name til noget. Men det er da mærkeligt, at id ikke vil godtage det, når name vil

rbl> problemet er at du ikke kan tildele id'er dynamisk på den måde. Jeg vil anbefale dig at kigge en ekstra gang, da det er den rigtige måde at gøre det på !!

http://samples.gotdotnet.com/quickstart/aspplus/doc/webdatalist.aspx
http://samples.gotdotnet.com/quickstart/aspplus/samples/webforms/customize/VB/datalist3.aspx

Ja, det er jo nemt nok at lave et link, der sender produktnummeret med som parameter, men problemet er jo, at jeg også skal have et "antal" med.

antal kommer jo fra din textbox

<asp:textbox id="txtMyTextbox" runat="server" />

du fanger den med

antal = txtMyTextbox.Text

ja, men når de bliver dynamisk lavet, så ved jeg jo ikke hvad textboxens id er før jeg har produktnummeret.

lad os sige din textbox hedder antal

For i=0 To MyDataList.Items.Count -1

    Dim CurrentTextBox As TextkBox
        CurrentTextBox = MyDataList.Items(I).FindControl("Save")
        "Nummer:" & i &" = "& CurrentTextBox.Text
Next

arg..

For i=0 To MyDataList.Items.Count -1

    Dim CurrentTextBox As TextkBox
        CurrentTextBox = MyDataList.Items(I).FindControl("antal")
        "Nummer:" & i &" = "& CurrentTextBox.Text
Next

det kræver selvfølgelig at du skriver alle dine produkter ud i et datalist, hvilket også er den rigtige måde at gøre det på... i det her tilfælde hedder dataliste't så MyDataList

Så får jeg jo bare indholdet af alle tekstbokse. Det jeg skal have fat i er den rigtige tekstboks, der står ud for den knap, der er trykket på.

det er sq lovligt at tænke selv... så istedet for at løbe dem alle sammen igennem bruger du MyDataList.SelectedItem.ItemIndex

denne rutine burde finde din textbox for det billede du har trykket på

        void Datalist_Select(object sender, System.EventArgs e)
        {
            TextBox thebox = new TextBox();

            thebox = Datalist.Items(Datalist.SelectedItem.ItemIndex).FindControl("antal");
            int antal = thebox.Text;
        }

sorry.. jeg fik rodet vb.net og c# sammen...

den her skulle virke

        void Datalist_Select(object sender, System.EventArgs e)
        {
            int antal = System.Convert.ToInt32(((CheckBox) Datalist.Items[Datalist.SelectedItem.ItemIndex].FindControl("antal")));
        }

jeg har lavet det med en repeater. Kan jeg bruge den i ovenstående kode i stedet for datalist ?

Nå, nu har jeg ændret det til en datalist. Nu får jeg en fejlmeddelelse, når jeg forsøger at få fat på tekstboksen:
System.NullReferenceException: Object reference not set to an instance of an object.

i linien:
thebox = ctrlrepeater.Items(datalist1.SelectedItem.ItemIndex).FindControl("antalField")

ups, linien der kommer fejl i ser sådan ud:
thebox = datalist1.Items(datalist1.SelectedItem.ItemIndex).FindControl("antalField")

Nu har jeg fået det til at virke:

sub bestil_Click(Obj As Object, e As System.Web.UI.ImageClickEventArgs)
  Dim antal as integer
  Dim varenr as String
  dim i as integer
  dim index as integer

  For i=0 To datalist1.Items.Count -1
    if obj is datalist1.Items(i).FindControl("knap") then
      index = i
    end if
  Next 
 
  dim varebox as textbox =  datalist1.Items(index).FindControl("varenrField")
  dim antalbox as textbox = datalist1.Items(index).FindControl("antalField")

  varenr = varebox.text
  antal = antalbox.text
end sub

Men efter at jeg har ændret min repeater til en datalist, så er alle borders på min tabel forsvundet. hvordan får jeg dem frem igen ?

rbl : Nå jeg kan se du har fået hjælp af cyberfessor...

Hvis du hellere vil have det som et datagrid så bare sig til..

Jeg har lige lavet det med et datagrid..

det var dejligt du fik det til at virke.. jeg har ikke haft tid til at checke min eksperten.dk før nu, men har da rodet lidt med det datalist selv (må ærligt indrømme at det jeg fortalte dig inat var ren teori), men som du kan se her http://www.burningice.dk/datalist.aspx (tryk på et navn), virker det skam også i praksis.

Ang. så ved jeg ikke om det er nogen i den retning som jeg har lavet du søger??

I så fald er det SeparatorStyle og SeparatorTemplate du skal kigge lidt på

ang din border skulle der selvfølgelig stå

gridlines kan også gøre det.. det er måske lidt mere hensigtsmæssigt i din sammenhæng?? Sæt den til both for at få border både vertikalt og horisontalt

Vidste faktisk ikke at man ikke kunne lave id dynamisk. Så det må du undskylde rbl :|. Men så ved jeg da os det.

-psycho->> ?? hvad snakker du om?? man kan ikke lave id dynamisk

???, prøv at læs hvad jeg skrev.. Vidste faktisk ikke, at man IKKE kunne lave id dynamisk.  :D

ah.. det er jo tidligt på morgenen ;) vidste ikke at man ikke... det lyder lidt sjovt... men ja, du har ret.. jeg læste forkert

heh, tidligt. Hvilken tidszone bor du i :D

Men jeg kan se det skal en del mindre linier til for at finde frem til varene og textboxen i et datalist..

Jeg har 2 linier i min click.

Dim txtbox As TextBox = e.Item.FindControl("Txtbox")
  Label.Text = "Vare Nr " & e.Item.Cells(0).Text & ". Tekst i textbox ' " & txtbox.Text & "'"

Nu virker det hele. Tak for hjælpen begge to. Hvis I vil have point, skal I oprette et svar. 30 point til deling er ikke meget, så jeg vil afsætte nogle flere, hvis I er interesseret

mindre = flere , og varene = varenr..

Heh, okay jeg bor i samme tidszone som dig :D

jeg har følgende kode til min klik

public void MyDataList_Select(Object sender, EventArgs e)
        {
            int index = MyDataList.SelectedItem.ItemIndex;

            MyLabel.Text += index.ToString() +" og i den tilhørende textbox stod der <b>"+ ((TextBox) MyDataList.Items[index].FindControl("txtText")).Text +"</b>";

        }

ok

rbl>> takker for point, og jeg håber du fik lært noget du kan bruge i din videre udvikling :)

tjo, jeg fik da lært at der er strengere regler for hvad man kan gøre i asp.net end i almindelig asp.

hehe... ja, det er rigtigt... eller... ikke strengere regler for hvad man kan, men mere hvordan det skal gøres.
Det er fordi at asp er et script sprog, hvor .net er et object orienteret programmeringssprog, i stil med JAVA. Dog er der uanede muligheder med asp.net i forhold til asp når man først får sat sig rigtig ind i tingene.

rbl : Tak for point