23 Antworten in diesem Thema

[paulaner]

Nabend Freunde der Nacht

 

Ich bräuchte mal wieder Unterstützung.

Und zwar versuche ich mit meinen mehr als bescheidenen Programmierkenntnissen ein Programm zu schreiben was im Grunde genommen eine Textdatei erstellen soll. Das ist auch noch im Rahmen meiner Möglichkeiten

das Ding ist, dass diese Textdatei auch gefüllt werden soll. UND ZWAR gibt es auf der GUI des Programms ein paar checkboxen. Wenn diese checkboxen .checked sind soll z.B folgende Zeile in die Textdatei geschrieben werden:

"addNAME blabla<>blablabla". Das ist ja eigentlich auch noch recht leicht machbar denk ich. Jetzt kommt meine Frage:

Ich müsste doch jetzt total viele IF Abfragen um alle .checked Kombinationen abzudecken oder habe ich einen Denkfehler?

Einfacher wäre es ja wenn es die Möglichkeit gibt dem Programm zu sagen:

Guck ob checkbox1.checked ist, wenn ja, schreibe in Zeile 1 "addNAME blabla<>blablabla".

Danach überprüfe ob checkbox2.checked ist und wenn ja dann schreibe in DIE NÄChSTE Zeile "addTelefonnummer blabla<>blablabla".

 

Da ich wie gesagt, nicht sehr viel Ahnung habe, wollte ich mal nachfragen, wie ihr das machen würdet?

 

 

Hoffe ich hab das EINIGERMAßEN verständnisvoll formuliert.

 

MfG

[Cube]

Erklärung is so lala.

 

Aber sag uns mal: Von wie vielen Checkboxen reden wir denn da so?

 

Grüße

[paulaner]

Sorry, bin heute irgendwie erledigt und verwirrt

 

Also wir reden von 20 Checkboxen und einer String Variablen die aus einem Textfeld erzeugt wird, welche noch mit in die Textdatei geschrieben werden muss.

[juPP]

So wie ich est verstanden hab willst du folgendes, Du hast ja bestimmt einen butten der dann deinen string in die datei schreibt. In dem ClickEvent von diesem Schreibe button fügst du folgenden Code ein:

            Hashtable haschisch = new Hashtable();

            haschisch.Add(checkBox2, "AAAAAAA\r\n");
            haschisch.Add(checkBox3, "CCCCCCC\r\n");
            haschisch.Add(checkBox4, "BBBBBBB\r\n");
            haschisch.Add(checkBox5, "CCCCCCC\r\n");


            foreach (DictionaryEntry xxx in haschisch)
            {
                CheckBox a = xxx.Key as CheckBox;

                if (a.Checked)
                {
                    final += xxx.Value;
                }
            }

            Console.WriteLine(this.final);

Damit fügst du einfach die ganzen Checkboxen deiner form und deinen gewünschten string einer hashtable hinzu, durchläufst diese dann in einer foreach und prüfst ob eine checkbox "Checked" ist, wenn ja fügst du den string an.

 

Hoffe es hilft.

 

ps: code ist c# aber sollte ja egal sein, kannste ja easy portieren

[Cube]

Mh.

Naja vlt kann man da irgendwas mit switch/Case drauß basteln.

Ansonsten wirst du an den If abfragen nicht dran vorbeikommen.

If Abfragen gibts diverse Kurzschreibweisen um dir etwas Tipp arbeit zu ersparen aber wirklich drumherum wirste denke ich nicht kommen.

Einfach mal danach googeln wenn du kurzschreibweisen suchst. (kenns aber nur aus c#)

 

Grüße

 

EDIT: Wie mein Vorposter schon schrieb: Das is natürlich keine schlechte Idee.

Selbiges prinzip kann man umbauen/umformen.

Da du mir wie ein anfänger aussiehst, wollte ich das vermeiden aber nun auch gut.

Und anstatt eben das ganze wie bei @juPP in die Konsole zu schreiben, eben in dein File.

[paulaner]

Vielen Dank euch beiden für die schnelle Antwort!

Ich werde deine Methode später auf jeden Fall  ausprobieren JuPP.

Vielen Dank.

 

Wenn es nicht funktioniert, gehts halt an die If-Abfragen

 

Ich geb auf jeden Fall mal feedback.

[juPP]

Es funktioniert auf jeden Fall und es wohl auch wesentlich eleganter, gerade wenn es 20 Checkboxen sind...

 

Habe dir zum besseren Verständnis, ein kleines Projekt angehangen was genau dein vorhaben realisiert.

 

Please Login HERE or Register HERE to see this link!

[paulaner]

Mega Korrekt von dir.

 

Vielen Dank!

Das ist fast genau das, was ich brauche. Die paar änderungen sollte ich hin bekommen

 

Tausend Dank!

[n1nja]

Du kannst die CheckBoxen auch alle den gleichen Eventhandler mitgeben,

sobald sich da dann was ändern frägst du beim sender nach, welche das ist.

 

Und dann ja wie meine Vorredner schon gesagt haben mit switch solltest es dann sortieren können.

 

 

Aber bei 20 CheckBoxen?

Würd ich eher sagen, das ist ein Design Fehler.

 

Denke auch daran, trenne GUI mit Daten.

 

Und falls du die Textbox brauchst, um den CheckState zu speichern, dann nehme gleich was anderes.

Benutz dazu die INI-Klasse oder speicher es als XML ab.

 

Dadurch hast du einen viel besseren Überblick und du sparst dir einiges an Code.

[SAR]

@juPP: Geniale Zweckentfremdung des HashTabels.

Vllt. für Anfänger leichter zu verstehen:

            foreach (Control co in this.Controls)
            {
                if (co is CheckBox)
                {
                    CheckBox ch = (CheckBox)co;

                    if (ch.Checked)
                        final += co.Name + "<>" + GetNextControl(co, true).Text + "\n"; 
                }
            }

Mfg.
SAR

Bearbeitet von SAR, 21 May 2015 - 00:31 Uhr.

[n1nja]

@SAR
Warum instanzierst die eine neue ChechBox und weist sie gleich von der "richten" die auf der Form ist zu?

Sollte doch ohne diese Zeile auch funktionieren.

Bearbeitet von n1nja, 21 May 2015 - 00:42 Uhr.

[SAR]

@juPP: Weil "Control co" vom Typ Control ist. Control hat leider nicht die Eigenschaft "Checked" zur Verfügung.


Mfg.
SAR

[n1nja]

Achso ja is richtig.
Man könnte es aber noch mit dynamischen Variablen machen

[SAR]

Man könnte es auch so machen, aber ist alles nur Zeilenklauberei.

            foreach (Control co in this.Controls)
            {
                if (co is CheckBox)
                {
                    if (((CheckBox)co).Checked)
                        final += co.Name + "<>" + GetNextControl(co, true).Text + "\n"; 
                }
            }

Was meinst du denn mit dynamischen Variablen?


Mfg.
SAR

[n1nja]

Ich bin nicht mehr am PC. Da is doof zu coden:)

