[C# ASP.NET] Inhoud Array van files tellen

[Verwijderd]

Ik heb een script gemaakt dat een array maakt aan de hand van de inhoud van een map met bestanden.
Wat ik nu wil is de inhoud tellen (dat wordt dan een int) en een random nummer genereren. Het heeft namelijk als doel om een random image te laten zien op de pagina.

code:

        ArrayList a = new ArrayList();
        DirectoryInfo info = new DirectoryInfo(Server.MapPath("~/Images"));
        FileInfo[] finfo = info.GetFiles();

       [u]// De inhoud van de array moet ik tellen en dan een random nummer genereren[/u]


        foreach (FileInfo fi in finfo)
        {
            a.Add("~/Images/" + fi.Name);
        }
        this.DataList2.DataSource = a;
        this.DataList2.DataBind();

Ik heb al heel wat en heel lang zitten knutselen. Bijna alle random image scripts zijn in VB geschreven maar ik ben nu eenmaal in c# begonnen .

Misschien is het zelfs ook wel handig om inplaats van een Datalist gewoon een asp image veld te gebruiken. Hopelijk kan iemand mij verder helpen om ook dat te realiseren, want ik bedenk me dat ik maar 1 plaatje wil en niet alles uit desbetreffende map

[ Voor 3% gewijzigd door Verwijderd op 15-02-2008 21:42 ]

[KoW]

Wat heb je zelf al geprobeerd dan ?
Het komt een beetje over alsof je wilt dat wij je script gaan schrijven.
Je zou het tellen natuurlijk in de foreach loop kunnen doen (zeker niet de netste en eenvoudigste oplossing, maar nu heb je niets)
Als je finfo intypt dan komt er een intellisense venstertje, heb je daar al in gekeken?

Voor een random nummer generatie zijn toch randomize functies uitgevonden? Dat bestaat in C# ook

[ Voor 11% gewijzigd door KoW op 15-02-2008 22:33 ]

[DoDo]

C#:

//Aantal elementen van de array
int amount = finfo.Length;

//Random getal tussen de 0 en de lengte
 int randomGetal = 0;
 Random r = new Random();
 randomGetal = r.Next(0, amount);

[Verwijderd]

KoW schreef op vrijdag 15 februari 2008 @ 22:29:
Wat heb je zelf al geprobeerd dan ?
Het komt een beetje over alsof je wilt dat wij je script gaan schrijven.
Je zou het tellen natuurlijk in de foreach loop kunnen doen (zeker niet de netste en eenvoudigste oplossing, maar nu heb je niets)
Als je finfo intypt dan komt er een intellisense venstertje, heb je daar al in gekeken?

Voor een random nummer generatie zijn toch randomize functies uitgevonden? Dat bestaat in C# ook

Klopt. Daar heb je gelijk in.
Ik heb geprobeerd te tellen dmv een foreach loop. Maar het wilde gewoon niet lukken en dat lag voornamelijk aan het feit dat ik verkeerde definities opgaf. Ook heb ik helemaal over het hoofd gezien dat je de Length van de array kan toewijzen aan een int.

Ik probeerde aan de hand van een aantal vbscripts de boel om te zetten, erg omslachtig in mijn geval.

[Verwijderd]

DoDo schreef op vrijdag 15 februari 2008 @ 22:32:

C#:

//Aantal elementen van de array
int amount = finfo.Length;

//Random getal tussen de 0 en de lengte
 int randomGetal = 0;
 Random r = new Random();
 randomGetal = r.Next(0, amount);

Bedankt, deze code zorgt dus voor een random getal aan de hand van de lengte.

Nu ga ik dit getal zelf niet gebruiken als bestandsnaam, maar de bedoeling is eigenlijk dat door middel van een loop door de array, dat als randomGetal is gevonden, het plaatje op die locatie uit de array op de site geplaatst word.

Zelf heb ik dit gemaakt maar ik krijg alle plaatjes op het scherm ipv 1.

C#:

//Loop door de array tot we de juiste hebben
        int Loopje = 1;

        foreach (FileInfo fi in finfo)
            if (Loopje == randomGetal)
            {
                a.Add("~/Images/Random/" + fi.Name);
            }
            else
            {
                Loopje++;
            }

Het werkt dus nog niet naar behoren. Heb ik iets fout gedaan in de foreach loop? Of kan het misschien anders.

[BM]

Verwijderd schreef op vrijdag 15 februari 2008 @ 23:04:
[...]


Bedankt, deze code zorgt dus voor een random getal aan de hand van de lengte.

Nu ga ik dit getal zelf niet gebruiken als bestandsnaam, maar de bedoeling is eigenlijk dat door middel van een loop door de array, dat als randomGetal is gevonden, het plaatje op die locatie uit de array op de site geplaatst word.

Zelf heb ik dit gemaakt maar ik krijg alle plaatjes op het scherm ipv 1.

C#:

//Loop door de array tot we de juiste hebben
        int Loopje = 1;

        foreach (FileInfo fi in finfo)
            if (Loopje == randomGetal)
            {
                a.Add("~/Images/Random/" + fi.Name);
            }
            else
            {
                Loopje++;
            }

Het werkt dus nog niet naar behoren. Heb ik iets fout gedaan in de foreach loop? Of kan het misschien anders.

probeer dit eens

C#:

a.Add("~/Images/Random/" + finfo[randomGetal].Name);

Ben je gelijk van die onnodige loop af

[ Voor 3% gewijzigd door BM op 15-02-2008 23:10 ]

[dominic]

Kleine tip. Gebruik in loops nooit string concatenation dmv:

_fout:

code:

string mystring = string1 + string2 + "bla" + string3

Een veel betere performance en memory footprint geeft:

_goed

code:

string mystring = new StringBuilder(string1).Append(string2).Append("bla").Append(string3).ToString();

[Verwijderd]

BM schreef op vrijdag 15 februari 2008 @ 23:10:
[...]


probeer dit eens

C#:

a.Add("~/Images/Random/" + finfo[randomGetal].Name);

Ben je gelijk van die onnodige loop af

Werkt als een zonnetje!
Ik heb slechts nog basiskennis op het moment dus wist niet zo snel dat je op die manier het randomGetal er even bij kon pakken .
Bedankt voor de reacties.

[Verwijderd]

dominic schreef op vrijdag 15 februari 2008 @ 23:12:
Kleine tip. Gebruik in loops nooit string concatenation dmv:

_fout:

code:

string mystring = string1 + string2 + "bla" + string3

Een veel betere performance en memory footprint geeft:

_goed

code:

string mystring = new StringBuilder(string1).Append(string2).Append("bla").Append(string3).ToString();

bedankt voor de tip. Gelukkig gaat het hier om een klein scriptje. Maar ik snap het en gaat het notitieblok in .

[whoami]

dominic schreef op vrijdag 15 februari 2008 @ 23:12:
Kleine tip. Gebruik in loops nooit string concatenation dmv:

_fout:

code:

string mystring = string1 + string2 + "bla" + string3

Een veel betere performance en memory footprint geeft:

_goed

code:

string mystring = new StringBuilder(string1).Append(string2).Append("bla").Append(string3).ToString();

Hmm....
Het hant er vanaf hoe 'groot' die loop is.
Een StringBuilder instantieren kost nl. ook.

[dominic]

whoami schreef op zaterdag 16 februari 2008 @ 10:31:
[...]

Hmm....
Het hant er vanaf hoe 'groot' die loop is.
Een StringBuilder instantieren kost nl. ook.

Doorgaans is 1 iteratie is al genoeg om over te stappen naar StringBuilder.

Bij string concatenation dmv + wordt voor iedere string de je aanmaakt een object een string-object aangemaakt (duh), maar bij het 'plakken' worden de tot dan toe aan elkaar geplakte strings ook weer in een nieuw string-object gegooid.

In feite zorgt iedere + voor een nieuw (en steeds groter wordend) string-object.

Ik zal even op zoek gaan naar de bron, want dit heb ik natuurlijk ook gewoon ergens gelezen.

[Verwijderd]

dominic schreef op zaterdag 16 februari 2008 @ 12:22:
[...]
Doorgaans is 1 iteratie is al genoeg om over te stappen naar StringBuilder.

Hoewel ik dat niet helemaal geloof, is er nog een ander nadeel.
Als ik een tekstbox probeer te vullen met string:

code:

this.Textbox.Text = "Hallo " + username;

of ik doe dit via een StringBuilder:

code:

StringBuilder sb = new StringBuilder();
sb.Append("Hallo ");
sb.Append(username);
this.Textbox.Text = "Hallo " + sb.ToString();

of jouw voorbeeld:

code:

this.Textbox.Text = (new StringBuilder()).Append("Hallo ").Append(username).ToString();

Dan kies ik toch voor de eerste... die vind ik namelijk wel een stuk simpeler.

[ Voor 9% gewijzigd door Verwijderd op 16-02-2008 12:40 ]

[Verwijderd]

In Delphi was/ben ik gewend om gewoon string concatenation te gebruiken (strings zijn niet immutable in Delphi), maar in C# gebruik ik vrijwel uitsluitend StringBuilders.

Klein voorbeeldje: een code generator die een class library genereert aan de hand van de tabellen in een database, resulterend in 65000 regels code.
Met concatenation duurde dat 20 minuten, met een StringBuilder.................. 4 seconden!

[dominic]

Verwijderd schreef op zaterdag 16 februari 2008 @ 13:48:
In Delphi was/ben ik gewend om gewoon string concatenation te gebruiken (strings zijn niet immutable in Delphi), maar in C# gebruik ik vrijwel uitsluitend StringBuilders.

Klein voorbeeldje: een code generator die een class library genereert aan de hand van de tabellen in een database, resulterend in 65000 regels code.
Met concatenation duurde dat 20 minuten, met een StringBuilder.................. 4 seconden!

Dat soort tests heb ik ook zelf uitgevoerd om te bewijzen tegenover mezelf dat het echt sneller zou zijn. Het scheelt inderdaad een heleboel.

[RobIII]

Ik ben het met Afterlife en dominic eensch; een stringbuilder is vele malen performanter. Maaaar, daar wil ik wel bij zeggen dat een micro-optimalisatie zoals deze (Debbus demonstreert dat mooi) natuurlijk compleet zinloos is als je effe een tekstbox wil vullen met 2 aaneengeplakte strings. Ook als dat misschien wel 5 keer voor komt. Pas bij flinke aantallen gaat het zich echt lonen en je code leesbaar(der) houden is ook wat waard. En daarmee heb ik omslachtig gezegd dat ik het al eensch was met whoami's post eerder

Maar we gaan off-topic

[ Voor 8% gewijzigd door RobIII op 16-02-2008 15:33 ]

[Verwijderd]

Geef elk object dat wat met strings doet z'n eigen StringBuilder. Dan is de overhead minimaal, en je performance optimaal.

[Alex]

Lijkt me ook weer overbodig . Een hoop domein object doen dingetjes als voor en achternamen samenvoegen. Een string.Concat of string.Format is dan toch écht wel wat goedkoper als al die objecten een StringBuilder geven. In alle gevallen wordt namelijk de ontiegelijk dure constructor van de StringBuilder aangeroepen(die met 3 argumenten). Deze doet een behoorlijke set met checks en data collectie(inclusief unsfe calls op het string-object).
Daarnaast leent het object zich niet echt voor reusage.

Ik geloof daarnaast niet in dergelijk regels. 'Elk' en 'Altijd' zijn absoluut uit den boze in computer science. Uitzonderingen zijn altijd te bekkenen en vaak dichterbij als je denkt.

[Verwijderd]

Uit mijn ervaring is de ctor van een StringBuilder minder duur dan 't boxing/unboxing van concatenations of string formats. Maar your mileage may vary...

[Alex]

Een hele snelle bench laat mij zien dat het pas boven de 10 miljoen interessant wordt, het verschil tussen het contatineren van 2 stringetjes. In het nadeel van de StringBuilder overigens.
Dus voor de hierboven beschreven gevallen, zeker als het weinig voorkomt is het niet zo interessant. Het wordt wat anders als je hele teksten op gaat bouwen.

[Verwijderd]

/me is wel benieuwd naar die bench...

[Hydra]

Als je eht echt goed wil doen, doe je het zo natuurlijk:

Java:

StringBuilder sb = new StringBuilder();
sb.Append("Hallo " + username);
this.Textbox.Text = sb.ToString();

[Alex]

Heb de test wat uitgebreid en iets representatiever gemaakt:
http://sitecore.alexiasof...and-string-concatenation/

[YakuzA]

Hydra schreef op maandag 18 februari 2008 @ 11:38:
Als je eht echt goed wil doen, doe je het zo natuurlijk:

Java:

StringBuilder sb = new StringBuilder();
sb.Append("Hallo " + username);
this.Textbox.Text = sb.ToString();

Aangezien het wel een C# topic is:

C#:

Textbox.Text = String.Format("Hallo {0}", username)

[riezebosch]

Las net afgelopen week op The Daily WTF dat String.Format intern ook gewoon een StringBuilder gebruikt (en heb het ook met Reflector gecontroleerd).

[sopsop]

YakuzA schreef op dinsdag 19 februari 2008 @ 12:00:
[...]

Aangezien het wel een C# topic is:

C#:

Textbox.Text = String.Format("Hallo {0}", username)

Daar is niks C# 's aan hoor. Gaat ook prima in VB.net.