ToAscii extension metode?

Hvorfor går du ikke blot således:

eller lidt kortere:

Der var ihvertfald det nemmeste omend det ikke er saa effektivt med mange replacements.

Funktionsnavnet er ihvertfald helt forkert, ved ToAscii forventer man at få ascii retur, men får istedet en string hvor Å er replacet med AA.

Hvad er det du gerne vil, måske der er en langt bedre løsning :)

Hej med jer

tak for jeres forslag. Metode navnet er også ændret siden til FromAscii :-). Nogen bedre foreslag....

Det er til at lave pæne URL'er med og lave æøå om .

Jeg har en StringExtension klasse, hvor jeg har en metode som virker, da den løber en char c igennem og laver Å om til Aa.

Eks.:

public static string ToAscii(this string s, string replaceOldChar, string replaceNewChar, bool isToLower)
        {
            StringBuilder sb = new StringBuilder();
            foreach (char c in s)
            {
                SpecialCharList(c, sb);

                if (replaceOldChar.Length > 0 && replaceNewChar.Length > 0)
                    sb.Replace(replaceOldChar, replaceNewChar);
            }
            if (isToLower)
                return sb.ToString().ToLower();

            return sb.ToString();
        }

public static void SpecialCharList(char c, StringBuilder sb)
        {
            switch (c)
            {
                case 'Å':
                    sb.Append("Aa");
                    break;
                case 'å':
                    sb.Append("aa");
                    break;
.........................

Den virker fint!

Så vil jeg lave en FromAscii, som laver Aa om til Å osv.

Men da char kun tager én karakter, er jeg lidt på den. Kan ikke finde en løsning på det som virker ihvertfald.

Kan jeg bruge noget af det jeg har lavet?

Det har ikke så meget med ASCII at gøre, så navnet er stadig lidt underligt.
Anyway, så er der jo princippet med single responsibility 8http://www.remondo.net/solid-principles-csharp-single-responsibility/), nu ved jeg godt at den hører mere klasser til en funktioner, men jeg synes nu alligevel det er lidt overkill at lave  if (isToLower)
                return sb.ToString().ToLower();

når du altid har muligheden for dette efter du har kaldt funktionen. Personligt synes jeg det er irriterende at kalde en funktion som kan alle mulige ting og som er bloated med parameter input.

Men hvorfor bruger du ikke bare .Replace som mbm2007 foreslår?

Hmmm... Janus_007 min ToAscii metode kunne måske blive Refactored lidt, men den gør præcis kun det jeg vil have den til.

Hvordan ville du Refactor den evt.?

Jeg har tilføjet et felt i DB tabel der hedder LoweredUriTopic.
Den indeholder de filtrede strenge til url. Eks.:

Da jeg fyldte år i søndags

bliver til:

da-jeg-fyldte-aar-i-soendags

Så kan jeg finde TitleID ude fra LoweredUriTopic.

Er det ikke den bedste måde at lave det på?

Hvad med den helt simple:

    public class ASCIITransscription2
    {
        private static readonly string[] unic = { "Æ", "Ø", "Å", "æ", "ø", "å" };
        private static readonly string[] dkeq = { "Ae", "Oe", "Aa", "ae", "oe", "aa" };
        public static string ToASCII(string s)
        {
            string res = s;
            for(int i = 0; i < unic.Length; i++)
            {
                res = res.Replace(unic[i], dkeq[i]);
            }
            return res;
        }
        public static string FromASCII(string s)
        {
            string res = s;
            for(int i = 0; i < dkeq.Length; i++)
            {
                res = res.Replace(dkeq[i], unic[i]);
            }
            return res;
        }
    }

?

Jeg ville nu ikke lave så meget i en funktion, men splitte det lidt mere op.

Anyway...
Men nu når du specifikt spørger til
"Da jeg fyldte år i søndags"
og det hele skal lowercases :)

Så ville jeg løse det sådan her:

var result = ReplaceDanishChars("Da jeg fyldte år i søndags");

string ReplaceDanishChars(string s)
{
  return s.ToLower()
              .Replace("æ", "ae")
              .Replace("å", "aa")
              .Replace("ø", "oe")
              .Replace(" ", "-");
}

Men i det hele taget er det jo meget svært sådan at skrive en funktion uden at kende den eksakte anvendelse. Der er flere måder det kan løses på og egentlig ikke noget "rigtig", det kommer an på så meget :)

Hehe først spørger du til hvordan man i en given streng erstatter alle ae,oe,aa med æ,ø,å... Så ligger jeg et svar som du tilsyneladende ignorerer i mere eller mindre grad og pludselig får vi flere detaljer om dit kodeprojekt, og du har nu fuldstændigt samme spørgsmål igen til hvordan man løser det at en char kun kan indeholde ét bogstav... Ydermere ser det nu også ud som om at du spørger ind til konverteringen fra æ,ø,å til ae,oe,aa ? Det vil sige det omvendte af hvad du startede med. Arne_v o Janus kommer også med nogle rigtig gode eksempler som ikke ser ud til at hjælpe. Kan du ikke være sød at præcisere dit problem? Det virker lidt som om du er en Programmør som gerne bare lige vil vise sin kode frem og hurtigt er igang med den næste :-) i al fald forvirrende for lille mig :)

arne_v og janus tak for jeres svar! Har været på ferie osv. og fik ikke givet point, inden jeg tog afsted. Vil i ikke skrive et svar så jeg kan give jer point?

svar