Programmierstil - Seite 2

RudiS · 03.10.2006, 15:39

Ich mache es auch so wie Chr!s, Basti, <Daniel> nur dass ich noch zusätzlich $rData für Daten vom Client ($_REQUEST, $_COOKIE, ...) verwende: z. B.

PHP-Code:

  <?php

$rData = ( isset( $_REQUEST['data'] ) )

    ? $_REQUEST['data']

    : '';

?>

So weiß ich welche Daten ich mit Vorsicht behandeln muss. Da diesen Daten sowieso Strings sind, ist das auch kein Problem und ich weiß $rData entspricht eigentlich $sData, nur ich muss aufpassen.
mfg RudiS

Basti · 04.10.2006, 10:24

Oh, das "r" habe ich auch. *g Allerdings fpr Ressourcen. Ich finde es wenig sinnig, diese Buchstabensuppe auch noch über den reinen Typ der Variable auf deren Inhalt oder Context etc. auszuweiten.

Es spricht zumindest für unübersichtliche Quelltexte, wenn sowas nötig wird. In meinen Augen besser, die einzelnen Methoden zu verkleinern (aufzuteilen), so dass man den Kontext immer im Blick hat.

Basti

Zergling-new · 04.10.2006, 12:34

Zitat:

Zitat von Fatal Error

<input type="text" name="input_vorname">
<input type="text" name="input_nachname">

in einer function die Parametervariablen so
function addKunden($param_vorname, $param_nachname){
// ...
}

Ich belasse die Formular-Eingaben eigentlich immer in $_POST oder $_GET, insofern ist es total eindeutig, woher die Daten kommen.
Sollte ich mal Variablen validieren, erzeuge ich eben eine neue Variable namens $firstname und es sollte ja wohl klar sein, woher $firstname kommt, wenn die PHP-Datei ein Formular verwaltet, in dem man u.a. den Nachnamen eintragen kann.
Benutzerprofile stehen meist sowieso in der $_SESSION, in sofern kommt es da selten zu Kollisionen.

Variablen nach ihrem Typ her zu benennen wie Basti tue ich im Moment nicht, ich bin aber ernsthaft am überlegen, da bei größeren Projekten so einfach klarer ist, was in was steht oder stehen könnte.

Mein dummy_new_customer Ordner sieht momentan so aus:

Code:

index.php -- Zentrale, über die alle Anfragen hereinkommen, verteilung auf core/
.htaccess -- steuert Mod-Rewrite

config -- Konfigurations-Einstellungen wie DEBUG an oder aus, E-Mail-Adresse des Admins, ..
core -- hier liegt pro Rubrik und Datei die PHP-Logik
img
inc
inc/global.php -- erzeugt alle wichtigen Pfadkonstanten
inc/classes
inc/functions
inc/smarty
inc/pear
inc/phpmailer
js -- JavaScript
style -- CSS Ordner
style/.htaccess -- erzwingt das Parsen von CSS Dateien über PHP, zwecks Pfadkonstanten
template -- Smarty-Templates, gleiche Ordner- und Dateistruktur wie core
tmp/template_c -- Smarty Cache

config, inc, template, tmp haben jeweils eine .htaccess mit "deny from all"
könnte sicher auch außerhalb des Webroots liegen, tut es aber im Moment noch nicht

<Daniel> · 04.10.2006, 13:15

Was mich jetzt ein wenig wundert ist, dass du einmal inc/smarty hast und noch mal template als einzelnen Ordner? Also in template sind die Templates, aber was ist dann in inc/smarty?

Gruß

freq.9 · 04.10.2006, 14:01

Zitat:

Zitat von Basti

Hi.

Ich mache durch Namenskonvetionen sichtbar, von welchem Type eine Variable ist:

PHP-Code:

  <?php

$iNumRows = 42;

$fFactor = 3.24;

$sLabel = 'Something';

$bIsValid = true;

$aRoles = array('array_key' => $mFoo);

$cMode = DEBUG;

$mFoo = ? (mixed/unknown)

$DaoFactory = new DaoFactoy();

?>

Was willst du nun aber genau sammeln? "Programmierstil" ist ja alle Mögliche...

Basti

f(loat), s(tring), i(nteger), b(ool) und a(rray) mache ich hier auch, jedoch finde ich "c" ein wenig unpassend, denn wieso schreibt man einen konstanten Wert aus einer Konstante in eine Variable? Wieso ist es dann eine Konstante? Meiner Meinung widerspricht das dem Sinn einer Konstante.

Dazu nutzen ich noch den Präfix o(bject) und r(esource).

Flor1an · 04.10.2006, 15:10

In inc/smarty liegt wohl ... na wer hätte es gedacht ... Smarty ?

Zergling-new · 04.10.2006, 15:12

So ist es:
in template/ liegen die Templates selber, also die *.tpl bzw. *.html Dateien und in inc/smarty/ liegt die Smarty-Engine

