Forum » Server-Side scripting » Script overzicht behouden
Jasper
Berichten: 887
avatar
Offline Stuur privebericht
hej leden van CP!

de titel zegt het eigenlijk al een beetje. hoe houd jij je script overzichtelijk?
je hoeft hier niet hele scripts te posten maar alleen begintags en klein beetje tekst.

zo doe ik het

Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<html>
  <head>
     <title> blabla </title>
  </head>

<body>
<?php

$variable = altijd_boven_aan_bij_mij
$variable2 = voorbeeldje

  echo "is altijd 1 TAB van de kant af"  ;
  echo " ; is altijd een paar spaties van de opdracht af" //commentaar direct achter de opdracht

?>
</body>
</html>


zoals je ziet laat ik ook veel ruimtes tussen de teksten etc. dit is alleen om het er netjes uit te laten zien

MVG, Puddinghoofd

Edit by Killingdevil: Topic verplaatst naar Scripting, en een highlight van gemaakt!
Laatst gewijzigd door Killingdevil op 2009-10-24 16:34:34
2009-10-23 11:06:09
Dit topic is 569 keer bekeken door 8 verschillende leden
Reacties op: "Script overzicht behouden"
Vorige pagina 1 | 2 | 3 | 4 | 5 Volgende pagina
Darsstar
Vreemd
Berichten: 1237
avatar
Offline Stuur privébericht
@NielsB
Waarom gebruik je zo'n vreemde indent?
De else van regel 16 zal alleen voor regel 17 wereken, toch zijn de volgende regels op het zelfde niveau als regel 17.

Ik probeer me sinds ik gebruik maar van Kohana ook aan de Kohana coding styles te houden.
Nieuw: http://v3.kohanaphp.com/guide/start.conventions
Oud: http://dev.kohanaphp.com/projects/kohana2/wiki/CodingStyle
combineer die twee (nieuw overschrijft oud)

Verder is het een goed idee om in bestanden die alleen maar php bevatten GEEN sluit tag (?> neer te zetten.
2009-10-24 17:36:01
NielsB
Webdeveloper
Berichten: 995
avatar
Offline Stuur privébericht
O ja sorry darsstar.
Ik bedoelde het ook voor alleen de volgende regel

Kan je uitleggen waarom geen sluit tag..
Pim deed het ook altijd en vond het iritant voor mijn dreamweaver die php dan weer fout las.

Ik ben nu druk bezig met OOP. En ben daarom ook volop bezig met gebruik te maken van commentaar en andere belangrijke dingen in de oop coding styles.
2009-10-24 17:52:52
Darsstar
Vreemd
Berichten: 1237
avatar
Offline Stuur privébericht
Het sluiten van PHP is niet nodig, volgens mij is het ook sneller (denk aan microseconden, ik denk het ergens gelezen te hebben, ooit) en als je PHP afsluit kan er alleen maar output na komen wat mogelijk headers already sent errors gaat geven, wat je wilt voorkomen.

Nieuwe reactie samengevoegd met originele reactie op 25.10.09 19:52:18:
BUMP! (via de database)

Dit is een interessant topic, ik weiger het zo snel dood te laten bloeden!

Code | Selecteer Alles
minimaliseren
1
ini_get('session.autostart') OR session_start();

Ik heb een maandje (of twee?) geleden geleerd dat zulke regels mogelijk zijn =]
Zulke regels kunnen erg handig zijn, en nemen minder ruimte in dan een if block!

PS. ik snap niet waarom de TS de punt-komma een paar spaties van de "opdracht" houd.
2009-10-24 18:03:29
RiiCk
ondergetekende.
Berichten: 1159
avatar
Offline Stuur privébericht
BUMP.

vond het wel een intressant topic.
2009-12-24 03:31:53
L.Groot
Rapifia Europe
Berichten: 3508
avatar
Offline Stuur privébericht
Ik gebruik Notepad++, nog nooit problemen mee gehad.
Verder script ik niet zo heel overzichtelijk.
Als ik een script moet bewerken, dan doe ik het wel netjes.
Bij mij is het gewoon oldskool:
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
<?php
$cp = 1;
if($cp == 1){
echo 'CP is gek!';
} else{
echo 'CP is chill!';
}
?>

Commentaar komt wel achter de opdracht regels, en de variabelen staan bij mij ook altijd bovenaan.
Inspringing vind ik gewoon irritant, dus dat doe ik zelf nooit.
2009-12-24 10:28:19
Darsstar
Vreemd
Berichten: 1237
avatar
Offline Stuur privébericht
Quote: larsisgoed9

Inspringing vind ik gewoon irritant, dus dat doe ik zelf nooit.


WHAT!?!

Inspringen is irritant?
Niet inspringen is irritant...

