Как улучшить данный код C?

Question

[prostiti]

Как сделать более быстрым/более лучшим данный код?
Я хочу сделать более быстрым код чем он сейчас, если это возможно.

#include <time.h>
#include <stdio.h>
#include <stdio.h>
#pragma warning(disable : 4996)
#pragma warning(disable : 4552)
#pragma warning(disable : 4013)
int main() {
	int start = clock(); int a = 1000; int b = 0, c = 0; int d = 1;
	FILE* f;
	while ("") {
		for (c < d; a > b; b++) {
			f = fopen("2200.txt", "a");
			fprintf(f, "\ntext");
			fclose(f);
		}
		for (c < d; a = b; c++) { b = 0; }
		if (c == d) {
			double elapsed = (clock() - start) / 1000.0;
			printf("\nTime elapsed in ms: %f", elapsed);
			getch();
		}
	}
}

Comments

[Rsa97]

Для начала хорошо бы просто сделать код правильным. Сейчас в нём какой-то бред.

[prostiti]

Rsa97, что именно в нём бред?
Как его сделать более правильным?

[Rsa97]

prostiti, Корректно записать циклы и выбросить всё лишнее.

[Василий Дёмин]

prostiti, Вот этот кусок, например, чистейший бред
for (c < d; a = b; c++) { b = 0; }

[prostiti]

Василий Дёмин, почему это бред?

[Василий Дёмин]

prostiti, Раз вам неочевидно, рекомендую почитать учебник по си, тут много подобных вопросов задавали, так что подборку учебников найдёте.

Объясняю, почему бред:
Цикл for состоит из трёх частей и самого тела. Первая часть - инициализация переменной, выполняется один раз перед началом цикла. В вашем случае это кусок
c < d

Данное выражение выполнится один раз перед началом цикла. Его результат не используется, следовательно можно убрать эту часть из цикла. Получаем:
for (; a = b; c++) { b = 0; }

Идём дальше. Следующая часть - условие работы цикла, если оно верно, цикл продолжает исполняться. В вашем коде это кусок
a = b
Его результат равен b, в теле цикла b присваивается ноль, то есть цикл выполнится максимум один раз, минимум - ноль. Полагаю, вам нужно поменять первую часть со второй местами. Третья часть - изменение переменной, которая влияет на результат условия. Тут у вас всё верно, при условии, что вы поменяете первую часть со второй. Перейдём к телу цикла. В b кладётся ноль. Сколько раз ноль не присваивай, будет один и тот же результат. Вывод - ваш цикл можно заменить на
b = 0, а код писал человек, который не понимает что делает и пишет всё, что придёт в голову с потолка

[Василий Дёмин]

Ответ на ваш вопрос - научится программировать, прочитать учебник по си, а потом только писать код

[Wataru]

Василий Дёмин, Да чего вы вообще на это время тратите. Чувак списал у кого-то задание, что-то случайно поменял, препод решение не принял. Теперь он хочет, чтобы мы ему все исправили.

[prostiti]

Василий Дёмин,
int a = 1000;
int b = 0;
int c = 0;
int d = 1;
Имелось ввиду, то, что:
Переменная a дело в том, что в C (Си) есть свой диапазон чисел тип, то, что максимум переменная в int может иметь не бесконечность, а пару млрд.
И из-за это была добавлена переменная d чтобы обходить данный диапазон.
Точно не помню как максимальный диапазон, но например он 1000, то, если переменная a будет = 2000, то d будет равняться 2 тем самым обойдёт данный диапазон
Переменная c была добавлена чтобы во сколько раз диапазон был увеличен...

[prostiti]

Wataru, может быть я просто школьник который ещё учится в школе?

[Василий Дёмин]

prostiti, Всё вышеперечисленное ничего не меняет. Учитесь основам программирования, потом читайте учебник по си, потом уже пишите код на си. Пока что ясен один факт - вы не умеете программировать вообще ни на каком языке.

[Василий Банников]

Начать следует с формирования нормального алгоритма, нормальных имён переменных, и исправления предупреждений.

Your code uses a function, class member, variable, or typedef that's marked deprecated.

'operator' : operator has no effect; expected operator with side-effect

The compiler encountered a call to an undefined function.

На счёт дурацких циклов, смысл которых крайне туманен вам уже выше пояснили.

Вот что вообще должна делать эта строчка? Зачем вы так её написали?
for (c < d; a = b; c++) { b = 0; }
Какой вообще сакральный смысл в переменных "a", "b", "c", "d", и "f" (куда кстати "e" пропала)? Почему их нельзя назвать исходя из их назначения?
elapsed и start вы хоть +/- нормально назвали же.

Переменная a дело в том, что в C (Си) есть свой диапазон чисел тип, то, что максимум переменная в int может иметь не бесконечность, а пару млрд.
И из-за это была добавлена переменная d чтобы обходить данный диапазон.
Точно не помню как максимальный диапазон, но например он 1000, то, если переменная a будет = 2000, то d будет равняться 2 тем самым обойдёт данный диапазон
Переменная c была добавлена чтобы во сколько раз диапазон был увеличен...

Если вам нужно считать до каких-то газилионов, то есть длинная арифметика, и готовые библиотеки, которые её реализуют.
Не нужно расширять диапазон при помощи дополнительных переменных.
Тем более что не видно, где этот самый расширенный диапазон используется.

Короче прежде чем оптимизировать код по скорости - его нужно оптимизировать по понятности.

[wisgest]

На вопрос невозможно дать однозначный ответ, так как непонятно, что этот код должен делать.