[oop] wat zou jij doen?

Door Ozzie PHP op 03-01-2014 20:09

2.004 views

Ola,

Een vraagje puur uit nieuwsgierigheid. Ik heb een stukje code:

<?php
if ($path->isDir()) $this->ensureDirectory(str_replace($directory, $directory_copy, $path->getPathname()));
else $this->copy($path->getPathname(), str_replace($directory, $directory_copy, $path->getPathname()));
?>
Op zich niks spannends aan, maar zoals je ziet zit er 2x dezelfde str_replace in. Dat vind ik eigenlijk best stom staan, 2x dezelfde riedel... Zouden jullie dit zo laten? Of voor die str_replace een variabele aanmaken?

<?php
$new_path = str_replace($directory, $directory_copy, $path->getPathname());
if ($path->isDir()) $this->ensureDirectory($new_path);
else $this->copy($path->getPathname(), $new_path);
?>
In voorbeeld 1 is de code wat langer en heb je dus 2x zo'n lange riedel. In voorbeeld 2 is de code korter en duidelijker, maar maak je onnodig een nieuwe variabele aan. Wat doen jullie in dit soort situaties? Wat zijn jullie afwegingen? Alvast bedankt voor de reacties.

L Local Dev

03-01-2014 20:19

Ik zou gaan voor optie 2, dit is duidelijker te lezen.

Wordt dit gebruikt voor een cli applicatie?

Ozzie PHP

03-01-2014 21:15

Dankjewel voor je reactie! Voor een wat? Wat is dat een cli applicatie?

Zouden meer mensen kiezen voor voorbeeld 2? Zijn er ook mensen die kiezen voor voorbeeld 1?

B Bart V B

03-01-2014 21:20

Een cli applicatie daar ben jij allergisch voor Ozzie.
Iets met Command Line Interface, dat ding wat jij niet op de server wil gebruiken. ;)

Ozzie PHP

03-01-2014 21:24

ah oke... dat ding, hihihihi :-))))

Welk voorbeeld heeft jullie voorkeur? 1 of 2?

N NOLot -

03-01-2014 21:35 gewijzigd op 03-01-2014 21:37

Ik zou voor optie twee kiezen. Zelf vind ik dat een stuk duidelijker.

Het zal vast iets zwaarder zijn qua performance maar daar ben ik eigenlijk nooit zoveel mee bezig

Ozzie PHP

03-01-2014 21:38

Oké thanks, dan kies je dus voor duidelijkheid ten koste van performance.

So far, 2 stemmen voor voorbeeld 2 en nog geen stemmen voor voorbeeld 1.

Graag nog meer stemmen, laat maar komen hoor :)

B Bart V B

03-01-2014 21:41 gewijzigd op 03-01-2014 21:44

Eigenlijk vind ik ze beiden niks.
Als ik zo code moet gaan debuggen dan pas ik.
Waarom gebruik je geen blokhaken?
Dit maakt je code i.i.g. leesbaarder.


<?php

if( $path->isDir() )
{
   $this->ensureDirectory( str_replace( $directory, $directory_copy, $path->getPathname() ) );
}
else 
{
   $this->copy( $path->getPathname() , str_replace( $directory, $directory_copy , $path->getPathname() ) );
}
?>


<?php

$new_path = str_replace( $directory, $directory_copy, $path->getPathname() );

if( $path->isDir() )
{
    $this->ensureDirectory( $new_path );
}
else 
{
    $this->copy( $path->getPathname(), $new_path );
}
?>

Persoonlijk zou ik voor de overzichtelijkheid optie 2 gaan.
Misschien net even 1 variabele meer, maar het maakt het verhaal wel beter leesbaar.
(Kuch, misschien komt het door die blokhaken.... ;) )

Ozzie PHP

03-01-2014 21:46

Lol... tja, ieder zo zo'n maniertjes. Als het op 1 regel zet, plaats ik het graag op 1 regel :)
Anders wordt het zo'n uitgerekte soep-zooi. Bij meerdere regels gebruik ik wel accolades, maar dan zo:

<?php
if () {
// code
} else {
// code
}
?>
Mja, kwestie van smaak en gewenning.

Nog meer stemmers? Laat je hoooooren.... :)

B Bart V B

03-01-2014 21:57 gewijzigd op 03-01-2014 22:10

Uiteraard mag jij het op jou manier doen.

Het viel me gewoon op. Normaal gesproken lees ik toch best snel code.
Maar het brijtje van een paar regels waar jij graag een stemronde over wenst, is toch wel prettig dat het een beetje standaard leesbaar is.
Zonder blokhaken is dat (vind ik persoonlijk) irritant.
Ook een spatie tussen if( $foo ) maakt het sneller leesbaar.
Dat is geen uitgerekte soep, maar meer collegiaal. ;)

Je hebt ook de optie (bijvoorbeeld in python) dat je moet inspringen, dat maakt het ook al duidelijker zonder blokhaken.
Maar dat doe je ook niet....
EDIT
even een voorbeeld:


n = raw_input("Integer? ")
n = int(n)
if n < 0:
    print "The absolute value of",n,"is",-n
else:
    print "The absolute value of",n,"is",n

Ozzie PHP

03-01-2014 22:20

Thanks bart. Ik doe inderdaad niet if( $foo ) maar if ($foo). Vind ik dan weer prettiger en meer mensen doen dat. Ik spring ikn dit geval niet in omdat het op dezelfde regel staat. Als ik accolades gebruik wel.

<?php
if (true) {
echo 'jaja';
} else {
echo 'nope';
}
?>
Duidelijker kan ik het niet maken...

Maar mijn eigenlijke vraag ging er dus over of je die str_replace in een variabele moet stoppen ja of nee. Ik had eigenlijk verwacht dat jullie voor voorbeeld 1 zouden gaan, in plaats van voorbeeld 2 omdat je in dit laatste geval onnodig een variabele kopieert. In het verleden werd dit altjd afgeraden.

Reageren

Inloggen om te reageren