🚀 go-pugleaf

Jacek Popławski napisał(a):
>> bo w rzeczywistych sytuacjach często są jeszcze zmienne lokalne, 
>> których wartości trzeba w jakiś sposób do funkcji przekazać.
> 
> Możesz podać kod rzeczywistej sytuacji? Bo cały czas padają dosyć 
> hipotetyczne przykłady.

mogę, chociaż pewnie usłyszę, że kod jest źle napisany, że przecież 
wszystko powinno być rozbite na malutkie funkcje. jasne, jeśli ktoś lubi 
bawić się w akademickie podejście, proszę bardzo. dla mnie bardziej 
czytelna jest jedna funkcja robiąca jedną rzecz, a nie wywołująca 
dziesiątki mniejszych funkcji _foo(), _schmoo(), _bar(), _bah(), _baz(), 
którym przekazuje się wskaźniki na kilka zmiennych lokalnych, na których 
funkcje operują.

int window_printat(WINDOW *w, int x, int y, const char *orig_format, 
void *orig_data, int fgcolor, int bold, int bgcolor, int status)
{
         int orig_x = x;
         int backup_display_color = config_display_color;
         char *format = (char*) orig_format;
         const char *p;
         struct format_data *data = orig_data;

	/* ... */

	while (*p && *p != '}' && x <= w->_maxx) {
                 int i, nest;

		/* ... */

                 for (i = 0; data && data[i].name; i++) {
                         int len;

			/* ... */

			if (!strncmp(p, data[i].name, len) && p[len] == '}') {
                                 char *text = data[i].text;
                                 int j;

				/* ... */

				goto next;
			}
		}

		if (*p == '?') {
			/* ... */
		}

next:

		/* ... */

		while (*p && nest) {
			/* ... */
		}
	}

	/* ... */

	return x - orig_x;
}

przy okazji szukania przykładowego kodu znalazłem całe mnóstwo sytuacji 
w stylu:

int foo(char *data)
{
	char *tmp;
	int value;
	void *ptr;

	tmp = malloc(123);

	while (*data) {
		switch (*data) {
			case 1:
				/* ... */

				if (costam())
					goto warning;

				break;

			case 2:
				/* ... */

				if (costam())
					goto error;

				break;

			/* ... */
		}
	}

warning:
	log("warning, blabla\n");
	free(tmp);
	return 0;

error:
	log("error, blabla\n");
	free(tmp);
	return -1;
}

jak można to zapisać nie używając goto, żeby było bardziej przejrzyste?

w.