Гость
Войти | Профиль | Очистить
Действия ...
Форумы / Java [игнор отключен] [закрыт для гостей] / Как правильно переделать код / 24 сообщений из 24, страница 1 из 1
01.11.2019, 08:17
    #39883961
skyboyy
skyboyy
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Я вывел в виде джейсона информацию о домене

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
60.
61.
62.
63.
64.
65.
66.
67.
68.
69.
70.
@RestController
@RequestMapping("api/whois")
public class DomainWhoIsController {

    @Autowired
    DomainRepository domainRepository;

    @Autowired
    ContactsRepository contactsRepository;

    @Autowired
    HostRepository hostRepository;

    @Autowired
    StatusRepository statusRepository;

    @PostMapping
    public Response whoIs(@RequestBody Request request) {
        System.out.println(request);
        DomainInfo domainInfo = domainRepository.findDomainInfo(request.getDomain());
        ContactInfo contactInfo = contactsRepository.findContactInfo(request.getDomain());
        List<HostInfo> hostInfo = hostRepository.findHostInfo(request.getDomain());
        List<StatusInfo> statusInfo = statusRepository.findStatusInfo(request.getDomain());

        return new Response(domainInfo, contactInfo, hostInfo, statusInfo);
    }

    private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo) {
        StringBuffer sb = new StringBuffer();
        sb
                .append("Domain Name............: ").append(domainInfo.getDomainName()).append(System.lineSeparator())
                .append(System.lineSeparator())
                .append("Organization Using Domain Name").append(System.lineSeparator())
                .append("Name...................: ").append(domainInfo.getName()).append(System.lineSeparator())
                .append("Organization Name......: ").append(domainInfo.getOrganizationName()).append(System.lineSeparator())
                .append("Street Address.........: ").append(domainInfo.getStreetAddress()).append(System.lineSeparator())
                .append("City...................: ").append(domainInfo.getCity()).append(System.lineSeparator())
                .append("State..................: ").append(domainInfo.getState()).append(System.lineSeparator())
                .append("Postal Code............: ").append(domainInfo.getPostalCode()).append(System.lineSeparator())
                .append("Country................: ").append(domainInfo.getCountry()).append(System.lineSeparator())
                .append(System.lineSeparator())
                .append("Administrative Contact/Agent").append(System.lineSeparator())
                .append("Nick Handle............: ").append(contactInfo.getHandle()).append(System.lineSeparator())
                .append("Phone Number...........: ").append(contactInfo.getVoice()).append(System.lineSeparator())
                .append("Fax....................: ").append(contactInfo.getFax()).append(System.lineSeparator())
                .append("Email Address..........: ").append(contactInfo.getEmail()).append(System.lineSeparator())
                .append(System.lineSeparator())
                .append("Nameserver in listed order").append(System.lineSeparator())
                .append(System.lineSeparator());

        hostInfo.forEach(hInfo -> sb
                .append("Server.................: ").append(hInfo.getName()).append(System.lineSeparator())
                .append("Ip address.............: ").append(hInfo.getIpaddr()).append(System.lineSeparator())
                .append(System.lineSeparator())
        );

        sb.append(System.lineSeparator());

        statusInfo.forEach(sInfo -> sb
                .append("Domain created: ").append(sInfo.getCreated()).append(System.lineSeparator())
                .append("Last modified: ").append(sInfo.getUpdated()).append(System.lineSeparator())
                .append("Domain status: ").append(sInfo.getStatus()).append(System.lineSeparator())
                .append("Registrar created: ").append(sInfo.getRegistarcr()).append(System.lineSeparator())
                .append("Current Registrar: ").append(sInfo.getRegistrar()).append(System.lineSeparator())
                .append(System.lineSeparator())
        );

        return sb.toString();
    }
}



Результат
Код: java
1.
2.
>curl -H "Content-Type: application/json" -d "{\"domain\":\"post.kz\"}" localhost:8080/api/whois
{"domainInfo":{"domainName":"post.kz","name":"AO Kazpochta","organizationName":"AO Kazpochta","streetAddress":"ul. Auezova,d. 13","city":"Astana","state":"Astana","postalCode":"010000","country":"KZ"},"contactInfo":{"handle":"HOST-000-015851","name":"post.kz","voice":"7-702-409 07 40","fax":"","email":"kazpost@post.kz"},"hostInfo":[{"oid":"10377785","name":"ns.telecom.kz","domain":"telecom.kz","ipaddr":"92.46.42.148","pver":"v4"},{"oid":"10377961","name":"ns.kazpost.kz","domain":"kazpost.kz","ipaddr":"193.193.247.146","pver":"v4"}],"statusInfo":[{"created":"2002-12-13 16:11:43","updated":"2016-04-19 11:27:11","status":"KAZNIC","registarcr":"HOST.KZ","registrar":"clientTransferProhibited"}]}