Ich hab bei meinen Tool vorhin ein Problem gehabt mit dem kompletten auslesen der Registry.
Da hab ich "dynamic" Vars benutzt:)

Da kannst du alle Methoden, Funktionen ranhängen ohne das der Compiler mäckert.

[SAR]

            foreach (dynamic co in this.Controls)
            {
                if (co is CheckBox)
                {
                    if (co.Checked)
                        final += co.Name + "<>" + GetNextControl(co, true).Text + "\n"; 
                }
            }

So würde das aussehen. Die Performance geht aber richtig in Keller.

Die erste Variante braucht bei einer checked-Checkbox weniger als 1ms (518 Ticks)
Die dynamic-Variante braucht 95ms (456451 Ticks).

Aber das dynamic-Schlüsselwort kannte ich nicht. Ich finde es aber direkt eklig.


Mfg.
SAR

[n1nja]

Das is mir nicht aufgefallen danke:)
Ich dachte es liegt an was anderen.

Aber was nicht alles an Daten anfallen bei der Registry.
mein PC hat wirklich knappe 120.000 Datensätze:)
Jeweils ein Array von 3 Strings.

Vllt lässt sich noch was ändern an der Performance.
Ich vermute mal, das es auf den zusammenhang mit ankommt. Mit Controls zu arbeiten während der Laufzeit ist ja ja sowieso belastend:)

[Cube]

Über sowas noch nie den Kopf darüber zerbrochen, zumal ich auch nie wirklich mehr als 5 checkboxen nutzte

 

Sars Ansatz find ich echt klasse.

Jeder setzt direkt an, wenn jede einzelne checkbox wirklich schon so in der GUI angelegt ist.

Vielleicht kann man sich aber davor schon eine etwas andere Struktur ausdenken.

 

Nehme an in VB.net gibt es auch CheckBoxListe.

Vielleicht kann man damit etwas handlicher arbeiten.

Mh wobei es hier denke ich auch aufs selbe rausläuft mh.

 

Macht das ganze doch nicht einfacher

 

Naja worauf ich hinaus will, vorrausgesetzt es wird noch nach weiteren Lösungsansätzen gesucht:

Vielleicht eine neue Idee zu gestalten, um von den 20 Checkboxen generell schon wegzukommen.

Ich mein, das erscheint mir doch etwas viel

 

Lg

Cube

[juPP]

@juPP: Geniale Zweckentfremdung des HashTabels.

Vllt. für Anfänger leichter zu verstehen:

            foreach (Control co in this.Controls)
            {
                if (co is CheckBox)
                {
                    CheckBox ch = (CheckBox)co;

                    if (ch.Checked)
                        final += co.Name + "<>" + GetNextControl(co, true).Text + "\n"; 
                }
            }

Mfg.
SAR

 

Haha ja... ging ja nur um die Methode, ob mans mit ner Hashtable macht oder mit DirectoryEntry oder mit einen mehrdimendionalen Array oder sogar einer eigenen Collection ist ja egal

[n1nja]

Also ich bin auch der Meinung wie Cube, das man schon am Design was ändern könnte.

Falls nicht, dann mit den Vorschlag von Sar.

 

Das mit dynamic hab ich versucht. Scheint wirklich bei Controls sehr langsam zu sein.

Man könnte hier aber auch wieder kürzen und direkt die CheckBoxen ansprechen:)

foreach (CheckBox cb in this.Controls)
{
   string txt = (cb.Checked) ? cb.Name+" true" : cb.Name+" false";
}