Pull to refresh

Comments 4

Позвольте высказать немного критики и сделать code review.Функцию sysCall_actuation можно упростить отделив бизнес логику от парсинга, парсинг при этом сделан через Ctrl+C/Ctrl-V и его можно упростить выделив повторяющуюся логику в фунуции.

Хорошим тоном является задание реальных имён компонентам, а не button1 и др.

И что? Где результаты моделирования? Я остался неудовлетворённым! Ждём видео о полученных результатах)

Никогда не встречал прямую кинематику в системах управления роботами. ИМХО мне больше всего нравится представление алгоритма в виде визуального графа.

C помощью string.gsub в принимаемой строке производим замену всех запятых на точки. Данное действие необходимо из-за русскоязычной локализации Windows.

Дело не столько в локализации Windows, сколько в том, что вы не учитываете на КЛИЕНТЕ, что она может быть разная, и используете FloatToStr без FormatSettings. Это нубский непрофессиональный подход ))

иначе говоря, вы не установили контракт между сервером и клиентом: что сервер принимает вот такие данные в таком-то виде/формате (т.е. если число - дробное/вещественное, то разделитель - точка (по всяким общепринятым стандартам)), и клиент должен этому следовать.

Upd. а, установили

Клиент на сервер будет передавать информацию в следующем виде:

[FK#J1=0.00,J2=0.00,J3=0.00,J4=0.00,J5=0.00,J6=0.00]

только клиент ему не следует, а вы "исправили" это на сервере, усложнив его

Sign up to leave a comment.

Articles