Но мне сказали там код надо переделать таким образом
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
@RestController
@RequestMapping("api/whois")
public class DomainWhoIsController {

    @Autowired
    DomainRepository domainRepository;

    @Autowired
    ContactsRepository contactsRepository;

    @Autowired
    HostRepository hostRepository;

    @Autowired
    StatusRepository statusRepository;

    @PostMapping
    public Response whoIs(@RequestBody Request request) {
        System.out.println(request);
        DomainInfo domainInfo = domainRepository.findDomainInfo(request.getDomain());
        ContactInfo contactInfo = contactsRepository.findContactInfo(request.getDomain());
        List<HostInfo> hostInfo = hostRepository.findHostInfo(request.getDomain());
        List<StatusInfo> statusInfo = statusRepository.findStatusInfo(request.getDomain());

        return new Response(domainInfo, contactInfo, hostInfo, statusInfo);
    }

    private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo) {
        StringBuffer sb = new StringBuffer();
        
        sb.append("{");
        sb.append(""/domain:"/");
        sb.append(domai
                nInfo.getDomainName());
        sb.append(",");
        //Как можно завершить этот код чтобы он выводил инфу
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 08:56
    #39883980
mad_nazgul
mad_nazgul
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
skyboyy,

Прошу прощения.
Зачем Вам функция "private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo)"?!
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 09:07
    #39883987
skyboyy
skyboyy
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
mad_nazgul,

Он помогает брать инфу о домене
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 09:39
    #39884003
mad_nazgul
mad_nazgul
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
skyboyymad_nazgul,

Он помогает брать инфу о домене

Где?
Просто не могу найти вызов этой функции.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 10:52
    #39884067
asv79
asv79
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
skyboyyЯ вывел в виде джейсона информацию о домене

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
60.
61.
62.
63.
64.
65.
66.
67.
68.
69.
70.
@RestController
@RequestMapping("api/whois")
public class DomainWhoIsController {

    @Autowired
    DomainRepository domainRepository;

    @Autowired
    ContactsRepository contactsRepository;

    @Autowired
    HostRepository hostRepository;

    @Autowired
    StatusRepository statusRepository;

    @PostMapping
    public Response whoIs(@RequestBody Request request) {
        System.out.println(request);
        DomainInfo domainInfo = domainRepository.findDomainInfo(request.getDomain());
        ContactInfo contactInfo = contactsRepository.findContactInfo(request.getDomain());
        List<HostInfo> hostInfo = hostRepository.findHostInfo(request.getDomain());
        List<StatusInfo> statusInfo = statusRepository.findStatusInfo(request.getDomain());

        return new Response(domainInfo, contactInfo, hostInfo, statusInfo);
    }

    private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo) {
        StringBuffer sb = new StringBuffer();
        sb
                .append("Domain Name............: ").append(domainInfo.getDomainName()).append(System.lineSeparator())
                .append(System.lineSeparator())
                .append("Organization Using Domain Name").append(System.lineSeparator())
                .append("Name...................: ").append(domainInfo.getName()).append(System.lineSeparator())
                .append("Organization Name......: ").append(domainInfo.getOrganizationName()).append(System.lineSeparator())
                .append("Street Address.........: ").append(domainInfo.getStreetAddress()).append(System.lineSeparator())
                .append("City...................: ").append(domainInfo.getCity()).append(System.lineSeparator())
                .append("State..................: ").append(domainInfo.getState()).append(System.lineSeparator())
                .append("Postal Code............: ").append(domainInfo.getPostalCode()).append(System.lineSeparator())
                .append("Country................: ").append(domainInfo.getCountry()).append(System.lineSeparator())
                .append(System.lineSeparator())
                .append("Administrative Contact/Agent").append(System.lineSeparator())
                .append("Nick Handle............: ").append(contactInfo.getHandle()).append(System.lineSeparator())
                .append("Phone Number...........: ").append(contactInfo.getVoice()).append(System.lineSeparator())
                .append("Fax....................: ").append(contactInfo.getFax()).append(System.lineSeparator())
                .append("Email Address..........: ").append(contactInfo.getEmail()).append(System.lineSeparator())
                .append(System.lineSeparator())
                .append("Nameserver in listed order").append(System.lineSeparator())
                .append(System.lineSeparator());

        hostInfo.forEach(hInfo -> sb
                .append("Server.................: ").append(hInfo.getName()).append(System.lineSeparator())
                .append("Ip address.............: ").append(hInfo.getIpaddr()).append(System.lineSeparator())
                .append(System.lineSeparator())
        );

        sb.append(System.lineSeparator());

        statusInfo.forEach(sInfo -> sb
                .append("Domain created: ").append(sInfo.getCreated()).append(System.lineSeparator())
                .append("Last modified: ").append(sInfo.getUpdated()).append(System.lineSeparator())
                .append("Domain status: ").append(sInfo.getStatus()).append(System.lineSeparator())
                .append("Registrar created: ").append(sInfo.getRegistarcr()).append(System.lineSeparator())
                .append("Current Registrar: ").append(sInfo.getRegistrar()).append(System.lineSeparator())
                .append(System.lineSeparator())
        );

        return sb.toString();
    }
}



