データベースへの接続時にNull参照例外

Question

DataBaseServiceクラスインスタンス経由でUserExistsメソッドを呼び出すと、すぐにデータベースに接続できません。 SqlConnectionに接続文字列を指定したにもかかわらず、インスタンスはnullです。

using System; 
using System.Collections.Generic; 
using System.Data; 
using System.Data.SqlClient; 
using System.Windows.Forms; 

namespace Somename 
{ 
    public class DataBaseService 
    { 
     // Setting up the Database Services 
     private SqlConnection myConnection; 
     private const string ConnectionString = "Data Source=Test-LAPTOP\\MSSQLSERVER1;Initial Catalog=myDatabaseTest;Integrated Security=True"; 

     private static readonly DataBaseService _instance; 

     static DataBaseService() 
     { 
      _instance = new DataBaseService(); 
      Initilize(); 
     } 

     public static DataBaseService Instance; 
     private static void Initilize() 
     { 
      try 
      { 
       Instance.myConnection = new SqlConnection(ConnectionString); 
       Instance.myConnection.Open(); 
      } 
      catch (Exception ex) 
      { 
       MessageBox.Show("Error while connecting to DB: " + ex.Message); 
       return; 
      } 
     } 

     public bool UserExists(string username) 
     { 
      var myQuery = "Select * from UserProfile where userId ="+username+""; 
      SqlCommand sqlCmd = new SqlCommand(myQuery, Instance.myConnection); 
      sqlCmd.CommandType = CommandType.Text; 
      //sqlCmd.CommandText = "Select * from Customers"; 
      SqlDataAdapter sqlDataAdap = new SqlDataAdapter(sqlCmd); 

      DataTable dtRecord = new DataTable(); 
      sqlDataAdap.Fill(dtRecord); 

      if (dtRecord == null) 
      { 
       return false; 

      } 
      else 
      { 
       return true; 
      } 
     } 
    } 
} 

Answer 1

あなたはあなたのコード、_instanceとInstanceにDataBaseServiceの2つのインスタンスを持っています。静的コンストラクタで_instanceを初期化しますが、Instanceを決して初期化しません。

あなたはおそらくどこでもあなたが現在Instanceを使用している、と完全にInstanceを取り除く_instanceを使用したい：

... 

private static readonly DataBaseService _instance; 

static DataBaseService() 
{ 
    _instance = new DataBaseService(); 
    Initilize(); 
} 

private static void Initilize() 
{ 
    try 
    { 
     _instance.myConnection = new SqlConnection(ConnectionString); 
     _instance.myConnection.Open(); 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show("Error while connecting to DB: " + ex.Message); 
     return; 
    } 
} 

... 

Answer 2

あなたはこのパターンに従うことをお勧めします：

private static DataBaseService _instance; 
// public property 
public static DataBaseService Instance 
{ 
    get 
    { 
     return _instance ?? (_instance = new DataBaseService()); 
    } 
}