hoaivan / Writing-Planet

1 stars 0 forks source link

review code importStudent #41

Closed hoaivan closed 12 years ago

hoaivan commented 12 years ago

StudentPortlet.importStudent():

  1. actionRequest.getPortletSession().setAttribute("impErrorList", errorList); <<< sao phải add vào portletSession? sao ko dùng SessionErrors.
  2. e.printStackTrace(); tại sao ko nên dùng debug kiểu này? nên dùng kiểu nào?
cuongmc commented 12 years ago
  1. actionRequest.getPortletSession().setAttribute("impErrorList", errorList); <<< sao phải add vào portletSession? sao ko dùng SessionErrors. Phần này cần xử lý báo lỗi riêng để hiển thị cho từng row nên sử dụng cách này thay vì sử dụng SessionErrors.
  2. e.printStackTrace(); tại sao ko nên dùng debug kiểu này? nên dùng kiểu nào? A miss do lúc code tự sinh try/catch không để ý, đã update lại src code phần này.