RError.com

RError.com Logo RError.com Logo

RError.com Navigation

  • 主页

Mobile menu

Close
  • 主页
  • 系统&网络
    • 热门问题
    • 最新问题
    • 标签
  • Ubuntu
    • 热门问题
    • 最新问题
    • 标签
  • 帮助
主页 / 问题 / 1480372
Accepted
invzbl3
invzbl3
Asked:2022-12-23 11:38:54 +0000 UTC2022-12-23 11:38:54 +0000 UTC 2022-12-23 11:38:54 +0000 UTC

如何在服务实现中重构 getAllProjects() 方法?

  • 772

我有以下服务,在一个项目中有一个实现:

@Service
public class ProjectServiceImpl implements ProjectService {
    
    @Autowired
    private ProjectRepository projectRepository;

    @Autowired
    private ProjectAdaptor projectAdaptor;

    @Override
    public List<Project> getAll() {
        return projectRepository.findAll();
    }

    @Override
    public List<Project> getAllProjects(Long companyId) {
        return projectRepository.getAll().stream()
                .filter(project -> project.getId() == companyId).findAny().orElse(null);
    }

    @Override
    public Project save(Project project) {
        return projectRepository.save(project);
    }

    @Override
    public List<ProjectDTO> findAllProducts() {
        List<Project> project = projectRepository.findAll();
        return projectAdaptor.databaseModelToUiDtoList(project);
    }
}

其中有一个方法 getAll():

@Override
public List<Project> getAll() { 
 return projectRepository.findAll();    
} 

并相应地getAllProjects():

@Override 
public List<Project> getAllProjects(Long companyId) { 
  return projectRepository
   .getAll() 
   .stream() 
   .filter(project -> project.getId() == companyId) 
   .findAny() 
   .orElse(null); 
}

在这方面,问题是如何重构上述方法,它重现了与类型相关的编译错误Object:

无法解析中的方法getId。Object

在此处输入图像描述

我的实体Project具有以下结构:

@Data
@Builder
@AllArgsConstructor
@NoArgsConstructor
@Entity(name = "project_entity")
@Table(name = "projects")
public class Project {

    public Project(ProjectDTO projectDTO) {
        this.name = projectDTO.getName();
        this.abbreviation = projectDTO.getAbbreviation();
        this.customer = projectDTO.getCustomer();
    }

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "project_id")
    private Integer id;

    private String name;

    private String abbreviation;

    private String customer;
}

我是否需要Lombok在实体上添加额外的注释来解决这个问题,或者有没有办法以更正确的方式将两种方法合二为一?

我将非常感谢你在这件事上的帮助。

java
  • 2 2 个回答
  • 30 Views

2 个回答

  • Voted
  1. Best Answer
    Ilya Lisov
    2022-12-23T13:48:49Z2022-12-23T13:48:49Z

    我还想指出,返回 null 不是好的做法,您可以将其全部包装在Optional<T>,并且findAny()在方法中它也getAllProjects()意味着从 Optional 包装器中的流返回一个值,但您的方法返回List<Project>。

    为避免您描述的问题:

    1. getAll()在通话中替换为findAll(). 此方法返回一个对象列表Project而不是Object。

    2. 或者转换为Projectlambda 表达式中的类型

      .filter(((Project)project) -> project.getId() == companyId)

    当然,第一种方法更可取。如果对你有帮助,请标记我的答案。

    • 1
  2. Bakuard
    2022-12-23T16:08:36Z2022-12-23T16:08:36Z
    1. 我建议你再仔细考虑一下——是否值得在内存中进行过滤。如果您严格保证它projectRepository.findAll()永远不会返回大量对象并且很少被调用,那么您使用 Stream 的内存过滤方法是合适的。否则,在存储库中创建一个单独的方法,该方法可以按 companyId 过滤项目并在服务内部调用它(或者您可以在存储库级别创建一个接受通用过滤器的方法,例如,通过 Criteria API)。

    2. 另一个重要的一点是,如果您的方法返回一个列表,那么它永远不应该返回 null。这是非常糟糕的做法。如果您没有对象 - 只需返回一个空列表。

    3. 目前尚不清楚方法projectRepository.findAll()和projectRepository.getAll(). 根据他们的名字,结论是他们做同样的事情。在这种情况下,您有代码重复。如果他们做不同的事情,那么重命名其中一个方法,以便立即清楚。

    4. 在服务级别进行交易是可取的。

    5. 按照方法命名的顺序。如果在存储库级别该方法被称为 findAll(),那么在服务级别它的包装器方法应该被完全相同地调用。

    • 1