neem nou de volgende code:
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
/**
     * Get the value of a field.
     *
     * @throws  Sprig_Exception  field does not exist
     * @param   string  field name
     * @return  mixed
     */
    public function __get($name)
    {
        if ( ! $this->_init)
        {
            // The constructor must always be called first
            $this->__construct();

            // This object is about to be loaded by mysql_fetch_object() or similar
            $this->state('loading');
        }

        if ( ! isset($this->_fields[$name]))
        {
            throw new Sprig_Exception(':name model does not have a field :field',
                array(':name' => get_class($this), ':field' => $name));
        }

        if (isset($this->_related[$name]))
        {
            // Shortcut to any related object
            return $this->_related[$name];
        }

        $field = $this->_fields[$name];

        if ($this->changed($name))
        {
            $value = $this->_changed[$name];
        }
        elseif (array_key_exists($name, $this->_original))
        {
            $value = $this->_original[$name];
        }

        if ($field instanceof Sprig_Field_ForeignKey)
        {
            if ( ! isset($this->_related[$name]))
            {
                $model = Sprig::factory($field->model);

                if ($field instanceof Sprig_Field_HasMany)
                {
                    if ($field instanceof Sprig_Field_ManyToMany)
                    {
                        if (isset($value))
                        {
                            if (empty($value))
                            {
                                return new Database_Result_Cached(array(), '');
                            }
                            else
                            {
                                $query = DB::select()
                                    ->where($model->pk(), 'IN', $value);
                            }
                        }
                        else
                        {                            
                            $query = DB::select()
                                ->join($field->through)
                                    ->on($model->fk($field->through, $field->foreign_field), '=', $model->pk(TRUE))
                                ->where($this->fk($field->through, $name), '=', $this->{$this->_primary_key});
                        }
                    }
                    else
                    {
                        if (isset($value))
                        {
                            $query = DB::select()
                                ->where($model->pk(), 'IN', $value);
                        }
                        else
                        {
                            $query = DB::select()
                                ->where($model->fk(NULL, $field->foreign_field), '=', $this->{$this->_primary_key});
                        }
                    }                    

                    $related = $model->load($query, NULL);

                    if ( ! $this->changed($name))
                    {
                        // We can assume this is the original value because no
                        // changed value exists
                        $this->_original[$name] = $field->value($related);
                    }
                }
                elseif ($field instanceof Sprig_Field_BelongsTo)
                {
                    $related = $model->values(array($model->pk() => $value));
                }
                elseif ($field instanceof Sprig_Field_HasOne)
                {
                    $related = $model->values(array($this->_model => $this->{$this->_primary_key}));
                }

                $value = $this->_related[$name] = $related;
            }
        }

        return $value;
    }

Irritant?
Niet echt...
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
/**
* Get the value of a field.
*
* @throws  Sprig_Exception  field does not exist
* @param   string  field name
* @return  mixed
*/
public function __get($name)
{
if ( ! $this->_init)
{
// The constructor must always be called first
$this->__construct();

// This object is about to be loaded by mysql_fetch_object() or similar
$this->state('loading');
}

if ( ! isset($this->_fields[$name]))
{
throw new Sprig_Exception(':name model does not have a field :field',
array(':name' => get_class($this), ':field' => $name));
}

if (isset($this->_related[$name]))
{
// Shortcut to any related object
return $this->_related[$name];
}

$field = $this->_fields[$name];

if ($this->changed($name))
{
$value = $this->_changed[$name];
}
elseif (array_key_exists($name, $this->_original))
{
$value = $this->_original[$name];
}

if ($field instanceof Sprig_Field_ForeignKey)
{
if ( ! isset($this->_related[$name]))
{
$model = Sprig::factory($field->model);

if ($field instanceof Sprig_Field_HasMany)
{
if ($field instanceof Sprig_Field_ManyToMany)
{
if (isset($value))
{
if (empty($value))
{
    return new Database_Result_Cached(array(), '');
}
else
{
    $query = DB::select()
        ->where($model->pk(), 'IN', $value);
}
}
else
{                            
$query = DB::select()
    ->join($field->through)
        ->on($model->fk($field->through, $field->foreign_field), '=', $model->pk(TRUE))
    ->where($this->fk($field->through, $name), '=', $this->{$this->_primary_key});
}
}
else
{
if (isset($value))
{
$query = DB::select()
    ->where($model->pk(), 'IN', $value);
}
else
{
$query = DB::select()
    ->where($model->fk(NULL, $field->foreign_field), '=', $this->{$this->_primary_key});
}
}                    

$related = $model->load($query, NULL);

if ( ! $this->changed($name))
{
// We can assume this is the original value because no
// changed value exists
$this->_original[$name] = $field->value($related);
}
}
elseif ($field instanceof Sprig_Field_BelongsTo)
{
$related = $model->values(array($model->pk() => $value));
}
elseif ($field instanceof Sprig_Field_HasOne)
{
$related = $model->values(array($this->_model => $this->{$this->_primary_key}));
}

$value = $this->_related[$name] = $related;
}
}

return $value;
}

Irritant?
HEEL ERG!
Dan gaan we nu de voor en de nadelen vergelijken...
Wat een verassing, indenting wint!

