Комментарии (9)

1. 

   nerumb

   20.10.2016 19:57

   #9870294

   Задача такова:
   
   — На основе данных БД построить дерево на веб-странице, все дерево сразу грузить было нельзя так, как слишком много данных.

   
   Но при этом вы используете заглушку для работы с БД. Не придираюсь, но думаю что не стоит вводить путаницу.
   
   

   Чтобы работал метод, нужно расставить аннотации в Node:

   
   Не обязательно, будет работать и без них. Аннотации нужны чтобы указать названия полей под которыми данные будут представлены в json.
   
   

   Это мой первый пост, который рассчитан на новичков в веб-программировании

   
   Куда проще было бы воспользоваться spring boot, dropwizard, vert.x (и т.п.). И кода было бы гораздо меньше, да и понятней все для новичков, без всякого blood enterprises деплоя на сервер приложений, jsp и т.п.
   

   
   

   1. 

      BioQwer

      21.10.2016 01:27

      #9870608

      Да вы правы. Основная задача отображение из БД, а для статьи на Хабр сделал простую заглушку.
      

      Не обязательно, будет работать и без них. Аннотации нужны чтобы указать названия полей под которыми данные будут представлены в json.

      
      Не всегда оно так, бывает задаешь правила для JacksonMapper и какие то куски API выходят из правил.
      

      Куда проще было бы воспользоваться spring boot, dropwizard, vert.x (и т.п.). И кода было бы гораздо меньше, да и понятней все для новичков, без всякого blood enterprises деплоя на сервер приложений, jsp и т.п.

      
      А jsp тут не используется и он мне сразу не понравился.
      Статью писал 2 года назад, тогда не знал про Spring, а когда узнал то ощутил blood enterprises деплой во всей красе.
      
      

