24.07.2017 / 17:03 | |
Death Пользователь Сейчас: Offline
Имя: Смерть Регистрация: 31.07.2015
| Инициализаторы вне конструкторовА вот это иногда самому стрёмно, но это удобно. Если в каждом конструкторе писать - не удобно, а если через this(...), то в самом конструкторе, где будет инициализироваться этот объект, то там будет много всего, и читабильность падает.
__________________
Смерть правит миром |
24.07.2017 / 17:17 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| А так из моего это только статичные переменные прямым доступом, как я понял.Всё перечисленное — твоё. Если ты откуда-то скопировал, перепечатал, но при этом не сделал нормально, то это всё равно твой косяк. Если в каждом конструкторе писать - не удобноРедко встречаю класс со множеством конструкторов. А даже если и встречаю, у них зачастую следует какая-то иерархия, например: public Point3D(Point2D p) {
this(p.x, p.y);
}
public Point3D(float x, float y) {
this(x, y, 0);
}
public Point3D(float x, float y, float z) {
this.x = x;
this.y = y;
this.z = z;
// тут можно инициализировать какие-то поля
}
и здесь в последнем конструкторе как раз и нужно разместить такой инициализатор как у тебя. А в твоём классе вообще один конструктор, так что удобно или нет, ты делаешь это неправильно. __________________
let live Изменено aNNiMON (24.07 / 17:20) (всего 1 раз) |
24.07.2017 / 18:32 | |
Death Пользователь Сейчас: Offline
Имя: Смерть Регистрация: 31.07.2015
| aNNiMON, Так. Скопированный код "вжил" в игру. Сделал обновление диномических объектов. Сделал что бы тип объекта задавался через enum. И просто добовлялся через сетку в метод add(GameWorld).
Так же инициализация объектов вне конструктора - исправил. А то что статичные объекты - я не вижу смысла использовать ТУТ паттерн скелетрон/скелестрон(могу ошибиться в названии). Хотя... Приметить можно, но не обезательно. Тут он не нужен. Нужна мера. Может в другом случае, но не тут. Тут удобнее на прямую создовать.(ну я привык уже так)
__________________
Смерть правит миром |
24.07.2017 / 18:41 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| Death, по-хорошему сейчас должна была бы идти ссылка на коммит со всеми перечисленными изменениями. На словах тяжело разобраться, что ты там сделал, может ещё хуже стало. И не скелетрон (интересное название, правда), а синглтон. Одиночка. От слова single. __________________
let live |
24.07.2017 / 18:47 | |
Death Пользователь Сейчас: Offline
Имя: Смерть Регистрация: 31.07.2015
| aNNiMON, Держи) Сейчас что не так стало? Синглтон - знаю. Читал. Но название - язык узлом при устном произношении. А "Скелетрон" - имя босса механического из Террарии Забыл сказать про метод load. Он у меня static везде. Так что тут интерфейс не применим. __________________
Смерть правит миром Изменено Death (24.07 / 18:48) (всего 1 раз)
Прикрепленные файлы: src.zip (40.36 кб.) Скачано 31 раз |
24.07.2017 / 19:02 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| Death, вот так надо было. Коммит. 10 минут ушло на создание репозитория и добавление изменений, Зато теперь наглядно видно, что ты сделал. __________________
let live |
24.07.2017 / 19:05 | |
Death Пользователь Сейчас: Offline
Имя: Смерть Регистрация: 31.07.2015
| aNNiMON, Я бы с радостью, но с телефона это не удобно. А с пк - на каждый раз подключать это долго и нудно и проблемно.
__________________
Смерть правит миром |
24.07.2017 / 19:12 | |
vl@volk Пользователь Сейчас: Offline
Имя: Владислав Откуда: Земля Регистрация: 26.12.2012
| Володя, тебе кажется. Тем более гит заливает только изменения, и ещё сжимает их. Бистро!
__________________
знает толк |
24.07.2017 / 19:12 | |
vl@volk Пользователь Сейчас: Offline
Имя: Владислав Откуда: Земля Регистрация: 26.12.2012
| А ты ищешь оправдание, чтобы не делать этого.
__________________
знает толк |
24.07.2017 / 19:13 | |
RblSb Пользователь Сейчас: Offline
Регистрация: 25.07.2011
| Death, создай хотя бы bash скрипт который будет забрасывать изменения в репозиторий по запуску, серьезно. Будет интересно посмотреть, может кто-то советы там даст. Любой кодер в какой-то момент должен с гитом разобраться, так что тебе нет смысла оттягивать.
Изменено RblSb (24.07 / 19:14) (всего 1 раз) |