Пересоздание кода

[Alex Cones]

Вчера, пока все системы переставлял, в голову пришла одна мысль о внутренней оптимизации моего GRAY FUR. Пока ставилась русификация все это обдумал на бумаге и прикинул, что ускорение вывода и обработки увеличится примерно на 20%, код станет понятнее и менее быдлокоднее. И теперь, пока обновляюсь до 10.10 сижу и думаю, стоит ли практически с нуля (разумеется, несколько используя старые наработки) переписать весь код? По времени займет около недели, но, думаю, результат будет довольно ущутимым как в плане понятности, так и в плане ускорения.

P.S. Много раз читал во всех статьях, что, мол, писать код с нуля есть плохо и неэкономия труда. А если оно того стоит?

P.P.S. Обновление до 10.10 будет проходить 10 часов. Это заговор

[Пепел Феникса]

А если оно того стоит?

если оно того стоит, то да.

сам прикинь стоит ли эта недели этих 20%?

[Levsha100]

В принципе в чем проблема, если ты делаешь это для себя для удовольствия, и это принесет профит в структуре кода то делай, хотя вообще это не есть хорошо.

Кстати, я вот думаю уже о 11.04 а ты только до 10.10 обновляешься

[Alex Cones]

сам прикинь стоит ли эта недели этих 20%?

В принципе с моей колокольни мне даже плевать на эти 20%. Мне скорее выгоднее сделать код более читабельным:

Case Msg Of
   WM_MOUSEMOVE   : Begin
                     Mouse := MakePoint(LParam);
                     If Length(Warehouse.Button) > 0 Then
                      For I := 0 To Length(Warehouse.Button) - 1 Do
                       Begin
                        If (Warehouse.Button[I].Exists)                           And
                           (Warehouse.Button[I].Visible)                          And
                           (Warehouse.Button[I].Form = GetFormIDByHandle(Handle)) And
                           (Warehouse.Form[GetFormIDByHandle(Handle)].Visible)    And
                           (Warehouse.Form[GetFormIDByHandle(Handle)].Exists)     Then
                          If Warehouse.Button[I].State <> BTN_STATE_DISABLED Then
                           If InRect(Mouse.X,
                                     Mouse.Y,
                                     Warehouse.Button[I].Left,
                                     Warehouse.Button[I].Top,
                                     Warehouse.Button[I].Left + Warehouse.Button[I].Width,
                                     Warehouse.Button[I].Top  + Warehouse.Button[I].Height) Then
                            Begin
                             If Not(Warehouse.Button[I].Pressed) Then Warehouse.Button[I].State := BTN_STATE_HOVER

                            End
                             Else Warehouse.Button[I].State := BTN_STATE_NORMAL;

                       End;
                     BufferDraw(GetFormIDByHandle(Handle),'',FALSE);
                     Application.Procs.OnMouseMove(GetFormIDByHandle(Handle), Name, Mouse.X, Mouse.Y);
                    End;

Ужас, правда?

а ты только до 10.10 обновляешься

Оооо, я вчера извращался - на ноуте поставил XP, а на съемный жесткий поставил убунту с грубом. Теперь гружусь с жесткого

[Пепел Феникса]

Мне скорее выгоднее сделать код более читабельным

переписать весь код?

в общем то вещи совместимые.

тем более если работаешь над проектом и дальше, то в какой то мере есть смысл сейчас переписать за неделю, чем потом за месяц.

в общем то все сводится к тому же вопросу:

сам прикинь стоит ли эта недели этих 20%?

(мое мнение что оно того стоит, + мое высказывание чуть выше)

Ужас, правда?

неа, у тя хоть код понятный более менее

[Alex Cones]

есть смысл сейчас переписать за неделю, чем потом за месяц.

Резонно.

Спасибо.

[Levsha100]

Ойойой, переписывай срочно код=)

[Виталий Желтяков]

Лично Я переписывал код своей игры (~ 100 Кб) ради оптимизации 4 раза с нуля. Было потрачено лишних 2 недели, но оно того стоило скорость скриптов возросла в среднем на 300%.
Но не это важно, важно то, что Я свободно могу использовать наработки уже оптимизированного кода в будущем.

[5naip]

не по теме, но у меня вообще 10.4 lts...
именно из-за lts.

[Levsha100]

А смысл в lts для простого юзера?
Для компаний- да, но не для юзеров.