Goede code opmaak

Op donderdag 27 december 2001 23:19 schreef elnino het volgende:
over ... ? ... : ..., ik vind persoonlijk dat je die dingen gewoon nooit moet gebruiken

Op donderdag 27 december 2001 23:30 schreef marcusk het volgende:

[..]

dat heb ik hier wel vaker zien langskomen als 'evil language construct'. maar wat is er nou mis mee?

Op donderdag 27 december 2001 23:33 schreef Jppr het volgende:
het is geen "nette" code, ofmdat je geen duidelijk overzicht hebt hoe het zit, over een vraagteken en een dubbele punt lees je snel heen.

Daarbij komt ook dat je maar een statement kan plaatsen, en als er eentje bij komt, je plots toch een hele if moet maken.

• Goto - dit maakt van je code een zooitje, en het voordeel van 'hogere programmeertalen' t.o.v. ASM is dat je for/while/do-lussen hebt, waarbij je Goto dus niet nodig hebt. Gelukkig kent PHP geen Goto...
• Global variabelen - De argumenten en de 'return' van een functie zijn bedoeld om waardes af te geven, niet willekeurige variabelen. Anders krijg je tijdens het debuggen zoiets van: "Hé waarom is die variabele i 10 i.p.v. 3?".
  In PHP raadt ik het sowieso aan om van de variabelen $REMOTE_HOST enzo eerst constanten te maken, die zijn namelijk wel overal beschikbaar, en bovendien die kunnen niet van waarde veranderen, dus onverwachte effecten zijn er ook niet.

Op vrijdag 28 december 2001 00:06 schreef mbravenboer het volgende:
Het is jammer dat niet meer talen een degelijke syntax hebben voor een if constructie als expressie. Die ? ... : ... zuigt uiteraard .

Er is in feite geen enkele reden waarom de gewone if syntax niet ook als expressie zou kunnen worden gebruikt...

Op donderdag 27 december 2001 23:30 schreef marcusk het volgende:
dat heb ik hier wel vaker zien langskomen als 'evil language construct'. maar wat is er nou mis mee?

Op vrijdag 28 december 2001 00:26 schreef Crazy_D het volgende:
Dat snap ik dus ook niet. Ik vind zo'n constructie ideaal vooral als het om iets gaan als
boolwaarde ? 1 : 2
of zo. Helemaal als het een argument voor een functie is.
Hoofdregel blijft wel dat het leesbaar moet zijn. Ik heb ze wel gezien dat er na de ? een lap code kwam, en daarna nog ergens een : met ook een lap erachteraan. Da's gewoon crap natuurlijk...
(de VB versie is Iif() en die gebruik ik erg vaak voor kleine dingen, maar zodra het iets meer als 1 waarde, maar ik er een gewone if van)

Op vrijdag 28 december 2001 00:34 schreef elnino het volgende:
Het is gewoon lastiger bij debuggen enzo. Het maakt code soms gewoon onoverzichtelijk, en zeker voor anderen.

Op vrijdag 28 december 2001 01:53 schreef OiSyN het volgende:
ja maar juist bij die statement kun je net zo goed zeggen

code:

1 2 3 4

if (blaat) a = 1; else a = 2;

bij dit wordt het wat anders:

1
2
3
4

int p1 = ( blaat1 ? 1 : 0 );
int p2 = ( blaat2 ? 2 : 1 );

functie(p1, p2);

Goede code opmaak:

PHP:

1 2 3 4 5 6 7 8 9 10 11 12 13

<? function gablaten($iets) { if($iets=="dat") { return "Dit"; } else { return "Dat"; } } ?>

Op vrijdag 28 december 2001 11:28 schreef Thargol het volgende:
De huidige kleuren zijn goed geschikt voor een witte achtergrond, niet voor paars.

Op vrijdag 28 december 2001 02:09 schreef OiSyN het volgende:
hier kun je gewoon geen if gebruiken

Op vrijdag 28 december 2001 11:28 schreef Thargol het volgende:
Qua layout heb ik niets op aan te merken, maar de kleuren zijn hopeloos. Groen op paars en rood op paars hebben gewoon te weinig contrast om goed leesbaar te zijn. En het lettertype zou ook wat groter mogen. De huidige kleuren zijn goed geschikt voor een witte achtergrond, niet voor paars.

