Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
private static Predicate<Token> GetMatchBracePredicate(Token token)
{
if (token is Token.BeginBrace)
return delegate(Token t) { return t is Token.BeginBrace; };
if (token is Token.BeginQuote)
return delegate(Token t) { return t is Token.BeginQuote; };
if (token is Token.BeginRound)
return delegate(Token t) { return t is Token.BeginRound; };
if (token is Token.BeginSquare)
return delegate(Token t) { return t is Token.BeginSquare; };
// ...
Обычно говнокодеры останавливаются на четвертом уровне сложности, и считают, что достигли вершины. Однако, есть еще пятый и шестой, а после шестого - дзен.
Ближе всего к первому уровню гость с двумя дженериками. Он две строки написал и пользуется, а ты постоянно переписываешь, дописываешь, копипастишь, исправляешь баги копипасте и теряешь время.
во всем нужен разумный подход. простые вещи можно и инлайном писать, а если код начинает требовать костылей, то значит нуждается в рефакторинге.
Периодический рефакторинг необходим всегда, потому что не всегда заранее можно сказать, по каким веткам в будущем будет бежать исполнение кода. что-то инлайнить, что-то вынести в отдельные методы и классы, к чему-то написать интерфейс...
С опытом, конечно, начинаешь некоторые вещи сразу видеть, как лучше сделать, но идеал, как водится, недостижим
совершенно идентичные логически записи. правда с помощью bool wereWolfTime() {return night&&fullMoon;} поясняется, что night и fullMoon это еще одна логическая единица. Но код "так плохо" вполне читаем
1.
private static Predicate<Token> GetMatchBracePredicate(Token token) { return t => t.GetType() == token.GetType(); }
2. private static bool MatchPredicate<T>(Token token) where T is Token { return typeof(T) == token.GetType(); }
1. return a;
2. return a ? true : false;
3. return a == true ? true : false;
4. if (a == true) return true; else return false;
5. if(!a == true) return false; else return true;
6. if(!a == true) return a == true; else return !a == false;
Обычно говнокодеры останавливаются на четвертом уровне сложности, и считают, что достигли вершины. Однако, есть еще пятый и шестой, а после шестого - дзен.
Периодический рефакторинг необходим всегда, потому что не всегда заранее можно сказать, по каким веткам в будущем будет бежать исполнение кода. что-то инлайнить, что-то вынести в отдельные методы и классы, к чему-то написать интерфейс...
С опытом, конечно, начинаешь некоторые вещи сразу видеть, как лучше сделать, но идеал, как водится, недостижим
"Рефакторинг -- улучшение существующего кода"
"Рефакторинг -- улучшение существующего кода"
по вашему, и по Фаулеру, я должен писать так
Например вот так плохо:
А вот так хорошо:
return a == true; //тут по идее a == false;
может мне к вам устроиться?
Дружите с ним.
Октябренок Палкин Петя
Захлебнулся при миньете
И посметрно, говорят,
Исключен из октябрят
видимо, автор хочет привязать его к пони
к жопе чтоле??
пзц вебкил, ты даже в девиациях лох
<img src="http://upload.wikimedia.org/wikipedia/commons/f/fe/Wiki-pegging.png"/> как-то так
if (night && fullMoon) {
takeSilverBullet();
killWerewolf();
if (wereWolfKilled()) {
someHouseKeeping();
}
}