Program

 private string KonverterMaanedsnr(int nr)
 {
   return CultureInfo.CurrentCulture.DateTimeFormat.GetMonthName(nr);
 }

Hej Michael,

Jeg gør udefra at det er i c# du gerne vil lave denne kode, siden at det er tagged til dette spørgsmål..

Hvis det er c#, så benytter du dig af følgende fremgangsmåde

1) skriv dit metode op

 private string KonverterMaanedsnr(int nr){
 
 }

2) skriv din kode inde i din metode

Herunder, skal du lave nogle if-else sætninger, siden du ved at der kun er 12 måneder, så er den første if-sætning om, tallet er imellem 1 og 12.. Hvis så nr ligger imellem 1 og 12 så går du dine næste if sætninger igennem, der tjekker om det er den første måned eller den anden, tredje eller fjerde måned osv.

i kode ville dette se ud på følgende måde:

 private string KonverterMaanedsnr(int nr)
         {
             if (nr >= 1 && nr <= 12)
             {
                 if (nr == 1)
                 {
                     return "Januar";
                 }
 
             }          
 
             return "Dette nr er ikke en måned";
         }

3) du bør kalde din funktion, når den skal benyttes enten, når en bruger trykker på en knap, hvis du benytter dig af gui, eller når du modtager et bestemt paramter i din konsol

4) dit program er nu færdigt, og du kan nyde resultatet

ps. Jeg skrev ikke alle linjers kode, da jeg regner med at det er fremgangs måde, du gerne ville have

Men, hvis du skulle få lyst til at se, hvordan et evt. GUI kunne se ud så har jeg lige vedhæftet det
Et hurtigt program

Brian: så længe, at du ikke benytter dig af pointers til funktioner i C eller C++, så betyder det ikke noget for afviklingstiden, om man benytter sig af en switch, statement eller en if-else statements. Da begge dele tager O(n) tid at køre igennem, dog ville du have ret hvis du lavede pointers til funktioner i C eller C++ at du ville kunne få din køretid ned på O(1).

Michael: Det er jo meget simpelt, det er bare at lave flere if-sætninger inde i din metode, sådanne så det kommer til at se ud på følgende måde:

 private string KonverterMaanedsnr(int nr)
         {
             if (nr >= 1 && nr <= 12)
             {
                 if (nr == 1)
                 {
                     return "Januar";
                 }
                 else if(nr == 2){
                     return "Febuar";
                 }
                 else if (nr == 3)
                 {
                     return "Marts";
                 }
                 else if (nr == 4)
                 {
                     return "April";
                 }
                 else if (nr == 5)
                 {
                     return "Maj";
                 }
                 else if (nr == 6)
                 {
                     return "Juni";
                 }
                 else if (nr == 7)
                 {
                     return "Juli";
                 }
                 else if (nr == 8)
                 {
                     return "August";
                 }
                 else if (nr == 9)
                 {
                     return "September";
                 }
                 else if (nr == 10)
                 {
                     return "Oktober";
                 }
                 else if (nr == 11)
                 {
                     return "November";
                 }
                 else if (nr == 12)
                 {
                     return "December";
                 }
             }          
 
             return "Dette nr er ikke en måned";
         }

Tænkte det, nok det lille hurtige program, jeg lavede var også visual studio c# kode ... Når man klikker på den eneste knap jeg har i mit program, bliver følgende metode kaldt

 private void button1_Click(object sender, EventArgs e)
         {
             output.Text = KonverterMaanedsnr(Convert.ToInt32(input.Text));
         }

Og det er på denne måde, at jeg benytter mig af KonverterMaanedsnr funktionen

Jeg tænker ikke på afviklingstiden, men på pæn kode. En enum vil trods alt gøre det nemmere at læse koden og det er netop sådan noget en enum skal bruges til. Der er if / else trods alt noget old school.

Michael, husk at der skal også valideres på om det indtastede er numerisk.

Hejsa Michael
Velkommen til udvikleren, og den nye verden af udvikling

Jeg ville foreslå at starte med GUI da det (for mig) er nemmest at starte ud med.

Lav derfor en windows applikation, og træk en textbox ind sammen med en knap som du giver en passende tekst.
Du dobbeltklikker så på knappen, hvorefter der automatisk laves en metode til den.
Vil blot sige at da jeg ikke sidder med en visual studio eller andet udviklerværktøj på denne maskine, er dette taget ud fra hukommelsen - men håber det giver lidt forståelse.

