"ส่วนตัว" และ "สาธารณะ" ในองค์ประกอบเชิงมุม

ถ้าฉันไม่ได้เพิ่มส่วนตัวก่อนfoo, loadBarและtextผมเชื่อว่าพวกเขาจะมีประชาชนไปโดยปริยาย

export class RandomComponent {
  @Input() foo: string;
  @Output() loadBar = new EventEmitter();
  text: string;
}

มีกรณีการใช้งานเมื่ออยู่publicในส่วนประกอบหรือไม่?

ด้วยเหตุผลด้านการห่อหุ้ม / ความปลอดภัยฉันควรเพิ่มprivateสิ่งเหล่านี้ทั้งหมดไว้ด้านล่างหรือไม่?

export class RandomComponent {
  @Input() private foo: string;
  @Output() private loadBar = new EventEmitter();
  private text: string;
}

ขอบคุณ

มีหลายสิ่งที่จะพูดในการตอบคำถามนี้นี่คือความคิดแรกที่เข้ามาในความคิดของฉัน:

ก่อนอื่นโปรดทราบว่าprivateเป็นเพียงโครงสร้างเวลาคอมไพล์ - ไม่สามารถบังคับใช้ในรันไทม์ได้ (ดูที่นี่และที่นี่สำหรับการอภิปรายที่เกี่ยวข้อง) ด้วยเหตุนี้โปรดตัดทอนความคิดที่privateว่าเป็นประโยชน์ในทางใดทางหนึ่งเพื่อความปลอดภัย นั่นไม่ใช่สิ่งที่เกี่ยวกับ

มันเป็นเรื่องเกี่ยวกับการห่อหุ้มและเมื่อคุณมีฟิลด์หรือวิธีการในส่วนประกอบของคุณที่คุณต้องการห่อหุ้มมันทำให้ชัดเจนว่าไม่ควรเข้าถึงจากที่อื่นคุณควรทำให้มันเป็นอย่างนั้นprivateนั่นคือสิ่งที่privateมีไว้สำหรับ: เป็นการบ่งบอกถึงเจตนาของคุณว่าไม่ควรแตะต้องสิ่งใดก็ตามจากภายนอกชั้นเรียน

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

ทั้งหมดนี้ใช้ได้กับ typescript โดยทั่วไป โดยเฉพาะใน Angular มีกรณีการใช้งานที่ถูกต้องอย่างแน่นอนสำหรับการมีสมาชิกสาธารณะในคลาสส่วนประกอบตัวอย่างเช่นเมื่อใช้รูปแบบคอนเทนเนอร์ / ส่วนประกอบ (หรือที่เรียกว่าสมาร์ท / ใบ้ ) โดยมีเด็ก "ใบ้" ฉีดพ่อแม่ "ฉลาด" ผ่านการฉีดตัวสร้าง สิ่งสำคัญอย่างยิ่งในการสื่อสารถึงเจตนาของคุณเกี่ยวกับสิ่งที่สมาชิกในผู้ปกครองควรและไม่ควรสัมผัสกับเด็ก ๆ มิฉะนั้นอย่าแปลกใจเมื่อคุณจับเด็กใบ้พวกนั้นมาหลอกอยู่ในตู้เหล้าของพ่อแม่

ดังนั้นคำตอบของฉันสำหรับคำถามของคุณ:

ฉันควรเพิ่มความเป็นส่วนตัวสำหรับทุกคนเช่นด้านล่างนี้หรือไม่

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

@drewmoore ให้คำตอบที่ดีในเรื่องส่วนตัว / สาธารณะนั้นขึ้นอยู่กับเจตนา แต่มีอีกสองสามสิ่งที่ควรพิจารณาเมื่อใช้ค่าส่วนตัวที่ฉีดเข้าไป:

• ทั้ง typescript ( noUnusedLocals ) และ tslint ( no-unused-variable ) จะรายงานข้อผิดพลาดหากคุณใช้@Import() private fooหรือconstructor(private foo) {}และใช้เฉพาะfooในเทมเพลตของคุณเท่านั้น
• ทอท. เคยชิน :

หากเราต้องการปล่อย TypeScript เป็นผลลัพธ์ของกระบวนการคอมไพล์ AoT เราต้องแน่ใจว่าเราเข้าถึงเฉพาะฟิลด์สาธารณะในเทมเพลตของส่วนประกอบของเรา **

• มันเป็นเรื่องที่ถือว่าเป็นรูปแบบการป้องกัน