E2 Code Überprüfung

[frog]

Hi
Ich wollte mal fragen was ich an diesen codemachen könnte damit er
1. funktioniert
und 2. besser aussieht

hier ist er

@name Iris Oeffner
@inputs On Reset Time
@outputs Iris1 Iris2 Iris3 Iris4 Iris5 Iris6 Iris7 Iris8 Iris9 Iris10 Iris11 Iris12 Iris13 Iris14 Iris15 Iris16 Iris17 Iris18 Iris19 Iris20
@persist Value

round(Time)

if(Reset == 1){Value ==0}

if(Value == 21){On == 0}
if(On == 1){interval(2)}

Time = Value 

if(Value == 1){Iris1 == 1}
if(Value == 2){Iris2 == 1}
if(Value == 2){Iris3 == 1}
if(Value == 3){Iris4 == 1}
if(Value == 4){Iris5 == 1}
if(Value == 5){Iris6 == 1}
if(Value == 6){Iris7 == 1}
if(Value == 7){Iris8 == 1}
if(Value == 8){Iris9 == 1}
if(Value == 9){Iris10 == 1}
if(Value == 10){Iris11 == 1}
if(Value == 11){Iris12 == 1}
if(Value == 12){Iris13 == 1}
if(Value == 13){Iris14 == 1}
if(Value == 14){Iris15 == 1}
if(Value == 15){Iris16 == 1}
if(Value == 16){Iris17 == 1}
if(Value == 18){Iris18 == 1}
if(Value == 19){Iris19 == 1}
if(Value == 20){Iris20 == 1}

Es sollen die 20 iris nacheinander aufgehen

[gamerpaddy]

Hi
Ich wollte mal fragen was ich an diesen codemachen könnte damit er
1. funktioniert
und 2. besser aussieht

hier ist er
....
Es sollen die 20 iris nacheinander aufgehen

müsste gehen ..

@name Iris Oeffner
@inputs On Reset Time
@outputs TimerON TimerReset Iris1 Iris2 Iris3 Iris4 Iris5 Iris6 Iris7 Iris8 Iris9 Iris10 Iris11 Iris12 Iris13 Iris14 Iris15 Iris16 Iris17 Iris18 Iris19 Iris20
@persist Value Time1
if(On > 1){TimerOn= 1}
Time1= round(Time)
Time= Value 
if(Reset == 1){TimerOn= 0, Value= 0, TimerReset= 1 }else{TimerReset= 0}

if(Value >= 21){On= 0}
if(On > 0){interval(2)}

if(Value == 1){Iris1= 1}else{Iris1= 0}
if(Value == 2){Iris2= 1}else{Iris2= 0}
if(Value == 2){Iris3= 1}else{Iris3= 0}
if(Value == 3){Iris4= 1}else{Iris4= 0}
if(Value == 4){Iris5= 1}else{Iris5= 0}
if(Value == 5){Iris6= 1}else{Iris6= 0}
if(Value == 6){Iris7= 1}else{Iris7= 0}
if(Value == 7){Iris8= 1}else{Iris8= 0}
if(Value == 8){Iris9= 1}else{Iris9= 0}
if(Value == 9){Iris10= 1}else{Iris10= 0}
if(Value == 10){Iris11= 1}else{Iris11= 0}
if(Value == 11){Iris12= 1}else{Iris12= 0}
if(Value == 12){Iris13= 1}else{Iris13= 0}
if(Value == 13){Iris14= 1}else{Iris14= 0}
if(Value == 14){Iris15= 1}else{Iris15= 0}
if(Value == 15){Iris16= 1}else{Iris16= 0}
if(Value == 16){Iris17= 1}else{Iris17= 0}
if(Value == 18){Iris18= 1}else{Iris18= 0}
if(Value == 19){Iris19= 1}else{Iris19= 0}
if(Value == 20){Iris20= 1}else{Iris20= 0}