Результат
Код: java
1.
2.
>curl -H "Content-Type: application/json" -d "{\"domain\":\"post.kz\"}" localhost:8080/api/whois
{"domainInfo":{"domainName":"post.kz","name":"AO Kazpochta","organizationName":"AO Kazpochta","streetAddress":"ul. Auezova,d. 13","city":"Astana","state":"Astana","postalCode":"010000","country":"KZ"},"contactInfo":{"handle":"HOST-000-015851","name":"post.kz","voice":"7-702-409 07 40","fax":"","email":"kazpost@post.kz"},"hostInfo":[{"oid":"10377785","name":"ns.telecom.kz","domain":"telecom.kz","ipaddr":"92.46.42.148","pver":"v4"},{"oid":"10377961","name":"ns.kazpost.kz","domain":"kazpost.kz","ipaddr":"193.193.247.146","pver":"v4"}],"statusInfo":[{"created":"2002-12-13 16:11:43","updated":"2016-04-19 11:27:11","status":"KAZNIC","registarcr":"HOST.KZ","registrar":"clientTransferProhibited"}]}




Но мне сказали там код надо переделать таким образом
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
@RestController
@RequestMapping("api/whois")
public class DomainWhoIsController {

    @Autowired
    DomainRepository domainRepository;

    @Autowired
    ContactsRepository contactsRepository;

    @Autowired
    HostRepository hostRepository;

    @Autowired
    StatusRepository statusRepository;

    @PostMapping
    public Response whoIs(@RequestBody Request request) {
        System.out.println(request);
        DomainInfo domainInfo = domainRepository.findDomainInfo(request.getDomain());
        ContactInfo contactInfo = contactsRepository.findContactInfo(request.getDomain());
        List<HostInfo> hostInfo = hostRepository.findHostInfo(request.getDomain());
        List<StatusInfo> statusInfo = statusRepository.findStatusInfo(request.getDomain());

        return new Response(domainInfo, contactInfo, hostInfo, statusInfo);
    }

    private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo) {
        StringBuffer sb = new StringBuffer();
        
        sb.append("{");
        sb.append(""/domain:"/");
        sb.append(domai
                nInfo.getDomainName());
        sb.append(",");
        //Как можно завершить этот код чтобы он выводил инфу


что за рукожоп тебе это сказал-представленный выше код-повод на выход из фирмы.а если тебя с помощью него обучают я бы сам оттуда ушел.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 11:01
    #39884080
Nixic
Nixic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Да ёпть)) Сделайте DTO'ошку включите туда
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 11:02
    #39884082
Nixic
Nixic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Блин)) ctrl+Enter))
включите туда DomainInfo domainInfo, ContactInfo contactInfo
и отдавайте это в респонсе, всё )
спринг сам поля разрулит и положит их в json
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 11:22
    #39884106
Nixic
Nixic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
 
Но мне сказали там код надо переделать таким образом
skyboyy
Но мне сказали там код надо переделать таким образом
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
@RestController
@RequestMapping("api/whois")
public class DomainWhoIsController {

    @Autowired
    DomainRepository domainRepository;

    @Autowired
    ContactsRepository contactsRepository;

