วันพฤหัสบดีที่ 13 ธันวาคม พ.ศ. 2555

Reviews code ( Gang 5 Students )

List of students ( Gang 5 Students )
      1.) ธวัชชัย
      2.) ปิยวัช
      3.) ศศิธร
      4.) จิรวัฒน์
      5.) ชาญศักดิ์

Reviews code : ปิยวัช
Commits : [ b4ee4c3 ] add/show/clear data
     ในไฟล์ views.py  การจัดย่อหน้าของ code มีบางบรรทัดยังไม่ตรงกัน ทำให้ code ดูไม่มีความสวยงาม บางครั้งอาจทำให้ยากต่อการดู แล้วก็มีการตั้งชื่อให้กับฟังก์ชั่นที่ยาวเกินไป ควรตั้งแบบสั้นๆ แต่ต้องตั้งให้สอดคล้องกับฟังก์ชั่นภายในที่สร้าง เพราะถ้าชื่อมีความยาวจนเกินไป อาจจะทำให้เกิดความผิดพลาดได้ในการพิมพ์โค้ด เพื่อเรียกใช้ฟังก์ชั่นนั้นมาใช้
     ในไฟล์ urls.py ผมชอบการเข้าถึง urls


Reviews code : ศศิธร
Commits : [  bdb29f4 ] process input from
     มีการลืมใส่เครื่องหมาย ดอลล่า ( $ ) มีหน้าที่ให้รู้ว่า เป็นตัวหยุด ประมาณว่าให้รู้ว่าเป็นการสิ้นสุดของการป้อน urls และการตั้งชื่อตัวแปร บางตัวมีการย่อตัวอักษร ( เขียนแบบย่อ ) อาจดูไม่มีความหมาย


Reviews code : จิรวัฒน์
Commits : [  2872457 ] ver 8 8 complate 
link : https://bitbucket.org/pondzacoct/w4-1/commits/28724577dcf84f2ec544e144a9680572e3ec0331
    การตั้งชื่อ commit มันยังดูไม่มีความหมายเท่าไร 
     ในการ commit ยังมีไฟล์ที่เป็น .py~ และ .html~ ( ควรทำการลบก่อนที่จะมีการ commit หรือตอนการ add ไฟล์ ควรเลือกไฟล์มา ไม่ควรที่จะ add มาหมดทั้งโฟรเดอร์ ) 
    การตั้งชื่อตัวแปร ควรตั้งชื่อ ขึ้นต้นด้วยตัวพิมพ์เล็ก มันจะดีกว่า


Reviews code : ชาญศักดิ์
Commits : [  73eca4c ] update homepage & template
     ในการ commit ยังมีไฟล์ที่เป็น .html~ 

ไม่มีความคิดเห็น:

แสดงความคิดเห็น