caesar cypher

**me77** · 04-02-2009

You were right it was the new line :-). I've fixed it and it works perfectly now.. thanks for all the help !

Let me know if I could've done this in a better way:

Code:

for(i=0;sent[i];i++){
                if(sent[i] == ' ')
                  sent[i] = ' ';
                else if((sent[i] + a) > 'Z')
                  sent[i] = 'A' + ((sent[i] + a) - 'Z')-1;
                  else if(!(isalpha(sent[i])))
                    sent[i] = ' ';
                else
                  sent[i] += a;

**quzah** · 04-02-2009

I'd swap the places of the two, so that you're checking to make sure it's alphabetic before you check to see if it's greater than Z. Better still, just:

Code:

if isalpha
    cypher it
else
    output as-is or as a space, pick one

Quzah.

**me77** · 04-03-2009

Alright thanks

Thread: caesar cypher

Thread Tools

Search Thread

Display

Similar Threads

Caesar Cipher

Another Caesar Cipher

Caesar Cypher

Help with Caesar cipher

My little Caesar cipher emulator