请大家帮我的代码风格挑挑毛病

我的Action文件是这样写的，不知道大家的Action怎么写？
用的和我一样吗？
高人帮指点一下，多谢
还有帮我挑挑代码风格方面的毛病……
先谢谢啦……


package user;

import org.apache.struts.action.ActionMapping;
import org.apache.struts.action.ActionForm;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import org.apache.struts.action.ActionForward;
import org.apache.struts.action.Action;
import javax.servlet.http.HttpSession;
import java.lang.*;

/*############################################################################
文件名称 : User_Action.java
创建日期 : 2006-10-20 By china_programmer
文件作用 : User模块的服务器程序
备注 :
############################################################################*/

public class User_Action extends Action {
public ActionForward execute(ActionMapping mapping, ActionForm form,
HttpServletRequest request,
HttpServletResponse response)
{

String m_p=""; // 跳转页面m_page = p
String m_acad_no=""; // 院系序号m_academy_no = acad_no
User_Function m_user = new User_Function();
User_Form m_form =(User_Form)form;
String m_teacher_name ="";
String m_no="";
m_teacher_name = m_form.getM_teacher_name();
m_no = m_form.getM_no();


HttpSession session = request.getSession(true); //申明一个Session变量
m_acad_no = (String)session.getAttribute("acad_no");//从页面传来Session的值
System.out.println(m_acad_no);

//异常处理判断
if (request.getParameter("p")==null)
{
m_p = "";
}
else
{
m_p = request.getParameter("p");
}

//判断不同条件的，跳到不同页面去
if (m_p.equals("view"))
{
System.out.print("输出为 view user table");
request.setAttribute("user_rs_list",m_user.m_get_user(m_acad_no));
return mapping.findForward("user");
}

//增加和修改，在同一个页面操作
if (m_p.equals("add") || m_p.equals("modify"))
{
System.out.print("输出为 add or modify");
return mapping.findForward("user_addmodify");
}

// 当增加动作提交
if (m_p.equals("add_submit"))
{
m_user.m_user_add(m_acad_no,m_no,m_teacher_name);
return mapping.findForward("user_add_submit");
}
// 当修改动作提交
if (m_p.equals("modify_submit"))
{
m_user.m_user_modify(m_acad_no,m_no,m_teacher_name);
return mapping.findForward("user_modify_submit");
}
//
//删除操作
if (m_p.equals("del"))
{
System.out.print("输出为 del");
return mapping.findForward("user_del");
}

return mapping.getInputForward();
}
}