Path: csiph.com!news.swapon.de!eternal-september.org!feeder.eternal-september.org!reader01.eternal-september.org!ipa.eternal-september.org!.POSTED!not-for-mail
From: Jukka Lahtinen <jtfjdehf@hotmail.com.invalid>
Newsgroups: comp.lang.java.programmer
Subject: Re: Interview question - better solution for MergePurgeAndReverseOrder()
Date: Sat, 30 Mar 2019 11:23:38 +0200
Organization: A noiseless patient Spider
Lines: 14
Message-ID: <m31s2ora05.fsf@sonera.fi>
References: <q7lbvc$152e$1@gioia.aioe.org> <And-20190329162634@ram.dialup.fu-berlin.de> <q7lihk$3ut$1@gioia.aioe.org>
Mime-Version: 1.0
Content-Type: text/plain
Content-Transfer-Encoding: 8bit
Injection-Info: ipa.eternal-september.org; posting-host="f2add765764eb7b8b20b32059a9dec47"; logging-data="3710"; mail-complaints-to="abuse@eternal-september.org";	posting-account="U2FsdGVkX19DgZCMcojLpZdKQUkUjVVP"
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
Cancel-Lock: sha1:dO1mX/584wpaB/1dSoEzaz6u+wY= sha1:9IF0N+QmAZw4iioLL6hUljh9vG8=
X-no-archive: yes
Xref: csiph.com comp.lang.java.programmer:38836

Graeme Geldenhuys <graemeg@example.net> writes:
> On 29/03/2019 15:30, Stefan Ram wrote:

>>   So one should check whether it is possible to refactor, i.e.,
>>   to split the method or improve the name.

> I was not allowed to change anything, other than implement the method.

You could have implemented it to call the methods resulting from the
split.
Besides, method names should start with a lowercase letter..

-- 
Jukka Lahtinen