Вниз  Совершенный код
- 24.07.2017 / 20:37
Death
  Пользователь

Death 
Сейчас: Offline
aNNiMON, Ааааааааа, ты так. Методы перегруженные. (вроде так называются). Да так я сделаю. Так даже быстрее будет)

Но... Копипаст... Можно избежать... Будет больше ветлений.

Кстати, заодно и Витаминыча вспомнил, геттер в интерфейс впихнул Вот тут гетер к месту в коде, а не сетеры к пятому месту.
__________________
 Смерть правит миром

Изменено Death (24.07 / 20:40) (всего 1 раз)
- 24.07.2017 / 20:52
Bogdan-G
  Пользователь

Bogdan-G 
Сейчас: Offline
Цитата Death:
Блин. Ну как вам это объеснить? МНЕ САМ ТЕЛЕФОН лет сто ПОДКЛЮЧАТЬ. Пока запустится программа для синхронизации Nokia, пока синхранизирутся, минут 5-10 уходит. А если так каждый раз ТОЛЬКО для залития
локальный гит репо не отменял никто.
- 25.07.2017 / 05:07
Витаминыч
  Супермодератор

Витаминыч 
Сейчас: Offline
Цитата Death:
Но... Копипаст... Можно избежать... Будет больше ветлений.
Тебе нравится копипастить? :gg: потому ты так расстроен красивым кодом? :gg:

aNNiMON, а с точки зрения логики архитектуры у него как там дела обстоят?
__________________
 わからない!!
- 25.07.2017 / 08:38
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
Витаминыч, за исключением глобальных статических полей-одиночек, вроде всё нормально.
__________________
 let live
- 25.07.2017 / 16:52
Death
  Пользователь

Death 
Сейчас: Offline
Пожалуйста проверти, всё ли хорошо.
Большие изменения в WorldObject.
Сделал плавное движение. Работает всё как надо, но всё ли логично? В плане производительности хорошо? Ничего лишнего нету?
__________________
 Смерть правит миром


Прикрепленные файлы:
src.zip (22.84 кб.) Скачано 66 раз
- 26.07.2017 / 17:52
Death
  Пользователь

Death 
Сейчас: Offline
Вчера начал делать плавное движение и ближе к вечеру прыжок.

Сегодня в 2-3 часа потратил на улучшение кода. Именно этого класса. Убрал 5 переменных, 2-4 метода. Многое переименовал и что-то за комментировал.

История от себя

Что не правельно сделано, оформлено или написано?
__________________
 Смерть правит миром

Изменено Death (26.07 / 17:53) (всего 1 раз)


Прикрепленные файлы:
WorldObject.java (8.24 кб.) Скачано 1046 раз
- 26.07.2017 / 23:43
Витаминыч
  Супермодератор

Витаминыч 
Сейчас: Offline
Жесть какая-то
__________________
 わからない!!
- 27.07.2017 / 01:14
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
Death, я понимаю, что тебе хочется высказаться, но уж в этой теме постарайся писать чётко и по делу.

И не надо такого: вот вам исходники с кучей кода, разберитесь тут сами и укажите мне на ошибки, а то мне лень самому думать, как сделать удобнее.
Если уж выкладываешь что-то более пяти классов, распиши подробнее какая у них связь, чем ты руководствовался, проектируя именно так и всё в таком духе.
Уважай других пользователей.

Проверку столкновений и управление можно, наверное, в другой класс вынести. Особенно проверку столкновений.
__________________
 let live

Изменено aNNiMON (27.07 / 01:28) (всего 2 раза)
- 27.07.2017 / 07:13
Death
  Пользователь

Death 
Сейчас: Offline
aNNiMON, :krasn:
Ладно. Спасибо.
__________________
 Смерть правит миром
- 9.08.2017 / 20:10
Витаминыч
  Супермодератор

Витаминыч 
Сейчас: Offline
  1. protected override void OnCreate(Bundle bundle)
  2. {
  3.         base.OnCreate(bundle);
  4.  
  5.         SetContentView (Resource.Layout.Main);
  6.  
  7.         Button buttonHello = FindViewById<Button>(Resource.Id.button1);
  8.         buttonHello.Click += ButtonHello_Click; // тут лямбда в примерах,
  9. }
  10. // но я сделал как IDE предложила
  11. private void ButtonHello_Click(object sender, System.EventArgs e)
  12. {
  13.         TextView tw = FindViewById<TextView>(Resource.Id.textView1);
  14.         tw.Text = GetString(Resource.String.ApplicationName);
  15. }
Мне кажется, что правильно так делать, чтобы потом при случае не искать место, где надо поменять логику определенной кнопки. Однако, в большинстве новых примеров все обмазываются лямбдами даже в таком случае
__________________
 わからない!!
Наверх  Всего сообщений: 233
Фильтровать сообщения
Поиск по теме
Файлы топика (7)