相关问题

  • wpcap 找不到指定的模块

  • 如何以编程方式从桌面应用程序打开 HTML 页面?

  • Android Studio 中的 R.java 文件在哪里?

  • HashMap 初始化

  • 如何使用 lambda 表达式通过增加与原点的距离来对点进行排序?

  • 最大化窗口时如何调整元素大小?

Sidebar

Stats

  • 问题 10021
  • Answers 30001
  • 最佳答案 8000
  • 用户 6900
  • 常问
  • 回答
  • Marko Smith

    我看不懂措辞

    • 1 个回答
  • Marko Smith

    请求的模块“del”不提供名为“default”的导出

    • 3 个回答
  • Marko Smith

    "!+tab" 在 HTML 的 vs 代码中不起作用

    • 5 个回答
  • Marko Smith

    我正在尝试解决“猜词”的问题。Python

    • 2 个回答
  • Marko Smith

    可以使用哪些命令将当前指针移动到指定的提交而不更改工作目录中的文件?

    • 1 个回答
  • Marko Smith

    Python解析野莓

    • 1 个回答
  • Marko Smith

    问题:“警告:检查最新版本的 pip 时出错。”

    • 2 个回答
  • Marko Smith

    帮助编写一个用值填充变量的循环。解决这个问题

    • 2 个回答
  • Marko Smith

    尽管依赖数组为空,但在渲染上调用了 2 次 useEffect

    • 2 个回答
  • Marko Smith

    数据不通过 Telegram.WebApp.sendData 发送

    • 1 个回答
  • Martin Hope
    Alexandr_TT 2020年新年大赛! 2020-12-20 18:20:21 +0000 UTC
  • Martin Hope
    Alexandr_TT 圣诞树动画 2020-12-23 00:38:08 +0000 UTC
  • Martin Hope
    Air 究竟是什么标识了网站访问者? 2020-11-03 15:49:20 +0000 UTC
  • Martin Hope
    Qwertiy 号码显示 9223372036854775807 2020-07-11 18:16:49 +0000 UTC
  • Martin Hope
    user216109 如何为黑客设下陷阱,或充分击退攻击? 2020-05-10 02:22:52 +0000 UTC
  • Martin Hope
    Qwertiy 并变成3个无穷大 2020-11-06 07:15:57 +0000 UTC
  • Martin Hope
    koks_rs 什么是样板代码? 2020-10-27 15:43:19 +0000 UTC
  • Martin Hope
    Sirop4ik 向 git 提交发布的正确方法是什么? 2020-10-05 00:02:00 +0000 UTC
  • Martin Hope
    faoxis 为什么在这么多示例中函数都称为 foo? 2020-08-15 04:42:49 +0000 UTC
  • Martin Hope
    Pavel Mayorov 如何从事件或回调函数中返回值?或者至少等他们完成。 2020-08-11 16:49:28 +0000 UTC

热门标签

javascript python java php c# c++ html android jquery mysql

Explore

  • 主页
  • 问题
    • 热门问题
    • 最新问题
  • 标签
  • 帮助

Footer

RError.com

关于我们

  • 关于我们
  • 联系我们

Legal Stuff

  • Privacy Policy

帮助

© 2023 RError.com All Rights Reserve   沪ICP备12040472号-5