Skip to content

Hw16 maven RMI message system#21

Open
deft1991 wants to merge 3 commits intomasterfrom
hw16-maven
Open

Hw16 maven RMI message system#21
deft1991 wants to merge 3 commits intomasterfrom
hw16-maven

Conversation

@deft1991
Copy link
Owner

@deft1991 deft1991 commented Dec 8, 2019

No description provided.

return new org.springframework.security.core.userdetails.User(user.getName(), user.getPassword(), null);
}
}
//package ru.deft.homework.service.impl;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

такого не должно быть в гите.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

и не понятно, как это изменение относится к этому ДЗ.

import java.util.HashMap;
import java.util.Map;

public class DBServiceImpl implements DBService {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удаляйте неиспользуемые классы.



public MessageSystem msClient() {
MessageSystem messageSystem = new MessageSystemImpl();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для Spring-ового приложения такой способ создания объектов не очень типичен.

* Created by sgolitsyn on 12/6/19
*/
@Slf4j
public class FrontendClient {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем тут этот класс?

*/
@Slf4j
@Component
public class MessageSystemClient implements IMessageSystem {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Класс тоже не используется?

@@ -0,0 +1,30 @@
//package ru.deft.homework.client;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем это комитить в гит?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants