Groovy/Grails วิธีทำให้คอนสตรัคเตอร์เป็นส่วนตัว - วิธีที่ถูกต้อง

เกี่ยวกับข้อยกเว้น:

ข้อยกเว้นเกิดขึ้นเนื่องจากตัวสร้างแผนที่ ด้วยคลาส Groovy ปกติ นี่ไม่ใช่ปัญหา แต่ Grails จะลงทะเบียนคลาสโดเมนเป็นต้นแบบ Bean จากนั้นจะแทนที่ตัวสร้างในเมตาคลาสเพื่อใช้การสร้าง bean และกลไกการเดินสายอัตโนมัติเพื่อรับอินสแตนซ์ บางสิ่งในตัวสร้างแผนที่และการเดินสายอัตโนมัติทำให้มีการสร้างถั่วสีว่างก่อนที่จะตั้งค่าโดยแผนที่

หากคุณเปลี่ยนรหัสเป็น:

Color c = Color.getColor('red')
Mood m = new Mood()
m.color = c
m.save()

ข้อยกเว้นควรหายไป

คุณอาจพิจารณายื่นปัญหา JIRA สำหรับกรณีการใช้งานเฉพาะนี้ แต่ฉันไม่รู้ว่าหรือ ไม่ใช่ทีม grails ที่จะถือว่านี่เป็นข้อบกพร่องหรือการตัดสินใจในการออกแบบ มันไม่ได้บันทึกไว้ที่ใดอย่างแน่นอน

เกี่ยวกับการออกแบบ:

ฉันเห็นด้วยกับ dmahapatro เกี่ยวกับการเปลี่ยนความรับผิดชอบด้านความสมบูรณ์ของข้อมูลไปยังฐานข้อมูลและข้อจำกัด GORM ของคุณโดยไม่ทราบอะไรมากนักเกี่ยวกับโมเดลของคุณ นั่นคือสิ่งที่พวกเขาทำเพื่อ

การหลีกเลี่ยงสิ่งนี้ทำให้เกิดรูปแบบการใช้งานที่ผิดปกติในโค้ด เช่น การรู้ว่าจะใช้ Color.getColor ซึ่งตรงข้ามกับการสร้างอินสแตนซ์คลาสโดเมนปกติ

อุดมการณ์เบื้องหลังแบบแผนเหนือกรอบการกำหนดค่าเช่น Grails คือการปฏิบัติตามแบบแผนให้มากที่สุดเท่าที่จะเป็นไปได้ เพื่อให้ใครก็ตามที่คุ้นเคยกับแบบแผนสามารถก้าวเข้ามาและรู้ทันทีว่าเกิดอะไรขึ้น

การใช้ name เป็นคีย์หลักสำหรับ Color จะทำให้คุณมีเอกลักษณ์เฉพาะตัว นอกจากนี้ findOrSaveBy* ยังสามารถใช้เพื่อแทนที่เมธอดคงที่แบบกำหนดเอง getColor ได้

หากการสร้าง name เป็นคีย์หลักไม่ใช่ตัวเลือกที่ใช้ได้ คุณสามารถเพิ่มข้อจำกัดตามที่บุคคลเกือบทั้งหมดกล่าวไว้ในความคิดเห็นต่อคำถามได้ :)

class Color {
  String name

  static mapping = {
      //column is optional
      id name: 'name', generator: 'assigned', type: 'string', column: 'NAME' 
  }

  static hasMany = [moods : Mood] //for example

  //DO NOT NEED THIS
  //This can be achieved by findOrSaveBy*
  /*static getColor(String name) {
    def color = Color.findByName(name.toLowerCase())
    color ? color : new Color(name).save(flush:true) 
  }*/

  def beforeValidate() {
    name = name.toLowerCase();
  }
}

ถ้าอย่างนั้นคุณก็ทำได้ดีมาก

def color = Color.findOrSaveByName('verylightgreen')
def mood = new Mood()

color.addToMoods(mood)
color.save()