    @Autowired
    HostRepository hostRepository;

    @Autowired
    StatusRepository statusRepository;

    @PostMapping
    public Response whoIs(@RequestBody Request request) {
        System.out.println(request);
        DomainInfo domainInfo = domainRepository.findDomainInfo(request.getDomain());
        ContactInfo contactInfo = contactsRepository.findContactInfo(request.getDomain());
        List<HostInfo> hostInfo = hostRepository.findHostInfo(request.getDomain());
        List<StatusInfo> statusInfo = statusRepository.findStatusInfo(request.getDomain());

        return new Response(domainInfo, contactInfo, hostInfo, statusInfo);
    }

    private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo) {
        StringBuffer sb = new StringBuffer();
        
        sb.append("{");
        sb.append(""/domain:"/");
        sb.append(domai
                nInfo.getDomainName());
        sb.append(",");
        //Как можно завершить этот код чтобы он выводил инфу


Во-первых, репозитории в контроллер не надо класть, нужно класть сюда сервисы, которые работают с этими репозитория.
Причем интерфейсы, а не реализации.
Код: java
1.
2.
3.
4.
    private DomainRepository domainRepository; // это должен быть сервис
    private ContactsRepository contactsRepository;// это должен быть сервис
    private HostRepository hostRepository;// это должен быть сервис
    private StatusRepository statusRepository;// это должен быть сервис



Во-вторых, все эти сервисы отдельно автоварить не нужно. Нужно добавить их так как указано выше, а затем добавить или метод или коструктор и автоварить его. Идея должна ругаться на это, можно к ней прислушиваться)

В-третьих, уже увидел, что вы добавили Response что-то аля DTO, как временный вариант, конечно, можно её сюда но по факту делаем сервис и возвращаем эту DTO из сервиса, который все эти данные соберет.

Можно, конечно, подробнее объяснить, но это долго. Думаю, на этапе создания сервисов для этих репозиториев, будет понятно, какой сервис создать для Response и как заавтоварить в него те самые сервисы.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 11:47
    #39884137
asv79
asv79
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Nixic,это точно,дефолтный репозиторий в контроллере
ведь самое смешное - кто то ему же дал этот кусок кода как учебный-тоесть там учитель судя по всему ,котрого самого учить надо

пс.Автор я тебе уже писал - изучи JAVA CORE ,JAVA ADVANCED ,EE и только потом пробуй на зуб спринг и рест сервисы

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

и ведь тебе уже тысячу раз ответили делай DTO все остальное спринг сделает сам
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 11:59
    #39884148
skyboyy
skyboyy
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
asv79,

Просто я сам запутался, просят одну потом другое, походу задания меняют направление
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 14:46
    #39884289
skyboyy
skyboyy
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
skyboyy,
сделал
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
60.
61.
62.
63.
64.
65.
66.
67.
68.
69.
70.
71.
72.
73.
74.
75.
76.
77.
78.
79.
80.
81.
82.
83.
84.
85.
86.
87.
88.
89.
90.
91.
92.
93.
94.
95.
96.
97.
package ru.whois.web;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RestController;
import ru.whois.model.*;
import ru.whois.repository.ContactsRepository;
import ru.whois.repository.DomainRepository;
import ru.whois.repository.HostRepository;
import ru.whois.repository.StatusRepository;

import java.util.List;

@RestController
@RequestMapping("api/whois")
public class DomainWhoIsController {

    @Autowired
    DomainRepository domainRepository;

    @Autowired
    ContactsRepository contactsRepository;

    @Autowired
    HostRepository hostRepository;

    @Autowired
    StatusRepository statusRepository;

    @PostMapping
    public String whoIs(@RequestBody Request request) {
        System.out.println(request);
        DomainInfo domainInfo = domainRepository.findDomainInfo(request.getDomain());
        ContactInfo contactInfo = contactsRepository.findContactInfo(request.getDomain());
        List<HostInfo> hostInfo = hostRepository.findHostInfo(request.getDomain());
        List<StatusInfo> statusInfo = statusRepository.findStatusInfo(request.getDomain());

        return report(domainInfo, contactInfo, hostInfo, statusInfo);
    }

