seedhartha / QuestPlayer

Android port of Quest Soft Player
http://qsp.su/
26 stars 5 forks source link

Доработка questplayer #14

Closed AlexandrKozlovskiy closed 3 years ago

AlexandrKozlovskiy commented 4 years ago

В этом pull request я предлагаю небольшие доработки и улучшения questplayer.

  1. Добавлена возможность выбирать игру,если в папке с игрой несколько файлов.

Конечно,я это реализовал не так,как Вы хотели,но это работает.

  1. Исправлен баг,когда на некоторых смартах (к примеру у меня) при открытии сохранённой игры не была видна внутренняя память телефона.

Этот баг был исправлен,благодаря тому,что я добавил скрытые extra в intent. Хотя при сохранении игры у меня такой проблемы не было,я добавил эту возможность на всякий случай и в метод,сохраняющий файл.

  1. Подредактированы и добавлена строка для contentDescription (В строке,отвечающей за название кнопки для ввода переменной,нужно добавить название переменной,чтобы пользователь понимал,какую именно переменную мы меняем.
  2. Для того,чтобы Вы могли это протестировать,я вернул мою модификацию замены строк,для прохождения игры "последняя дорога".

Пожелания к новой версии

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

seedhartha commented 4 years ago

Частично смержил как https://github.com/seedhartha/QuestPlayer/commit/268fe70938eae490980c03db29c6689b91168298, https://github.com/seedhartha/QuestPlayer/commit/663cebee4171ee65c29e7d170880ccffc08a6427 и https://github.com/seedhartha/QuestPlayer/commit/2b412f1f81326480e844003c8fbcc2ae4026abdf, спасибо. Всё ещё не уверен насчёт описания кнопки пользовательского ввода: это не только изменение конкретной переменной, но одна из функциональных возможностей плеера.

AlexandrKozlovskiy commented 4 years ago

Всеволод,тогда нужно дать понять пользователю,что он вводит и что должно произойти. К примеру если мы меняем какую-то переменную,то можно просто написать,к примеру,изменение переменной такой-то. Если ввод не только меняет переменную,то,на мой взгляд,нужно описать,что ещё она меняет.

AlexandrKozlovskiy commented 4 years ago

Кстати в игре чашка кофе на aeroqsp,когда я её распаковываю архиваторам,чтобы получить qsp и играю на android,у меня отображаются кракозябры где-то в первых двух ссылках в дополнительном описании,т.е в ссылке с рисунками и в ссылке,при нажатии на которую,мы можем начинать соединять предметы. Примерно такая же ситуация в игре неожиданные приключения,когда мы хотим познакомиться с ведьмочкой. Вроде в нативном коде что-то есть,связанное с кодировкой,но поскольку я не знаю c и c++,я не смогу разобраться,если проблема действительно в нативном коде.