Dat is in elk geval de mening van goede scripters...

Als je het irritant vind om telkens tabs te zetten, kies dan een andere editor...
2009-12-24 10:58:45
Jannick
nyan nyan nyan nyan
Berichten: 2080
avatar
Offline Stuur privébericht
+1 voor Darsstar, identing ftw. Je code word echt onleesbaar zonder. Waar ik ook een hekel aan heb is dit:
Code | Selecteer Alles
minimaliseren
1
2
3
4
<?
$i = 0;
if($i = 0) { echo "I is nul."; } else { echo "$I is niet nul.";}
?>


In plaats van
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
9
10
<?
$i = 0;
if($i = 0)
    {
        echo "I is nul.";
    }
    else {
        echo "I is niet nul.";
    }
?>
2009-12-24 13:05:48
RiiCk
ondergetekende.
Berichten: 1159
avatar
Offline Stuur privébericht
net een stukje van profiel gemaakt. Zouden jullie het anders doen qua form geving? Ik zelf vind dit de fijnste manier om te werken.

Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
<?PHP
include_once "config.php";

logincheck();

mysql_query("UPDATE `leden` SET `online`=NOW() WHERE `login`='". $data['login'] ."'") or die (mysql_error());
?>
<html>
    <head>
        <title>Major-Crime</title>
        <link rel="stylesheet" type="text/css" href="css-v1.css" />
    </head>
    <?php
        if(isset($_GET['naam'])) {
            $dbres = mysql_query("SELECT * FROM `leden` WHERE `login`='". mysql_real_escape_string($_GET['naam']) ."'") or die (mysql_error());
            $prof = mysql_fetch_array($dbres);
            
            if(empty($prof['familie']))
            {
                $familie = "Geen familie!";
            } else 
            {
                $familie = '<a href="famprof.php?fam='. $prof['familie'] .'">'. $prof['familie'] .'</a>';
            }
            
            $rank1 = array("","Noob");
            $rank1 = $rank1[$prof['rank']];
            $famrank = array("Zoekend!","Lid","Bankbeheerder","Onderbaas","Don");
            $famrank = $famrank[$prof['famrank']];
    ?>
    <table width="100%">
        <tr>
            <td>
                <h3>Profiel - <? echo $prof['login']; ?></h3>
            </td>
        </tr>
    </table>
    <table width="100%">
        <table width="50%" align="left">
            <tr>
                <td>
                    <h3>Algemeen</h3>
                </td>
            </tr>
            <tr>
                <td>Gebruikersnaam</td>
                <td><? echo $prof['login']; ?></td>
            </tr>
            <tr>
                <td>Rank</td>
                <td><? echo $rank1; ?></td>
            </tr>
            <Tr>
                <td>Bank</td>
                <td>&euro;<? echo round($prof['bank']); ?></td>
            </tr>
            <tr>
                <td><h3>Familie</h3></td>
            </tr>
            <tr>
                <td>Familie</td>
                <td><? echo $familie; ?></td>
            </tr>
            <tr>
                <td>Familie rank</td>
                <td><? echo $famrank; ?></td>
            </tr>
        </table>
        <table width="50%" align="right">
            <tr>
                <td>
                    <h3>Uitrusting</h3>
                </td>
            </tr>
            <tr>
                <td>Wapen</td>
                <td><? echo $prof['wapen']; ?></td>
            </tr>
            <tr>
                <td>Bescherming</td>
                <td><? echo $prof['bescherming']; ?></td>
            </tr>
            <tr>
                <td>Vliegtuig</td>
                <td><? echo $prof['vliegtuig']; ?></td>
            </tr>
            <tr>
                <td>Kogels</td>
                <td><? echo $prof['kogels']; ?></td>
            </tr>
        </table>
    </table>
        <?php
        }
        ?>
</html>
2009-12-24 13:09:18
Jannick
nyan nyan nyan nyan
Berichten: 2080
avatar
Offline Stuur privébericht
Mijn persoonlijke voorkeur is om <tr> en <td></td></tr> etc naast elkaar te zetten, slechte eigenschap maar zo doe ik het altijd. Zo is inderdaad wel overzichtelijker, misschien een goed idee om het aan te leren
Wat me trouwens opvalt is, dat je geeneens een <body> gebruikt... Doe je dat expres?
2009-12-24 13:12:28
RiiCk
ondergetekende.
Berichten: 1159
avatar
Offline Stuur privébericht
Doe ik inderdaad expres ja. Ik werk niet met iframes ofzo dus je bent ze nergens voor nodig. Daarnaast scheelt het natuurlijk weel ruimte om zo veel mogelijk scripts zo klein mogelijk te maken. Zo is mijn loguit misschien 6 regels lang ofzo
2009-12-24 13:14:14
Reageer op: "Script overzicht behouden"
Vorige pagina 1 | 2 | 3 | 4 | 5 Volgende pagina
Je kan niet reageren omdat je niet bent ingelogd. Inloggen of Aanmelden