How to redirect from servlet to JSP/Servlet with out executing the below code

[santhosh.R gowda]

Dear All

I'm using the servlet to upload the file.

The Feature of this Servlet is if it exceeds the size limit (ex:- 15Mb) it should redirect to Jsp/servlet so here im putting the condition of file size check if it is more than 15 mb it should redirect to another jsp or else it it should allow to upload

   
if(condition)
{
  response.sendRedirect("../failure.jsp")
}
else
{
  continue....
}
 

but im getting the size with in 2 second's but its not redirecting the servlet if it exceeds it ll take time around 20 minutes
So please give me hand on this issue

[Deepak Bala]

Post the entire code. It is not possible to understand your logic unless you do that

[santhosh.R gowda]

Dear Deepak Bala

I ll give you the logic

 



import java.io.File;

import java.io.IOException;
import java.util.Iterator;
import java.util.List;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletConfig;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

import org.apache.commons.fileupload.FileItem;
import org.apache.commons.fileupload.FileItemFactory;
import org.apache.commons.fileupload.FileUploadException;
import org.apache.commons.fileupload.disk.DiskFileItemFactory;
import org.apache.commons.fileupload.servlet.ServletFileUpload;



public class myservlets extends HttpServlet {
	private static final long serialVersionUID = 1L;
       
   
    public myservlets() {
        super();
        // TODO Auto-generated constructor stub
    }

	
	public ServletConfig getServletConfig() {
		// TODO Auto-generated method stub
		return null;
	}

	
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
	}

	
	
	
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		
		if(ServletFileUpload.isMultipartContent(request))
		{
			FileItemFactory factory = new DiskFileItemFactory();
			ServletFileUpload upload = new ServletFileUpload(factory);
			try {
				Iterator iter = upload.parseRequest(request).iterator();
				while(iter.hasNext())
				{
					FileItem item = (FileItem) iter.next();
					RequestDispatcher reqRes=item.getSize()<15l*1024l*1024l?request.getRequestDispatcher("../success.jsp"):request.getRequestDispatcher("../failure.jsp");
					reqRes.forward(request, response);
				}
				
				
			} catch (FileUploadException e) {
				e.printStackTrace();
			}
			System.out.println("hi dude");
		}
		
	
	}	

}

here im redirecting to either one of the jsp file and even though it redirects its printing "hi dude" message .so my question is how to redirect it with out printing " hi dude" message

[Deepak Bala]

Please edit the code since some lines will surely not compile.

The redirect method is not a return statement. Lines of code after this statement will also be run by the JVM

[santhosh.R gowda]

The redirect method is not a return statement. Lines of code after this statement will also be run by the JVM

So how to avoid the JVM not to execute the code after redirect method

The code is compiling and running i think there is no error ....

[Deepak Bala]

So how to avoid the JVM not to execute the code after redirect method

Why do you want to do this ? Why do you have code after the redirect method ?

[santhosh.R gowda]

Is there any solution or what ...?
or else i want to put it in if else block

[santhosh.R gowda]

 List items = upload.parseRequest(req);   

It is taking time in parsing the request

can we over come this ....

[David Newton]

If you want to return after the redirect, use the "return" keyword.

[santhosh.R gowda]

Dear David Newton

If you want to return after the redirect, use the "return" keyword.

Can you elobarate this statement

[David Newton]

I suppose. In this code, doMoreStuff() will execute.

method() {
    doStuff;
    if (foo) {
        sendRedirect();
    }
    doMoreStuff();
}

In this code, it won't.

method() {
    doStuff;
    if (foo) {
        sendRedirect();
        return;
    }
    doMoreStuff();
}

That's about as elaborate as I can get, though.

[santhosh.R gowda]

Thanks David Newton
I understood the return word concept

But in my previous code

 

Iterator iter = upload.parseRequest(request).iterator();  
 

This statement is taking too much of time(3 minutes or more ) but when i execute the same code in my standalone application its taking few secongs(2) how to over come this problem

[Seetharaman Venkatasamy]

i dont know why this much time (3 mins) ? [may be network traffic]. showing a progressbar help you ?

ProgressBar

[Mark E Hansen]

Can't you check the content-length header of the request, and forward based on the size reported there?