Как вернуться к началу строки?

Question

[OccamaRazor]

Нужно зашифровать argv[1] с помощью argv[2]. В наличии 2 аргумента, которые вводятся пользователем, оба аргумента являются строками. Соответственно первый аргумент получает условный текст который нужно будет зашифровать, например "ENCODE ME", второй аргумент это и есть тот самый ключ, с помощью которого происходит кодирование, например "KEY". Так вот, хотелось бы чтобы "KEY" применялся посимвольно и не переставал работать после однократного выполнения. Из-за нехватки опыта не получается сделать именно это, подскажите, как можно реализовать. Много вариантов перепробовал, пытался сделать цикл чтобы после того как слово закончиться возвращаться к нулевому символу и прогонять заново но не вышло, так как не могу придумать чем заменить в Си, indexOf в JS.

for (int i = 0; i < argv[1][i]; i++)
	{
		if (isalpha(argv[1][i]))
		{
			if (argv[1][i] >= 'A' && argv[1][i] <= 'Z')
			{
				int shift = argv[2][i] - 65;

				printf("%c + %c '%.3d' == ", argv[1][i], argv[2][i], shift);

				char rightNumber = argv[1][i] + shift;

				if (rightNumber > 'Z')
					rightNumber -= 26;

				printf("%c\n", rightNumber);
			}
		}
	}

Answer 1

[Алексей]

Все argv[2][i] замените на argv[2][i % strlen(argv[2])] или заведите отдельную переменную под новый индекс.
Это стандартная практика хождения по кругу - взятие остатка от деления индекса на диапазон.
PS. Условие в цикле i < argv[1][i] очень сомнительное.

Comments

[OccamaRazor]

Благодарю за ответ. Прошу прощения, если можно, подробнее об условии...

[Алексей]

OccamaRazor: Вы, вероятно хотели написать цикл по всем символам строки, то есть for (int i = 0; i < strlen(argv[1]); i++), а то что вы написали, может и будет работать, но только пока длина строки менее 32 символов, как только строка превысит эту длину, ваше условие может споткнуться на пробеле, так как его код 32.

[OccamaRazor]

Алексей: да, верно, не знал об этой ошибке, огромное спасибо.

Answer 2

[Александр]

#include <string.h>
#include <stdio.h>
#include <ctype.h>

int main(int argc, char *argv[])
{
    int incode_len = strlen(argv[1]);
    int key_len = strlen(argv[2]);
    for (int i = 0, k = 0; i < incode_len; i++, k++)
    {
        if( k >= key_len) k = 0;
        if (isalpha(argv[1][i]))
        {
            if (argv[1][i] >= 'A' && argv[1][i] <= 'Z')
            {
                int shift = argv[2][k] - 65;
                printf("%c + %c '%.3d' == ", argv[1][i], argv[2][k], shift);
                char rightNumber = argv[1][i] + shift;
                if (rightNumber > 'Z') rightNumber -= 26;
                printf("%c\n", rightNumber);
            }
        }
    }
}

Comments

[OccamaRazor]

Благодарю, очень компактный код, буду учится так писать, надеюсь в будущем тоже освою

Answer 3

[abcd0x00]

Пример кода с циклической строкой

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[])
{
    const char *s, *start, *end, *cur;
    int n, i;

    if (argc == 3) {
        s = argv[1];
        n = atoi(argv[2]);
    } else {
        s = "12345";
        n = 50;
    }

    start = end = s;
    if (*end)
        while (*(end + 1))
            end++;

    cur = start;
    for (i = 0; i < n; i++) {
        putchar(*cur);
        cur = (cur < end) ? cur + 1 : start;
    }
    putchar('\n');

    return 0;
}

Вывод

[guest@localhost c]$ .ansi t.c -o t
[guest@localhost c]$ ./t
12345123451234512345123451234512345123451234512345
[guest@localhost c]$ ./t abc 80
abcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcabcab
[guest@localhost c]$ ./t a 10
aaaaaaaaaa
[guest@localhost c]$ ./t ab 10
ababababab
[guest@localhost c]$ ./t abc 10
abcabcabca
[guest@localhost c]$

Answer 4

[Rsa97]

#include <string.h>
#include <stdio.h>

int main(int argc, char *argv[])
{
  unsigned char *text, *key;
  unsigned char shift, rightNumber;
  for (text = argv[1], key = argv[2]; *text; text++, key++) {
    if (0 == *key)
      key = argv[2];
    if (*text >= 'A' && *text <= 'Z') {
      shift = *key - 'A';
    rightNumber = *text + shift;
    if (rightNumber > 'Z')
      rightNumber -= ('Z' - 'A');
    printf(""%c + %c '%.3d' == "%c\n", *text, *key, shift, rightNumber);
  }
}

Comments

[Александр]

Можно еще малень шлифануть:
#include
#include

int main (int argc, char *argv[])
{
unsigned char *key = argv[2];
unsigned char rightNumber, shift;
for (; *argv[1]; argv[1]++, key++) {
if (0 == *key)
key = argv[2];
if (*argv[1] >= 'A' && *argv[1] <= 'Z') {
shift = *key - 'A';
rightNumber = *argv[1] + shift;
if (rightNumber > 'Z')
rightNumber -= ('Z' - 'A');
printf ("%c + %c '%.3d' == %c\n", *argv[1], *key, shift, rightNumber);
}
}
}

[Rsa97]

Александр: Нет. Чем меньше обращений к массиву по индексу, тем лучше. В худшем случае argv[1] будет перевычисляться при каждом использовании.

[Александр]

Rsa97: Согласен. Об этом не подумал.