Struktura funkcji - nazewnictwo - zasada jednej operacji Opcje

[jackraymund]

Witam,
walczę z nazewnictwem

[PHP] pobierz, plaintext 
$companyId = $this->updateCompanySetToArrayAndGetId($matchedData['company']);//wywolanie
public function updateCompanySetToArrayAndGetId($aCompanyName)
	{
	$isCompanyExistInArray = in_array($aCompanyName,$this->companyArray);
	if(!$isCompanyExistInArray)
	  {
	  $companyId = $this->sqlUpdater->updateCompanyAndGetId($aCompanyName);
	  $this->companyArray[$companyId] = $aCompanyName;
	  return $companyId;
	  }
	else
	  return array_search($aCompanyName,$this->companyArray);
	}
[PHP] pobierz, plaintext 

Z zasady, jedna funkcja, jedna operacja,ten kod jest okropny, lecz zarazem nazwa tej funkcji jest okropna(pomijając tą zasadę).
Natomiast załóżmy że chciałbym zrobić to na tej zasadzie i zrobić to w miarę optymalnie.(chodzi o to, aby nie wywołać funkcji z sqlUpdater, gdy rekord znajduje się aktualnie w tablicy).

Jestem otwarty na wszelakie pomysły.

Pozdrawiam

[NickOver]

@up Według mnie komentarzy powinno nie używać się w kodzie, nazwy funkcji lub metod powinny mówić wszystko.

[PHP] pobierz, plaintext 
private function prepareUpdateCompany($companyName){
if (!$this->isInArray($companyName){
$this->companyArray[$companyId] = $companyName;
return $this->sqlUpdater->updateCompanyAndGetId($companyName);
} else {
return array_search($aCompanyName,$this->companyArray);
}
 
private function isInArray($companyName){
return in_array($companyName,$this->companyArray);
}
[PHP] pobierz, plaintext 

BTW Rozumiem że to kod piany na szybko, przed faktoryzacją, ale albo używasz notacji węgierskiej albo nie.