Василий Мельников

Квадратные скобки это более удобный и читабельный вариант записи
a[i] обозначает ровно *(a+i)

Поэтому законен и валиден код

1[a] = 42;

Для обращения в первому элементу массива например.

Иногда нет необходимости аводить отдельную переменную для индекса, и тогда код без скобок может быть предпочтительнее:

size_t i = 0;
while (a[i] != 0) {
  a[i] = 3;  /// <- тут если без оптимизации лишняя операция с индексами :)
  ++i;
}

// или

while (*a != 0) {
  *a = 3;
  ++a;
}
// правда тут надо понимать, что a изменилась после цикла

Всё зависит от целей. В подавляющем большинстве случаев можно использовать int и не беспокоиться о перерасходе памяти.
Задумывается стоит если таких переменных миллиарды или ресурсы очень ограничены.
Я бы скорее задумывался о возможности переполнения, не всегда результат операции с переменными конкретного типа в него помещается. Это важнее, чем мистическая экономия.
И еще важно чтобы типы в выражении соответствовали друг другу и ожиданию программиста.

Ну начнем с того, что строку можно перевернуть не выделяя дополнительной памяти. Во вторых возвращать из функции указатель на выделенный блок памяти без лишней необходимости не стоит. Иначе как минимум нужно будет сделать функцию освобождения этого блока.
Что касается циклов. Делать так можно, если никого в команде это не напрягает, но на мой взгляд не нужно. Просто современные компиляторы отлично оптимизируют код, и практической необходимости в подобных упражнениях нет, а ошибиться гораздо легче. Не зря в новые стандарты добавлены циклы по диапазонам и прочие плюшки.

Ну как минимум использование перекрестных переменных уже нелогично. Лучше функциями это обернуть. Как бы инкапсулировать в модуль.
Правильный способ это на бумаге нарисовать модули/объекты системы и обозначить как они взаимодействуют между собой и внешним миром. И уже исходя из этого разбивать. В идеале чем больше модуль похож на черный ящик тем лучше с точки зрения системности.
Как лакмусовая бумажка может использоваться то, что из одного модуля вдруг надо копаться в кишках другого, или знать о том как там что-то реализовано.
Как пример именно для Си - стандартная библиотека. Открываешь файл получаешь указатель на структуру FILE но тебе, абсолютно не важно какие там поля, создается она по malloc или заранее созданный объект из некоего массива таких объектов. Это уже скрыто реализацией. Есть функции которые работают с этим указателем и всё.
Как плохой пример "не очень черного ящика" можно рассмотреть strtok строка которая передается на вход будет модифицироваться и портится, что может быть не очевидным из семантики. Но сделано в угоду скорости и использования памяти.

Как уже предлагалось выше:

int main(void)
{
	int i = 1;
	system(("echo $" + std::to_string(i)).c_str());
	return 0;
}

Ну либо, если компилятор не знает таких новомодных штук как to_string
то заменяется на itoa, sprintf, stringstream и т.д. по вкусу...

А так действительно стоит почитать книжки умные.