java-park-mail-ru / 2016-02-unnamed-1-bmp

Азартная игра
0 stars 1 forks source link

РК4 #11

Closed ValerGit closed 8 years ago

ValerGit commented 8 years ago

Сделала два потока для гейм механики и один для вебсокет сервиса

d-frolov commented 8 years ago

Вы переименовывали пакеты в проекте?

У меня на многих файлах ругается на неправильное название пакета Пример UserDataSet package base.datasets; - ругается, что папка base/dataSets

проверьте пожалуйста все java файлы проекта

d-frolov commented 8 years ago

При старте сервера

ERROR StatusLogger No log4j2 configuration file found. Using default configuration: logging only errors to the console. Exception in thread "main" org.hibernate.internal.util.config.ConfigurationException: Could not locate cfg.xml resource [dbconfig.xml] at org.hibernate.boot.cfgxml.internal.ConfigLoader.loadConfigXmlResource(ConfigLoader.java:53) at org.hibernate.boot.registry.StandardServiceRegistryBuilder.configure(StandardServiceRegistryBuilder.java:163) at org.hibernate.cfg.Configuration.configure(Configuration.java:259) at main.Main.main(Main.java:49)

ValerGit commented 8 years ago

ERROR StatusLogger No log4j2 configuration file found. Using default configuration: logging only errors to the console. Exception in thread "main" org.hibernate.internal.util.config.ConfigurationException: Could not locate cfg.xml resource [dbconfig.xml] at org.hibernate.boot.cfgxml.internal.ConfigLoader.loadConfigXmlResource(ConfigLoader.java:53) at org.hibernate.boot.registry.StandardServiceRegistryBuilder.configure(StandardServiceRegistryBuilder.java:163) at org.hibernate.cfg.Configuration.configure(Configuration.java:259) at main.Main.main(Main.java:49)

Пожалуйста, отметьте папку setups как sources

2016-05-23 19 27 54
d-frolov commented 8 years ago

Да, с этим все ок. За картинку отдельное спасибо :)

ValerGit commented 8 years ago

Я исправила регистр названия папок ... но гитхаб испытывает какие-то проблемы с базами данных и сейчас я не могу увидеть свои коммиты здесь, даже проксирование не помогает(

d-frolov commented 8 years ago

Да, у меня тоже подвисает. Вечером еще раз гляну.

ValerGit commented 8 years ago

папки в нижнем регистре наконец 😊

d-frolov commented 8 years ago

Код - внушает :)

А вы гейммеханику не тестируете? Не боитесь багов?

ValerGit commented 8 years ago

Я осознаю их необходимость ... но ведь ручное тестирование выдает желаемые результаты ...)

upddd!!! Мой тиммейт покроет свою часть кода тестами

d-frolov commented 8 years ago

добавьте в pom.xml org.jetbrains.annotations зависимость. Версия аннотаций - 13. 15 не надо.

d-frolov commented 8 years ago

Логирование ошибок во всем проекте:

  1. e.printStackTrace(); убрать и забыть как страшный сон
  2. Везде логирование ошибок должно быть в виде logger.error("Some message", e); То есть вы с одной стороны добавляете какое-то свое кастомное сообщее проясняющее, что именно логически произошло, с другой строны передавая Exception e - вы на выводе в логе получите полный стектрейс ошибки
ValerGit commented 8 years ago

Предлагаем всему SkyForge попробовать https://sbattle.ru/

d-frolov commented 8 years ago

Да чего уж там - молодцы! :)