初期化配列

Question

は、私が

完全に働いた

string[] matchingFiles = Directory.GetFiles(FilePath, FileNamePattern); 

それを行うためのこの方法を使用していたが、今私はDirectory.GetFilesを囲みたいこと/試しに呼び出しますキャッチブロック、しかし、私はまた、文字列配列の宣言を持っていないので、それは正しい範囲でtryブロックの外でそれを使用することはできません。私がしようとした場合でも、この：私は、文字列配列を初期化していない

string[] matchingActiveLogFiles; 
      try 
      { 
       matchingFiles = Directory.GetFiles(FilePath, FileNamePattern); 
      } 
      catch (Exception ex) 
      { 
       //logerror 
      } 

は、私はエラーを持っています。だから私はtryブロックの外側に文字列配列を宣言する必要がありますこの状況で何がベストプラクティスですか？そしてもしそうなら、どのように？

Answer 1

これはあなたの配列を初期化します：

string[] matchingActiveLogFiles = {}; 
      try 
      { 
       matchingFiles = Directory.GetFiles(FilePath, FileNamePattern); 
      } 
      catch (Exception ex) 
      { 
       //logerror 
      } 

しかし、私は思ったんだけど、あなたはどのようなエラーを取得していますか？初期化されていない配列であっても、上記のコードは動作するはずです。また、1行目に「matchingActiveLogFiles」、4行目に「matchingFiles」があることに気付きました。おそらくそれはあなたの問題ですか？

Answer 2

私は、一般的にアウトのparamsを持つメソッドを嫌いますが、これはTry方法のために良い候補のように思える：

bool TryGetMatchingLogFiles(out string[] matchingFiles) 
{ 
    matchingFiles = null; 
    try 
    { 
    matchingFiles = Directory.GetFiles(FilePath, FileNamePattern); 
    return true; 
    } 
    catch (Exception ex) 
    { 
    //logerror 
    return false; 
    } 
} 

使用法：

string[] matchingActiveLogFiles; 
if (TryGetMatchingLogFiles(out matchingActiveLogFiles)) 
{ 
    // Use matchingActiveLogFiles here 
} 

または代わりに、単にnullにあなたの変数を初期化：

string[] matchingActiveLogFiles = null; 
try ... 

Answer 3

最初に初期化：

string[] matchingActiveLogFiles = new string[0]; 

Answer 4

問題はあなたの名前です。あなたはmatchingActiveLogFilesを定義していますが、matchingFilesを割り当てています。

Answer 5

名は、文字列配列のために異なっている一つは、他のあなたは、その変数が必要とされているスコープで変数を宣言する必要があり matchingFiles

string[] matchingActiveLogFiles; 
    try 
    { 
     matchingActiveLogFiles = Directory.GetFiles(FilePath, FileNamePattern); 
    } 
    catch (Exception ex) 
    { 
       //logerror 
    } 

Answer 6

あるmatchingActiveLogFilesです。

• 変数がtryブロックに必要な場合は、そこに配置します。
• tryブロックの外側に必要な場合は、コードがファイルの内容を取得できない場合はどうしたらよいでしょうか？それをエラー時に設定してください。

Answer 7

あなたは

string[] matchingActiveLogFiles = {}; 
this is what is called a dynamic array it's totally functional and I've had ZERO issues with declaring arrays this way.. 

List<string> matchingFiles= new List<string>(); 
try 
{ 
    matchingFiles.Add(Directory.GetFiles(FilePath, FileNamePattern)); 
    matchingActiveLogFiles = matchingFiles.ToArray(); 
} 
catch (Exception ex) 
{ 
    //logerror 
}