wen ncht dan lass das if(On > 0){interval(2)} weg

[frog]

Muss nicht der input "TimerON" nicht "TimerOn" heißen bzw alle anderen so ändern?

		ungefähr so?		 
		@name Iris Oeffner
@inputs On Reset Time
@outputs TimerOn TimerReset Iris1 Iris2 Iris3 Iris4 Iris5 Iris6 Iris7 Iris8 Iris9 Iris10 Iris11 Iris12 Iris13 Iris14 Iris15 Iris16 Iris17 Iris18 Iris19 Iris20
@persist Value Time1
if(On > 1){TimerOn= 1}
Time1= round(Time)
Time= Value 
if(Reset == 1){TimerOn= 0, Value= 0, TimerReset= 1 }else{TimerReset= 0}

if(Value >= 21){On= 0}
if(On > 0){interval(2)}

if(Value == 1){Iris1= 1}else{Iris1= 0}
if(Value == 2){Iris2= 1}else{Iris2= 0}
if(Value == 2){Iris3= 1}else{Iris3= 0}
if(Value == 3){Iris4= 1}else{Iris4= 0}
if(Value == 4){Iris5= 1}else{Iris5= 0}
if(Value == 5){Iris6= 1}else{Iris6= 0}
if(Value == 6){Iris7= 1}else{Iris7= 0}
if(Value == 7){Iris8= 1}else{Iris8= 0}
if(Value == 8){Iris9= 1}else{Iris9= 0}
if(Value == 9){Iris10= 1}else{Iris10= 0}
if(Value == 10){Iris11= 1}else{Iris11= 0}
if(Value == 11){Iris12= 1}else{Iris12= 0}
if(Value == 12){Iris13= 1}else{Iris13= 0}
if(Value == 13){Iris14= 1}else{Iris14= 0}
if(Value == 14){Iris15= 1}else{Iris15= 0}
if(Value == 15){Iris16= 1}else{Iris16= 0}
if(Value == 16){Iris17= 1}else{Iris17= 0}
if(Value == 18){Iris18= 1}else{Iris18= 0}
if(Value == 19){Iris19= 1}else{Iris19= 0}
if(Value == 20){Iris20= 1}else{Iris20= 0}	

und könnte man nicht den timer schon im E-gate intrigieren?

[marZsius]

machs doch über ein array, hab jetz grad keine zeit n beispiel zu schreiben, wird nachgeliefert...

so da:

@name Iris Oeffner
@inputs On Reset
@outputs Iris1 Iris2 Iris3 Iris4 Iris5 Iris6 Iris7 Iris8 Iris9 Iris10 Iris11 Iris12 Iris13 Iris14 Iris15 Iris16 Iris17 Iris18 Iris19 Iris20
@persist Value:array Zeit

interval(500)

if(Reset == 1)
{
Value = array()
Zeit = 0
}

if(Zeit == 21)
{
stoptimer("interval")
}

if(On == 1)
{
Zeit++
Value:setNumber(Zeit,1)
}

Iris1 = Value:number(1)
Iris2 = Value:number(2)
Iris3 = Value:number(3)
Iris4 = Value:number(4)
Iris5 = Value:number(5)
Iris6 = Value:number(6)
Iris7 = Value:number(7)
Iris8 = Value:number(8)
Iris9 = Value:number(9)
Iris10 = Value:number(10)
Iris11 = Value:number(11)
Iris12 = Value:number(12)
Iris13 = Value:number(13)
Iris14 = Value:number(14)
Iris15 = Value:number(15)
Iris16 = Value:number(16)
Iris17 = Value:number(17)
Iris18 = Value:number(18)
Iris19 = Value:number(19)
Iris20 = Value:number(20)

jetzt gehts, habs getestet

p.s: die zeit in der Klammer beim Interval wird in millisekunden (1/1000) gemessen, also falls du 2 sekunden abstand zwischen den öffnungen haben wolltest, müsstest du 2000 eingeben =)