TranThiPhuongAnh

[daihv96]

• Tên biến, đối số ... nên được viết rõ ràng theo đúng quy định để người đọc dễ hiểu, tránh hiểu sai.
  

  int3117-2018/TranThiPhuongAnh/Shape/src/DrawDiagram/Diagram.java

  Line 52 in af756f1

  Layer l = new Layer();

• Mã nguồn nên được format giúp code dễ đọc, đỡ rối mắt.
  

  int3117-2018/TranThiPhuongAnh/Shape/src/DrawDiagram/Diagram.java

  Line 25 in af756f1

  for(int i=0; i<listLayer.size() ; i++){

• Không nên để dòng trống trong mã nguồn với những trường hợp này.
  

  int3117-2018/TranThiPhuongAnh/Shape/src/DrawDiagram/Layer.java

  Line 43 in af756f1

• Phương thức "main" nên dùng ở 1 tệp riêng.
  

  int3117-2018/TranThiPhuongAnh/Shape/src/DrawDiagram/Diagram.java

  Line 50 in af756f1

  public static void main(String[] args){

• Tên package nên được dùng hoàn toàn là chữ in thường.
  

  int3117-2018/TranThiPhuongAnh/Shape/test/DrawDiagramTestJunit/DiagramTest.java

  Line 6 in af756f1

  package Cau3;

• Kiểm thử Junit test chưa tốt, nên xóa một số import không sử dụng đến ví dụ như:
  

  int3117-2018/TranThiPhuongAnh/Shape/test/DrawDiagramTestJunit/LayerTest.java

  Line 8 in af756f1

  import org.junit.After;

  
  

  int3117-2018/TranThiPhuongAnh/Shape/test/DrawDiagramTestJunit/LayerTest.java

  Line 10 in af756f1

  import org.junit.Before;