In Rule eine Variable ändern innerhalb von (if)

[sonic]

Hallo Community,

vielleicht verzettel ich mich aktuell. Ich möchte gerne eine bereits gefüllte Variable innerhalb eine IF Abfrage ändern.

Der Plan:
Es kommt ein Anruf an auf folgender Nummer 00001, dann soll diese Nummer durch einen String ersetzt werden.

Code: Alles auswählen

rule "Anruf Notification"
when
    Item M_Tel_Status changed to RINGING
then

    var incCall = M_Tel_EingehendAnruf.state as StringListType
    val callerNumber = incCall.getValue(1)
    var calledNumber = incCall.getValue(0)

    
    if (calledNumber === "00001") {
        var calledNumber = "Praxistelefon"
    }
    
    if (calledNumber === "00002") {
        var calledNumber = "Privattelefon"
    }
    

    var msg = "Anruf von " + callerNumber + " fuer " + calledNumber

    M_Persist_Notify.postUpdate(msg)
    sendBroadcastNotification(msg)
 
end

Die Regel funktioniert grundsätzlich, nur wird die Nummer angezeigt und nicht durch den String "Praxistelefon" oder "Privattelefon" ersetzt.

Ich freue mich auf Eure Hilfe und schonmal besten Dank
Grüße
Bernhard

[peter-pan]

Hast du den Vergleich mal mit zwei == versucht

Code: Alles auswählen

if (calledNumber == "00001") {

[sonic]

Ja habe ich. Die 3 = waren eher Verzweifelunt

[udo1toni]

Punkt 1: Du darfst nicht das Schlüsselwort var voranstellen, das legt eine neue lokale Variable an, die allerdings nur innerhalb des if-Blocks gilt, also außerhalb schon wieder verworfen wird. Ich könnte mir gut vorstellen, dass hier ein Override stattfindet, das darf natürlich nicht sein.
Punkt 2: Bist Du sicher, dass es sich wirklich um einen String handelt? Um das tatsächlich sicherzustellen, wäre die Rule so sinnvoller:

Code: Alles auswählen

rule "Anruf Notification"
when
    Item M_Tel_Status changed to RINGING
then
    val incCall = M_Tel_EingehendAnruf.state as StringListType
    val callerNumber = incCall.getValue(1)
    var calledNumber = incCall.getValue(0).toString // String erzwingen

    switch(calledNumber) {
        case "00001" : calledNumber = "Praxistelefon"
        case "00002" : calledNumber = "Privattelefon"
    }
    val msg = "Anruf von " + callerNumber + " für " + calledNumber

    M_Persist_Notify.postUpdate(msg)
    sendBroadcastNotification(msg)
end

Das switch-Konstrukt spart hier nur wenig Code, ist aber etwas effizienter als die if-Anweisungen.

[sonic]

Guten Morgen.
DANKE udo1toni,

es funktioniert nicht nur, ich habe auch wieder was dazu gelernt. TOP.

Allen einen schönen Tag!