[C#] Meerdere textboxen creëren

[Verwijderd]

Heej,

Ik ben bezig met een applicatie, maar ik kom er niet uit hoe ik meerdere textboxen kan creëren.
Ik heb het volgende:

code:

// aantal textboxen dat moet worden gecreëerd.
private static int iStopwatchStopCount = 3;

TextBox[] a_textboxes = new TextBox[iStopwatchStopCount];

code:

private void CreateTextBoxes()
        {
            // alle textboxen op het form verwijderen op 1 na.
            for (int iI = this.Controls.Count - 1; iI >= 0; iI--)
            {
                if (this.Controls[iI] is TextBox && Controls[iI].Name != "tbRunningTime")
                {
                        this.Controls.RemoveAt(iI);
                }
            }

            // tekstboxen aanmaken en weegeven op het form. <--- dit werkt niet!
            int iCount = 0;
            int iCoordinaatX = 12;
            int iCoordinaatY = 84;
            foreach (TextBox textbox in a_textboxes)
            {
                TextBox tekstbox = new TextBox();
                textbox.Location = new Point(iCoordinaatX, iCoordinaatY);
                textbox.Name = "tbSaveTime" + iCount.ToString();
                textbox.Size = new Size(100, 20);
                textbox.TabIndex = iCount;
                textbox.Visible = true;
                Controls.Add(tekstbox);
                iCount++;
                iCoordinaatY += 40;
            }

Wat de bedoeling is, is dat als de functie CreateTextBoxes word opgeroepen er tekstboxen op het form worden aangemaakt. Het aantal textboxen dat moet worden aangemaakt staat in de integer iStopwatchStopCount.

Groetjes,

Bob

[remco_k]

Ik ken C# niet zo goed, maar moet je niet de Parent en/of Owner zetten op "this"?

Verder doe je rare dingen. Je declareerd een array, a_textboxes en daar meteen new TextBox[iStopwatchStopCount]; ??

Later loop je met een loopje door die array heen, en maak je alsnog dat aantal textboxen aan.

Je doet:

C++:

TextBox tekstbox = new TextBox(); // Let op: "tekstbox" met ks --- en je had de textboxen toch al gemaakt?
textbox.Location = ... // Let op: "textbox" met een x ? 
// ...
Controls.Add(tekstbox); // en weer met ks...

Naamgeving is niet goed.
"tekstbox" heb je enkel aangemaakt met "new TekstBox();" en geen enkele property gezet.

Er is meer te verbeteren, maar eerst maar even aan de praat krijgen.

[ Voor 32% gewijzigd door remco_k op 15-07-2009 14:29 . Reden: Code tags en comment toegevoegd ]

[boe2]

Je werpt een hoop ongeinitialiseerde tekstboxen in een array maar doet er vervolgens niets mee. Wat je in de foreach lus probeert te doen is me ook niet volledig duidelijk. Volgens mij wil je dit (schoolvoorbeeld) doen:

C#:

for(Int32 i = 0; i < a_textboxes.Length; i++)
{
    a_textboxes[i] = new TextBox();
    a_textboxes[i].Location = new Point(iCoordinaatX, iCoordinaatY);
    //enzovoort
}

remco_k: "this" slaat hoogstwaarschijnlijk op de form class waarin hij werkt.

[ Voor 19% gewijzigd door boe2 op 15-07-2009 14:34 ]

[Verwijderd]

Ik heb het al opgelost, ik heb het gedaan zoals Boeboe zij.

Groetjes en bedankt,

Bob

[Phyxion]

Boeboe schreef op woensdag 15 juli 2009 @ 14:27:
Je werpt een hoop ongeinitialiseerde tekstboxen in een array maar doet er vervolgens niets mee. Wat je in de foreach lus probeert te doen is me ook niet volledig duidelijk. Volgens mij wil je dit (schoolvoorbeeld) doen:

C#:

for(Int32 i = 0; i < a_textboxes.Length; i++)
{
    a_textboxes[i] = new TextBox();
    a_textboxes[i].Location = new Point(iCoordinaatX, iCoordinaatY);
    //enzovoort
}

remco_k: "this" slaat hoogstwaarschijnlijk op de form class waarin hij werkt.

foreach was ook prima mogelijk geweest omdat ze toch allemaal al bestaan.

[Woy]

Phyxion schreef op donderdag 16 juli 2009 @ 08:38:
[...]

foreach was ook prima mogelijk geweest omdat ze toch allemaal al bestaan.

Nee, ze bestaan nog niet, er bestaat alleen een lege array. Er is nog geen enkele Textbox gecreerd.

[Phyxion]

Woy schreef op donderdag 16 juli 2009 @ 09:30:
[...]

Nee, ze bestaan nog niet, er bestaat alleen een lege array. Er is nog geen enkele Textbox gecreerd.

Jawel ->

C#:

        private const int iStopwatchStopCount = 3;
        TextBox[] a_textboxes = new TextBox[iStopwatchStopCount];

        public Form1()
        {
            InitializeComponent();
            foreach (TextBox textBox in a_textboxes)
                Console.WriteLine("Wij hebben een textbox voor Woy");
        }

Output:

code:

Wij hebben een textbox voor Woy
Wij hebben een textbox voor Woy
Wij hebben een textbox voor Woy

Works like a charm

[BM]

Ja, en al die TextBox objecten waar je doorheen loopt zijn null

[Woy]

Idd BM

C#:

        private const int iStopwatchStopCount = 3;
        TextBox[] a_textboxes = new TextBox[iStopwatchStopCount];

        public Form1()
        {
            InitializeComponent();
            foreach (TextBox textBox in a_textboxes)
                textBox.Text = "Mooie NullReferenceException Hier!!!";
        }

Zal een mooie NullReferenceException gooien.

[whoami]

Phyxion schreef op donderdag 16 juli 2009 @ 12:04:
[...]

Jawel ->

C#:

        private const int iStopwatchStopCount = 3;
        TextBox[] a_textboxes = new TextBox[iStopwatchStopCount];

        public Form1()
        {
            InitializeComponent();
            foreach (TextBox textBox in a_textboxes)
                Console.WriteLine("Wij hebben een textbox voor Woy");
        }

Output:

code:

Wij hebben een textbox voor Woy
Wij hebben een textbox voor Woy
Wij hebben een textbox voor Woy

Works like a charm

Tuurlijk werkt dat gelijk een 'charm'.
Je itereert gewoon over ieder item in de array, en je gaat gewoon bij iedere iteratie iets gaan outputten.
Echter, ieder item in die array is 'NULL', want nog niet geinitialiseerd....

Zoals Woy al zegt: je gaat een NullRef exception krijgen als je iets met een textbox zou willen doen.

[Phyxion]

Ach ja, foreach is readonly / immutable.
My bad

[Woy]

Phyxion schreef op donderdag 16 juli 2009 @ 13:37:
Ach ja, foreach is readonly / immutable.
My bad

foreach heeft niks met readonly /immutable te maken. Het itereert gewoon over een IEnumerable, en cast eventueel naar het gewenste type.

Het gaat erom dat bij het creeeren van een array de items erin niet gecreerd worden. Alle items in een array worden standaard op default() geinitialiseerd, en bij een referencetype is dat gewoon NULL.

[Phyxion]

Woy schreef op donderdag 16 juli 2009 @ 13:51:
[...]

foreach heeft niks met readonly /immutable te maken. Het itereert gewoon over een IEnumerable, en cast eventueel naar het gewenste type.

Het gaat erom dat bij het creeeren van een array de items erin niet gecreerd worden. Alle items in een array worden standaard op default() geinitialiseerd, en bij een referencetype is dat gewoon NULL.

Ik doelde op het feit dat het niet mogelijk is om te assignen in foreach omdat ze readonly zijn,

C#:

foreach (TextBox textBox in textBoxes)
   textBox = new TextBox();

Is niet mogelijk omdat textBox readonly is en dus immutable.

[Canaria]

Waarom gebruik je eigenlijk een array? Je kunt toch ook in de for-loop doen wat je wilt en iStopwatchStopCount als max waarde voor de iteratie gebruiken? Ik zou overigens een Collection<T> gebruiken ipv een array.

[Hobbles]

Phyxion schreef op donderdag 16 juli 2009 @ 13:54:
[...]

Ik doelde op het feit dat het niet mogelijk is om te assignen in foreach omdat ze readonly zijn,

C#:

foreach (TextBox textBox in textBoxes)
   textBox = new TextBox();

Is niet mogelijk omdat textBox readonly is en dus immutable.

textBoxes is immutable dacht ik? De elementen die je uit een collectie haalt via een enumerator zijn normaliter niet readonly...

[curry684]

En Woy zegt terecht dat dat niet boeit, omdat je hoe dan ook over nulls itereert. Je claimde dat een array instantieren ook z'n content instantieert en dat is gewoon onzin.

[Woy]

Phyxion schreef op donderdag 16 juli 2009 @ 13:54:
[...]

Ik doelde op het feit dat het niet mogelijk is om te assignen in foreach omdat ze readonly zijn,

C#:

foreach (TextBox textBox in textBoxes)
   textBox = new TextBox();

Is niet mogelijk omdat textBox readonly is en dus immutable.

textBox is inderdaad een readonly local variabele ( niet immutable ), maar als dat het niet was geweest dan had je er nog niks aan gehad, want hij verwijst naar het element, niet naar de locatie in de collectie waar het element staat
[/miereneuken]

Hobbles schreef op donderdag 16 juli 2009 @ 14:02:
[...]


textBoxes is immutable dacht ik? De elementen die je uit een collectie haalt via een enumerator zijn normaliter niet readonly...

textBoxes is ook niet immutable, maar als je toch wijzigingen aan de collectie gaat toepassen gaat het wel mis idd het element zelf is inderdaad niet immutable, maar de variabele zelf is wel readonly.

[ Voor 26% gewijzigd door Woy op 16-07-2009 14:07 ]

[curry684]

Hobbles schreef op donderdag 16 juli 2009 @ 14:02:
[...]

textBoxes is immutable dacht ik? De elementen die je uit een collectie haalt via een enumerator zijn normaliter niet readonly...

De elementen niet nee, maar de Current property van IEnumerator wel (terecht). Omdat foreach een native shorthand language construct om IEnumerable is gooit C# een speciale foutmelding als je aan de foreach iteration variable probeert te assignen.

[Woy]

curry684 schreef op donderdag 16 juli 2009 @ 14:07:
[...]
Omdat foreach een native shorthand language construct om IEnumerable is gooit C# een speciale foutmelding als je aan de foreach iteration variable probeert te assignen.

Ja namenlijk een compiler foutmelding

[Phyxion]

Woy schreef op donderdag 16 juli 2009 @ 14:05:
[...]

textBox is inderdaad een readonly local variabele ( niet immutable ), maar als dat het niet was geweest dan had je er nog niks aan gehad, want hij verwijst naar het element, niet naar de locatie in de collectie waar het element staat
[/miereneuken]

[...]

textBoxes is ook niet immutable, maar als je toch wijzigingen aan de collectie gaat toepassen gaat het wel mis idd het element zelf is inderdaad niet immutable, maar de variabele zelf is wel readonly.

De collection is immutable, ik mag wel een cursus uitleggen nemen

[curry684]

Woy schreef op donderdag 16 juli 2009 @ 14:09:
[...]

Ja namenlijk een compiler foutmelding

Maar een andere, en daarmee speciaal, dan als je gewoon aan IEnumerable.Current assignt

code:

error CS0200: Property or indexer 'System.Collections.Generic.IEnumerator<System.Windows.Forms.TextBox>.Current' cannot be assigned to -- it is read only
error CS1656: Cannot assign to 'tbx' because it is a 'foreach iteration variable'

[ Voor 6% gewijzigd door curry684 op 16-07-2009 14:12 ]

[Woy]

Phyxion schreef op donderdag 16 juli 2009 @ 14:11:
[...]
De collection is immutable, ik mag wel een cursus uitleggen nemen

Ook dat klopt niet. De collection is niet immutable. De volgende code compileert en runt ook gewoon

C#:

List<string> myList = new List<string>();

myList.Add("test");

foreach (String s in myList)
{
    myList.Add(s);
    break;
}

Als je de break weghaalt zal je echter een InvalidOperationException krijgen, want de Iterator is invalid geworden door het wijzigen van de collection.

[Hobbles]

Wat Woy zegt dus (is wat ik voorheen bedoelde maar mijn uitleg was duidelijk iets te beknopt)

[Phyxion]

Woy schreef op donderdag 16 juli 2009 @ 14:19:
[...]

Ook dat klopt niet. De collection is niet immutable. De volgende code compileert en runt ook gewoon

C#:

List<string> myList = new List<string>();

myList.Add("test");

foreach (String s in myList)
{
    myList.Add(s);
    break;
}

Als je de break weghaalt zal je echter een InvalidOperationException krijgen, want de Iterator is invalid geworden door het wijzigen van de collection.

textBoxes is volgens mij immutable want dat is geen Collection maar een array.

[Canaria]

C#:

List<TextBox> textBoxes = new List<TextBox>();

for (int i=0; i<stopwatchStopCount; i++)
{
     // create textBox
     TextBox textBox = new TextBox();

     // do stuff with textBox

     textBoxes.Add(textBox);
}

[ Voor 45% gewijzigd door Canaria op 16-07-2009 14:48 ]

[Woy]

We gaan wel redelijk off-topic maar toch:

Immutable betekend ( van Wiki geplukt )

In object-oriented and functional programming, an immutable object is an object whose state cannot be modified after it is created.

De state van een array kun je wel degelijk veranderen, want je kunt er wel andere elementen in stoppen. Dus een array is niet immutable.

Je hebt wel gelijk dat je de lengte van een Array niet kunt veranderen ( Array.Resize zal gewoon een nieuwe array maken )

[whoami]

Phyxion schreef op donderdag 16 juli 2009 @ 14:24:
[...]

textBoxes is volgens mij immutable want dat is geen Collection maar een array.

Een Array is nochtans een collection.

Woy schreef op donderdag 16 juli 2009 @ 14:43:
We gaan wel redelijk off-topic maar toch:

Immutable betekend ( van Wiki geplukt )

[...]

De state van een array kun je wel degelijk veranderen, want je kunt er wel andere elementen in stoppen. Dus een array is niet immutable.

Idd.

De enige reden waarom je met een foreach je array niet kunt wijzigen, is omdat je Iterator dan ongeldig wordt, omdat de achterliggende collectie gewijzigd is.
Echter, dit heeft niets met immutability te maken, zoals eerder al gezegd werd.
Foreach gaat nl. de GetEnumerator method van de IEnumerable waarover 'ge-foreached' wordt gaan aanroepen, en gebruikt dan de IEnumerator die gereturned wordt gaan gebruiken om over ieder element van die collectie te gaan itereren.
Logisch dan dat die IEnumerator niet meer geldig is, als je de collectie nadien aangepast hebt.

[ Voor 65% gewijzigd door whoami op 16-07-2009 16:14 ]

[curry684]

whoami schreef op donderdag 16 juli 2009 @ 16:06:
[...]

Logisch dan dat die IEnumerator niet meer geldig is, als je de collectie nadien aangepast hebt.

Hangt van de onderliggende collection-implementatie af natuurlijk, met een linked list is er geen enkel bezwaar om gewoon door te enumereren

[Sebazzz]

curry684 schreef op donderdag 16 juli 2009 @ 14:11:
[...]

Maar een andere, en daarmee speciaal, dan als je gewoon aan IEnumerable.Current assignt

code:

error CS0200: Property or indexer 'System.Collections.Generic.IEnumerator<System.Windows.Forms.TextBox>.Current' cannot be assigned to -- it is read only
error CS1656: Cannot assign to 'tbx' because it is a 'foreach iteration variable'

Waarschijnlijk omdat de compiler intern de eerste fout opvangt, en omdat de situatie een foreach is, deze fout omvormt naar de tweede fout.(want het staat dan in context).