Szyfrowanie podanego tekstu, szyfrem podstawieniowym C#

Question

Witam. Mam problem z następującym zadankiem: 

Napisz program, który szyfruje podany przez użytkownika tekst prostym szyfrem podstawieniowym zwanym „gaderypoluki”. Nazwa pochodzi od jednego z najczęściej używanych kluczy GA-DE-RY-PO-LU-KI. Klucz ten zawiera pary znakowych zamienników umieszczonych między myślnikami. Litery, których nie ma w kluczu pozostawia się w szyfrowanym tekście bez zmian. Przykładowo tekst „DRZEWO” po zaszyfrowaniu ma postać „EYZDWP”.

Napisałem już częściowo program, ale mam problem z tym, że druga pętla wykonuje się tylu krotnie, ile ma znaków szyfr (szyfr.Length). Chodzi mi o to, żeby podstawieniu literki, program zwiększył zmienną "a" i analizował a później z petli drugiej przeszedł do pierwszej. Liczę na jakieś wskazówki :) 

Console.WriteLine("Podaj tekst do zaszyfrowania: ");
            string tekst = Console.ReadLine();
            string szyfr = "GADERYPOLUKI";
            string tekstSzyfr = String.Empty;
            for (int a = 0; a < tekst.Length; a++)
            {
                for (int b = 0; b < szyfr.Length; b++)
                {
                    if (tekst[a] == szyfr[b]) //sprawdzenie czy literka z podanego tekstu wystepuje w szyfrze.
                    {
                        int indeks = szyfr.IndexOf(szyfr[a]); //sprawdzenie indeksu literki.
                        if (indeks % 2 == 0) //true to wartość parzysta. 

                            tekstSzyfr = tekstSzyfr + szyfr[indeks + 1];
                        else
                            tekstSzyfr = tekstSzyfr + szyfr[indeks - 1];
                        

                    }
                    else
                        tekstSzyfr = tekstSzyfr + tekst[a];
                    
                } 
            }
            Console.WriteLine(tekstSzyfr);
            Console.ReadKey();

 

Answer 1

Napisałem takie coś:

Console.WriteLine("Podaj tekst do zaszyfrowania: ");
            string tekst = Console.ReadLine();
            string szyfr = "GADERYPOLUKI";
            string tekstSzyfr = String.Empty;
            tekst = tekst.ToUpper();
            for (int a = 0; a < tekst.Length; a++)
            {
                for (int b = 0; b < szyfr.Length; b++)
                {
                    if (tekst[a] == szyfr[b])
                    {
                       if(b % 2 == 0)
                       {
                            tekstSzyfr += szyfr[b+1];
                        }
                        else
                        {
                            tekstSzyfr += szyfr[b-1];
                        }
                      }
                   }
                   if(tekstSzyfr.Length-a == 0)
                   {
                      tekstSzyfr += tekst[a];
                    }
}
            Console.WriteLine(tekstSzyfr);
            Console.ReadKey();

Działa zgodnie z założeniem.

Answer 2

Na twoim miejscu zamiast szyfr trzymać w stringu stworzyłbym słownik. Potem analizujesz każdy znak z tekstu czy jest w słowniku.

Answer 3

Zacznijmy od tego, że pętla w pętli, a nich warunek w warunku (czterokrotne zagnieżdżenie) nigdy nie jest dobrym pomysłem.

Poczytaj o LINQ w C#, jest to bardzo złożone narzędzie i bardzo pomocne w takim przypadku co u Ciebie.

https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/concepts/linq/introduction-to-linq-queries

Oraz poczytaj o słownikach:

https://docs.microsoft.com/en-us/dotnet/api/system.collections.generic.dictionary-2?view=netframework-4.8

 

var szyfr = new Dictionary<char, char> {
  {'G', 'A'},
  {'D', 'E'},
  ...
};

Przykładowy kod może wyglądać tak:

Edit:

Przepraszam, ten poprzedni kod nie działa, bo string jest immutable (w przeciwieństwie do C++). Za chwilę podam inny kod.

 

Comments

Bez sensu używać LINQ czy słowników do tak małego ciągu. Więcej zajmie biblioteka LINQ niż cały kod to robiący. Gdyby tych par-kluczy było więcej to tak, jak najbardziej. Zapewne to przykład z jakiegoś zadania do przećwiczenia umiejętności posługiwania się pętlami i warunkami. :)

---

W sumie sam teraz widzę, że kod trochę długi się zrobił, ale wrzucam jak by to wyglądało w LINQ:

            string tekst = "DRZEWO";
            var szyfr = new Dictionary<char, char> {
                {'G', 'A'},
                {'D', 'E'},
                {'R', 'Y'},
                {'P', 'O' },
                {'L', 'U' },
                {'K', 'I' }
            };

            var nowyTekst = new StringBuilder();

            foreach (char t in tekst)
            {
                var item = szyfr.FirstOrDefault(s => s.Key == t || s.Value == t);
                
                if (item.Key == t)
                    nowyTekst.Append(item.Value);
                else if (item.Value == t)
                    nowyTekst.Append(item.Key);
                else
                    nowyTekst.Append(t);
            }

            Console.WriteLine(nowyTekst);

 

---

@JakSky, nie zgodze sie z Tobą. Biblioteka linq jest bardzo wydajna tak samo jak slownik. Natomiast korzystając ze słowników jest latwiejsze w czytaniu niż z indeksów w tablicach zgnierzdzonych w pętlch... 

---

ale nie w takim prostym przypadku

---

Ale lepiej uczyć się od początku, na prostych przykładach poprawnej składni. Po to jest nauka języka, żeby go dobrze poznać i robić tak jak się powinno.