search

it-kmitl-2018 / group3-backend

1 stars 0 forks source link

[Add] TradeSettlement, TradeSettlementHeader, TradePaymentTerms Model and unit test #32

Closed patcharapon-j closed 6 years ago

patcharapon-j commented 6 years ago

Add new following model class and its' unit test.

Please note the problem with class name length will be refactored after @champillon class.

champillon commented 6 years ago

คะแนนครับ 3.75 คะแนน x 2 Promotion => 7.5 คะแนน 5.1 การ design เป็น components เพื่อทำให้คนอื่นทำงานร่วมกันต่อได้ง่าย => 1 คะแนน (ยังไม่ได้แก้ component อะไรเพิ่มจาก PR เดิมนะ) 5.2 คุณภาพของการตั้งชื่อตัวแปร, ฟังก์ชั่น และความเรียบร้อยของ code => 0.5 คะแนน (ชื่อตัวแปรน่าเกลียด ถึงยังไม่สอนแต่ขอหักไว้ก่อน) 5.3 ความยากเชิงเทคนิคที่นำมาประยุกต์ใช้ เช่น แปลง XML Schema เป็น Java code => 1 คะแนน (ใช้ Lombok) 5.4 รายละเอียดการตั้งชื่อใน PR name และ commit log รวมทั้งรายละเอียดที่บรรยายใน issue ของ PR นั้นๆ => 1 คะแนน 5.5 ความสอดคล้องกันกับ PR อื่นๆของทีม ที่ทำงานร่วมกันได้ดี => 0.25 คะแนน (ทำให้ code สะอาด เพื่อนๆ maintain ง่าย)