ループでより効率的なConsole.Writelineブロックを作成するのに助けが必要

Question

ちょっと私はちょっとしたエクササイズのように、誰かが応募する必要がある仕事、場所、連絡方法を書き出す小さなコンソールアプリケーションを作った。 私のコードを見ると、Console.WriteLineステートメントを書き出すことは効率的であるのを助けるために関数を作ったことを考えると非常に効率的です。

私は、これはループを使用しますが、それはループを別の変数で各時間を作る方法がわからないイムことであろう達成するための最善の方法を考えていた（例えばphn1、phn2変数）

みんなありがとう！

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 
using System.Threading.Tasks; 

namespace PlacestoApplyfor 
{ 
    class Program 
    { 
     static void Main(string[] args) 
     { 
      //Displays the actual program in descending order. 
      //Used functions as a faster method to change variable names on the fly 
      //Whilst keeping its format clean and in descending order 
      Console.WriteLine("--Name of Places--"); 
      ApplyFor("EdwayApps" ,"Appxperts","Genie App Studio","Appster"); 
      Console.WriteLine("--Name of Locations--"); 
      LocationsInOrder("3/424 St kilda", "101/27 Little Collins St", "Contact Online only", "2/377 LonsDale ST"); 
      Console.WriteLine("--Companies Numbers-- "); 
      PhoneNumbers(043990976,1300939225,0421336722,1800709291); 
     } 

     static void ApplyFor(string num1, string num2, string num3, string num4) { 
     //Literally just the job names 
     Console.WriteLine(num1); 
     Console.WriteLine(num2); 
     Console.WriteLine(num3); 
     Console.WriteLine(num4); 
     } 

     // Job Locations 

     static void LocationsInOrder(string loc1, string loc2, string loc3, string loc4) { 
     Console.WriteLine(loc1); 
     Console.WriteLine(loc2); 
     Console.WriteLine(loc3); 
     Console.WriteLine(loc4); 
     } 

     //Contact - references 

     static void PhoneNumbers(int phn1, int phn2, int phn3, int phn4) { 
     Console.WriteLine(phn1); 
     Console.WriteLine(phn2); 
     Console.WriteLine(phn3); 
     Console.WriteLine(phn4); 

     } 
    } 
} 

Answer 1

paramsキーワードはあなたの友達です：

static void PhoneNumbers(params int[] phn) 
{ 
    foreach(var num in phn) 
     Console.WriteLine(num); 
} 

それからちょうど行うことができます。でも

PhoneNumbers(1,2,3,4,5,6,7,8,9); 

または配列を渡す：

PhoneNumbers(new int[] {1,2,3,4,5,6,7,8,9}); 

Answer 2

なぜそんなにconsole.wを書いているあなたその代わりのriteline（） は、クラスに次のコード

を書く

static void Main(string[] args) 
    { 
     Console.WriteLine("--Name of Places--"); 
     ApplyFor("EdwayApps", "Appxperts", "Genie App Studio", "Appster"); 
     Console.WriteLine("--Name of Locations--"); 
     LocationsInOrder("3/424 St kilda", "101/27 Little Collins St", "Contact Online only", "2/377 LonsDale ST"); 
     Console.WriteLine("--Companies Numbers-- "); 
     PhoneNumbers(043990976, 1300939225, 0421336722, 1800709291); 
     Console.ReadLine(); 
    } 
    static void ApplyFor(string num1, string num2, string num3, string num4) 
    { 
     //Literally just the job names 
     Console.WriteLine(num1 + "\n"+num2 + "\n" + num3 + "\n " + num4); 
    } 
    // Job Locations 
    static void LocationsInOrder(string loc1, string loc2, string loc3, string loc4) 
    { 
     Console.WriteLine(loc1 + "\n" + loc2 + "\n" + loc3 + "\n " + loc4); 
    } 
    //Contact - references 
    static void PhoneNumbers(int phn1, int phn2, int phn3, int phn4) 
    { 
     Console.WriteLine(phn1 + "\n" + phn2 + "\n" + phn3 + "\n " + phn4 +"\n"); 
    } 

Answer 3

あなたは「すべての行を書き、改行用セパレータを構成するために一つだけの機能で行うことができます

\ n "または"、 "をカンマで区切って1行に入力します。 "トピック"を印刷機能の最初のパラメータとして追加して、 " - トピック - "記号内に展開することもできます。 paramsを使用して

static void Main(string[] args) 
{ 
    //Displays the actual program in descending order. 
    //Used functions as a faster method to change variable names on the fly 
    //Whilst keeping its format clean and in descending order 
    Print("Name of Places", "EdwayApps", "Appxperts", "Genie App Studio", "Appster"); 
    Print("Name of Locations", "3/424 St kilda", "101/27 Little Collins St", "Contact Online only", "2/377 LonsDale ST"); 
    Print("Companies Numbers", 043990976, 1300939225, 0421336722, 1800709291); 

    Console.ReadKey(); 
} 

static void Print(string topic, params object[] array) 
{ 
    Console.WriteLine("-- {0} -- \n\n {1}\n", topic, string.Join("\n ", array)); 
} 

Answer 4

はロイドが言ったように涼しいですが、あなたは）Console.WriteLineをを（最適化しようとする場合には、そのために何度も.WriteLineを呼び出すループをしないことができます。 1行

static void PhoneNumbers(params int[] phn) 
{ 
    Console.WriteLine(String.Join(" ", phn)); 
} 

か、値の固定数で1行のすべてのそれらの値を書きたい場合には、方法

static void PhoneNumbers(params int[] phn) 
{ 
    Console.WriteLine(String.Join(Environment.NewLine, phn)); 
} 

同じことばかりスペース区切りString.Formatを見て、String.Joinちょうどこれを行う

static void PhoneNumbers(params int[] phn) 
{ 
    Console.WriteLine(String.Format("{0} {1} {2} {3}", phn)); 
}