    private String report(DomainInfo domainInfo, ContactInfo contactInfo, List<HostInfo> hostInfo, List<StatusInfo> statusInfo) {
        StringBuffer sb = new StringBuffer();
        sb
                .append("{");

        sb
                .append("\"domainInfo\":{")
                .append("\"domainName\":\"").append(domainInfo.getDomainName()).append("\",")
                .append("\"name\":\"").append(domainInfo.getName()).append("\",")
                .append("\"organizationName\":\"").append(domainInfo.getOrganizationName()).append("\",")
                .append("\"streetAddress\":\"").append(domainInfo.getStreetAddress()).append("\",")
                .append("\"city\":\"").append(domainInfo.getCity()).append("\",")
                .append("\"state\":\"").append(domainInfo.getState()).append("\",")
                .append("\"postalCode\":\"").append(domainInfo.getPostalCode()).append("\",")
                .append("\"country\":\"").append(domainInfo.getCountry()).append("\"")
                .append("}");

        sb.append(",\"contactInfo\":{");

        sb
                .append("\"handle\":\"").append(contactInfo.getHandle()).append("\",")
                .append("\"voice\":\"").append(contactInfo.getVoice()).append("\",")
                .append("\"fax\":\"").append(contactInfo.getFax()).append("\",")
                .append("\"email\":\"").append(contactInfo.getEmail()).append("\"")
                .append("}");

        sb.append(",\"hostInfo\":{");
        hostInfo.forEach(hInfo -> sb
                .append("[")
                .append("\"oid\":\"").append(hInfo.getOid()).append("\",")
                .append("\"name\":\"").append(hInfo.getName()).append("\",")
                .append("\"domain\":\"").append(hInfo.getDomain()).append("\",")
                .append("\"ipaddr\":\"").append(hInfo.getIpaddr()).append("\",")
                .append("\"pver\":\"").append(hInfo.getPver()).append("\"")
                .append("],")
        );
        sb.append("}");

        sb.append(",\"statusInfo\":{");
        statusInfo.forEach(sInfo -> sb
                .append("[")
                .append("\"created\":\"").append(sInfo.getCreated()).append("\",")
                .append("\"updated\":\"").append(sInfo.getUpdated()).append("\",")
                .append("\"status\":\"").append(sInfo.getStatus()).append("\",")
                .append("\"registarcr\":\"").append(sInfo.getRegistarcr()).append("\",")
                .append("\"registrar\":\"").append(sInfo.getRegistrar()).append("\"")
                .append("],")
        );
        sb.append("}");

        sb.append("}");

        return sb.toString();
    }
}
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 14:52
    #39884293
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Про Velocity будем говорить? Не?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 14:52
    #39884294
Nixic
Nixic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Ну, йоперный театр! убирай(те) репозитории из контроллера

В нем должны быть сервисы, которые возвращают объект для отправки клиенту, всё.
Ни репозиториев, ни мапперов, ни конвертеров, ничего лишнего, если это есть, то создаем сервис и кидаем всё это туда
И тем более никаких
private String report(...) {
over_dohrena_coda
}
методов.

А это что за дичь? Нечем заняться? Реально? :)) Вы зачем свой костыль делаете, если всё уже сделано
 
.append("\"name\":\"").append(domainInfo.getName()).append("\",")
.append("\"organizationName\":\"").append(domainInfo.getOrganizationName()).append("\",")
.append("\"streetAddress\":\"").append(domainInfo.getStreetAddress()).append("\",")
.append("\"city\":\"").append(domainInfo.getCity()).append("\",")
.append("\"state\":\"").append(domainInfo.getState()).append("\",")
.append("\"postalCode\":\"").append(domainInfo.getPostalCode()).append("\",")
.append("\"country\":\"").append(domainInfo.getCountry()).append("\"")
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 15:12
    #39884303
asv79
asv79
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Я считаю дальнеешая помощь только усугубит ситуацию.
просто пойми со стороны опытных прогеров- ты выглядишь как 5 летний штангист,который пришел на чемпионат мира по штанге среди взрослых и пытается дернуть штангу 200 кг- а когда не получается -просит ему помочь окружающих -вот именно так ты сейчас выглядишь- это не твоя весовая категория .Твоя задача писать интрефейсы,их имплементации,пузырьковые сортировки и тд

Ну и мой искренний совет
Скачай книжки по java -первая которую ты должен прочитать это философия java,вторая Чистый код
третья -промышеленное программирование
четвертая спринг в дейтсвии
и только после этого приходи сюда за вопросами

попутно с чтением советую java rush
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 15:16
    #39884308
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
StringBuilder еще спрашивают на собесах хотя javac начиная там с 7 или с 8 уже умеет сам цепочку
конкатенаций пересобирать в неявный StringBuilder.

