Улучшение структуры кода

[Elrmrnt-Kritik]

Часто замечаю код примерно такого вида:

#define MAX_HOUSES			1000
function_name(value)
{
	for(new i = MAX_HOUSES; i != 0; --i)
	{
		if(i == value)
		{
			strmid(gHouse[i][hOwner], "none", 0, MAX_PLAYER_NAME, MAX_PLAYER_NAME);
			new string[60+(-2+MAX_PLAYER_NAME)+(-2+4)+1];
			format(string, sizeof(string), "UPDATE `houses` SET `owner` = '%s' WHERE `id` = '%d' LIMIT 1", gHouse[i][hOwner], i);
			mysql_tquery(dbHandle, string, "", "");
		}
                return 1;
	}
}

Однако, сам пишу код следующего вида:

#define MAX_HOUSES			1000
function_name(value)
{
	for(new i = MAX_HOUSES; i != 0; --i)
	{
		if(i != value)
			continue;
		strmid(gHouse[i][hOwner], "none", 0, MAX_PLAYER_NAME, MAX_PLAYER_NAME);
		new string[60+(-2+MAX_PLAYER_NAME)+(-2+4)+1];
		format(string, sizeof(string), "UPDATE `houses` SET `owner` = '%s' WHERE `id` = '%d' LIMIT 1", gHouse[i][hOwner], i);
		mysql_tquery(dbHandle, string, "", "");
	}
        return 1;
}

[hr]
Я считаю, что гораздо удобнее в начале задать условия выхода из данной итерации. Их сразу будет видно. Плюс к этому я обошелся без лишних скобок. Но все же интересно узнать, как будет лучше (я так на более простом примере сравнивал, в принципе, по скорости работы разницы нет). И да, приложил код, который бы реализовывал и в своем моде. В некотором роде специально, вдруг что предложить сможете по данной реализации (функцию, например, какую-нибудь другую посоветуете вместо strmid, либо иначе объявить максимальное количество домов).

Заранее большое спасибо)

[DeimoS]

Если говорить про скорбки и условие - разницы существенной нет. Если говорить в целом о коде, то первый вариант не совсем рабочий, ибо return стоит в теле цикла и обработка кода оборвется на первой итерации.
И скобки в данном случае не лишние. Они вообще не могут быть лишними, если по предназначению используются :)

Касаемо предложений. Убери инициализацию массива

new string[60+(-2+MAX_PLAYER_NAME)+(-2+4)+1];  

из цикла. Зачем каждую итерацию по новой создавать и удалять массив?

[ziggi]

function_name(value)
{
    if (!(0 <= value < MAX_HOUSES)) {
        return 0;
    }
    strcpy(gHouse[value][hOwner], "none", MAX_PLAYER_NAME);
    new string[60+(-2+MAX_PLAYER_NAME)+(-2+4)+1];
    format(string, sizeof(string), "UPDATE `houses` SET `owner` = '%s' WHERE `id` = '%d' LIMIT 1", gHouse[value][hOwner], i);
    mysql_tquery(dbHandle, string, "", "");
    return 1;
}

stock strcpy(dest[], src[], const size = sizeof(dest))
{
    dest[0] = '\0';
    return strcat(dest, src, size);
}  

strcpy не обязателен, но раз попросил.