Вы звоните fileType.trim()
, не проверяя, является ли он нулевым или нет.
Вещи, которые вы не просите, но должны быть упомянуты.
В Apache Commons есть класс StringUtils, который отлично подходит для проверки пустых строк,
поэтому вместо написания
if (null !=request.getParameter("batchid"))
вы бы написали
if (StringUtils.isNotEmpty(request.getParameter("batchid"))
Поскольку вам, вероятно, интересно узнать, содержит ли оно что-то, кроме пробелов.
Кроме того, ваш
else if (mode.equalsIgnoreCase("download"))
указывает, что вы включаете mode
(что я не уверен, если вы также проверили на null
). В Struts 1.x вы можете использовать DispatchAction
, в котором вы указываете параметр, например, mode, который будет именем вызываемого метода, поэтому в вашем примере вы реализуете метод
public ActionForward download(...) {
...
}
, который будет вызван, когда "download".equals(request.getParameter("mode")
.
Также обратите внимание, что запись "download".equals(request.getParameter("mode")
вместо request.getParameter("mode").equals("download")
спасает вас от потенциального нулевого указателя