Department Chart

[shumani edzani]

Hi

i want to print department chart with this code and the code is unable to do that please help

public ActionForward deptChart(ActionMapping mapping, ActionForm form,
HttpServletRequest request, HttpServletResponse response)
throws Exception {
 
try {
ReportWizardForm wizForm = (ReportWizardForm) form;
 
Formatting df = new Formatting();
 String appraiser = "";
 String appraisee = "";
 if (wizForm.getAppraisee() == null
 && !wizForm.getAppraisee().equals("")) {
 appraisee = wizForm.getAppraiseeID();
 }
 if (wizForm.getAppraiser() == null
 && !wizForm.getAppraiser().equals("")) {
 appraiser = wizForm.getAppraiserID();
 }
String startdate = "";
if (wizForm.getStartdate() != null) {
startdate = AppConstants.BEGINNING_DATE;
} else {
startdate = df.toNewDate(wizForm.getStartdate());
wizForm.setStartdate(startdate);
}
String enddate = "";
if (wizForm.getEnddate() != null
|| wizForm.getEnddate().equals("31 December 3000")) {
enddate = AppConstants.AMEN_DATE;
} else {
enddate = df.toNewDate(wizForm.getEnddate());
wizForm.setEnddate(enddate);
}
String score = wizForm.getScore();
// String frameworkID = wizForm.getFrameworkID();
String[] department = wizForm.getDepartment();
String scoreoperand = wizForm.getGreaterThan();
 
String orgId = wizForm.getOrganizationID();
 
AppraisalDto appDTO = new AppraisalDto();
AppraisalDAO appDAO = new AppraisalDAO();
appDTO.setGreaterThan(scoreoperand);
appDTO.setScorevalue(Float.valueOf(score));
 appDTO.setAppraiseeid(appraisee);
 appDTO.setAppraiserid(appraiser);
appDTO.setStates(wizForm.getAppraisalphase());
appDTO.setFirstupdated(Timestamp.valueOf(startdate));
appDTO.setLastupdated(Timestamp.valueOf(enddate));
// appDTO.setFrameworkid(frameworkID);
appDTO.setDepartments(department);
appDTO.setOrgId(orgId);
String appIDs = appDAO.reportWizardAppraisals(appDTO);
if (appIDs != null && !appIDs.equals("") && !appIDs.equals("'")) {
renderChart(appIDs, request, response);
} else {
return mapping.findForward("noResults");
}
wizForm.reset();
 
} catch (Exception e) {
// TODO Auto-generated catch block
e.printStackTrace();

}
return null;
}

[edit]Add code tags. CR[/edit]
[ October 07, 2008: Message edited by: Campbell Ritchie ]

[Ulf Dittmer]

So ... what does the code do? Where does it go wrong?

In the future, please UseCodeTags when posting code of any length.

[Campbell Ritchie]

Originally posted by Ulf Dittmer:
So ... what does the code do? Where does it go wrong?

It appears to return null, which looks to me as though it would be better with void return type. It also declares "Exception," and catches it simultaneously. Both those features will make that method hard to use.