1. 

   aol-nnov

   21.10.2016 01:11

   #9870600

   {
   id: «string» // required
   parent: «string» // required

   
   
   это всё «платина». все заборы исписаны такими примерами. вот, кабы nested sets там, или closure table…
   вот их рендеринг — это интересно!

   
   

   1. 

      BioQwer

      21.10.2016 01:29

      #9870610

      +1

      Оно на heroku
      Позже начал играться с облаками вот оно там и осталось. Единственное сейчас там по 5к элементов генерируется.
      

1. 

   grossws

   21.10.2016 04:35

   #9870670

   +3

   public static String toJSON_String(Map map) throws JsonProcessingException

   Закачайте вашу java 1.4 обратно в Интернет, она там уже закончилась. Если что, 5 появилась в 2004, 12 (sic!) лет назад (и тогда же появились generics, да). Уже случился EOL Java 7 в прошлом году. Это даже не говоря про нормальное именование, в Java принят camelCase.

   
   

   //Класс Синглтон

   Загляните в effective java, прочитайте как обычно реализуют синглтоны, особенно в concurrent окружении, коим являются сервлеты. private static LocalDataImpl ourInstance — это некорректный вариант. Есть довольно удобный вариант с enum, но в 1.4 его ещё не завезли, если правильно помню. Но есть и вариант с private static final полем для экземпляра и private конструктором, хотя этот вариант не абсолютно надёжен.

   
   

   Если же смотреть глобально, то статья — такое днище, что маловероятно, что кто-нибудь сумеет постучать снизу.

1. 

   fogone

   21.10.2016 18:37

   #9872160

   +1

   Чуть более современный вариант серверной части

   class TreeNode {
       private String id;
       private String parentId;
       private String text;
       private boolean hasChildren;
   
       public TreeNode(String id, String parentId, String text, boolean hasChildren) {
           this.id = id;
           this.parentId = parentId;
           this.text = text;
           this.hasChildren = hasChildren;
       }
   
       public String getId() {
           return id;
       }
   
       public String getParentId() {
           return parentId;
       }
   
       public String getText() {
           return text;
       }
   
       public boolean hasChildren() {
           return hasChildren;
       }
   
       public boolean isChildOf(String parentId) {
           boolean isRoot = this.parentId == null && parentId == null;
           boolean isChild = this.parentId != null && this.parentId.equals(parentId);
   
           return isRoot || isChild;
       }
   }
   
   
   @ResponseBody
   @RequestMapping("/tree")
   class TreeController {
   
       private final TreeRepository treeRepository;
   
       @Autowired
       public TreeController(TreeRepository treeRepository) {
           this.treeRepository = treeRepository;
       }
   
       @RequestMapping("root")
       public List<TreeNode> root() {
           return treeRepository.findRoot();
       }
   
       @RequestMapping("node/{id}")
       public TreeNode get(@PathVariable("id") String id) {
           return treeRepository.findById(id);
       }
   
       @RequestMapping("node/{id}/children")
       public List<TreeNode> children(@PathVariable("id") String id) {
           return treeRepository.findChildrenByParentId(id);
       }
   
   }
   
   
   interface TreeRepository {
   
       TreeNode findById(String id);
   
       default List<TreeNode> findRoot() {
           return findChildrenByParentId(null);
       }
   
       List<TreeNode> findChildrenByParentId(String parentId);
   
   }
   
   class FakeTreeDataGenerator implements TreeDataGenerator {
   
       private final int count;
       private final int deep;
       private final IdGenerator idGenerator;
   
       public FakeTreeDataGenerator(IdGenerator idGenerator, int count, int deep) {
           this.idGenerator = idGenerator;
           this.count = count;
           this.deep = deep;
       }
   
       @Override
       public List<TreeNode> createTree() {
           return createFakeNodes(null, deep).collect(Collectors.toList());
       }
   
       private Stream<TreeNode> createFakeNodes(String parentId, int deep) {
           return IntStream.range(0, count)
               .boxed()
               .map(i -> idGenerator.generateId().toString())
               .flatMap(id -> {
                   boolean hasChildren = deep > 0;
                   Stream<TreeNode> nodeStream = Stream.of(
                       new TreeNode(id, parentId, id + " text", hasChildren));
                   return hasChildren ? 
                       Stream.concat(nodeStream, createFakeNodes(id, deep - 1)) : nodeStream;
               });
       }
   
   }
   
   class GeneratedTreeRepository implements TreeRepository {
   
       interface TreeDataGenerator {
   
           List<TreeNode> createTree();
   
       }
   
       private final Map<String, TreeNode> repository;
   
       @Autowired
       public GeneratedTreeRepository(TreeDataGenerator treeDataGenerator) {
           this.repository = treeDataGenerator
               .createTree()
               .stream()
               .collect(Collectors.toConcurrentMap(TreeNode::getId, Function.identity()));
       }
   
       @Override
       public TreeNode findById(String id) {
           return repository.get(id);
       }
   
       @Override
       public List<TreeNode> findChildrenByParentId(String parentId) {
           return repository.values().stream()
               .filter(treeNode -> treeNode.isChildOf(parentId))
               .collect(Collectors.toList());
       }
   
   }
   
   @SpringBootApplication
   public class TreeApplication {
   
       @Bean
       public TreeDataGenerator treeDataGenerator() {
           return new FakeTreeDataGenerator(new JdkIdGenerator(), 10, 5);
       }
   
       @Bean
       public TreeRepository treeRepository(TreeDataGenerator treeDataGenerator) {
           return new GeneratedTreeRepository(treeDataGenerator);
       }
   
       @Bean
       public TreeController treeController(TreeRepository treeRepository) {
           return new TreeController(treeRepository);
       }
   
       public static void main(String[] args) {
           SpringApplication.run(TreeApplication.class, args);
       }
   
   }
   

   
   

   
   

   1. 

      fogone

      21.10.2016 18:56

      #9872198

      Прошу прощения, @Autowired на конструкторах явно лишние

   
   

   1. 

      fogone

      21.10.2016 19:48

      #9872278

      +1

      Тоже самое но чуть более лаконично на котлине

      data class TreeNode(val id: String, val parentId: String?,
                          val text: String, val hasChildren: Boolean) {
      
          fun isChildOf(parentId: String?): Boolean {
              val isRoot = this.parentId == null && parentId == null
              val isChild = this.parentId != null && this.parentId == parentId
      
              return isRoot || isChild
          }
      
      }
      
      interface TreeRepository {
      
          fun findById(id: String): TreeNode?
      
          fun findRoot(): List<TreeNode> = findChildrenByParentId(null)
      
          fun findChildrenByParentId(parentId: String?): List<TreeNode>
      
      }
      
      interface TreeDataGenerator {
      
          fun createTree(): List<TreeNode>
      
      }
      
      class GeneratedTreeRepository(treeDataGenerator: TreeDataGenerator) : TreeRepository {
      
          private val repository = treeDataGenerator.createTree().associateBy(TreeNode::id)
      
          override fun findById(id: String): TreeNode? = repository[id]
      
          override fun findChildrenByParentId(parentId: String?): List<TreeNode> =
                  repository.values.filter { treeNode -> treeNode.isChildOf(parentId) }
      
      }
      
      class FakeTreeDataGenerator(val idGenerator: IdGenerator, val count: Int, val deep: Int) : TreeDataGenerator {
      
          override fun createTree(): List<TreeNode> = createFakeNodes(null, deep)
      
          private fun createFakeNodes(parentId: String?, deep: Int): List<TreeNode> =
                  if (deep >= 0) {
                      (0..count)
                              .map { i -> idGenerator.generateId().toString() }
                              .map { id -> TreeNode(id, parentId, id + " text", deep > 0) }
                              .flatMap { node -> createFakeNodes(node.id, deep - 1) + node }
                  } else emptyList()
      }
      
      @ResponseBody
      @RequestMapping("/tree")
      class TreeController(val treeRepository: TreeRepository) {
      
          @RequestMapping("root")
          fun root(): List<TreeNode> =
                  treeRepository.findRoot()
      
          @RequestMapping("node/{id}")
          fun get(@PathVariable("id") id: String): TreeNode? =
                  treeRepository.findById(id)
      
          @RequestMapping("node/{id}/children")
          fun children(@PathVariable("id") id: String): List<TreeNode> =
                  treeRepository.findChildrenByParentId(id)
      
      }
      
      @SpringBootApplication
      open class TreeApplication {
      
          @Bean
          open fun treeDataGenerator(): TreeDataGenerator =
                  FakeTreeDataGenerator(JdkIdGenerator(), 10, 5)
      
          @Bean
          open fun treeRepository(treeDataGenerator: TreeDataGenerator): TreeRepository =
                  GeneratedTreeRepository(treeDataGenerator)
      
          @Bean
          open fun treeController(treeRepository: TreeRepository): TreeController =
                  TreeController(treeRepository)
      
      }
      
      fun main(args: Array<String>) {
          SpringApplication.run(TreeApplication::class.java, *args)
      }
      

      
      

1. 

   AlexeyKh89

   22.10.2016 15:09

   #9873120

   ObjectMapper должен быть один, а не на каждый вызов метода.