Search Results

return ((c-97) & 26) I think you don't want to use the operator '&', but the modulus operator '%', the first one is a bit-level operator. They affect the operands bit by bit, so the logical operatio …

Five functions for Vigenere seem excessive to me. Well, everything is a matter of preferences. …

In order to use a variable in any loop or function, we must declare it outside any of them. If your program does not use auxiliary files, it is sufficient to declare it inside main and outside any cod …

First of all, saying that trying to change a variable of type char * is not a good idea, however we can do this in vigenere because the function get_string () is using the dynamic allocation of memory …

A really strange problem, everything is the fault of the following statement: int j; We have not initialized the variable j, after using gdb j it is set to zero inside the following statement: j = …

In vigenere change the letter with each letter of the text in clear, when we reach the final character of the keyword we start again with the first character, this suggests the use of the module with the … /vigenere <key>.\n"); return 1; } We have isolated a block of code that does a task, and it ends there when the check is done, we do not worry about it anymore. …

i<= strlen(keyword) keep in mind that keyword begins with the zero subscript so that if its length is given by strlen the one index is strlen(keyword) - 1, the correct statement would be: i < strle …

The problem is in the string keyletter, I suppose that it is destined to store the uppercase version of key, however you do: string keyletter = 0; This is to make the pointer keyletter point to NUL …

isalpha() do not accept a pointer but only characters, hence segfault. You must write isalpha(k[i]),the simplest thing is to use a for loop to iterate over each character, besides there are other erro …

Isalpha takes as an argument a character, it is easy to use a for loop to iterate over each character in a string: string name = "my name"; // char* name for( int k = 0, s = strlen (name); k < …

isalpha(argv[1]) Is the responsible declaration of segfault, argv[1] is a pointer to a string, isalpha() has as its argument a character. If you do k = argv [1]; you can access each individual chara …

string Final = (((UI[i] - 'A') + tolower(key[a % keyl]) - 'a') % 26) + 'A'; It is an incorrect assignment: (((UI[i] - 'A') + tolower(key[a % keyl]) - 'a') % 26) + 'A'; The above line is an integ …

isalnum() checks for an alphanumeric character, argv[1] is not a character but a string (exactly a pointer to string),the right way would iterate over the length of argv [i] to check each character in …

else { printf ("%c", p[i]); } FOR loop will not consider whitespace, but if printed, anyway your formula is not correct, note that the formula for Vigenere is exactly the same as for Caesar, but …

if (isupper(letter[i])) ..... else if (islower(letter[i])) ..... you do not consider the formulas when if (isupper((key[i]))...