Op vrijdag 28 december 2001 13:30 schreef klokop het volgende:
...en doen we in HTML dan ook keurig inspringen na een <td>? Als in:

Op vrijdag 28 december 2001 13:56 schreef klokop het volgende:
Dat is dus gewoon niet leesbaar.

Op vrijdag 28 december 2001 11:28 schreef Thargol het volgende:

[..]

Qua layout heb ik niets op aan te merken, maar de kleuren zijn hopeloos. Groen op paars en rood op paars hebben gewoon te weinig contrast om goed leesbaar te zijn. En het lettertype zou ook wat groter mogen. De huidige kleuren zijn goed geschikt voor een witte achtergrond, niet voor paars.

Op vrijdag 28 december 2001 14:07 schreef Jppr het volgende:

[..]

die kleuren kiezen wij zelf niet, die worden door het forum hier gemaakt met syntax highlighting. (highlight_string() voor wie het zich afvraagt: http://www.php.net/manual/en/function.highlight-string.php)

<td>
Bla<br>
Wop<br>
Koeien<br>
</td>

Op vrijdag 28 december 2001 14:12 schreef Taradino het volgende:

[..]

Maar dit is één van de weinige dingen die ACM/Kees wel zouden kunnen veranderen. De kleuren staan namelijk gewoon in je php config file, en je kan ze ook aanpassen. Idee?

Op vrijdag 28 december 2001 01:53 schreef OiSyN iets over die mooie operator

1
2
3
4
5

If blaat Then 
    foo=blabla 
Else
    foo=blablabla
EndIf

Dus "<td>[enter]Bla" is dus anders als "<td>Bla", maar weer hetzelfde als "<td> Bla". Met name in Netscape 4.x krijg je hiermee vaak ongewenste layout effecten.

Op vrijdag 28 december 2001 14:56 schreef X-Fire het volgende:

[...]
Als we het toch over nette code hebben moet dat niet zijn:

code:

1 2 3 4 5

<td> Bla<br /> Wop<br /> Koeien<br /> </td>

Deze week heb ik dat op w3c gelezen. Elke Tag moet gesloten worden is de aanbeveling.
[...]

Op vrijdag 28 december 2001 15:07 schreef klokop het volgende:
Maar dan natuurlijk:

code:

1 2 3 4 5

<td> Bla<br /> Wop<br /> Koeien<br /> </td>

1
2
3
4
5

<td>
Bla<br>
Wop<br>
Koeien<br>
</td>

1
2
3
4
5

<td>
Bla<br />
Wop<br />
Koeien<br />
</td>

Op vrijdag 28 december 2001 14:23 schreef farlane het volgende:
zou je willen dat VB ook zo'n operator had. Scheelt 10% code.

1
2
3
4
5
6
7

<table>
    <tr>
      <td>
        blah blah
      </td>
    </tr>
</table>

1
2
3
4
5

<table>
    <tr>
      <td>blah blah</td>
    </tr>
</table>

Op vrijdag 28 december 2001 15:13 schreef mbravenboer het volgende:
Sorry maar ik moet ff gapen .

Dit gaat nergens meer over!

Op vrijdag 28 december 2001 15:13 schreef mbravenboer het volgende:
Sorry maar ik moet ff gapen .

Dit gaat nergens meer over!

Op vrijdag 28 december 2001 15:34 schreef mietje een lang verhaal:
...
Toen Wu Li weer bij zinnen was en op de grond zittend zijn buil betastte sprak de Meester: "Zo geindenteerd was Uw hoofd door Heilige Wetten, dat Uw Geest erdoor belemmerd werd. U behoefde node een exdentatie!". Dit horende werd Wu Li de Verlichting ten deel.
^{Copyrighted 2001 by /me}

Op vrijdag 28 december 2001 15:13 moest mbravenboer gapen:
Sorry maar ik moet ff gapen .

Dit gaat nergens meer over!

novice asked the Master: "Here is a programmer that never designs, documents, or tests his programs. Yet all who know him consider him one of the best programmers in the world. Why is this?"

The Master replies: "That programmer has mastered the Tao. He has gone beyond the need for design; he does not become angry when the system crashes, but accepts the universe without concern. He has gone beyond the need for documentation; he no longer cares if anyone else sees his code. He has gone beyond the need for testing; each of his programs are perfect within themselves, serene and elegant, their purpose self-evident. Truly, he has entered the mystery of the Tao."

Op vrijdag 28 december 2001 15:21 schreef Crazy_D het volgende:

Hmm die heb je. Iif(voorwaarde, truewaarde, falsewaarde)

Op zaterdag 29 december 2001 16:39 schreef farlane het volgende:

[..]

Is dat een VB operator?

1
2
3
4
5
6
7
8

Function Iif (exp as Boolean, vTrue as Variant, vFalse as Variant) as Variant

    If exp Then
      Iif = vTrue
    Else
      Iif = vFalse

Exit Function

Op zaterdag 29 december 2001 16:39 schreef farlane het volgende:
Is dat een VB operator?

IIf Function

Returns one of two parts, depending on the evaluation of an expression.

Syntax

IIf(expr, truepart, falsepart)

The IIf function syntax has these named arguments:

Part Description
expr Required. Expression you want to evaluate.
truepart Required. Value or expression returned if expr is True.
falsepart Required. Value or expression returned if expr is False.


Remarks

IIf always evaluates both truepart and falsepart, even though it returns only one of them. Because of this, you should watch for undesirable side effects. For example, if evaluating falsepart results in a division by zero error, an error occurs even if expr is True.

Op zaterdag 29 december 2001 16:54 schreef Crazy_D het volgende:

Helaas is het Remarks stukje (dus) ook waar, en dat vind ik wel erg spijtig... (of dat in VB.net beter is weet ik eigenlijk niet, heb 'm hier niet geinstalleerd staan).

Ander vraagje: ik heb dus bvb drie files article.php, author.php en category.php. Daarmee wordt contenet uit de db gehaald en ge-output.
Maar in de admin area moeten ook files bestaan om die zaken te editten. Is het dan "slecht" om deze admin files ook article.php, enzo te noemen, ook al staan ze in een andere dir?

-----------------------------------------------------------
Slecht? Neuh niet echt, maar ik vind het zelf handig om het dan bijv. edit_article.php, edit_category.php te gebruiken. Dan zie je meteen wat het doet, itt bij de eerste benaming.

Op zaterdag 29 december 2001 17:32 schreef Jppr het volgende:
nog even hierover:
[..]

om de article bvb volledig te kunnen adminnen moet ik er kunnen toevoegen, verwijderen en editten.

Wat is dan netter?

* drie files: add_article.php, edit_article en rem_article ofzo

* 1 file: admin_article.php waarin de drie functies onderscheiden worden door bvb een var $func ofzo...

Op zondag 30 december 2001 13:05 schreef Jppr het volgende:
het gaat niet meer echt over nette code, maar heeft er toch nog iets mee te maken:

standaard hebben we als url bvb
.../article.php?aid=17

nu kun je er vrij makkelijk van maken:
.../article/17

nu, wat vinden jullie pers. mooier/professioneler/...

Op zondag 30 december 2001 13:12 schreef whoami het volgende:

[..]

Volgens mij zijn beiden niet goed. Het article-id mag je nl niet kunnen afleiden uit de url.

Op zondag 30 december 2001 13:12 schreef whoami het volgende:

[..]

Volgens mij zijn beiden niet goed. Het article-id mag je nl niet kunnen afleiden uit de url.

Op donderdag 27 december 2001 23:19 schreef elnino het volgende:
Maar met die nieuwe phpBB Coding Standard ben ik het wel mee eens, behalve dan met het gedeelte over ... ? ... : ..., ik vind persoonlijk dat je die dingen gewoon nooit moet gebruiken, dan kan je het ook niet fout doen. Een if/else-statement is dan wel langer, maar dan wel 100 keer duidelijker.

1
2
3
4

// Initialize package count
int     l_PackageCount  = (p_SendHeader ? 1 : 0);

// Now calculate how many other packages to send

1
2
3
4
5
6
7

MyStringClass MyClass::SomeFunction()
{
// Calculate some booleans

// Return output string based on values
return ((BoolA && BoolB) ? "valid" : "invalid");
}

Op zondag 30 december 2001 13:43 schreef Taradino het volgende:

[..]

En dat is waarom?

Hier heb ik toch nog nooit van gehoord.

Op zaterdag 29 december 2001 16:51 schreef OiSyN het volgende:

nee het is gewoon een functie