ПЛК150 + СП270Т

[_Александр_]

Здравствуйте!
Выношу на ВАШЕ рассмотрение мой проект работы системы регулирования водогрейного котла. Проект рабочий. Отопительный сезон ПЛК отработали без сбоев. Но мне, как начинающему работать в программировании будет полезно узнать ВАШИ замечания. Что делаю может быть не так, как сделал бы квалифицированный программист.

[Yegor]

Код — жуть. Отступы, пустые строки и пробелы вокруг операторов от балды вообще расставлены. Как у вас: http://pastebin.com/zbe6EdWf, как надо: http://pastebin.com/EquvpVBB

Вот так нехорошо делать:

          IF  iNagruzka > 100  THEN
                   xMeoGazZacrytie := FALSE ;
                            xMeoGazBlok  := TRUE;
           END_IF

                     IF  iNagruzka < 100  THEN
                               xMeoGazZacrytie := TRUE ;
                                       xMeoGazBlok  := FALSE;
                   END_IF

Можно же проще:

xMeoGazBlok := iNagruzka > 100;
xMeoGazZacrytie := iNagruzka < 100;

Есть фрагменты ещё забавнее:

IF xBlokGaz   = TRUE

Зачем тут «= TRUE» вообще?

Очень много бесполезных и даже неправильных комментариев:

Reguljtor_Vozduxa(); (*Объявление программы регулятора воздуха*)

Не объявление, а вызов. Это комментарий типа наклейки «Руль» на руле автомобиля.

Многие переменные объявлены по два раза. Имена неудачные. Например, в PLC_PRG:

rVozduxVxod AT %ID6.0: REAL := 4; (*Давление воздуха перед горелкой 4..20 mA.*)

Во-первых, в конфигурации эта переменная уже привязана к %ID6.0. Во-вторых, если это давление, то почему не написать VozduxDavlenie4_20? А лучше и вовсе AirPressure4_20. И уж совсем прекрасно будет назвать его AirPressure, убрать блок Shcala_Vozdux и задать шкалу на вкладке Module Parameters в конфигурации. Или вот iZdnGaz... Почему, например, не HMI_GasSetpoint? И не надо было бы этих комментариев:

Задание нагрузки введённое с панели оператора имеющее тип WORD преобразуется в тип INT для подключения на вход модуля CHARCURVE

Было бы и так понятно по «HMI», что значение берётся с панели. И про эти преобразования в то да в сё — зачем? Это какой-то хитрый трюк, непонятный из кода? Это имеет непосредственное отношение к задаче управления?

Дальше. Константы должны быть константами. Такие переменные, как timPauzaGaz и timImpulsCaz следует объявлять в разделе VAR CONSTANT.

В программах Reguljtor_Nagruzki и Reguljtor_Vozduxa у вас велосипед гистерезиса и комментарий для восьмиклассника о том, как он работает. Почему-то CHARCURVE вы догадались использовать, а HYSTERESIS проигнорировали.

Двоичные операторы GT и LT

Это операторы сравнения, а не двоичные операторы. Двоичные — AND, OR, NOT, XOR.

Интерфейс терпимый, только цвета не такие буйные возьмите.

Ладно, хватит пока.

[_Александр_]

Код — жуть.
Спасибо. Я для того и выложил, чтобы потыкали носом.
Вы не против, если ещё возникшие вопросы буду адресовать Вам персонально?

[Yegor]

Пожалуйста.

[_Александр_]

Внесу Ваши замечания. HYSTERESIS применить на вход регулятора?
... убрать блок Shcala_Vozdux и задать шкалу на вкладке Module Parameters в конфигурации...
Значит шкалу можно задавать так, как на рисунке?

[Yegor]

HYSTERESIS применить на вход регулятора?

Гистерезис вместо всех этих GT/LT в регуляторах, да.

Шкалу можно задавать так.

[_Александр_]

Гистерезис вместо всех этих GT/LT в регуляторах, да.

Шкалу можно задавать так.

Спасибо. Жаль ПЛК свободного нет. Будет только через пару месяцев.

[capzap]

Спасибо. Жаль ПЛК свободного нет. Будет только через пару месяцев.

что такого в Вашей программе уникального, что нельзя все проверить в эмуляции?

[_Александр_]

что такого в Вашей программе уникального, что нельзя все проверить в эмуляции?

Как проверить диапазон у шкалы заданной со входа ПЛК в эмуляции?

[capzap]

вручную вводить ниже диапазона, равно нижней границе, в диапазоне и по верху те же манипуляции, если нужна динамика я подключал другое устройство в качестве слейва, там формировал аналоговый сигнал и выдавал его по модбас, а в редактируемом проекте у меня в конфигураторе диапазон 4-20 и есть фб который переводит эти значения в требуемый диапазон, границы которого можно менять из визуализации. После отладки меняю переменные из добавленного эмулятора сигналов на реальные входа контроллера