[Alg] Slechtste programmeervoorbeelden deel 3

[Zoijar]

Zag deze net, nice. Was wel effectief

C++:

if (g_vm["file"].as<std::string>() == std::string("/home/smit/lucy.bin")) {
   glRotatef(-90, 1,0,0);
}
glTranslatef(-g_avg_x, -g_avg_y, -g_avg_z);

[TJHeuvel]

Ik moet voor een stage opdracht een licentie systeem vernieuwen.
Dus licentie, alles moet goed beveiligd zijn en compleet voutloos.

Een paar voorbeelden:

PHP:

if($formsubmit<>""){
    //form submitted, check licensecode etc..
    //$odbLicenses = new dbLicenses();
    $checkmail = new CheckEmail();
    $SMTPSERVER = "localhost";
    switch(false){
        case $checkmail->check($email, $SMTPSERVER):
            $message = "Please check your email address it seems to be invalid.";
            break;

    }

$Formsubmit <> ""? Is dat niet hetzelfde als empty($Formsubmit)?

En die switch statement? Waar slaat dat op

PHP:

        $sCheckNewsletter = '';
        if($newsletter=='true')
        $sCheckNewsletter = ' CHECKED';

Nogal vaag, maar Newsletter wordt ook nergens gedefinieerd.

Nog wat leuke Javascript:

PHP:

function CheckRequiredFields(srequiredfields, TheForm){
        var aREQUIREDFIELDS = srequiredfields.split(",");
        for (i=0; i<aREQUIREDFIELDS.length;i++){
            sField = aREQUIREDFIELDS[i];
            sValue = "";
            eval("sValue=TheForm."+sField+".value;");
            if(sValue==""){
                eval("TheForm."+sField+".focus();");
                alert(MSGREQUIRED);
                return false;           
            }
                
        }
        
        return true;        
    }
//Aanroepen doe je dus zo:
CheckRequiredFields("sname,email,license,reference", TheForm);

Eval?

Ooh, en dan het formulier:

PHP:

echo"<tr><font size='2' face='arial'><b>Reseller Request - Pricelist Request</b>";
if(isset($_POST['btnsubmit']))
echo "<tr>Thank you for your request.</tr><tr>We are verifying your details and will contact you regarding your request</tr>";

if(!isset($_POST['btnsubmit']))
{
  echo "<tr><td>";

Juist ja, elke regel heeft een nieuwe Echo.
En er wordt minstens 5x gekeken of dat formulier verstuurd is.

[ Voor 0% gewijzigd door TJHeuvel op 01-10-2008 12:53 . Reden: Typo ]

[!null]

licencie?

[Haan]

CyCloneNL schreef op woensdag 01 oktober 2008 @ 12:44:
Ik moet voor een stage opdracht een licencie systeem vernieuwen.
Dus licencie, alles moet goed beveiligd zijn en compleet voutloos.

Nog wat leuke Javascript:

PHP:

function CheckRequiredFields(srequiredfields, TheForm){
        var aREQUIREDFIELDS = srequiredfields.split(",");
        for (i=0; i<aREQUIREDFIELDS.length;i++){
            sField = aREQUIREDFIELDS\[i];
            sValue = "";
            eval("sValue=TheForm."+sField+".value;");
            if(sValue==""){
                eval("TheForm."+sField+".focus();");
                alert(MSGREQUIRED);
                return false;           
            }
                
        }
        
        return true;        
    }
//Aanroepen doe je dus zo:
CheckRequiredFields("sname,email,license,reference", TheForm);

Geniaal, zo ga ik voortaan ook methoden schrijven die meerdere strings als input hebben

[Grijze Vos]

terabyte schreef op dinsdag 30 september 2008 @ 22:48:
[...]

Zo'n website met veel flash ('strak design'), lage performance ('brakke code'), ononderhoudbaar (''Beter een goed uitgedachte website met [...] brakke code') en alleen werkend in IE ('Beter een goed uitgedachte website met [...] brakke code')?

Ik vraag me af hoe serieus ik jou moet nemen als je praat over PHP versus Python en dan aan komt met "alleen in IE", je hebt neem ik aan wel gehoord van het scheiden van je code en je opmaak?

Ik kan je overigens ook een mooi stukje Python laten zien waar je van over je nek gaat hoor, heb ik met een projectgroep een tijdje aan mogen sleutelen. (Resultaat: complete rewrite waarbij 3% van de code werd hergebruikt, bij de Acceptance Test bleek die 3% ook nog niet eens foutloos te zijn.)

Oh, en ik ken nog wel een hoogleraar die het e.e.a. te ranten heeft over de off-side rule waar Python en Haskell zo gretig gebruik van maken.

[TJHeuvel]

Hoe zou jij zoiets oplossen? Wat is wel een goede oplossing?

PHP:

 function CheckForm(Form)
 {
   for(var i = 0;i < Form.Elements.Length;i++)
  {
    if(Form.Elements[i].value == '')
    {
      alert('Vul aub alle velden in!');
       return;
    }
  } 
  Form.submit();
 }

Maar hoe geef je dan optionele velden aan.

[.oisyn]

Zoijar schreef op woensdag 01 oktober 2008 @ 10:23:
Zag deze net, nice. Was wel effectief

C++:

if (g_vm["file"].as<std::string>() == std::string("/home/smit/lucy.bin")) {
   glRotatef(-90, 1,0,0);
}
glTranslatef(-g_avg_x, -g_avg_y, -g_avg_z);

Heh, in Tomb Raider: Legend stond dit:

C++:

    // TR7 specific hack: portals in vtrain units are too small, which results in culling issues, so we're going to double them in size.
    bool tr7bigportalhack = !strnicmp(pStreamUnit->baseAreaName, "vtrain", 6) || !strnicmp(pStreamUnit->baseAreaName, "vehone", 6);

    ISceneCell * pCell = pStreamUnit->pCellGroup->GetCell(0);
    for (uint32 i = 0; i < pTerrain->numStreamUnitPortals; i++)
    {
        if (tr7bigportalhack)
        {
            const Vector3 * quad = pTerrain->streamUnitPortals[i].quad;
            Vector3 portalVerts[4]; // can be a local buffer because scene copies the verts anyway
            Vector3 midpoint = quad[0];
            for (uint32 p = 1; p < 4; p++)
                midpoint += quad[p];
            midpoint *= 0.25f;
            for (uint32 p = 0; p < 4; p++)
                portalVerts[p] = 2.f * quad[p] - midpoint;   //this means: midpoint + (quad[p] - midpoint) * 2.f;
            pCell->CreateDynamicPortal(0, portalVerts, 4);
        }
        else
        {
            pCell->CreateDynamicPortal(0, pTerrain->streamUnitPortals[i].quad, 4);
        }
    }

Je moet toch wat als op het laatste moment bepaalde portals te klein blijken geauthored en je niet de mogelijkheid hebt om de content aan te passen

[MichelVH]

CyCloneNL schreef op woensdag 01 oktober 2008 @ 12:44:
$Formsubmit <> ""? Is dat niet hetzelfde als empty($Formsubmit)?

Nee Ook voor de string '0' returnt empty true.

[TJHeuvel]

MichelVH schreef op woensdag 01 oktober 2008 @ 13:09:
[...]
Nee Ook voor de string '0' returnt empty true.

Oh, wat is het dan? Gewoon hetzelfde als !=? Waarom zit dat dan er twee keer in?

[NMe]

[alg] Slechtste programmeervoorbeelden deel 4