이벤트가 null이되는 이유는 무엇입니까? (객체 참조가 객체의 인스턴스로 설정되지 않음)

Question

버튼을 클릭하면 프로세스를 실행하는 클래스 b의 인스턴스를 만들 수 있도록 버튼, 레이블 및 진행률 막대가있는 양식이 있습니다. 프로세스가 완료되면 EventHandler를 호출하여 메인 폼의 레이블에 "done"을 표시합니다!

이렇게하려면 대리인 (SetStatus)의 이벤트 (SetStatusEvent)를 만들었습니다. 그리고이 이벤트 핸들러 (usbforProcessExited) 외부에서이 이벤트를 호출 할 때 그것을 잘 보이지만 내가 usbforProcessExited에서 호출 할 때 오류 제공 -

object reference not set to an instance of an object 

기본 양식을

public partial class main : Form 
{ 
    b rsSet = new b(); 

    public main() 
    { 
     InitializeComponent(); 
     rsSet.SetStatusEvent += new RemoteS.SetStatus(updateStatus); 
    } 

    private void button1_Click(object sender, EventArgs e) 
    { 
     rsSet.FormatUSB(); 
    } 

    private delegate void UpdateStatus(int i, string str, Color clr); 

    private void SetStatus(int i, string str, Color clr) 
    { 
     this.progressBar1.Value = i; 
     this.lbl_status.ForeColor = clr; 
     this.lbl_status.Text = str; 
    } 

    private void updateStatus(int i, String msg, Color color) 
    { 
     object[] p = GetInokerPara(i, msg, color); 
     BeginInvoke(new UpdateStatus(SetStatus), p); 
    } 

    private object[] GetInokerPara(int progress, string msg, Color color) 
    { 
     object[] para = new object[3]; 
     para[0] = progress; 
     para[1] = msg; 
     para[2] = color; 

     return para; 
    } 
} 

클래스 B

class b 
{ 
    public delegate void SetStatus(int i, string msg, Color color); 
    public event SetStatus SetStatusEvent; 

    System.Diagnostics.Process usbfor = new System.Diagnostics.Process(); 

    public void FormatUSB() 
    { 

     usbfor.StartInfo.FileName = @"usbformat.bat"; 
     usbfor.EnableRaisingEvents = true; 
     usbfor.Exited += new EventHandler(usbforProcessExited); 
     usbfor.Start(); 
    } 

    public void usbforProcessExited(object sender, EventArgs f) 
    { 
     SetStatusEvent(100, "DONE", Color.Green); //ERROR HERE! (object reference not set to an instance of an object 
    } 
} 

어디에서 문제가 발생합니까?

Answer 1

당신은 경쟁 조건을 가지고있다.

SetStatusEvent을 구독하면 usbfor.Start()으로 전화해야합니다.

관련된 문제는 이벤트가 다른 스레드에서 실행된다는 것입니다. 이벤트 처리기가 Invoke/BeginInvoke을 수동으로 호출하지 않고 양식을 수정할 수 있도록 프로세스를 시작하기 전에 rsSet.SynchronizingObject을 설정해야합니다.가입자가없는 경우

Answer 2

아무도 아직 구독하지 않은 경우 이벤트는 null입니다. 는 그래서 같은 null 평등 제어하는 ​​좋은 방법입니다 :

외부 가 잘 작동 이유를 가지고,의

public void usbforProcessExited(object sender, EventArgs f) 
    { 
     if(SetStatusEvent!=null) 
      SetStatusEvent(100, "DONE", Color.Green); 
    } 

,이 라인 :

rsSet.SetStatusEvent += new RemoteS.SetStatus(updateStatus); 

때문에 가입과의 initilizaton 행사.

에서으로 전화하면 구독이 없기 때문에 이벤트는 null입니다.

public void usbforProcessExited(object sender, EventArgs f) 
    { 
     var ev = SetStatusEvent; //[1] 
     if(ev!=null) //[2] 
      ev(100, "DONE", Color.Green); 
    } 

그래서 짝수 라인 [사이에서 CLR에서 원자 그 할당 작업 아이오와를, 기억의 이벤트에 null 참조 확인을 처리하는 더 많은 스레드 안전한 접근을 제공 할 의견에 따라

편집

1] 및 [2] 다른 사람이 이벤트를 null로 재설정하면 ev은 여전히 ​​유효하며 코드는 충돌하지 않고 실행됩니다. 이것이 바람직한 동작인지 아니면 구체적인 케이스에 따라 다르므로, 이것은 thread-safe 방식으로 이벤트에 대한 null 참조 제어를 관리하는 또 다른 옵션입니다.

usbforProcessExited

이 b의 생성자에 가입됩니다 그리고 당신이 rsSet.SetStatusEvent += new RemoteS.SetStatus(updateStatus)를 호출하기 전에 호출 될 수 있습니다

Answer 3

이벤트는, null입니다. (더미 가입자가 아무것도하지 않고) 선언 할 때

1. 이벤트를 초기화 :

   public event SetStatus SetStatusEvent = delegate { }; 
   

2. 스레드 안전한 방법으로 (올리기 전에 널위한 이벤트를 확인

   는 두 가지 해결책이 있습니다) :

   public void usbforProcessExited(object sender, EventArgs f) 
   { 
       SetStatus setStatus = SetStatusEvent; 
       if (setStatus != null) 
       { 
        setStatus(100, "DONE", Color.Green); 
       } 
   } 
   

Answer 4

존 소총은 # 6.0 c를 당신은 또한 사용할 수에서 저를 가르쳤다 :

SetStatusEvent?.Invoke(100, "DONE", Color.Green);;