variabelen uitlijnen?

Door Ozzie PHP op 21-08-2012 06:42

3.493 views

Hallo,

Al meerdere malen loop ik hier tegenaan en ik ben benieuwd hoe jullie dit doen. Als je een aantal variabelen onder elkaar zet, lijn je deze dan netjes uit of niet?

Dus doe je gewoon dit:

<?php
$name = $user->getName();
$age = $user->getAge();
$gender = $user->getGender();
?>

Of doe je dit?

<?php
$name = $user->getName();
$age = $user->getAge();
$gender = $user->getGender();
?>

De laatste optie ziet er mooier uit, maar kost meer werk. De ene keer denk ik, ik doe het allemaal netjes. Dan denk ik weer, nee toch maar niet... en dan even later denk ik... toch maar wel :-)

Maar als ik het in PHP netjes doe, dan zou ik het in m'n css bestanden bijvoorbeeld ook netjes moeten doen... en dat is helemaal een boel werk! Dan zou je zoiets krijgen:


.eenclass {
  border     : 1px solid red;
  height     : 100%;
  margin-left: 15px;
  width      : 500px;
}

Kortom, ik ben benieuwd of jullie je variabelen wel of niet uitlijnen..!

Chris PHP

21-08-2012 07:36

Ik zelf doe het niet, ik zorg dat het overzichtelijk en netjes is. Dit kan ook zonder uitlijning, echter ziet het er met uitlijning wel netter uit.

Maar let wel dat dit natuurlijk alleen voor jezelf is, het hoort uiteindelijk te werken en ziet in principe niemand die code :D

Ozzie PHP

21-08-2012 08:14

Nee, ben ik met je eens... maar het leest wel makkelijker. Echter, als je dus consequent wil zijn, zou je het in je css code eigenlijk ook moeten doen en da's zo'n boel werk!

Eddy E

21-08-2012 08:32

Ik doe dit zelf niet.
In 'indent' wel de HTML zo goed mogelijk.

Dus dit:


<!DOCTYPE html>
<html>
	<head>
		<!-- Title -->
		<title>Overzicht &bull; HGBH</title>
		
		<!-- Meta informatie, and Charset-->
		<meta http-equiv="cache-control" content="public">
		<meta charset="ISO-8859-15">
		<meta name="robots" content="all">
		<meta name="copyright" content="&copy; 2011 Hervormde Gemeente Boven-Hardinxveld en diens vertegenwoordigers">
		<meta name="description" content="Website van de Hervormde Gemeente te Boven-Hardinxveld">
		<meta name="keywords" content="Hervormde, Gemeente, Boven-Hardindxveld, Hardinxveld, Kerk, Hervormd, PKN, Gereformeerde, Bond" >
		<meta name="viewport" content="width=device-width">
		
		<!-- Base href -->
		<base href="http://www.hervormdegemeentebovenhardinxveld.nl">
			
		<!-- Cascade Style Sheet -->
		<link href="css/algemeen.css" rel="stylesheet" type="text/css" media="all">
		<link href="css/overzicht.css" rel="stylesheet" type="text/css" media="all">

		<!-- Javascript -->
		<script src="./js/jquery-1.7.2.min.js"></script>
	</head>
	
	<body>	
		<div id="container">	
			<div id="header"></div>	
			<h1 class="print_only">Overzicht &bull; HGBH</h1>
			<nav id="menu" class="screen_only">

Etc.

In CSS spring ik in als het erbij hoort, dus net een mappen-structuur:

#footer
	{
	margin-top: -50px;
	padding: 0;
	height: 360px;
	background: url(afbeeldingen/achtergrondfooter.png) top left no-repeat;
	margin-left: 0px;
	width: 962px;
	overflow: hidden;
	}
	#footer p
		{
		display: block;
		float: left;
		width: 200px;
		margin: 0;
		padding: 0;
		
		color: #000;
		padding-top:140px;
		}

	#footer p.knoppen
		{
		display: block;
		position: relative;
		top: 115px;
		right: 10px;
		margin: 0;
		padding: 0;
		float: right;
		}	
		#footer p.knoppen a
			{
			display: block;
			float: right;
			height: 38px;
			width: 32px;
			overflow: hidden;
			text-indent: -9999px;
			margin: 0;
			padding: 0;
			background-position: 0px 3px;
			background-repeat: no-repeat;
			margin-left: -10px;
			background: url(afbeeldingen/iconen/ZW/sprite.png) top left no-repeat;
			}		
		
			#footer p.knoppen #boven
				{
				background-position: 0px -64px;
				}
			#footer p.knoppen #zoom
				{
				background-position: 0px -384px;
				}

(ingekorte versie).

In PHP declareer ik mijn variabelen gewoon zo:

<?php
$name = $user->getName();
$age = $user->getAge();
$gender = $user->getGender();
?>

(dus geen gezeur met inspringen)

Ozzie PHP

21-08-2012 08:37

