Search Results

The correct way isif(argc !=2 || ! isalpha(key[j]))

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 …

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() 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 …

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 …

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 = …

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 …

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

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 …

"The values returned are nonzero if the character c falls into the tested class, and a zero value if not." We can use that to build the if condition, the correct way would be: if (! isalpha (key [a]) …

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

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 …

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 …

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 …

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 …