Ganske rigtigt som Kevin siger så ønsker du at lave din metode først. Jeg ville dog løse det lidt anderledes. Faktisk ville jeg nok bruge noget DateTime, men da jeg ikke har en Visual Studio og ikke kan huske syntaksen i hovedet så derfor jeg har valgt denne.

 private string KonverterMaanedsNr(int maanedsNr)
 {
     if (maanedsNr < 0 || maanedsNr > 12)
         return maanedsNr + " er ikke et månedsnr";
 
     string returnMessage = "Det er";
     string[] maaneder = {"Januar", "Februar", "Marts", "April", "Maj", "Juni", "Juli", "August", "September", "Oktober", "November", "December"};
     returnMessage += maaneder[maanedsNr -1].ToString();
     returnMessage += " måned";
 }

Det du ønsker at gøre er at tjekke først om der er indskrevet noget og derefter om det indskrevne er et tal. Da det er en tekstbox kan man jo indskrive enhver ønsket tekst som fx "abcd" eller hvad man nu ønsker.

 private void button1_Click(object sender, EventArgs e)
 {
     string indskrevetText = Textbox1.Text;
     //Fjern alle whitespaces (Trim) og derefter se på længden af teksten
     if (indskrevetText.Trim().Length == 0)
     {
         MessageBox.Show("Du skal indskrive en måned");
         return;
     }
 
     int tal = -1;
     //Undersøg om det indskrevne er et hel-tal
     if (!int.TryParse(indskrevetText, tal))
     {
         MessageBox.Show("Det indskrevne er ikke et tal");
         return;
     }
     string maaned = KonverterMaanedsNr(tal);
     MessageBox.Show(maaned);
 }

Håber det bringer dig lidt tættere på en løsning

Mit bud ville være:

 using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
 
 namespace ConsoleApplication1
 {
     class Program
     {
         static void Main(string[] args)
         {
             int maanedNumerisk;
 
             if (int.TryParse(Console.ReadLine(), out maanedNumerisk) && maanedNumerisk >= 1 && maanedNumerisk <=12)
                 Console.WriteLine((Maaneder)(maanedNumerisk));
             else
                 Console.WriteLine("Måned skal være numerisk og fra 1 til 12!");
         }
 
         enum Maaneder
         {
             Januar = 1,
             Februar = 2,
             Marts = 3,
             April = 4,
             Maj = 5,
             Juni = 6,
             Juli = 7,
             August = 8,
             September = 9,
             Oktober = 10,
             November = 11,
             December = 12
         } 
 
     }
 }

Egentlig imod et princip omkring, at levere færdige skoleopgaver. Det lærer man ikke så meget af Michael ;-)

I stedet for if else som forslået af Kevin, vil jeg foreslå en enumerator (enum) og hvis du ikke vil det, vil en swith/case være bedre end en if/else

Hej Kevin

Tak for svaret. Jeg vil meget gerne have det hele, da jeg er newbie

Hej Brian

Hvordan skal jeg sammensætte koden?

Og det er forresten visual studio og C# kode

Tænker bare at en enum, er lidt et overkill for et lille projekt sådanne som dette. Men, hvis det var et større projekt, ville jeg nok også lave en enum til dette.. Det kan godt være, at if-else er lidt old school; men det er oftest nemmere for begyndere at forstå

så betyder det ikke noget for afviklingstiden, om man benytter sig af en switch, statement eller en if-else statements. Da begge dele tager O(n) tid at køre igennem

Big O notation har intet med afviklingstiden at gøre, men beskriver blot kompleksiteten af en algorithme.


Anyway... et par alternativer der lidt kortere:

 static string KonverterMaanedsnr(int n)
 {
     return
           n == 1 ? "Januar"       : n == 2 ? "Febuar"
         : n == 3 ? "Marts"        : n == 4 ? "April"
         : n == 5 ? "Maj"          : n == 6 ? "Juni"
         : n == 7 ? "Juli"         : n == 8 ? "August"
         : n == 9 ? "September"    : n == 10 ? "Oktober"
         : n == 11 ? "November"    : n == 12 ? "December"
 
         : "Dette nr er ikke en måned";
 }

 static string KonverterMaanedsnr(int n)
 {
     const string noMatch = "Dette nr er ikke en måned";
 
     var months = new string[12] 
     { 
         "Januar", "Febuar", "Marts", 
         "April", "Maj", "Juni", 
         "Juli", "August", "September", 
         "Oktober", "November", "December" 
     };
 
     return n >= 1 && n <= 12 ? months[n - 1] : noMatch;
 }