gavr97 / path-search Goto Github PK
View Code? Open in Web Editor NEWPath Search project
Path Search project
В ридми нет всех разделов, необходимых для успешной сдачи 1 КТ.
Должно быть что-то в духе AStar или ASearch или что-то в этом духе (но не repо, как сейчас).
Комит [cf03d0b]
Зачем weightmovement включен в Node? Это концептуально не правильно, все веса переходов, которые ведут к ноду должны учитываться в его g-значении. Weightmovement это по сути стоимость (вес) перехода из соседа в текущий нод. Эта стоимость (вес) должна вычисляться при генерации соседей при раскрытии вершины способом аналогичным тому как вычисляется эвристическое значение. Далее вычисленный вес прибавляется к g-значению раскрываемой вершины и получается g-значение соседа-последователя. PROFIT.
Достаточно странная структура Node. В качестве key логично использовать свертку по индексам (и делать это там, где нужно, а не хранить key как часть структуры данных). В качестве родителя логичнее всего хранить указатель (на нод, естественно).
Похоже есть дублирование в классе OPEN, т.к. используется одновременно и unordered_map и set, состоящие из Node. Очевидно это сделано для комбинирования свойств контейнеров (быстрый поиск и доступ к ноду по координатам с помощью unordered_map, и быстрый доступ к минимальному по f-значению ноду в set). Но, насколько я вижу, это достигается за счёт полного дублирования нодов, что весьма плохо по памяти. Советую подумать в сторону: unordered_map (с ключом в виде свертки индексов) для основного хранилища + отдельная поле для хранения минимального по f-значению элемента.
Наличие отдельного класса для CLOSE выглядит избыточным. Достаточно объявить std::unordered_map<unsigned, Node> close полем класса Astar. Ведь никакого «нестандартного» функционала от этого контейнера в ходе поиска не требуется (фактически требуется только добавлять туда элементы и искать по ключу). Но в принципе, для улучшения читаемости кода, можно оставить и оберточный класс.
Структуру Output лучше назвать searchResult.
Переменные allowDiag, cutCorners, allowSqueeze лучше объявить как bool (для удобства восприятия кода) и, где необходимо пользоваться конструкциями if (cutCorners), а не (if cutCorners==0) как сейчас.
Стоки 156-158 в astar.cpp выглядят дублирующими. Мы проверили на наличие нода в OPEN, а потом ещё раз проверяем (в строке 161).
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.