C# Kort kod, behöver hjälp

Skoluppgift:

Skriv en metod som har:
enbart access från den egna klasssen
ett heltal som inparameter
en textsträng som returvärde.
returnera en valfri text som du slagit ihop med inparametern

http://i.imgur.com/KdgNSnx.png

Jag är nybörjare och har klurat ett tag nu. Jag får inga error messages men när jag kör programmet skrivs ingenting ut.

Extremt tacksam för svar, gärna en förklaring istället för ett "facit"
Tacksam för svar

Jag tycker att man antingen gör som han gjort, eller använder stringbuilder. Att köra string.format är både "svårt" att läsa, och svårt att underhålla (Tänk om du vill lägga till/ta bort en grej).

När man gör det så kort som han har gjort tycker jag det funkar utmärkt, men ska man dra igenom någon loop och göra något mer avancerat, t.ex. concaternera några hundra stränger, ja, då funkar stringbuilder bra (och är läsbarare).

Det måste ju upp till bra många tusen repetitioner innan man ser någon som helst prestandaskillnad, spara in på gungorna istället för karusellen

Jag tycker att man antingen gör som han gjort, eller använder stringbuilder. Att köra string.format är både "svårt" att läsa, och svårt att underhålla (Tänk om du vill lägga till/ta bort en grej).

När man gör det så kort som han har gjort tycker jag det funkar utmärkt, men ska man dra igenom någon loop och göra något mer avancerat, t.ex. concaternera några hundra stränger, ja, då funkar stringbuilder bra (och är läsbarare).

Det måste ju upp till bra många tusen repetitioner innan man ser någon som helst prestandaskillnad, spara in på gungorna istället för karusellen

Man kan använda sig av "Replace" men "stringbuilder" är mycket snabbare.

Källa på det?

Senast jag testade stringbuilder krävs det en hel del manipulation av strängar för att det skall vara värt det av prestandaskäl. Är det inte en större mängd sting-operationer / manipulering som skall göras så är det snabbare att undvika stringbuilder och manipulera dem själv.

Replace methods benchmark, 20 chars

Method A - String Replace: 5.60 seconds
Method B - StringBuilder Replace: 0.62 seconds [faster]

Replace methods benchmark, 1000 chars

Method A - String Replace: 21.80 seconds
Method B - StringBuilder Replace 4.89 seconds [faster]

Just det, det är när man gör över 2000 då är stringbuilder snabbare. källa http://www.dotnetperls.com/replace

Detta är ju dock läsbarare, eftersom det är så man läser.

Salesman + " har sålt " + Produkt + " till kunden " + CustomerName

I ditt exempel, som är mycket vanligt, så vet man ju inte vad {0} är för någonting utan att flytta blicken hela vägen till slutet på raden, räkna antalet kommatecken och sen flytta tillbaka blicken. Det är inte lika irriterande om man bara har 3 parametrar, men om man har 8, eller 11 etc blir det riktigt jobbigt. Särskilt om man har lagt till en ny variablel någon gång och inte orkat uppdatera alla siffrorna, så att det helt plötsligt står en 9:a i mitten:

string.format("Nu skall vi jämföra värde {0} med värde {1} som borde bli värde {2} för att se om {3} lyser i {4} till och med {9} för {5} som inte har lika många {6} beroende {7} {8} ", x,y,z, solen, vaderstreck, datum, tidspann, vad, oj, da)

Skriv en liten kodsnutt som beskriver vilka operationer dina ögon behöver utföra för att tolka string.Format jämfört med + concat så är svaret ganska tydligt. Bara för att man har gjort på ett visst sätt väldigt länge betyder ju inte att det är bäst.

String Interpolation
Prior to C# 6.0, you could concatenate two or more strings together in
one of the following ways:

string firstName = "Michael";
string lastName = "Crump";
WriteLine("Name : " + firstName + " " + lastName);
WriteLine(string.Format("Name : {0} {1}", firstName, lastName));

In C# 6.0, there is a cleaner format, as shown in the first WriteLine call.
Also, you can place expressions directly in the string literal to evaluate
an expression:

string firstName = "Michael";
string lastName = "Crump";
int orderNumber = 250000;

WriteLine("Name : \{firstName} \{lastName}");

WriteLine("Name : \{firstName} \{lastName}\nDiscount :\{orderNumber == 250000 ? " You
get 25% off your order!" : ""}");

In this sample, the console returns the following information, as it
evaluates the orderNumber is equal to 250,000; otherwise won’t
print anything