<Daniel> · 04.10.2006, 15:17

Ahso, ok alles klar :)

RudiS · 04.10.2006, 16:10

@Basti: Hab nochmal nachgedacht, ich finde du hast Recht. Verwende jetzt $r... für Ressourcen und nicht mehr für die $_REQUEST Daten.
mfg RudiS

Basti · 04.10.2006, 21:37

Zitat:

Zitat von freq.9

[...] jedoch finde ich "c" ein wenig unpassend, denn wieso schreibt man einen konstanten Wert aus einer Konstante in eine Variable? Wieso ist es dann eine Konstante? Meiner Meinung widerspricht das dem Sinn einer Konstante.

Ich verwende praktisch gar keine Konstanten und nur als Elemente einer kleinen Wertemenge, z.B. als Property der Result-Objekte, die die Controller zurückgeben SUCCESS, NOVIEW, NORIGHT, FORWARD, REDIRECT. Was dann da drinnen steht ist mir Wurscht, aber natürlich liegt der Wert eben als Eigenschaft in der entsprechenden Klasse vor und wird von deren Konstruktor auch als erstem Parameter übernommen.

Klar, in der Regel liegen in Konstanten Integer, denen dann mit Bitverschiebung etc. zu Leibe gerückt wird. Aber in dem Kontext gebrache ich sie nicht.

Oh ... Kommando zurück. Hab grad mal mein Framework nach "$c" durchsucht. Da ist garnichts mehr. Die Controller-Result-Typen hatte ich $sType genannt.

Wie auch immer...

Basti

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Der "richtige" Programmierstil	Matze	PHP Tipps 2007	15	18.01.2007 14:37
Frage zu Programmierstil	inu	PHP Tipps 2006	8	26.05.2006 08:11
Frage zu eurem Programmierstil		PHP Tipps 2005	10	27.03.2005 05:25

03.10.2006, 15:39
RudiS Erfahrener Benutzer Registriert seit: 08.06.2004 Beiträge: 865	Ich mache es auch so wie Chr!s, Basti, <Daniel> nur dass ich noch zusätzlich $rData für Daten vom Client ($_REQUEST, $_COOKIE, ...) verwende: z. B. PHP-Code: `<?php $rData = ( isset( $_REQUEST['data'] ) ) ? $_REQUEST['data'] : ''; ?>` So weiß ich welche Daten ich mit Vorsicht behandeln muss. Da diesen Daten sowieso Strings sind, ist das auch kein Problem und ich weiß $rData entspricht eigentlich $sData, nur ich muss aufpassen. mfg RudiS __________________ Kunst kommt von Können und nicht von wollen, denn sonst würde es ja Wunst heißen.


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

04.10.2006, 10:24
Basti Erfahrener Benutzer Registriert seit: 18.07.2004 Beiträge: 2.162 PHP-Kenntnisse: Fortgeschritten	Oh, das "r" habe ich auch. *g Allerdings fpr Ressourcen. Ich finde es wenig sinnig, diese Buchstabensuppe auch noch über den reinen Typ der Variable auf deren Inhalt oder Context etc. auszuweiten. Es spricht zumindest für unübersichtliche Quelltexte, wenn sowas nötig wird. In meinen Augen besser, die einzelnen Methoden zu verkleinern (aufzuteilen), so dass man den Kontext immer im Blick hat. Basti

04.10.2006, 13:15
<Daniel> Erfahrener Benutzer Registriert seit: 03.04.2006 Beiträge: 222	Was mich jetzt ein wenig wundert ist, dass du einmal inc/smarty hast und noch mal template als einzelnen Ordner? Also in template sind die Templates, aber was ist dann in inc/smarty? Gruß

04.10.2006, 15:10
Flor1an ¯\_(ツ)_/¯ Registriert seit: 18.06.2008 Beiträge: 8.814 PHP-Kenntnisse: Fortgeschritten	In inc/smarty liegt wohl ... na wer hätte es gedacht ... Smarty ? __________________ ▇█▓▒░◕‿‿◕░▒▓█▇

04.10.2006, 15:12
Zergling-new Erfahrener Benutzer Registriert seit: 21.05.2008 Beiträge: 9.937	So ist es: in template/ liegen die Templates selber, also die .tpl bzw. .html Dateien und in inc/smarty/ liegt die Smarty-Engine

04.10.2006, 15:17
<Daniel> Erfahrener Benutzer Registriert seit: 03.04.2006 Beiträge: 222	Ahso, ok alles klar :)

04.10.2006, 16:10
RudiS Erfahrener Benutzer Registriert seit: 08.06.2004 Beiträge: 865	@Basti: Hab nochmal nachgedacht, ich finde du hast Recht. Verwende jetzt $r... für Ressourcen und nicht mehr für die $_REQUEST Daten. mfg RudiS __________________ Kunst kommt von Können und nicht von wollen, denn sonst würde es ja Wunst heißen.