C# CheckListBox zuweisen

[Peter (Gast)]

Hallo,

zur Visualisierung von einem Modbus RTU will ich eine UInt16 einer 
CheckListBox zuweisen. Habe folgendes probiert aber es funktioniert 
nicht. Hat was mache ich falsch?
So will ich die Funktion aufrufen und zuweisen.

cbl_control = UInt16_To_CheckListBox( Modbus_Data );


        private CheckedListBox UInt16_To_CheckListBox(UInt16 Input)
        {
            string[] Bit_List = new string[16];
            CheckedListBox Output = new CheckedListBox();
            Bit_List[0] = "Bit 0";
            Bit_List[1] = "Bit 1";
            Bit_List[2] = "Bit 2";
            Bit_List[3] = "Bit 3";
            Bit_List[4] = "Bit 4";
            Bit_List[5] = "Bit 5";
            Bit_List[6] = "Bit 6";
            Bit_List[7] = "Bit 7";
            Bit_List[8] = "Bit 8";
            Bit_List[9] = "Bit 9";
            Bit_List[10] = "Bit 10";
            Bit_List[11] = "Bit 11";
            Bit_List[12] = "Bit 12";
            Bit_List[13] = "Bit 13";
            Bit_List[14] = "Bit 14";
            Bit_List[15] = "Bit 15";

            Output.Items.AddRange(Bit_List);

            Output.SetItemChecked(0, Convert.ToBoolean(Input & (1 << 0)));
            Output.SetItemChecked(1, Convert.ToBoolean(Input & (1 << 1)));
            Output.SetItemChecked(2, Convert.ToBoolean(Input & (1 << 2)));
            Output.SetItemChecked(3, Convert.ToBoolean(Input & (1 << 3)));
            Output.SetItemChecked(4, Convert.ToBoolean(Input & (1 << 4)));
            Output.SetItemChecked(5, Convert.ToBoolean(Input & (1 << 5)));
            Output.SetItemChecked(6, Convert.ToBoolean(Input & (1 << 6)));
            Output.SetItemChecked(7, Convert.ToBoolean(Input & (1 << 7)));
            Output.SetItemChecked(8, Convert.ToBoolean(Input & (1 << 8)));
            Output.SetItemChecked(9, Convert.ToBoolean(Input & (1 << 9)));
            Output.SetItemChecked(10, Convert.ToBoolean(Input & (1 << 10)));
            Output.SetItemChecked(11, Convert.ToBoolean(Input & (1 << 11)));
            Output.SetItemChecked(12, Convert.ToBoolean(Input & (1 << 12)));
            Output.SetItemChecked(13, Convert.ToBoolean(Input & (1 << 13)));
            Output.SetItemChecked(14, Convert.ToBoolean(Input & (1 << 14)));
            Output.SetItemChecked(15, Convert.ToBoolean(Input & (1 << 15)));
            return Output;
        }

[GoMaD A. (gomad_a)]

Mach aus:

Peter schrieb:
> CheckedListBox Output = new CheckedListBox();

CheckedListBox Output = cbl_control;

Ist es das Ergebnis welches du willst?

[Peter (Gast)]

Ziel ist es jedes der 16 Bits des UInt16 als Kästchen anzuzeigen. Diese 
Funktion aber auch an anderen Stellen verwenden zu können.

[Markus (Gast)]

Ich würde vielleicht eher mal folgendes versuchen:

    //löschen: cbl_control = UInt16_To_CheckListBox( Modbus_Data );
    UInt16_To_CheckListBox( cbl_control, Modbus_Data );

    private void UInt16_To_CheckListBox(CheckedListBox Output, UInt16 Input)
    {
        ....
    }

Grüße
Markus

[Flugmops (Gast)]

Peter schrieb:
> aber es funktioniert nicht.

Hmm... das ist ja eine recht umfassende Problembeschrebung.

Du erzeugst bei jedem Aufruf eine neue CheckedListBox und weist sie 
einer Variablen zu. Falls da nicht der entscheidende Code fehlt: Warum 
sollte das die UI interessieren? Erzeuge doch einfach eine 
CheckedListBox im UI-Editor und verwende dann diese in 
UInt16_To_CheckListBox (der Name wäre dann aber nicht ganz richtig).

Peter schrieb:
> Diese Funktion aber auch an anderen Stellen verwenden zu können.

Um die selbe CheckedListBox zu verwenden oder wirklich immer wieder ein 
neues Control zu erzeugen? Würmer, Nase ...

[Markus (Gast)]

Ach ja, anstelle von

Convert.ToBoolean(Input & (1 << 0))

könntest Du auch kompakter

(Input & (1 << 0)) != 0

schreiben.

Außerdem kannst Du die 16 Zeilen bequem in eine For-Schleife umwandweln. 
Die Methode wird kompakter und leichter änderbar.

Und wenn Du dann noch eine Methode mit den Parametern Input und 
BitNumber und einem bool Return-Wert aus dem obigen Ausdruck machst...

Grüße
Markus

[Peter (Gast)]

Hallo,

danke schon mal. Bin neu in der C# Welt und wusste nicht das man die 
Variablen einfach als Ein und Ausgangsvariablen benutzen kann. So 
funktioniert  es jetzt.

        private void UInt16_To_CheckListBox(UInt16 Input, CheckedListBox Output)
        {
            Output.SetItemChecked(0, Convert.ToBoolean(Input & (1 << 0)));
            Output.SetItemChecked(1, Convert.ToBoolean(Input & (1 << 1)));
            Output.SetItemChecked(2, Convert.ToBoolean(Input & (1 << 2)));
            Output.SetItemChecked(3, Convert.ToBoolean(Input & (1 << 3)));
            Output.SetItemChecked(4, Convert.ToBoolean(Input & (1 << 4)));
            Output.SetItemChecked(5, Convert.ToBoolean(Input & (1 << 5)));
            Output.SetItemChecked(6, Convert.ToBoolean(Input & (1 << 6)));
            Output.SetItemChecked(7, Convert.ToBoolean(Input & (1 << 7)));
            Output.SetItemChecked(8, Convert.ToBoolean(Input & (1 << 8)));
            Output.SetItemChecked(9, Convert.ToBoolean(Input & (1 << 9)));
            Output.SetItemChecked(10, Convert.ToBoolean(Input & (1 << 10)));
            Output.SetItemChecked(11, Convert.ToBoolean(Input & (1 << 11)));
            Output.SetItemChecked(12, Convert.ToBoolean(Input & (1 << 12)));
            Output.SetItemChecked(13, Convert.ToBoolean(Input & (1 << 13)));
            Output.SetItemChecked(14, Convert.ToBoolean(Input & (1 << 14)));
            Output.SetItemChecked(15, Convert.ToBoolean(Input & (1 << 15)));
        }

[Flugmops (Gast)]

Wenn du Code oft wiederholst, sollte etwas klingeln ...

private void SetCheckListBox(ushort input, CheckedListBox target)
{
  for(ushort i = 0; i < 16; i++)
  {
    target.SetItemChecked(i, (input & (1 << i)) != 0);
  }
}

Nicht so wichtig, aber als Hinweis: Parameternamen werden in C# 
praktisch immer klein geschrieben und Methodennamen in "Camel Case" 
(keine Unterstriche).