3

背景:我在教育环境中工作,去年夏天,我们的一位开发人员使用 Spring MVC 和 Hibernate 设计并构建了一个 Java Web 应用程序。它在 9 月推出了新的术语,用户非常高兴,因为它取代了尘土飞扬的旧 Blackboard 插件。应用程序的主要功能用于为学生设定目标、给他们留言和为学生创建报告。

快进几个月,最初的开发人员已经继续前进,该应用程序正在经历一些成长的痛苦。

用例场景:教师登录后,他们会看到他们的主屏幕,其中包含他们所教授的课程列表,以及当前所选课程的目标、消息和报告的概述,以及学生注册列表课程。如果课程包含少量目标等,则可以快速返回信息。但是随着信息量的增长,加载它所花费的时间似乎呈指数级增长。

经过调查,我想我已经找到了原因。我参加了一个示例课程并查看了报告以了解发生了什么。我发现数据库以毫秒为单位返回了相关数据,浏览器以毫秒为单位呈现它,但是在浏览器等待从它返回的数据之间有 12 秒。对数据库查询完成和前端接收响应之间的对象所做的唯一事情是转换为 DTO。

代码:这是报表对象在 DAO 层中的样子

@Entity
@Table(name = "REPORTS")
public class Report implements Serializable
{

    /**
     * 
     */
    private static final long   serialVersionUID    = -7659637777880535914L;

    @Id
    @GeneratedValue
    @Column(name = "REPORT_ID", insertable = true, updatable = false, nullable = false, unique=true)
    private Integer             reportID;

    @Column(name = "DATE_CREATED", insertable = true, updatable = false, nullable = false)
    private GregorianCalendar   dateCreated;

    @Column(name = "DATE_MODIFIED", insertable = true, updatable = true, nullable = true)
    private GregorianCalendar   dateModified;

    @Column(name = "TITLE", insertable = true, updatable = true, nullable = false, length=1000)
    private String              title;

    @Column(name = "CURRENT_PERFORMANCE_GRADE", insertable = true, updatable = true, nullable = false)
    private String              currentPerformanceGrade;

    @Column(name = "TARGET_GRADE", insertable = true, updatable = true, nullable = false)
    private String              targetGrade;

    //VARCHAR(MAX) as this is the main body of the tutor report comments. Here the tutor can write as much content as they like.
    @Column(name = "TUTOR_COMMENTS", insertable = true, updatable = true, nullable = false, columnDefinition="VARCHAR(MAX)")
    private String              tutorComments;
//getters and setters below
}

那里还有其他字段,例如报告链接到的用户、课程、编写报告的导师等,但为了简单起见,我在这里省略了它们。

public class ReportDTO implements Serializable
{

/**
 * 
 */
private static final long   serialVersionUID    = 2795129355073929139L;

private Integer             reportID;

private String              dateCreated;

private String              dateModified;

private String              title;

private String              currentPerformanceGrade;

private String              targetGrade;

private String              tutorComments;
//getters and setters below
}

因此,主要区别在于日期对象已成为日期格式的字符串,而不是 GregorianCalendar 对象,因此日期的前端显示具有良好的可读格式。这是转换为 DTO 所涉及的示例。服务层中的 Single 方法获取 DAO 对象,从中获取相关字段,将它们设置在新构建的 DTO 对象中,根据需要进行转换(例如,公历到日期格式的字符串)并返回 DTO:

public ReportDTO convertToDto(Report daoReport) throws Exception
{

    ReportDTO dtoReport = new ReportDTO();
    try
    {
                    if(daoReport.getReportID() != null)
        {
            dtoReport.setReportID(daoReport.getReportID());
        }
                    if(daoReport.getDateCreated() != null)
        {
            dtoReport.setDateCreated(ReportServiceImpl.ISO_DATE_TIME_FORMAT.format(daoReport.getDateCreated().getTime()));

        }

        if(daoReport.getDateModified() != null)
        {
             dtoReport.setDateModified(ReportServiceImpl.ISO_DATE_TIME_FORMAT.format(daoReport.getDateModified().getTime()));

        }

        if(daoReport.getTitle() != null)
        {
            dtoReport.setTitle(daoReport.getTitle());

        }
                     if(daoReport.getCurrentPerformanceGrade() != null)
        {
              dtoReport.setCurrentPerformanceGrade(daoReport.getCurrentPerformanceGrade());

        }

        if(daoReport.getTargetGrade() != null)
        {
            dtoReport.setTargetGrade(daoReport.getTargetGrade());

        }

        if(daoReport.getTutorComments() != null)
        {
            dtoReport.setTutorComments(daoReport.getTutorComments());

        }
                    return dtoReport;
    }
    catch(Exception e)
    {
        Exception myException = new Exception("Exception was thrown while converting a persistent Report object to it's data transport equivalent", e);
        throw myException;
    }

问题:毕竟,我的问题是,这是从 DAO 转换为 DTO 的正确方法吗?自从他离开后,我一直在关注他的代码,任何新添加的内容都是以同样的方式完成的。将对象返回到前端而不转换它们,我会在 >300 毫秒而不是 12 秒内看到结果。

我知道他从这里为该项目学习了 Spring MVC,所以他不是经验丰富的 Spring 开发人员,我也不是,而且从我们看到如此大的请求时间这一事实来看,我们一定是做错了什么。

4

2 回答 2

3

好的,正如 beny23 提到的,Hibernate 是延迟加载(最初加载 PK 列表,然后在对数据执行某些操作时加载其余部分)

我使用的解决方案是创建一个非休眠连接以使用普通 JDBC 连接读取数据,查询还转换了数据,以便它以我需要的格式返回(日期为字符串等)所以我没有t 必须转换为 dto。通过这种方式,我将一些工作卸载到数据库中,并为我的应用程序省去了这样做的麻烦。

于 2013-01-28T10:55:18.017 回答
1

这可能不是您的问题的原因(12s 很大),但仍然值得一提。

简单DateFormat类不是线程安全的:

日期格式不同步。建议为每个线程创建单独的格式实例。如果多个线程同时访问一个格式,它必须在外部同步。

所以不要将它们存储在全局类属性中,否则你可能会遇到奇怪的问题。一个简单的事情是在使用之前实例化一个 ( Simple) 。DateFormat

另请参阅有关 SimpleDateFormat的有趣博客文章。

于 2013-01-21T19:50:14.433 回答