Когда коту делать нечего… Люблю 2гис и уже много лет пользуюсь, но вектор развития несовсем понятен в последнее время.
Вместо того, чтобы сделать версию для ipad/mac/linux, сделать нормальное отображение направления движения в ios (как в android сделано; на текущий момент в какую сторону не иди, он все равно показывает кружочек), нормальное построение маршрута (насколько я помню сейчас строит только через общественный транспорт, «пешком» нельзя никак построить), вместо этого прикрутили зачем-то 3D-карты. Непонятно.
А ещё раздражает, что хром пытается за меня думать. Например, когда я пытаюсь зайти на биллинг провайдера, находящийся в локальном домене .zet, он редиректит сначала в гугл с вопросом «Вы точно хотели туда зайти?». Маразм.
Та же херня с назойливыми предложениями перевести страничку и сохранить пароль, причем отключение этих функций срабатывает не всегда с первого раза.
Кто вообще будет писать десктопное приложение на рубях, тем более используя через одно место реализованный qt-ruby? Ещё и кроссплатформенное. Какое-то рассуждение о сферическом коне в вакууме.
github.com/troydm/AnimeWallpaperDownloader/pull/1
Я, следуя порыву, начал рефакторить, но сумел вовремя остановиться.
Для более глубокого рефакторинга надо смотреть, что там за страницы оно отдаёт и т.д.
Примеры:
1. В руби стандартом являются отступы в 2 пробела (styleguide)
2. Неплохо было бы вычищать лишние пробелы — это очень портит диффы.
3. Имена переменных это главный фейл. Ещё в школе по-моему рассказывают, что называть переменные «a, b, c» — это плохо.
4. Повторяющийся код надо извлекать в соответствующие методы.
5. Меня смущают счётчики и метод, которым парсятся страницы, но по этому поводу я ничего не могу сказать конкретного, надо вникать в то, как работает приложение.
7. Зачем там хеши нужны я так и не понял. Заменил на массивы.
8. Ну и прочие C-измы. То есть приложение написано в обычном процедурном стиле как это делается в C, без использования всех возможностей ruby.
ты не понял. цель поста — попиарить pvs studio, а не что либо поправить. Поэтому и отправлять никто ничего не будет, очевидно же. В прошлый раз правда за такой отклик мне влупили полгода r/o, так что считай реклама проплачена.
Вместо того, чтобы сделать версию для ipad/mac/linux, сделать нормальное отображение направления движения в ios (как в android сделано; на текущий момент в какую сторону не иди, он все равно показывает кружочек), нормальное построение маршрута (насколько я помню сейчас строит только через общественный транспорт, «пешком» нельзя никак построить), вместо этого прикрутили зачем-то 3D-карты. Непонятно.
А ещё раздражает, что хром пытается за меня думать. Например, когда я пытаюсь зайти на биллинг провайдера, находящийся в локальном домене .zet, он редиректит сначала в гугл с вопросом «Вы точно хотели туда зайти?». Маразм.
Та же херня с назойливыми предложениями перевести страничку и сохранить пароль, причем отключение этих функций срабатывает не всегда с первого раза.
Я, следуя порыву, начал рефакторить, но сумел вовремя остановиться.
Для более глубокого рефакторинга надо смотреть, что там за страницы оно отдаёт и т.д.
Примеры:
1. В руби стандартом являются отступы в 2 пробела (styleguide)
2. Неплохо было бы вычищать лишние пробелы — это очень портит диффы.
3. Имена переменных это главный фейл. Ещё в школе по-моему рассказывают, что называть переменные «a, b, c» — это плохо.
4. Повторяющийся код надо извлекать в соответствующие методы.
5. Меня смущают счётчики и метод, которым парсятся страницы, но по этому поводу я ничего не могу сказать конкретного, надо вникать в то, как работает приложение.
7. Зачем там хеши нужны я так и не понял. Заменил на массивы.
8. Ну и прочие C-измы. То есть приложение написано в обычном процедурном стиле как это делается в C, без использования всех возможностей ruby.
Ну в общем рад, если помог. :)