このコードで使用される変数を減らすにはどうすればよいですか？

Question

私は以下のコードで使用される変数の量を減らそうとしています。
理想的には、変数を再利用して、LatestRipMeVersion string[]変数を作成しないでください。
Skip(1)とFirst()を行うために配列を作成する必要がありますか？
最終的にLatestRipMeの値に達することはできますか？

private void RipMeGetLatestVersion_Process() 
{ 
    //Get the .json file and save it in LatestRipMe string 
    LatestRipMe = ClientRipMe.DownloadString("http://www.rarchives.com/ripme.json"); 

    //Create an array from the previously saved string, and skip the first line 
    LatestRipMeVersion = LatestRipMe.Split(Environment.NewLine.ToCharArray()).Skip(1).ToArray(); 

    //Put the array back in a string, and select only the first line 
    LatestRipMe = LatestRipMeVersion.First(); 

    //The characters which need to be removed 
    char[] LatestRipMeTrim = { ' ', ' ', '"', 'l', 'a', 't', 'e', 's', 't', 'V', 'e', 'r', 's', 'i', 'o', 'n', '"', ' ', ':', ' ', '"', '"', ',' }; 

    //Trim the above declared characters 
    LatestRipMe = LatestRipMe.Trim(LatestRipMeTrim); 

    //Show the remaining string content in the TextBlock 
    LatestRipMeVersionText.Text = "Latest RipMe version: " + LatestRipMe; 
} 

Answer 1

あなたは置き換えることができ

LatestRipMe = LatestRipMe.Split(Environment.NewLine.ToCharArray())[1]; 

あるいは

LatestRipMe = LatestRipMe 
      .Split(Environment.NewLine.ToCharArray())[1] 
      .Trim(new char[] { ' ', 'a', 'b' }); // etc ... 

意見に

//Create an array from the previously saved string, and skip the first line 
    LatestRipMeVersion = LatestRipMe.Split(Environment.NewLine.ToCharArray()).Skip(1).ToArray(); 

    //Put the array back in a string, and select only the first line 
    LatestRipMe = LatestRipMeVersion.First(); 

：賢い
スタイルは、それが、醜いデバッグすることは困難であるとサイトが期待しているものを返さないという例外をスローします（インターネットに接続していないなど）。あなたの明確で文書化されたステップをステップコードで置き換えるという願望を理解していないのは、プログラミングの課題がなければ、変数の数を減らすことだけです。

Answer 2

private void RipMeGetLatestVersion_Process() 
{ 
    LatestRipMe = ClientRipMe.DownloadString("http://www.rarchives.com/ripme.json"); 

    LatestRipMeVersion = LatestRipMe.Split(Environment.NewLine.ToCharArray())[1]; 

    LatestRipMe = LatestRipMe.Trim(" \"latestVersion\" : \"\",".ToCharArray()); 

    LatestRipMeVersionText.Text = "Latest RipMe version: " + LatestRipMe; 
} 

Answer 3

これは、あなたが望むものを与えるはずですが、このコードで間違っている可能性があることに注意してください。

private void RipMeGetLatestVersion_Process() 
    { 
     LatestRipMeVersion = ClientRipMe 
      .DownloadString("http://www.rarchives.com/ripme.json") 
      .Split(Environment.NewLine.ToCharArray()) 
      .Skip(1) 
      .ToArray(); 

     LatestRipMe = LatestRipMeVersion.First() 
      .Trim(' ', ' ', '"', 'l', 'a', 't', 'e', 's', 't', 'V', 'e', 
      'r', 's', 'i', 'o', 'n', '"', ' ', ':', ' ', '"', '"', ','); 
     LatestRipMeVersionText.Text = "Latest RipMe version: " + LatestRipMe; 
    } 

Answer 4

多くの人が既にあなたに言ったように、このコードには多すぎることがあります。 JSONパーサーを使用し、コードの命名と構造化にいくつかの標準的な規則に従ってください。私はあなたのコードがあなたがしたいことをまだしていないと感じています。下記の私のバージョンを見てください。これは、あなたが正しく欲しいものを行います（jsonの属性latestVersionの値を取得します）。これは良いコードですが、あなたの持っているものよりもはるかに優れています。