Вниз  Protection Framework
- 5.01.2014 / 12:14
AHiLLA
  Пользователь

AHiLLA 
Сейчас: Offline
А зачем удаляете?:gg:
- 5.01.2014 / 12:30
ВитаминКО
  Супермодератор

ВитаминКО 
Сейчас: Offline
Два метра архивом :gg:
__________________
 わからない!!
- 5.01.2014 / 20:35
samodelkin
  Пользователь

samodelkin 
Сейчас: Offline
Добавлена исправленная модифицированная версия и исходники.

Изменено samodelkin (5.01 / 20:36) (всего 3 раза)
- 5.01.2014 / 21:16
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
samodelkin, я сейчас жестко раскритикую твою игру и твои действия. Но лучше ты это услышишь сейчас, чем во время оценивания жюри твоей игры.
Итак:
- Когда выкладываешь исходники, удаляй оттуда всё лишнее. Папка bin ни к чему, gen тоже сама создастся. Пустые папки assets, libs удаляй.

- Если готовишься выкладывать исходники людям, потрудись сделать их читабельными. Это означает, что в коде должны быть комментарии, он должен быть хорошо отформатирован и т.д. Ты же, когда качаешь примеры игр или приложений сразу получаешь красиво оформленный понятный код, вот и для других старайся также делать.
Тем более в Eclipse это делается всего парой нажатий в меню Source - Format.

- Всегда оптимизируй размер своих приложений.
Зачем в каждую папку drawable одинаковые файлы картинок бросать? Чтоб не масштабировалось? Лучше кинь всё в "drawable-nodpi", тогда оно не будет масштабироваться. Размер apk существенно уменьшится.
И про PngOptimizer не забудь.

- Куда ты музыку кинул? В папку drawable?? Для этого нужно специальную папку raw создать, а не в первую попавшуюся кидать.

- Ты считаешь себя крутым программистом, но ты даже не используешь модификаторы доступа должным образом. Почему метод, который используется только в одном классе объявлен как public? Почему все поля твоих классов имеют модификатор по-умолчанию, а не private или protected?

Теперь конкретно по коду:
Удали это
  1. public void onBackPressed() {
  2.     System.exit(0);
  3. }
и это
  1. @Override
  2. public boolean onCreateOptionsMenu(Menu menu) {
  3.     return true;
  4. }
В "this.getWindowManager()" удали "this.", это здесь не нужно.
"public class Tank extends Object" угадай, что здесь лишнее?

Вот это:
  1. public boolean getEnd(){
  2.     boolean b=false;
  3.     if (enemyTank.getX()<110 | enemyPlane.getX()<110) b=true;
  4.     return b;
  5. }
можно переписать так:
  1. private boolean getEnd() {
  2.     return (enemyTank.getX() < 110 || enemyPlane.getX() < 110);
  3. }

Удали папку res-layout, ты всё равно не используешь xml-разметку. Удали две папки values-sw???dp, лишние значения в файлах папки values.

По коду у меня много претензий, но я уже устал писать :-D

:ps: Всё вышесказанное попросту накипело. Не воспринимай это как критику, наоборот, считай это хорошим знаком, так как у тебя есть возможность к окончанию конкурса исправиться и сделать нормальный продукт.

Вот, что получилось у меня после пары часов рефакторинг твоего проекта.
Исходники прилагаю. Сравнивай, как было, как есть и учись - у тебя хорошая игра, но просто надо приложить немного усилий и сделать её нормально :)
__________________
 let live

Изменено aNNiMON (5.01 / 23:46) (всего 5 раз)


Прикрепленные файлы:
ProtectionFrame(…).apk (300.25 кб.) Скачано 267 раз
ProtectionFrame(…).zip (285.7 кб.) Скачано 40 раз
- 5.01.2014 / 22:03
Vapigor
  Пользователь

Vapigor 
Сейчас: Offline
aNNiMON, Я почти ничё не понял, но кажись ты его поломал полностью...
Хм, такое чувство, что я уже писал точно так же и по похожей ситуации. Насчёт крутого программиста, это вообще в яблочко... :lol:
__________________
 Делай шум!
- 6.01.2014 / 06:51
samodelkin
  Пользователь

samodelkin 
Сейчас: Offline
aNNiMON, спасибо большое за совет.
- 6.01.2014 / 07:46
1JavaMods1
  Пользователь

1JavaMods1 
Сейчас: Offline
ProtectionFramework.zip (2668kb) Samodelkin edition vs. ProtectionFramework.zip (285kb) aNNiMON Edition. Ужас.:hack:
- 6.01.2014 / 09:54
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
1JavaMods1, ну я из тех людей, которые заядло экономят трафик и место на жестком диске компьютера. :gg:
__________________
 let live
- 6.01.2014 / 10:26
1JavaMods1
  Пользователь

1JavaMods1 
Сейчас: Offline
aNNiMON, эх, а ЖД наверняка на 1 тб? :gg:
- 6.01.2014 / 10:43
ВитаминКО
  Супермодератор

ВитаминКО 
Сейчас: Offline
aNNiMON (06.01.2014/09:54)
1JavaMods1, ну я из тех людей, которые заядло экономят трафик и место на жестком диске компьютера. :gg:
Та же фигня :gy:
__________________
 わからない!!
Наверх  Всего сообщений: 66
Фильтровать сообщения
Поиск по теме
Файлы топика (6)