C# Rectangle creux utilisant Enumerable.Repeat()

Question

J'ai reçu l'affectation de créer un rectangle vide en utilisant Enumerable.Repeat au lieu de boucles imbriquées. Je l'ai fait comme ça.

string str; 
int rows = 5; 
int cols = 15; 

for (int i = 1; i <= rows; i++) 
{ 
    if (i == 1 || i == rows) 
    { 
     str = string.Concat(Enumerable.Repeat("*", cols)); 
     Console.WriteLine(str); 
    } 
    else 
    { 
     str = string.Concat(Enumerable.Repeat(" ", cols - 2)); 
     str = string.Concat("*", str, "*"); 
     Console.WriteLine(str); 
    } 

Il fonctionne très bien, mais la partie creuse, je l'ai conçu ne semble pas LE CHEMIN aller si vous savez ce que je veux dire ... Quelle serait la façon la plus efficace d'aller? Exclusion de l'utilisation de StringBuilder.

Answer 1

Vous pouvez mettre en cache les lignes et les Join:

string top = new string('*', cols); 
string body = "*" + new string(' ', cols - 2) + "*"; 

string result = string.Join(Environment.NewLine, 
    top, 
    string.Join(Environment.NewLine, Enumerable 
    .Repeat(body, rows - 2)), 
    top); 

Consol.Write(result); 

Answer 2

Dans ce cas, il existe un constructeur de chaîne qui constitue un meilleur choix. Voici juste un segment de l'original pour démontrer:

if (i == 1 || i == rows) 
{ 
    str = string.Concat(new string('*', cols)); 
    Console.WriteLine(str); 
} 

De plus, vous pouvez simplifier le tout en plaçant les rangées supérieure et inférieure à l'extérieur de la boucle:

int rows = 5; 
int cols = 15; 

Console.WriteLine(new string('*', cols); 
foreach(string line in Enumerable.Repeat("*".Concat(new string(' ', cols-2)).Concat("*"), rows-2)) 
{ 
     Console.WriteLine(line); 
} 
Console.WriteLine(new string('*', cols);