View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002673 | SAS.Планета | Рефакторинг / Refactoring | public | 03-04-2015 11:15 | 04-04-2015 17:54 |
| Reporter | vdemidov | Assigned To | vdemidov | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | resolved | Resolution | fixed | ||
| Product Version | 141212 | ||||
| Target Version | 150915 | Fixed in Version | 150915 | ||
| Summary | 0002673: Сделать простенькую замену TTileIteratorByRect на основе record | ||||
| Description | В программе есть много мест, где нужно пройтись по прямоугольнику тайлов построчно. В некоторых местах сейчас используются два фора, в некоторых TTileIteratorByRect, но это явно большой оверхед по созданию объекта и тд. Хотя достаточно записи на стеке и пары инлайн функций. | ||||
| Tags | No tags attached. | ||||
|
|
Переделал везде где нашел. Если кто-то найдет перебор тайлов прямоугольника простыми форами, то сообщите или сами исправьте. |
|
|
А зачем оставил TotalCount и Reset? Мысль конечно была правильная, но как-то не до конца упростил. |
|
|
Ну, Reset вообще ни на что не влияет, а TotalCount в нескольких местах использовался, так что я решил что вычисление одного произведения это не сильно большой оверхед. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 03-04-2015 11:15 | vdemidov | New Issue | |
| 03-04-2015 14:06 | vdemidov | Status | new => confirmed |
| 03-04-2015 20:16 | vdemidov | Note Added: 0015530 | |
| 03-04-2015 20:16 | vdemidov | Status | confirmed => resolved |
| 03-04-2015 20:16 | vdemidov | Fixed in Version | => 150915 |
| 03-04-2015 20:16 | vdemidov | Resolution | open => fixed |
| 03-04-2015 20:16 | vdemidov | Assigned To | => vdemidov |
| 04-04-2015 13:03 | vasketsov | Note Added: 0015531 | |
| 04-04-2015 17:54 | vdemidov | Note Added: 0015532 | |
| 08-08-2025 13:25 | zed | Category | Рефакторинг => Рефакторинг / Refactoring |