[alg] Slechtste programmeervoorbeelden deel 4

[Xesxen]

PHP:

class x {
    function __construct(){
        $this->db = new Database('mysql:host=localhost;dbname=databasename',
            'username',
            'password',
...

Come'on, er is een configfile met wachtwoorden erin hoor

[Verwijderd]

Firesphere schreef op donderdag 20 juni 2013 @ 14:36:
[...]

Vermoedelijk zat je precies in een deploymentmoment en heeft er iemand een dom deploymentmoment gekozen?

Hmm, maar dat is ook redelijk te marginaliseren door je webserver naar een "current" symlink te verwijzen, dan eerst je code te kopieren of te fetchen en daarna je symlink naar de nieuwe code te verwijzen.

Ik zou met zo'n groot publiek in ieder geval zorgen dat dit soort momenten in ieder geval zo min mogelijk voorkomen. Neemt niet weg dat er misschien evengoed wel een goede reden achter zit.

[HeSitated]

Na eerst meer dan 70 regels foutmeldingen:

<b>Warning</b>: imagecreatefromstring() [<a href='function.imagecreatefromstring'>function.imagecreatefromstring</a>]: Empty string or invalid image in <b>/home/gunbv/domains/<DOMAIN>.nl/public_html/catalog/model/tool/watermark.php</b> on line <b>12</b><br />
<b>Warning</b>: fopen(/home/gunbv/domains/<DOMAIN>.nl/public_html/system/logs/error.txt) [<a href='function.fopen'>function.fopen</a>]: failed to open stream: Permission denied in <b>/home/gunbv/domains/<DOMAIN>.nl/public_html/system/library/log.php</b> on line <b>12</b><br />
<br />
<b>Warning</b>: fwrite(): supplied argument is not a valid stream resource in <b>/home/gunbv/domains/<DOMAIN>.nl/public_html/system/library/log.php</b> on line <b>14</b><br />
<br />
<b>Warning</b>: fclose(): supplied argument is not a valid stream resource in <b>/home/gunbv/domains/<DOMAIN>.nl/public_html/system/library/log.php</b> on line <b>16</b><br />
<b>Warning</b>: imagesx(): supplied argument is not a valid Image resource in <b>/home/gunbv/domains/<DOMAIN>.nl/public_html/catalog/model/tool/watermark.php</b> on line <b>17</b><br />
<b>Warning</b>: fopen(/home/gunbv/domains/<DOMAIN>.nl/public_html/system/logs/error.txt) [<a href='function.fopen'>function.fopen</a>]: failed to open stream: Permission denied in <b>/home/gunbv/domains/<DOMAIN>.nl/public_html/system/library/log.php</b> on line <b>12</b><br />
<br />

Krijg je bij een klik met de rechter-muisknop de volgende melding:

Kopi�ren van afbeeldingen is niet toegestaan Copyright� <DOMAIN>.nl

Dan wil je het een en ander melden, krijg je:

Warning: fsockopen() [function.fsockopen]: unable to connect to mail.bginventaris.nl:2525 (Connection refused) in /home/gunbv/domains/<DOMAIN>.nl/public_html/vqmod/vqcache/vq2-system_library_mail.php on line 171
Warning: fopen(/home/gunbv/domains/<DOMAIN>.nl/public_html/system/logs/error.txt) [function.fopen]: failed to open stream: Permission denied in /home/gunbv/domains/<DOMAIN>.nl/public_html/system/library/log.php on line 12

Notice: Error: Connection refused (111) in /home/gunbv/domains/<DOMAIN>.nl/public_html/vqmod/vqcache/vq2-system_library_mail.php on line 174

Lijkt me het melden hier meer dan waard.....

[ Voor 16% gewijzigd door HeSitated op 19-07-2013 21:29 ]

[Kajel]

In een webapp waar ik aan werk, moeten we wat waardes doorsturen naar een service/formulier welke niet door ons beheerd wordt. Aan die kant gebeuren er heel vreemde dingen met encoding, waardoor wij ons in bochten moeten wringen. Dit heeft geleid tot het volgende stuk code met hilarische comments van een collega van me

Java:

private void addParameter(StringBuilder builder, String name, Object value) {
        if (value == null) {
            return;
        }

        if (String.valueOf(value).trim().length() == 0) {
            return;
        }

        builder.append('&').append(name).append('=');
        // in theory, we should URL-encode the values
        // unfortunately, the customer-service form does not understand this; it apparently
        // goes through great pains 'understanding' unencoded characters, going so far as to
        // even understand the unescaped '+'.
        // therefore, we do not encode our values.
        // somewhere, an encoding-kitten is crying :'-(

        // UPDATE: it gets worse. The customer-service form doesn't understand ANY encoded value,
        // not even the lovely %20 (space); and, as said before, not the '+' either.
        // as a desparate workaround, we remove all spaces from the value; this might annoy our
        // customers, but I hope they prefer 'vanderDongen' over 'van%20der%20Dongen'.
        // The encoding-kitty is now attempting suicide.
        String encodedValue = String.valueOf(value);        // haha, look at me still calling it an encoded value
        encodedValue = encodedValue.replaceAll(" ", "");

        builder.append(encodedValue);
    }

[Haan]

C#:

public interface IFoo
{
    IList<IdValuePair> BarIds { get; }
}

public class IdValuePair
{
    public Guid Id { get; set; }
    public string Value { get; set; }
}

Lijkt verdacht veel op een datastructuur die standaard in .NET aanwezig is..

[Afvalzak]

Kajel schreef op woensdag 24 juli 2013 @ 11:38:
...

Whaha, vooral die

code:

// haha, look at me still calling it an encoded value

.

[Firesphere]

The poor encoding kitty!

[mcDavid]

Kajel schreef op woensdag 24 juli 2013 @ 11:38:
In een webapp waar ik aan werk, moeten we wat waardes doorsturen naar een service/formulier welke niet door ons beheerd wordt. Aan die kant gebeuren er heel vreemde dingen met encoding, waardoor wij ons in bochten moeten wringen. Dit heeft geleid tot het volgende stuk code met hilarische comments van een collega van me

Java:

private void addParameter(StringBuilder builder, String name, Object value) {
        if (value == null) {
            return;
        }

        if (String.valueOf(value).trim().length() == 0) {
            return;
        }

        builder.append('&').append(name).append('=');
        // in theory, we should URL-encode the values
        // unfortunately, the customer-service form does not understand this; it apparently
        // goes through great pains 'understanding' unencoded characters, going so far as to
        // even understand the unescaped '+'.
        // therefore, we do not encode our values.
        // somewhere, an encoding-kitten is crying :'-(

        // UPDATE: it gets worse. The customer-service form doesn't understand ANY encoded value,
        // not even the lovely %20 (space); and, as said before, not the '+' either.
        // as a desparate workaround, we remove all spaces from the value; this might annoy our
        // customers, but I hope they prefer 'vanderDongen' over 'van%20der%20Dongen'.
        // The encoding-kitty is now attempting suicide.
        String encodedValue = String.valueOf(value);        // haha, look at me still calling it an encoded value
        encodedValue = encodedValue.replaceAll(" ", "");

        builder.append(encodedValue);
    }

geniaal!
wel goed commentaar, zo is het iig voor iedereen duidelijk waarom zo'n stuk ranzige code erin staat. Voor je het weet wordt je fix anders goed bedoeld zomaar verwijderd.

[Styxxy]

Haan schreef op woensdag 24 juli 2013 @ 11:52:
[...]
Lijkt verdacht veel op een datastructuur die standaard in .NET aanwezig is..

Och joh, .NET klasses (KeyValuePair) is te risky om te gebruiken. Beter zelf maken, kan niet mis lopen! (Overigens is een Dictionary<Guid, String> niet hetzelfde als een List<KeyValuePair<Guid, String>>.)

[Kajel]

Afvalzak schreef op woensdag 24 juli 2013 @ 12:07:
[...]


Whaha, vooral die

code:

// haha, look at me still calling it an encoded value

.

Ik verdenk mijn collega er sterk van dat hij die variabele naam expres heeft laten staan om die comment er nog even achter te kunnen plakken

[Alain]

Gisteren heb ik een webapplicatie "gefixed" die het niet meer deed. Bleek dat er op 3 plekken hardcoded een proxy server ingesteld was. Daarnaast faalt de hele applicatie als de RSS reader niet wil laden.

Gelukkig is dit alleen een intern hebbedingetje en leveren we geen webapplicatie's.

[Phoib]

PHP:

 function sanitize($sql) {
    //return mysql_real_escape_string($sql);
    return $sql;
}

Hoezo de programmeur deed aan veiligheid.

of in een andere class

PHP:

 function makeCountryDropDown() {
    return "/*Hier een string met HTML opties voor landen, slechts 10k characters lang*/";
}

Echt enig...

[.oisyn]

Sowieso een rare naam voor de parameter van sanitize(). Wat je wilt sanitizen stop je in een SQL statement, maar wat je wilt sanitizen is zelf geen SQL. Tenzij je SQL statements in je database wilt gaan stoppen oid

[ Voor 17% gewijzigd door .oisyn op 25-07-2013 23:47 ]

[dcm360]

SQL-statements de database in, en vervolgens kan je in jouw code de statements handig uit de database trekken om ze weer uit te voeren. Gaaf toch?
(en ergens bekruipt mij het gevoel dat iemand dit ooit echt zo gebouwd heeft. Waarschijnlijk in combinatie met PHP).

[BM]

Haan schreef op woensdag 24 juli 2013 @ 11:52:

C#:

public interface IFoo
{
    IList<IdValuePair> BarIds { get; }
}

public class IdValuePair
{
    public Guid Id { get; set; }
    public string Value { get; set; }
}

Lijkt verdacht veel op een datastructuur die standaard in .NET aanwezig is..

Been there, done that, got the t-shirt. Wat heet, zoiets heb ik gister nog gedaan Niet zonder reden overigens, de 1e versie gebruikte keyvaluepairs en een dictionary, maar daar werkten anderen stukjes niet goed mee samen. En dit biedt (in ons geval) nog iets meer ruimte voor uitbreidingen later ook.

[HMS]

BM schreef op vrijdag 26 juli 2013 @ 07:38:
[...]

Been there, done that, got the t-shirt. Wat heet, zoiets heb ik gister nog gedaan Niet zonder reden overigens, de 1e versie gebruikte keyvaluepairs en een dictionary, maar daar werkten anderen stukjes niet goed mee samen. En dit biedt (in ons geval) nog iets meer ruimte voor uitbreidingen later ook.

C#:

var dictionary = new Dictionary<Guid, string>();
IList<KeyValuePair<Guid, string>> list = dictionary.ToList();

IList interface even helemaal uitgetypt voor clarity, aub geen comments over het gebruik van 'var'

[ Voor 9% gewijzigd door HMS op 26-07-2013 12:15 ]

[BM]

HMS schreef op vrijdag 26 juli 2013 @ 12:14:
[...]

C#:

var dictionary = new Dictionary<Guid, string>();
IList<KeyValuePair<Guid, string>> list = dictionary.ToList();

IList interface even helemaal uitgetypt voor clarity, aub geen comments over het gebruik van 'var'

Dat had gekunt, en waarschijnlijk ook wel gewerkt. Weet ook wel dat ik een omslachtige oplossing gekozen heb, maar wel 1 die in ons geval voordelen gaat hebben als we zaken aan ons systeem toevoegen

[HeSitated]

Hmm, lees even dit

[HMS]

In principe had je ook nog de Tuple class kunnen gebruiken .
MSDN: Tuple Class (System)

edit:
Hmm, had nog niet gerefresht .

HeSitated schreef op vrijdag 26 juli 2013 @ 12:36:
Hmm, lees even dit

Wel een duidelijk verschil in performance tussen een Class en een Struct. Maar of het uitmaakt is natuurlijk wel afhankelijk van je use case. Meestal ga je die nanosecondes toch niet merken.

[ Voor 59% gewijzigd door HMS op 26-07-2013 12:42 ]

[MBV]

Zonder 3 regels commentaar met o.a. verwijzing naar die url zou ik het sowieso weghalen als ik het tegenkwam.

[Phoib]

.oisyn schreef op donderdag 25 juli 2013 @ 23:47:
Sowieso een rare naam voor de parameter van sanitize(). Wat je wilt sanitizen stop je in een SQL statement, maar wat je wilt sanitizen is zelf geen SQL. Tenzij je SQL statements in je database wilt gaan stoppen oid

Het erge is, dit was een van de duidelijkere namen...

De rest staat vol met $r, $e, $m...

[HeSitated]

HMS schreef op vrijdag 26 juli 2013 @ 12:39:
Wel een duidelijk verschil in performance tussen een Class en een Struct. Maar of het uitmaakt is natuurlijk wel afhankelijk van je use case. Meestal ga je die nanosecondes toch niet merken.

Om die performance vergelijking was het mij niet te doen, zeker niet meer dan een aardige bijkomstigheid.

Maar met een Tuple kun je zoveel meer dan met een KVP...

[azerty]

Ik vraag me af waarom ik dit ooit bedacht heb -_-

code:

$SE = array("sometruerandomnamethatwillneverevereveroccurortherewillbeaproblemwiththedisplayofit");
                if($fill && $filldata["teller"] != "0"){
                    $SE = explode("|",$filldata["SE"]);
*snip*
                }               
                if($SE[0] != "sometruerandomnamethatwillneverevereveroccurortherewillbeaproblemwiththedisplayofit"){

[NMe]

Met ruim 10k replies is dit topic intussen wel vol. Tijd voor een nieuwe!

[alg] Slechtste programmeervoorbeelden deel 5