Hello everyone, 

I'm pretty new to the programming world and I would like to have some more experienced opinions to my programming skills

For a project I need to use a function multiple times. 

first = sealing with a button 

second = sealing with 2 sensors high in Auto mode 

third = sealing with 1 sensor high and 1 sensor low. 

My main question with this is, am I doing it the right way? Because the machine buggs out sometimes when the power has been off for a longer time.

Is there an easy way to do this better or easier? ? 

I think I am using the IF statement to much as well, so if there are people who have tips for me on how I can program more efficient, I would like to read them! 

I've inserted my program in the appendix for everybody to check. 

Using Tia Portal V17.0 

If you would like to see how the code runs: 

https://youtu.be/7oxLi7dkQbE?si=P41C1z_qw6x8r1ZQ

 I'm hoping for some useful replies! 

Kind regards,

Jobbbooo

Hallo Job.

As a conversation starter there are 4 things to mention.

1. The timers function on a off-to-on edge trigger. The timer does not see the FALSE -> ON transition when the timer is in a code block or if-statement that blocks scanning.

2. Loosing the EN on a code block means that the code is not scanned. This is not a great idea if there is a timer inside that code block. Disabling the EN makes sense if the code is a short running code (like code that runs one scan). Longer running code is best to keep EN at the TRUE state. A block input can give permissions for segments to output a result.

3. Building a sequence of one timer triggering another timer repeated for several timers/steps is not a good idea. If one timer in the sequence of tasks would fail, the whole sequence falls apart. Better to use something like an INT tag to keep track of the sequence steps. To continue from a previous step after a stop is then more practical.

4. The instance of a timer does not need to be a global instance. Inside an FB, that timer can be a shared (local) instance of the FB. Memory management becomes more easy as the timer tags now belongs to the memory of the FB.

Tot later.

Hello Qwazee,

Thank you for your Reply!!

I'm currently working on bettering the code using yours and Jacek's suggestions!

But I still have some questions:

1. Yesterday I tried inserting the EN on the auto blocks again, but when I did the machine started an endless loop doing the first strap.
Is it a better option to insert the code that is in front of the EN inside the code?

2. I'm trying to insert the INT values in the code for the timers but I think that I don't understand you quite well. I've inserted a picture in the appendix on how I thought about how to do it. Is it about the same way you would use it, or am I totally wrong?

3. Since most of my timers are having the same value, would it be wise for the memory usage to re-use the timers that have the same value in other blocks?

Thank you for your time and I'm looking forward to your reply!

Kind regards,

Jobbbooo