Хороший стиль программирования

[Levsha100]

Не могу не привести несколько кусков кода с которыми довелось сегодня поработать.

// частицы
  massX:array[0..1000] of integer;
  massY:array[0..1000] of integer;

Про имена массивов молчу. Нужно было организовать структуру Particle это тру и это раз. Во-вторых тут не использовался динамический массив, хотя нужно. И еще если дин. массив не нужен(в этом случае все таки нужен), то число частиц нужно было бы запхать в константу с осмысленным именем.

// f = (q1*qi*k) \r
  Fx:=(strtofloat(edit10.Text) * strtofloat(edit13.Text) * strtofloat(edit11.Text))/(rx*rx);

Самое страшное что может быть- несоответствие комментариев и кода.
Глобальная переменная Fx-зло, я бы организовал объект и в нем определенные свойства. Нет проверки введенных значений и сопутствующих действий. Нет проверки деления на ноль. Неосмысленные названия объектов(эдитов). Прямое считывание с компонентов- полнейшее зло, необходимо организовать глобальные переменные и их обновлять после успешного ввода новых значений. + код неправильно работает, ибо учитывается только модуль силы.


И такой весь код.
Мой мозг чуть не взорвался, и это только _4_ строки кода.

[mutabor]

Ты слишком категоричен, это с возрастом пройдет, см. тут

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

[MaTBeu]

// f = (q1*qi*k) \r
  Fx:=(strtofloat(edit10.Text) * strtofloat(edit13.Text) * strtofloat(edit11.Text))/(rx*rx);

Это вы небось из какой-то студневской лабы взяли и решили покритиковать) Потому что в нормальных программах(не в лабах) таких комментариев не пишут.
Я скажу больше - это наследие мнимой "легкости" в освоении Делфи. Люди учатся просто создавать формочки, бросать на них кнопочки и прочие рюшики, не задумываясь о том, какая структура у этого самого окна и всех его компонентов, не задумываясь, что форма - это класс, и каждый компонент на ней - тоже объект класса. То же самое с Билдером - попытка "облегчить" освоение С++.
Я не холиварщик и не собираюсь разводить срач по поводу что лучше. Я просто говорю, из-за чего возникают вот такие ситуации с кодом.

[Levsha100]

Я скажу больше - это наследие мнимой "легкости" в освоении Делфи.

Вооот, полностью согласен.

[MaTBeu]

// частицы
  massX:array[0..1000] of integer;
  massY:array[0..1000] of integer;

Это еще ничего, вот я занимался разработкой плагина под Adobe Acrobat, и исследовал Adobe SDK. Так вот там дела обстоят просто плачевно -
Типы данных называются
ACCB1, ACCB2, ACCB3... и так далее - множество абстрактных типов у которых переменные называются как-то типа m1, m432, a43 и так далее.
И это же в коммерческом продукте... ужас.

[veter_s_morya]

вот я занимался разработкой плагина под Adobe Acrobat

А как вы в коде копаетесь? Я понял не дизасемблированием. А как?

[BOBAH13]

Наверное open source тогда ясно почему все не ясно, зачем давать все расписанное, ведь легко создать аналог. Продукт ведь коммерческий

[MaTBeu]

Сообщение от veter_s_morya

А как вы в коде копаетесь? Я понял не дизасемблированием. А как?

SDK открытый, на то он и SDK))) Просто доки нужно курить к нему. В доках все расписано, но для понимания все равно сложно.