Da's apart met dat inspringen in css. Dat heb ik nog nooit gezien.

Write Down

21-08-2012 10:54

Ik doe het half en half. Ik maak zeker en vast gebruik van inspringingen, bv:


<?php
//even maar iets verzonnen
if($_SERVER['REQUEST_METHOD'] == 'POST') {
	$naam = $_POST['naam'];
	$adres = $_POST['adres'];
	$geboortedatum = $_POST['geboortedatum'];
}
?>

In mijn CSS code doe ik iets gelijkaardig. Bij HTML heb je eigenlijk niet zo veel manieren om in te springen, is allemaal vrij logisch hoe je daar inspringt.

Wat ik wel vaak doe, nadat het project klaar is zeg maar is het volgende: Ik maak een kopie van de bestanden en ik minify de CSS code en JS code. De HTML-code niet. Want het is altijd handig om even snel in de broncode iets te kunnen zoeken. Terwijl dat in de CSS code minder voorkomend is. Tenzij je natuurlijk van iemand anders stukjes wilt rippen :+

- Mark -

21-08-2012 12:09

Bij vars let ik er niet op want er staat in mijn geval altijd wel 1 of meerdere regels commentaar tussen.

Bij een array doe ik het wel om er voor te zorgen dat het gemakkelijker te lezen is.

Ozzie PHP

21-08-2012 12:23

Write Down op 21/08/2012 10:54:51

Ik doe het half en half. Ik maak zeker en vast gebruik van inspringingen, bv:
<?php
//even maar iets verzonnen
if($_SERVER['REQUEST_METHOD'] == 'POST') {
	$naam = $_POST['naam'];
	$adres = $_POST['adres'];
	$geboortedatum = $_POST['geboortedatum'];
}
?>

Zo zou ik het ook ongeveer doen, alleen dan zo:


<?php
//even maar iets verzonnen
if($_SERVER['REQUEST_METHOD'] == 'POST') {
	$naam          = $_POST['naam'];
	$adres         = $_POST['adres'];
	$geboortedatum = $_POST['geboortedatum'];
}
?>

[size=xsmall]Toevoeging op 21/08/2012 12:37:35:[/size]

Ook nog een leuke...

Doe je dit:

<?php
if ($iets) {
// code
// code
// code
// nog meer code...
$user = $data;
} else {
$user = 'Piet';
}
?>

of

<?php
$user = 'Piet';
if ($iets) {
// code
// code
// code
// nog meer code...
$user = $data;
}
?>

Chris PHP

21-08-2012 12:38

Ozzie PHP op 21/08/2012 12:23:55

Zo zou ik het ook ongeveer doen, alleen dan zo:


<?php
//even maar iets verzonnen
if($_SERVER['REQUEST_METHOD'] == 'POST') {
	$naam          = $_POST['naam'];
	$adres         = $_POST['adres'];
	$geboortedatum = $_POST['geboortedatum'];
}
?>

Dit ziet er zo netjes uit, omdat dit korte statements en declaraties zijn. Maar wordt het straks wat langer en uitgebreider krijg je een hele andere situatie.

Het is uiteindelijk een persoonlijke smaak van schrijven, en hoe je het zelf het liefst leest. Ik probeer onnodige whitespaces zoveel mogelijk te vermijden in php scripts, uiteraard heeft dit geen invloed op de werking van het script. Maar ik vind een script dat 30 regels heeft, waarvan 15 witregels een beetje te :D

Ozzie PHP

21-08-2012 12:48 gewijzigd op 21-08-2012 12:48

Als de code langer wordt en er komen regels tussen dan bekijk ik de situatie weer opnieuw...

<?php
//even maar iets verzonnen
if($_SERVER['REQUEST_METHOD'] == 'POST') {
$naam = $_POST['naam'];
$adres = $_POST['adres'];
$geboortedatum = $_POST['geboortedatum'];
// nog wat code...
// nog wat code...
// nog wat code...
// nog wat code...
$huisnummer = $_POST['huisnummer'];
$adres = $_POST['adres'];
$land = $_POST['land'];
}
?>

- Mark -

21-08-2012 12:50

Ik heb juist het tegenovergestelde, Ik heb liever wat meer witregels dan de hele hoop op elkaar gepropt. Ik heb zelfs meer witregels dan regels met code over het algemeen. Het is wel zo dat ik alles consequent doe dus ik heb er geen onnodige witregels in. Het maximum aantal witregels achter elkaar is 2.


<?php

	# even maar iets verzonnen

		if($_SERVER['REQUEST_METHOD'] == 'POST') {
		
		# bla bla bla.

    		$naam = $_POST['naam'];


		# bla bla bla.

    		$adres = $_POST['adres'];


		# bla bla bla.
    
    		$geboortedatum = $_POST['geboortedatum'];
    		
		}

?>

@ Ozzie's tweede vraag, Ik ga voor optie 2.

Reageren

Inloggen om te reageren