По данному конкретному кейсу не готов сказать. Но можно декомпилировать бинарь и посмотреть.

С чисто эстетической точки зрения "+" писать короче чем .append.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 15:39
    #39884332
забыл ник
забыл ник
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
maytonStringBuilder еще спрашивают на собесах хотя javac начиная там с 7 или с 8 уже умеет сам цепочку
конкатенаций пересобирать в неявный StringBuilder.

По данному конкретному кейсу не готов сказать. Но можно декомпилировать бинарь и посмотреть.

С чисто эстетической точки зрения "+" писать короче чем .append.

скоро должны разродиться triple quotes, что тогда будут на собесах спрашивать, бедные?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 15:57
    #39884349
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
+String interpolation. Уже давно ждут.

По барабану будет што Groovy что Java. Кстати вот многие строковые функции - must have.
И любой язык плавно и постепенно к ним приходит. Вот с операцией + конечно некрасиво вышло.
Математически некрасиво.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 16:07
    #39884357
Nixic
Nixic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Мне лично из темы работы со строками вот этого не хватает в Jave порой. Ооочень редко надо и было бы удобно
https://stackoverflow.com/questions/51874352/template-literals-in-java
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 16:07
    #39884358
chpasha
chpasha
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
забыл никскоро должны разродиться triple quotes
разве не уже в sdk 13?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 16:19
    #39884369
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
Nixic, яж говорю String Interpolation

https://docs.scala-lang.org/overviews/core/string-interpolation.html

Штука мне помогла недавно когда надо было на 700 постгресовских табличек нагенерить триггеров для захвата событий.
Я кое-как склеил DDL скриптик за пол-дня. Делать тоже самое на Java.... это тоже самое что писать колбасу из append()
наподобие той что сделал автор топика.

Брр..
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 16:30
    #39884380
забыл ник
забыл ник
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
chpashaзабыл никскоро должны разродиться triple quotes
разве не уже в sdk 13?
Я не слежу детально... в свободное время иногда почитываю. А паттерн матчинг тоже мимо?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 16:34
    #39884383
PetroNotC Sharp
PetroNotC Sharp
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
По PHP соскучились)))
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 17:10
    #39884410
andreykaT
andreykaT
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
maytonNixic, яж говорю String Interpolation

https://docs.scala-lang.org/overviews/core/string-interpolation.html

Штука мне помогла недавно когда надо было на 700 постгресовских табличек нагенерить триггеров для захвата событий.
Я кое-как склеил DDL скриптик за пол-дня. Делать тоже самое на Java.... это тоже самое что писать колбасу из append()
наподобие той что сделал автор топика.

Брр..
да в скале это прекрасная штука. но эээ ты можешь юзать шаблонизаторы например ))

в скале вот не ясно можно сделать "hello $user"

а потом куда нибудь передать где оно интерполируется, вместо s"hello $user" это ж я так понимаю какой то типа объект или что то типа того. а не просто сахар
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
01.11.2019, 18:55
    #39884462
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как правильно переделать код
andreykaTmaytonNixic, яж говорю String Interpolation

https://docs.scala-lang.org/overviews/core/string-interpolation.html

Штука мне помогла недавно когда надо было на 700 постгресовских табличек нагенерить триггеров для захвата событий.
Я кое-как склеил DDL скриптик за пол-дня. Делать тоже самое на Java.... это тоже самое что писать колбасу из append()
наподобие той что сделал автор топика.

Брр..
да в скале это прекрасная штука. но эээ ты можешь юзать шаблонизаторы например ))

в скале вот не ясно можно сделать "hello $user"

а потом куда нибудь передать где оно интерполируется, вместо s"hello $user" это ж я так понимаю какой то типа объект или что то типа того. а не просто сахар
В моём кейсе всё чики-пики.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / Java [игнор отключен] [закрыт для гостей] / Как правильно переделать код / 24 сообщений из 24, страница 1 из 1
Целевая тема:
Создать новую тему:
Автор:
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
созд. / загр.: 276ms
Закрыть
start [/forum/topic.php?fid=59&mobile=1&tid=2121041]:
 0ms
get settings:
 12ms
get forum list:
 14ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 151ms
get topic data:
 13ms
get forum data:
 3ms
get page messages:
 59ms
get tp. blocked users:
 2ms
others: 14ms
total:  276ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]