StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

POHow to simplify my implementation of a basic Caesar Shift Encryption algorithm c#?
primarykey
Id
5337080
data
AcceptedAnswerId
0
AnswerCount
5
ClosedDate
CommentCount
3
CommunityOwnedDate
CreationDate
2011-03-17T09:20:15.270
FavoriteCount
0
LastActivityDate
2011-03-17T13:41:52.283
LastEditDate
LastEditorUserId
0
OwnerUserId
635434
ParentId
0
PostTypeId
1
Score
2
ViewCount
1151
LastEditorDisplayName
text
Body
So I have recently decided that my coding style is somewhat clunky. The trouble is I never seem to be able to get to a stage where I can figure out the way of simplifying it into less lines of more efficient code. I was in a team coding situation the other day trying to code a word wrapping feature using TDD. When I was in the driving seat I spent most of my time with String.Split() and if statements this and if that. The guy I was coding with asked why so complex and went with the simpler recursive one liner that returned the values required and with a few conditions to boot him out of the recursive loop when he was done. So my question is thus – Below is some code that I have written to do a Caesar Shift Encryption on a string input using only lower case alphabet and no spaces. The unit tests pass and I believe I have implemented the various conditions that may occur. In a nut shell how would you guys simplify the code below to make it more readable and more efficient? I appreciate the help on this, because at the end of the day I need to make my coding style less verbose and more simplistic and can’t figure out the best place to start. Cheers Code in C#: <pre><code> public static string Encrypt(string inputString, int shiftPattern) { StringBuilder sb = new StringBuilder(); char[] alphabet = { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z' }; //y = x + 3 (mod 26) foreach (var letter in inputString.ToLower()) { if (!alphabet.Contains(letter)) { return "The " + letter + " Character was not in the sample set, please ensure you only use letters"; } var res = Array.IndexOf(alphabet, letter) + (shiftPattern % 26); if (res >= 26) { res = res - alphabet.Length; sb.Append(alphabet[res]); } else if (res < 0) { res = alphabet.Length + res; sb.Append(alphabet[res]); } else sb.Append(alphabet[res]); } return sb.ToString(); } </code></pre>
Tags
<c#>
Title
How to simplify my implementation of a basic Caesar Shift Encryption algorithm c#?
singulars
PostAcceptedAnswerId
1. This table or related slice is empty.
PostParentId
1. This table or related slice is empty.
PostTypePostTypeId
1. PTQuestion
UserLastEditorUserId
1. This table or related slice is empty.
UserOwnerUserId
1. USMorn
plurals
PostLinksPostIdRelatedPostId
1. This table or related slice is empty.
PostLinksRelatedPostIdPostId
1. This table or related slice is empty.
PostsAcceptedAnswerId
1. This table or related slice is empty.
PostsParentIdCreationDate
1. PO
 singulars
 PostTypePostTypeId
 PTAnswer
2. PO
 singulars
 PostTypePostTypeId
 PTAnswer
3. PO
 singulars
 PostTypePostTypeId
 PTAnswer
VotesPostIdCreationDate
1. VO
 singulars
 PostPostId
 POHow to simplify my implementation of a basic Caesar Shift Encryption algorithm c#?
 UserUserId
 This table or related slice is empty.
 VoteTypeVoteTypeId
 VTUpMod
2. VO
 singulars
 PostPostId
 POHow to simplify my implementation of a basic Caesar Shift Encryption algorithm c#?
 UserUserId
 This table or related slice is empty.
 VoteTypeVoteTypeId
 VTUpMod
CommentsPostId

Querying!

Guidance

A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.