Вот допустим, есть такой код.


Тут вся соль в том, что этот метод В ЛЮБОМ случае должен возвращать правильный ответ (tiles[yarray, xarray]), и никогда не возвращать null. Вот я пытаюсь предотвратить это путём джамперов. Вопрос чисто теоретический. Не является ли подобный способ нарушающим какие-то правила кодинга? 

Globals.ServiceRecursionLoopBreaker() - пустой метод.

Это сообщение отредактировал(а) Enteropoly - 18.6.2009, 13:34

Enteropoly, зачем тут вообще goto - там и else справится? и как работает магический метод ServiceRecursionLoopBreaker?

И зачем его вызывать?  
Я думаю, что ты гораздо быстрее получил бы помощь, если бы объяснил, что хочешь сделать.

В общем, суть такова. Данная функция ищет клетку на поле по положению мыши. Ессно, результат тут может быть единственный. Так как функция возвращает некий тип (эту самую клетку), то, соответственно, возвратить мы тут можем:

- верный ответ
- неверный ответ
- null

Неверный ответ нам не нужен. null нам тоже не нужен (потому что возвращать null в массив клеток = оставлять дыру). Так как в функции присутствуют всякие условия и рекурсия, нужно обрубить все выходы из функции кроме того, который выдаёт правильный результат. 

В данном случае я тупо прерываю на полпути работу функции (после того, как правильная клетка подсчитана) и через goto перехожу на "подставной" метод-пустышку, код которого:


Вот такие дела =))   

ЗЫ. Всё это, конечно же, работает как часы =) Вопрос в том, считается ли это нормальным =)

ЗЗЫ. Сейчас я выхожу из функции простым ретурном:

вопрос именно про goto.

Это сообщение отредактировал(а) Enteropoly - 18.6.2009, 16:31

После чего всё это возвращается в вызвавший код, который продолжает выполняться, то есть функция всё равно возвращает null, а эффект от вызова пустышки тоже null.

Искать клетку рекурсивно не стоит, при большом количестве клеток приведёт к переполнению стека, да и вообще, память грузить ни к чему, просто вычисляй её математически. 
Я так понимаю клетка у тебя не квадратная, поэтому опиши поподробнее, какая клетка, как это описано в твоём классе и как ты хранишь информацию об этих клетках(одномерный массив, двумерный массив, коллекция или как-то ещё).

diadiavova, я и вычисляю всё математически. Программа у меня вроде работает как положено, вопрос был чисто теоретический =)


Странно, у меня потом вроде ничё не возвращалось...


Из МСДН.

Для чего служит goto я в курсе. Я писал не о гоуту, а о вызове пустышки. После того как вызываемая функция отработает, управление снова передаётся к вызвавшей., то есть инструкция

всё равно выполнится. Поэтому смысла вызова пустышки вообще нет.
Что до теоретической части, то в твоём случае вместо гоуту надо писать  return null;. goto обычно используют для досрочного выхода из глубоких циклов.

Добавлено через 2 минуты и 13 секунд

null - это и есть "ничё"

Мы, кажется, немного о разных вещах вообще думаем =)

Если я тебя не понял, то тут 2 варианта, либо у меня для этого слишком маленький моск, либо ты просто плохо объяснил. В первом случае я "умываю руки", во втором - попытайся сформулировать что тебе не понятно так чтобы я понял.

Enteropoly, если я правильно понял, то тебя интересует "законность" использования go to. в умных книжках пишут, что от использования этого оператора следует воздерживаться любыми способами, потому как он путает код(создает "спагетти код"), который сложно читать. так что с точки зрения "философии" то использование go to является нарушением. но с точки зрения реализации все ложится на плечи и совесть разработчика.

Экскалупатор,   

Enteropoly,  ни с помощью goto ни с помощью вызова метода-пустышки, как ты говоришь, ты не выходишь из рекурсии.


что тебе мешает вместо goto бла бла бла сразу написать return null ?

Кроме того я вообще не понимаю смысла твоего рекурсивного вызова. Смысл рекрсивного вызова в использовании возвращаемого значения. А у тебя после этого вызова оно не запомниается , а просто возвращается null.

По всей видимости у тебя все работает только потому что у тебя yarray всегда четное.

Кроме того постарайся не использовать рекурсию. Она усложняет читабельность и управляемость. Есть ряд задач , где ее применение оправдано , но это не та задача.

Добавлено через 1 минуту и 20 секунд
Да и насчет goto, Забудь это слово. Табу не просто так наложили. Если интересно более подробно , то поищи по форуму. Обсуждали не раз.

Ну в принципе, так и есть. Если оно нечётное, то я его делаю чётным и запускаю метод заново. Это происходит потому, что у меня таким образом ищутся шестиугольники, стоящие плотно друг к другу (а поиск идёт по двумерному массиву). Так как расположить их один под другим нельзя (тот, что ниже на половину сдвинут по отношению к верхнему), а поиск объекта на экране по координатам у меня достаточно простой, я делаю так:
- Ищу по координате Y на какой ряд клеток приходится найденный результат
- Если это нечётный ряд, то входная координата X сдвигается (т.к. и сам ряд как бы сдвинут)
- Теперь запускаем заново метод и находим клетку правильно

В результате, я попросту не могу вернуть null, иначе получится ерунда. И неправильный ответ я тоже вернуть не могу. А вернуть что-то необходимо. Вот и получается, что мне нужно как-то полюбому выбраться из метода до того, как он дойдёт до дефолтного return null. Я и делал через goto =)

Про goto теперь ясно. Спасибо всем, особенно Экскалупатору. 

PS. Про эту программу есть тема-побратим.

Это сообщение отредактировал(а) Enteropoly - 20.6.2009, 03:07