Gardery Poluki - proszę o pomoc

0

Witam,
mam do napisania Gardery Poluki,
dotychczas udało mi się napisać tyle:
jakieś uwagi, wskazówki, feedback, co poprawić, w jakim kierunku iść, co jest źle ?

 static void zamiana(string args)
{
 string klucz = "GARDERYPOLUKI";
 string znak = "";
 string  slowo = "MANDARYNA";
 public void zamiana()
  {
    for (int i = 0; i <= klucz.Length; i++)
    {

    if (klucz[i]==znak) && (i % 2 == 0)
    { 
       return klucz[i + 1];
    }
    
    else{
       return klucz [i - 1];
        
     }
     return  znak;
}
    
    }
  public void szyfr()
{
str  s2 = "";  
for (int i = 0; i <= slowo.Length; i++)
    {

     s2 = s2+ zamiana (k, slowo[i]);
        print s2;
    }
}
    
    zamiana();
    szyfr();
}
2

1.Formatowanie kodu.
2.Sensowne nazewnictwo identyfikatorów (w tym parametrów metod).
3.Pętlę powinieneś iterować od 0 do slowo.Length-1, bądź inaczej: wykorzystać operator <, aniżeli <=.

0

dziękuję bardzo !
a ogólnie algorytm dorbze napisany? zadziała, jak powinien?

0

Przede wszystkim twoj klucz jest zly przez co szyfrowanie jest niejednoznaczne (R odpowiada jednoczesnie D i E).

Poza tym to sie nawet nie skompiluje (metoda void zamiana() a w srodku return?)

0

dziękuję za uwagi, kwestie kompilacji dopracuje, ale jakieś podpowiedzi, jak poprawić klucz, by dobrze szyfrowalo?

1 użytkowników online, w tym zalogowanych: 0, gości: 1