[alg] Slechtste programmeervoorbeelden deel 4

[CodeCaster]

Ik was pas wat aan het lezen over hoe sessies in PHP ook alweer precies werken (met name het moment van aanmaken van het sessiebestand, bij session_start() of pas bij het schrijven naar $_SESSION? Bij het eerste, blijkbaar).

Daarbij kwam ik de volgende aangedragen juweeltjes tegen:

quote: http://www.php.net/manual....session-start.php#107812

I had a problem with the session_start(); like this: "Warning: session_start(); [function.session-start]: Cannot send session cache limiter - headers already sent".

This is my php code:

PHP:

session_start();

if ($_SESSION['nome'] == ""){
die('Você não pode acessar essa pagina!');
} else {
}

The problem is the "" that we need to chango to ''.

Correct code:

PHP:

session_start();

if ($_SESSION['nome'] == ''){
die('Você não pode acessar essa pagina!');
} else {
}

Want het maakt natuurlijk helemaal niet uit dat je een niet-bestaande index probeert te bereiken, die de Notice: Undefined index: nome uitpoept, zodat de cookies niet meer verstuurd kunnen worden.

Ik durf te wedden dat 'ie in de tussentijd gewoon wat heeft lopen rommelen met die $_SESSION['nome'] en er wél een waarde in heeft gezet, en vervolgens met de moed der wanhoop z'n enkele quotes vervangen door dubbele waarna het plotseling werkte, doordat de notice verdween en dus de sessie-cookie-header wel weggeschreven kon worden.

Of deze dan:

quote: http://www.php.net/manual/en/book.session.php#85021

Persistence of session data in included file also has one more aspects if you are including a file using/over the http connection.
Session will not be available without refresh (it's a separate thread even if you are on the same domain/server), which is again a correct behavior, but if you are including a file from local file system then it should have session data if you are setting and accessing the $_SESSION variable even before page refresh because it is available globally.

PHP:

session_start(); 
$_SESSION['example']="yes"; 

Include("otherfile");

Session variable will be available in included file.

PHP:

session_start(); 
$_SESSION['example']="yes"; 

Include("http://domain.com/otherfile.php");

Session will NOT be available in other file.
BUt if both files are on same server then after refresh values will be available.

Gewoon totaal niet snappen dat je met het includen van een PHP-script over een URL juist de uitvoer van dat script insluit, wat ook gewoon in de handleiding staat:

quote: http://php.net/manual/en/function.include.php

(Over "URL fopen wrappers"): This is not strictly speaking the same thing as including the file and having it inherit the parent file's variable scope; the script is actually being run on the remote server and the result is then being included into the local script.

En het pas beschikbaar zijn van de sessiedata na een refresh als de scripts wel op dezelfde server draaien, bijvoorbeeld include('http://localhost/foobar.php'), toeschrijven aan het uitgevoerd worden door verschillende threads.

Nee, want die zouden hetzelfde sessiebestand lezen en daar ook op locken gedurende het hele request, en dus over dezelfde sessiedata beschikken. De URL-wrapper die het tweede script opent doet echter simpelweg niet aan cookies, dus krijgt een eigen sessie. Vervolgens is de sessiedata uit de include dus gewoon verloren (de volgende include genereert immers net zo goed weer een nieuwe sessie), dus z'n statement over de refresh slaat ook op kaas.

Volgens mij kunnen ze gewoon beter die hele comments-sectie verwijderen van de manual-pages. Hoewel, af en toe is het prima vermaak.

[HeSitated]

Vandaag Oracle Database 10g Express Edition geprobeerd te installeren met een oudere versie.

Krijg ik de volgende foutmelding:

[ValHallASW]

Pop quiz: wat is de goede sorteervolgorde van baa - báb - bac - bba?

spoiler:

Dat hangt er vanaf! De correcte sorteervolgorde in het Nederlands is baa - báb - bac - bba, maar die in het IJslands is baa - bac - báb - bba!

Vraag het je favoriete programmeertaal ook 'ns. Als je het naïef in Python doet dan doe je bijvoorbeeld:

code:

>>> print ' '.join(sorted([u"baa", u"báb", u"bac", u"bba"]))
baa bac bba báb

en dat is dus níet goed. OK, wat doe je dan? Je zou, bijvoorbeeld, je in kunnen lezen over collations, de ICU-library kunnen installeren en vervolgens dit doen:

code:

>>> print ' '.join(sorted([u"baa", u"báb", u"bac", u"bba"],  cmp=icu.Collator.createInstance(icu.Locale('is_IS')).compare))
baa bac báb bba

Maar waarom zou je dat doen als je ook dit kunt doen (let overigens ook op de functionele scheiding, en noteer het aantal verschillende dingen dat deze functie doet - vooral de dingen die niet uit de naam blijken):

quote: (quote want niet-europese tekens die niet werken in code-blokken)

def icelandic_sort(categorylist):
    finallradeh,s_radeh=[],[]
    alphabets=u' 0123456789aábcdðeéfghiíjklmnoópqrstuúvwxyzþæö' #Space is first then numbers and then correct Alphabet letter and after all of letters is ZWNJ 
    for radeh in categorylist:
            radeh=radeh.replace(u'[[category:',u'').replace(u']]',u'')
            coderadeh=radeh
            for i in range(0,len(alphabets)):
                alphabet=alphabets[i]
                if i<10:
                    j='0'+str(i)
                else:
                    j=str(i)
                coderadeh=coderadeh.replace(alphabet,j)
            s_radeh.append(coderadeh+u'0000000000000000000@@@@'+radeh)
    s_radeh=list(set(s_radeh))
    sortedradeh=sorted(s_radeh)
    for radeh in sortedradeh:
        radeh=u'[[category:'+radeh.split(u'@@@@')[1]+u']]'
        finallradeh.append(radeh)
    return finallradeh

(aangepast van een Perzisch origineel, maar ik dacht dat jullie iets europesere letters zouden kunnen waarderen)

Scheelt toch maar weer een library installeren.

[ Voor 3% gewijzigd door ValHallASW op 11-05-2012 22:17 ]

[Herko_ter_Horst]

ValHallASW schreef op vrijdag 11 mei 2012 @ 22:13:
Pop quiz: wat is de goede sorteervolgorde van baa - báb - bac - bba?

Dit soort vragen kun je ook stellen voor diverse String-manipulaties zoals toUpperCase(...) en toLowerCase().

[ValHallASW]

Herko_ter_Horst schreef op vrijdag 11 mei 2012 @ 22:27:
Dit soort vragen kun je ook stellen voor diverse String-manipulaties zoals toUpperCase(...) en toLowerCase().

http://www.moserware.com/...ode-pass-turkey-test.html ;-)

Ook nog eentje om over na te denken als je met niet-ascii-meuk werkt:

code:

>>> u'ä' == u'ä'
False
>>> u'ä'
u'\xe4'
>>> u'ä'
u'a\u0308'

code:

>>> import unicodedata
>>> unicodedata.normalize("NFC", u"\xe4") == unicodedata.normalize("NFC", u"a\u0308")
True

Die moet ik nog altijd een keertje gaan toepassen op gebruikersnaamveldjes ;-)

[Verwijderd]

PHP:

function index()
{
    if ($this->ion_auto->logged_in() || 2+2==4)
    {
        $this->load->view('upload');
    }

    ... snip ...
}

Bron

[ Voor 19% gewijzigd door Verwijderd op 14-05-2012 23:56 ]

[Soultaker]

ValHallASW schreef op vrijdag 11 mei 2012 @ 22:59:

code:

>>> import unicodedata
>>> unicodedata.normalize("NFC", u"\xe4") == unicodedata.normalize("NFC", u"a\u0308")
True

Die moet ik nog altijd een keertje gaan toepassen op gebruikersnaamveldjes ;-)

Da's ook niet zaligmakend hoor.

>>> s = u"Sοultakеr"
>>> t = u"Soultaker"
>>> unicodedata.normalize("NFC", s) == unicodedata.normalize("NFC", t)
False

offtopic:
Waarom displayt bovenstaande code niet goed in code/cmd/pre tags?

[RayNbow]

Soultaker schreef op dinsdag 15 mei 2012 @ 02:27:

offtopic:
Waarom displayt bovenstaande code niet goed in code/cmd/pre tags?

Omdat code/cmd/pre tags een slecht programmeervoorbeeld zijn. Prima ontopic dus.

[.oisyn]

Soultaker schreef op dinsdag 15 mei 2012 @ 02:27:

offtopic:
Waarom displayt bovenstaande code niet goed in code/cmd/pre tags?

Omdat niet-latin-1 tekens worden omgezet naar html entities voordat de RML parser er überhaupt overheen gaat, waarna ze vervolgens verbatim worden afgedrukt omdat het code/cmd/pre tags zijn.

GoT is niet UTF-8

[ Voor 3% gewijzigd door .oisyn op 15-05-2012 10:32 ]

[CMG]

Verwijderd schreef op maandag 14 mei 2012 @ 23:56:

PHP:

function index()
{
    if ($this->ion_auto->logged_in() || 2+2==4)
    {
        $this->load->view('upload');
    }

    ... snip ...
}

Bron

zal wel een FxCop (achtige) tool willen omzijlen die || true) niet accepteerd Maar heel slecht om zo je login te bypassen

[R4gnax]

ValHallASW schreef op vrijdag 11 mei 2012 @ 22:13:
Scheelt toch maar weer een library installeren.

Af en toe ben ik toch zo blij met het feit dat .NET dit allemaal compleet afgeregeld heeft en ingebouwd heft zitten in het culture-sensitive zijn van string comparison en string equality comparison. Zeg wat je wilt van Microsoft, maar daar hebeen ze toch echt goed over nagedacht en een heel mooi stuk werk neergezet.

Jammer genoeg is dat ze met DateTime dan weer niet helemaal gelukt.

[ValHallASW]

https://github.com/django...backends/__init__.py#L843

Python:

    def convert_values(self, value, field):
        """Coerce the value returned by the database backend into a consistent type that
        is compatible with the field type.
        """
        internal_type = field.get_internal_type()
        if internal_type == 'DecimalField':
            return value
        elif internal_type and internal_type.endswith('IntegerField') or internal_type == 'AutoField':
            return int(value)
        elif internal_type in ('DateField', 'DateTimeField', 'TimeField'):
            return value
        # No field, or the field isn't known to be a decimal or integer
        # Default to a float
        return float(value)

"OOP, zo moet het dus niet"

Daar zit je dan met een veld dat 'IntervalField' heet en netjes datetime.timedelta's uitspuugt. En nee, die laten zich niet casten naar float

[CMG]

ValHallASW schreef op dinsdag 15 mei 2012 @ 22:36:
https://github.com/django...backends/__init__.py#L843

Python:

    def convert_values(self, value, field):
        """Coerce the value returned by the database backend into a consistent type that
        is compatible with the field type.
        """
        internal_type = field.get_internal_type()
        if internal_type == 'DecimalField':
            return value
        elif internal_type and internal_type.endswith('IntegerField') or internal_type == 'AutoField':
            return int(value)
        elif internal_type in ('DateField', 'DateTimeField', 'TimeField'):
            return value
        # No field, or the field isn't known to be a decimal or integer
        # Default to a float
        return float(value)

"OOP, zo moet het dus niet"

Daar zit je dan met een veld dat 'IntervalField' heet en netjes datetime.timedelta's uitspuugt. En nee, die laten zich niet casten naar float

Zal wel aan mij liggen, maar deze volgorde lijkt me ook niet heel handig:

if internal_type == 'DecimalField':
return value
elif internal_type and

Doet dus eerst een check tegen een waarde (gaat er van uit dat waarde van type sting is en niet null, en als het die string niet is, gaat ie checken of ie wellicht null is... ben niet bekend met de taal, maar als ie null is, krijg je in eerste instantie meteen een exception, of de is null check in de elif is totaal overbodig ).

[RayNbow]

CMG schreef op woensdag 16 mei 2012 @ 16:28:
Doet dus eerst een check tegen een waarde (gaat er van uit dat waarde van type sting is en niet null, en als het die string niet is, gaat ie checken of ie wellicht null is... ben niet bekend met de taal, maar als ie null is, krijg je in eerste instantie meteen een exception, of de is null check in de elif is totaal overbodig ).

Hoezo dat? In Java geeft het volgende toch ook geen exception?

Java:

if (null == "hallo") {
    /* ... */
}

[MBV]

Punt 1: dit is python, geen java. En in python krijg je volgens mij een exception als het (bijvoorbeeld) een integer of None is.
Punt 2: in java heeft de == operator op strings (zonder .intern()) bijzonder weinig nut, altijd string.equals(otherString) gebruiken (die dus wel een NPE geeft als string null is.

[ValHallASW]

MBV schreef op donderdag 17 mei 2012 @ 20:30:
Punt 1: dit is python, geen java. En in python krijg je volgens mij een exception als het (bijvoorbeeld) een integer of None is.

Python:

>>> None == "string"
False
>>> 0 == "string"
False

De code klopt gewoon hoor. De reden dat 'ie pas bij de tweede een null-check doet is omdat

code:

>>> None.endswith("g")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'NoneType' object has no attribute 'endswith'

uiteraard niet werkt.

[RayNbow]

MBV schreef op donderdag 17 mei 2012 @ 20:30:
Punt 2: in java heeft de == operator op strings (zonder .intern()) bijzonder weinig nut, altijd string.equals(otherString) gebruiken

Dat weet ik...

(die dus wel een NPE geeft als string null is.

...vandaar dat ik dus niet .equals() gebruikte.

Punt 1: dit is python, geen java. En in python krijg je volgens mij een exception als het (bijvoorbeeld) een integer of None is.

ValHallASW heeft het hierboven al aangekaart hoe het zit in Python, maar jouw opmerking hier deed een vraag in mij rijzen:

code:

x == y

Bestaan er talen waarin je een x en/of y kunt vinden, zodanig dat de bovenstaande vergelijking een exception gooit?

[ValHallASW]

RayNbow schreef op donderdag 17 mei 2012 @ 21:05:

code:

x == y

Bestaan er talen waarin je een x en/of y kunt vinden, zodanig dat de bovenstaande vergelijking een exception gooit?

code:

>>> class Test(object):
...    def __eq__(self, other):
...       raise Exception()
...
>>> Test() == Test()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 3, in __eq__
Exception

Dat is natuurlijk een beetje flauw, maar dit is geen geheel onredelijke code:

code:

>>> class Test(object):
...     a = 1
...     def __eq__(self, other):
...         return self.a == other.a
...
>>> Test() == None
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 4, in __eq__
AttributeError: 'NoneType' object has no attribute 'a'

[ Voor 29% gewijzigd door ValHallASW op 17-05-2012 21:17 ]

[Verwijderd]

offtopic:
Is deze deeel van de topic niet een beetje groot aan het worden?

[Voutloos]

Nee, we beginnen pas net.

[MBV]

offtopic:
deze deel?

@RayNBow: in elke taal waar je een operator== kan definieren, en je vergeet op null te checken. Ik was bij python even in de war, wat ik erg irritant vind is dat "blaat" + 5 een exception geeft. Het is een script-taal, doe dan ook even die automatische cast naar string alsjeblieft, net zoals bijvoorbeeld java...

[Solopher]

JavaScript:

            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'NL',name: 'nl',xtype: 'textfield',anchor: '96%'}]
            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'EN',name: 'en',xtype: 'textfield',anchor: '96%'}]
            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'DE',name: 'de',xtype: 'textfield',anchor: '96%'}]
            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'FR',name: 'fr',xtype: 'textfield',anchor: '96%'}]
            }],

Right oftewel altijd 0.2 en 96%. En het leuke is; IE ging op zijn bek omdat er een , teveel instaat.

[_JGC_]

MBV schreef op vrijdag 18 mei 2012 @ 12:47:

offtopic:
deze deel?

@RayNBow: in elke taal waar je een operator== kan definieren, en je vergeet op null te checken. Ik was bij python even in de war, wat ik erg irritant vind is dat "blaat" + 5 een exception geeft. Het is een script-taal, doe dan ook even die automatische cast naar string alsjeblieft, net zoals bijvoorbeeld java...

Dat soort dingen vind ik juist irritant. Bij "blaat" + 5 zou daar dan blaat5 uitkomen, maar als je "1" + 1 doet, is het maar net afhankelijk van de taal of dat 2 of 11 wordt.

[boe2]

MBV schreef op vrijdag 18 mei 2012 @ 12:47:

offtopic:
deze deel?

@RayNBow: in elke taal waar je een operator== kan definieren, en je vergeet op null te checken. Ik was bij python even in de war, wat ik erg irritant vind is dat "blaat" + 5 een exception geeft. Het is een script-taal, doe dan ook even die automatische cast naar string alsjeblieft, net zoals bijvoorbeeld java...

Nee, nee, nee

[ZaPPZion]

_JGC_ schreef op vrijdag 18 mei 2012 @ 13:40:
[...]

Dat soort dingen vind ik juist irritant. Bij "blaat" + 5 zou daar dan blaat5 uitkomen, maar als je "1" + 1 doet, is het maar net afhankelijk van de taal of dat 2 of 11 wordt.

Ehm, of een error, dat is eigenlijk nog het meest logisch wat mij betreft

[RayNbow]

MBV schreef op vrijdag 18 mei 2012 @ 12:47:
@RayNBow: in elke taal waar je een operator== kan definieren, en je vergeet op null te checken.

Maar is dit dan niet een bug in de implementatie van ==?

(Of het een bug is hangt natuurlijk af van het contract van ==. Het probleem is alleen dat het contract niet altijd gedocumenteerd is of bestaat, waardoor dus eigenlijk elk mogelijk gedrag goed is.)

Ik was bij python even in de war, wat ik erg irritant vind is dat "blaat" + 5 een exception geeft. Het is een script-taal, doe dan ook even die automatische cast naar string alsjeblieft, net zoals bijvoorbeeld java...

In Python zijn operators instance methods. Een automatische cast zou betekenen dat str.__add__ van het bestaan van ints moet weten, wat dus resulteert in sterkere coupling tussen str en int.

[MBV]

Python heeft toch ducktyping? Iedereen die aan een string wil worden geconcateneerd moet een method toString() implementeren, dan hoeft __add__ alleen maar [wateverobject].__tostring__() of zoiets aan te roepen. Maar dat zal vast te simpel gedacht zijn.

Anyway, met meer java- dan python-ervaring vond ik dat nogal irritant, dat jullie java irritant vinden moeten jullie weten

[ValHallASW]

MBV schreef op zaterdag 19 mei 2012 @ 10:18:
Python heeft toch ducktyping? Iedereen die aan een string wil worden geconcateneerd moet een method toString() implementeren, dan hoeft __add__ alleen maar [wateverobject].__tostring__() of zoiets aan te roepen. Maar dat zal vast te simpel gedacht zijn.

Dit is de tweede keer dat je een definitie bedenkt, roept dat python aan die definitie voldoet en vervolgens tot de verkeerde conclusie komt.

Ja, je kunt python een 'scripttaal' (waar nogal wat verschillende definities van zijn) noemen, maar de stap 'oh, volgens mij hoort een scripttaal altijd x, y en z te doen, *dus* is dit gedrag van python raar' slaat nergens op.

Wat je nu hier beschrijft heeft bijzonder weinig met duck typing te maken: dit kan in niet-duck typed-talen juist ook zolang maar gedefinieerd is dat de interface van elk object een toString() heeft. Duck typing betekent in deze context juist dat als je een object maakt dat zich 'als string' gedraagt (en dan is het dus /niet/ logisch naar str te casten), het aaneenplakken blijft werken.

Zie ook: duck typing

[RayNbow]

MBV schreef op zaterdag 19 mei 2012 @ 10:18:
Python heeft toch ducktyping?

Ja, maar dat betekent niet dat alles zich gedraagt als een string.

Iedereen die aan een string wil worden geconcateneerd moet een method toString() implementeren, dan hoeft __add__ alleen maar [wateverobject].__tostring__() of zoiets aan te roepen. Maar dat zal vast te simpel gedacht zijn.

Python heeft de keuze gemaakt om str.__add__ te laten werken op strings. Wil je iets anders aan een string plakken, dan moet je de conversie zelf uitvoeren, of beter, string formatting gebruiken:

Python:

s = 'foo %d' % 3

Zie ook:

>>> import this
The Zen of Python, by Tim Peters

Beautiful is better than ugly.
Explicit is better than implicit.
Simple is better than complex.
Complex is better than complicated.
Flat is better than nested.
Sparse is better than dense.
Readability counts.
Special cases aren't special enough to break the rules.
Although practicality beats purity.
Errors should never pass silently.
Unless explicitly silenced.
In the face of ambiguity, refuse the temptation to guess.
There should be one-- and preferably only one --obvious way to do it.
Although that way may not be obvious at first unless you're Dutch.
Now is better than never.
Although never is often better than *right* now.
If the implementation is hard to explain, it's a bad idea.
If the implementation is easy to explain, it may be a good idea.
Namespaces are one honking great idea -- let's do more of those!

[KnoppenSpook]

Solopher schreef op vrijdag 18 mei 2012 @ 13:34:

JavaScript:

            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'NL',name: 'nl',xtype: 'textfield',anchor: '96%'}]
            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'EN',name: 'en',xtype: 'textfield',anchor: '96%'}]
            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'DE',name: 'de',xtype: 'textfield',anchor: '96%'}]
            },{
                border: false,
                columnWidth: 0.2,
                layout: 'form',
                items: [{fieldLabel: 'FR',name: 'fr',xtype: 'textfield',anchor: '96%'}]
            }],

Right oftewel altijd 0.2 en 96%. En het leuke is; IE ging op zijn bek omdat er een , teveel instaat.

Ah, wat mooi. ExtJS!
Handig he, die default en configDefault .

Trailing comma's are a bitch...

[HeSitated]

Voor wie het nog niet weet, maar ook bij grote bedrijven werken soms behoorlijke

Gisteren wilde ik twee externe hd behuizingen bestellen bij Amazon.de, bestelling geplaatst met one-click systeem, prima. Ooh ooohhps, kabeltjes vergeten, dus bestelling geannuleerd.

Vervolgens geprobeert de bestelling nog een keer teplaatsen, nu mét kabels, krijg ik de melding, dat dat niet mag naar het opgegeven adres....

Ok, het was toch al laat (en ik wilde om 5:00 uur opstaan), ik doe het morgen wel weer...

Uiteraard kon ik vandaag wel alles bestellen en naar mijn eigen huisadres laten sturen, maar wacht even.....

Ondanks dat ik het drempelbedrag voor gratis thuis sturen ruimschoots had overschreden, moet ik nu omwille van die kabeltjes € 9,00 bestelkosten betalen...

Daarom de kabels maar weer verwijderd, die haal ik dan wel hier ergens....

[GekkePrutser]

HeSitated schreef op maandag 21 mei 2012 @ 22:42:
Ondanks dat ik het drempelbedrag voor gratis thuis sturen ruimschoots had overschreden, moet ik nu omwille van die kabeltjes € 9,00 bestelkosten betalen...

Waarschijnlijk kwamen die kabeltjes niet van Amazon.de zelf vandaan, maar van een van de Amazon Marketplace aanbieders. Dan geldt dat gratis verzenden niet.

Vaak kan je wel kiezen om (eventueel tegen een iets hogere prijs) bij Amazon zelf te kopen. Aan de rechterkant heb je dan een knop met 'Alle Angebote', 'xx neu' en dan zie je een overzichtje. Hij laat standaard de goedkoopste aanbieder van een produkt zien, en dat is niet noodzakelijk Amazon zelf. Maar vaak staat die er wel tussen. En dan geldt dat gratis verzenden vaak wel.

Dat is dan ook geen programmeerfout maar wel logisch, die kabels komen dan ook echt ergens anders vandaan bij een bedrijf dat geen gratis verzenden biedt naar Nederland. Maar ze zouden het wel wat duidelijker aan kunnen geven.

offtopic:
Maargoed ik kom nu wel heel erg op het gebied van het shopping forum, sorry

[ Voor 5% gewijzigd door GekkePrutser op 21-05-2012 23:23 ]

[R4gnax]

KnoppenSpook schreef op maandag 21 mei 2012 @ 14:06:
[...]


Ah, wat mooi. ExtJS!
Handig he, die default en configDefault .

Trailing comma's are a bitch...

Te zien aan wat voor data er in gaat zou het ook nog wel eens slim kunnen zijn om dat gewoon data-driven te doen, ipv statisch up-front te vullen.

Iemand anders trouwens ook zo'n bloody rothekel aan hoe hele stukken van ExtJS zich aan de standaard lifecycle gepredikt in de API docs (lijken te) houden, maar dat je op de één of andere manier het toch weer treft om problemen te krijgen met juist die ene component waar het (5 ancestor classes terug omhoog) toch net eventjes anders gaat. ::Urgh::

[Firesphere]

Instacompile! Ik ben naar bed. Beertimer is "on schedule"!
Van de week even de api fixen, dan de ios n android app....

And... I gave away too much. sleep tight folks

[KnoppenSpook]

R4gnax schreef op dinsdag 22 mei 2012 @ 00:07:
[...]


Te zien aan wat voor data er in gaat zou het ook nog wel eens slim kunnen zijn om dat gewoon data-driven te doen, ipv statisch up-front te vullen.

Iemand anders trouwens ook zo'n bloody rothekel aan hoe hele stukken van ExtJS zich aan de standaard lifecycle gepredikt in de API docs (lijken te) houden, maar dat je op de één of andere manier het toch weer treft om problemen te krijgen met juist die ene component waar het (5 ancestor classes terug omhoog) toch net eventjes anders gaat. ::Urgh::

Veel tegen aangelopen ja. Maar ik vind het bij ExtJS 4.1 wel een stuk beter. Zit wel meer lijn in.

[Grijze Vos]

Paul schreef op vrijdag 11 mei 2012 @ 10:11:
[...]
Als je alleen de 6, 10 en 12 nodig hebt en er geen collega voor handen is dan is die oplossing (bedrijfseconomisch) beter dan gaan zoeken wat de betere manier is

Als er (permanent) geen collega voor handen is om je daarmee te helpen dan is het beter een baan te zoeken waar je wel iets kan leren van je collega's. (imho)

[Davio]

Een parameter als het aantal milliseconden meegeven als rekensom om het aantal minuten duidelijk te maken, goed of slecht?

C#:

var t = new Timer { Interval = 30 * 60 * 1000 };
// of
t = new Timer { Interval = 1800000 };
// of
const int HALFUUR_IN_MS = 30 * 60 * 1000;
t = new Timer { Interval = HALFUUR_IN_MS };

Om nou een methode te hebben die minuten omrekent naar milliseconden is ook weer zo overbodig.

[Zeebonk]

Die vraag doet mij herinneren aan een artikel op TheDailyWTF: http://thedailywtf.com/Articles/Sanity-Check.aspx waarbij dit zelfde vraagstuk in de comments wordt besproken.

De meeste snappen de eerste suggestie wel, de tweede is vaak al lastiger. De derde lijkt mij de beste suggestie, maar komt dicht in de buurt van:

C#:

const int ONE = 1;

Een interval heeft vaak een bepaalde betekenis, bijvoorbeeld de tijd tussen twee controles op nieuwe e-mail. Het mooiste vind ik om een constante daarom op de volgende manier te defineren:

C#:

const int UPDATE_INBOX_INTERVAL_IN_MS = 30 * 60 * 1000;

Waarbij de naam van de constante de constante duidelijk beschrijft en er aardig is af te leiden hoeveel minuten het duurt.

[Daspeed]

Davio schreef op woensdag 23 mei 2012 @ 16:33:
Een parameter als het aantal milliseconden meegeven als rekensom om het aantal minuten duidelijk te maken, goed of slecht?

C#:

var t = new Timer { Interval = 30 * 60 * 1000 };
// of
t = new Timer { Interval = 1800000 };
// of
const int HALFUUR_IN_MS = 30 * 60 * 1000;
t = new Timer { Interval = HALFUUR_IN_MS };

Om nou een methode te hebben die minuten omrekent naar milliseconden is ook weer zo overbodig.

TimeSpan heeft een aantal handige static methods, waaronder FromMinutes:

C#:

var milliseconds = TimeSpan.FromMinutes(30).TotalMilliseconds;

[Davio]

Maar dan heb ik het gevoel dat ik eerst vooruit en dan weer achteruit ga.
Ik bedoel, ik heb niet echt een timespan nodig om 30 minuten uit te rekenen.

Als ik het met een const doe dan kan ik er in ieder geval een zinnige naam aangeven en hoef ik het maar op één plek te zien.

[ValHallASW]

Davio schreef op woensdag 23 mei 2012 @ 17:36:
Als ik het met een const doe dan kan ik er in ieder geval een zinnige naam aangeven en hoef ik het maar op één plek te zien.

Behalve dat HALF_UUR_IN_MS geen duidelijke titel is, dat is net zoiets als TIEN=10. Je variabele moet immers iets zeggen over wat de bedoeling van de waarde is, niet over wat de waarde zélf is!

RELOAD_INTERVAL = 30 * 60 * 1000;

lijkt me prima leesbaar. Eventueel kan je het nog verduidelijken met

RELOAD_INTERVAL = 30 * 60 * 1000; // ms

of

RELOAD_INTERVAL_MS = 30 * 60 * 1000.
ik moet meer dan twee posts terug lezen voor ik reageer

[ Voor 5% gewijzigd door ValHallASW op 23-05-2012 18:33 ]

[Verwijderd]

Zeebonk schreef op woensdag 23 mei 2012 @ 16:52:
Die vraag doet mij herinneren aan een artikel op TheDailyWTF: http://thedailywtf.com/Articles/Sanity-Check.aspx waarbij dit zelfde vraagstuk in de comments wordt besproken.

De meeste snappen de eerste suggestie wel, de tweede is vaak al lastiger. De derde lijkt mij de beste suggestie, maar komt dicht in de buurt van:

C#:

const int ONE = 1;

Een interval heeft vaak een bepaalde betekenis, bijvoorbeeld de tijd tussen twee controles op nieuwe e-mail. Het mooiste vind ik om een constante daarom op de volgende manier te defineren:

C#:

const int UPDATE_INBOX_INTERVAL_IN_MS = 30 * 60 * 1000;

Waarbij de naam van de constante de constante duidelijk beschrijft en er aardig is af te leiden hoeveel minuten het duurt.

In het laatste geval zou je er nog even

C#:

//30 minutes.

achter kunnen zetten

[ValHallASW]

Verwijderd schreef op woensdag 23 mei 2012 @ 19:28:
In het laatste geval zou je er nog even

C#:

//30 minutes.

achter kunnen zetten

En vervolgens moet het verkort worden naar 20 minuten, en dan staat er opeens

C:

const int UPDATE_INBOX_INTERVAL_IN_MS = 20 * 60 * 1000; // 30 minutes

Als je dat graag wilt doen, doe dan iets als

C:

const int UPDATE_INBOX_INTERVAL_IN_MINUTES = 30;
const int UPDATE_INBOX_INTERVAL_IN_MS = UPDATE_INBOX_INTERVAL_IN_MINUTES * 60 * 1000;

[Alex)]

Wat maken jullie er allemaal lelijke code van zeg

C#:

/// <summary>
/// The answer to life, the universe and everything.
/// </summary>
private static readonly dynamic UpdateInterval = TimeSpan.FromMinutes(42);

/// <summary>
/// Wie dit leest is gek.
/// </summary>
public static void Henk()
{
    var koekjes = new Timer { Interval = UpdateInterval.TotalMilliseconds; }
}

Wegwezen met die HOOFDLETTERS, reken + sommetjes en allerlei andere vage meuk

[ Voor 7% gewijzigd door Alex) op 23-05-2012 22:31 ]

[Zeebonk]

Daarom posten we het allemaal netjes in dit topic, wat jouw suggestie hier doet is mij dan ook een raadsel

[ZaPPZion]

Hebben we geen types meer tegenwoordig ? UpdateInterval is een errrr?

[Damic]

Krijg ik nu 2520000 koekjes? Ja ik ben gek ;D

[Alex)]

ZaPPZion schreef op woensdag 23 mei 2012 @ 22:12:
Hebben we geen types meer tegenwoordig ? UpdateInterval is een errrr?

Fixed

[YopY]

Ruby / Scala / etc to the rescue

Scala:

var duration = 5 minutes

[ Voor 34% gewijzigd door YopY op 23-05-2012 22:40 ]

[.oisyn]

Alex) schreef op woensdag 23 mei 2012 @ 20:51:
Wat maken jullie er allemaal lelijke code van zeg

Zegt de meneer die compleet zonder enige valide reden dynamic gebruikt.

[Alex)]

.oisyn schreef op woensdag 23 mei 2012 @ 22:42:
[...]

Zegt de meneer die compleet zonder enige valide reden dynamic gebruikt.

[Sebazzz]

Alex) schreef op woensdag 23 mei 2012 @ 23:05:
[...]

[afbeelding]

Dat zou ik ook zeggen als ik niet achteraf zou willen toegeven dat ik het fout heb

/steeksteek

[Voutloos]

Doe mij de oplossing van Zeebonk maar. Een constante met sematische naam, met waarde opgeschreven als sommetje.

Klaar is Kees. Allemaal methods gebruiken omdat je zelf niet met 60 en 1000 durft te vermenigvuldigen is mij echt te Enterprisey.

offtopic:
Let op: Dit is nog geen vrijbrief om zelf berekeningen van dates of intervals te implementeren. Daar gebruik je libraries voor en voor iedereen die denkt 'wel ff zelf een paar minuutjes bij een bepaalde datetime op te kunnen tellen' zou een verplichte levenslange dienstplicht of plantsoenendienst moeten gelden.

[.oisyn]

Niet om 't een of ander, maar ik denk wel dat ik het beter kan dan menig library *kuch*.net*kuch*

[ Voor 13% gewijzigd door .oisyn op 23-05-2012 23:42 ]

[Verwijderd]

Natuurlijk maak je een klasse die Timer overerft met de onderstaande code:

C#:

    public class ClockTimer : System.Timers.Timer
    {

        public double IntervalInMinutes
        {
            get
            {
                return Interval / 1000 / 60;
            }
            set
            {
                Interval = value * 1000 * 60;
            }
        }

    }

[.oisyn]

Nee ik zou er toch echt een webservice voor gebruiken hoor

[YopY]

.oisyn schreef op donderdag 24 mei 2012 @ 09:42:
Nee ik zou er toch echt een webservice voor gebruiken hoor

pfsh, J2EE stack, draaien op WebSphere in de cloud.

[ValHallASW]

C:

namebuffer[0] = 48 + (((numberOfCookies)/1000) % 10);
namebuffer[1] = 48 + (((numberOfCookies)/100) % 10);
namebuffer[2] = 48 + (((numberOfCookies)/10) % 10);
namebuffer[3] = 48 + ((numberOfCookies) % 10);

en dan óók nog 48 gebruiken ipv '0'

[ Voor 8% gewijzigd door ValHallASW op 24-05-2012 13:42 ]

[vinnux]

Verwijderd schreef op donderdag 24 mei 2012 @ 07:38:
Natuurlijk maak je een klasse die Timer overerft met de onderstaande code:

C#:

    public class ClockTimer : System.Timers.Timer
    {

        public double IntervalInMinutes
        {
            get
            {
                return Interval / 1000 / 60;
            }
            set
            {
                Interval = value * 1000 * 60;
            }
        }

    }

Ik zou toch eerder voor een extension method gaan in dit geval.
(Momenteel zijn er nog geen extension properties in .NET 4.0)

C#:

namespace ConsoleApplication1 {
    public class Program {
        static void Main(string[] args) {
            System.Timers.Timer timer = new System.Timers.Timer();
            timer.SetIntervalInMinutes(10);
            timer.GetIntervalInMinutes();
        }
    }

    public static class TimerHelper {

        public static double GetIntervalInMinutes(this System.Timers.Timer timer) {
            return timer.Interval / 1000 / 60;
        }

        public static void SetIntervalInMinutes(this System.Timers.Timer timer, int minutes) {
            timer.Interval = minutes * 1000 * 60;
        }
    }
}

[Davio]

Alleen het feit al dat het hier een volle pagina over gaat is voor mij voldoende bewijs dat het toch niet zo 'clear cut' is.

Ik zit ook een beetje te stoeien met StyleCop en de ingebakken Code Analysis van Visual Studio.
Sommige rules vind ik gewoon complete onzin.

Zoals overal overbodige comments boven zetten. Bijvoorbeeld bij een constructor

C#:

public class Monkey
{
  /// <summary>
  ///  Creates a new instance of Monkey.
  /// </summary>
  public Monkey(string name)
  {
    Name = name;
  }
}

Of bij event handlers, dat je moet zeggen waar sender voor staat en het argument e van de EventArgs.

En ik ben ook van de regions afgestapt, om nu in klasses met 1 constructor die constructor in een region te zetten... Ik gebruik regions nu alleen nog maar om bij elkaar horende dingen te groeperen, bijvoorbeeld 20 DelegateCommands onder elkaar.

[ZaPPZion]

Die overbodige comments is wel iets waar je misschien later wat aan kan hebben. Stel dat een Monkey kind kan zijn met van andere Monkey, wat is dan nu de string naam die je meegeeft in de constructor? De naam van de Monkey of die van een ouder? (ik verzin maar ff wat)

[ Voor 6% gewijzigd door ZaPPZion op 24-05-2012 21:14 ]

[Firesphere]

If the code is hard to write, it should be hard to read. Do not comment.

[Verwijderd]

vinnux schreef op donderdag 24 mei 2012 @ 15:10:
[...]

Ik zou toch eerder voor een extension method gaan in dit geval.
(Momenteel zijn er nog geen extension properties in .NET 4.0)

C#:

namespace ConsoleApplication1 {
    public class Program {
        static void Main(string[] args) {
            System.Timers.Timer timer = new System.Timers.Timer();
            timer.SetIntervalInMinutes(10);
            timer.GetIntervalInMinutes();
        }
    }

    public static class TimerHelper {

        public static double GetIntervalInMinutes(this System.Timers.Timer timer) {
            return timer.Interval / 1000 / 60;
        }

        public static void SetIntervalInMinutes(this System.Timers.Timer timer, int minutes) {
            timer.Interval = minutes * 1000 * 60;
        }
    }
}

Dat kan natuurlijk ook

[Jaap-Jan]

ValHallASW schreef op donderdag 24 mei 2012 @ 13:40:

C:

namebuffer[0] = 48 + (((numberOfCookies)/1000) % 10);
namebuffer[1] = 48 + (((numberOfCookies)/100) % 10);
namebuffer[2] = 48 + (((numberOfCookies)/10) % 10);
namebuffer[3] = 48 + ((numberOfCookies) % 10);

[afbeelding]
en dan óók nog 48 gebruiken ipv '0'

Op school hebben we ook weleens zoiets geschreven als tweedejaars, ik heb zelfs de exacte code nog.

C:

char* printNum(unsigned char number) {
  unsigned char array[4];
  array[0] = ((number/(10 * 10)) % 10) + 48;
  array[1] = ((number/10) % 10) + 48;
  array[2] = (number % 10) + 48;
  array[3] = '\0';
  return &array;
}

't Was voor een microprocessor en met printf() zou het programmageheugen in één keer vol zijn.

[ Voor 6% gewijzigd door Jaap-Jan op 24-05-2012 22:24 ]

[Verwijderd]

Hier staan ook wel leuke dingen tussen!

Zoals:

C#:

static bool IsDataFOK(string filetext)
{
     bool res = false;
     if (string.IsNullOrEmpty(filetext) || !filetext.Contains(",")) goto end;

     res = true
     end:
     return res;
}

[R4gnax]

Verwijderd schreef op donderdag 24 mei 2012 @ 22:35:
Hier staan ook wel leuke dingen tussen!

Zoals:

C#:

static bool IsDataFOK(string filetext)
{
     bool res = false;
     if (string.IsNullOrEmpty(filetext) || !filetext.Contains(",")) goto end;

     res = true
     end:
     return res;
}

Met dat soort code kun je er redelijkerwijs vanuit gaan dat je data inderdaad fokked is.


Ach. Het kan ook erger zijn. Je kunt ook Google zijn en al sinds versie 18 van je browser een fout in de constructor inlining optimalisatie hebben zitten van je JavaScript engine, waardoor je in elk geval jQuery (en daarmee potentieel zo'n 50% van het internet) op non-deterministische wijze als een tikkende tijdbom om zeep helpt.

Oops!

[ Voor 28% gewijzigd door R4gnax op 24-05-2012 22:56 ]

[RayNbow]

R4gnax schreef op donderdag 24 mei 2012 @ 22:50:
Ach. Het kan ook erger zijn. Je kunt ook Google zijn en al sinds versie 18 van je browser een fout in de constructor inlining optimalisatie hebben zitten van je JavaScript engine, waardoor je in elk geval jQuery (en daarmee potentieel zo'n 50% van het internet) op non-deterministische wijze als een tikkende tijdbom om zeep helpt.

Ik vind Issue 128527: REGRESSION: Tooltips don't appear anymore een ergere bug en veroorzaakt minstens 3x per week voor ergernissen (xkcd).

[Davio]

ZaPPZion schreef op donderdag 24 mei 2012 @ 21:13:
Die overbodige comments is wel iets waar je misschien later wat aan kan hebben. Stel dat een Monkey kind kan zijn met van andere Monkey, wat is dan nu de string naam die je meegeeft in de constructor? De naam van de Monkey of die van een ouder? (ik verzin maar ff wat)

Nou, comments moeten wel iets zinnigs zeggen.
Als ze alleen maar de naam van de methode of de property herhalen, is het vrij nutteloos.

C#:

/// <summary>
/// Gets or sets the magic number, ORLY??
/// </summary>
public int MagicNumber { get; set;}

[Verwijderd]

Davio schreef op vrijdag 25 mei 2012 @ 08:16:
[...]

Nou, comments moeten wel iets zinnigs zeggen.
Als ze alleen maar de naam van de methode of de property herhalen, is het vrij nutteloos.

C#:

/// <summary>
/// Gets or sets the magic number, ORLY??
/// </summary>
public int MagicNumber { get; set;}

Toch vind ik dat soort dingen wel handig. Zeker als je alleen de documentatie hebt (b.v. van een library), dan is het fijn om te weten dat die property echt alleen gets en sets doet en verder niets anders. Zelfs bij libraries die ik zelf gemaakt heb is dat nog handig, aangezien ik niet telkens terug wil naar de code.

[YopY]

Verwijderd schreef op vrijdag 25 mei 2012 @ 08:42:
[...]


Toch vind ik dat soort dingen wel handig. Zeker als je alleen de documentatie hebt (b.v. van een library), dan is het fijn om te weten dat die property echt alleen gets en sets doet en verder niets anders. Zelfs bij libraries die ik zelf gemaakt heb is dat nog handig, aangezien ik niet telkens terug wil naar de code.

Maar zelfs dan blijft de stelregel: Als je commentaar moet schrijven om je code te begrijpen, moet je toch even achter je oren krabben of je code zelf niet gefixed moet worden.

Op het moment doen we niet / nauwlijks aan commentaar; waarschijnlijk krijg ik WTFs over een paar maanden . Sterker nog, krijg ik nu al, zo van 'waarom staat er een width: 398 in deze css regel? :?'

voor de gein eens in de Java code gedoken. Ik zie auto-generated comments (of automatisch, of uit automatisme) die weinig toevoegen . Voorbeeld (geen directe c/p ivm het feit dat dat niet mag )

Java:

/**
 * Finds a Product by a query
 *
 * @param query
 *        A search string for used to find matching Products
 * @return The Products found
 */
 @GET
  public Collection<Product> searchProducts(@QueryParam("query") String query) {
    if (isEmpty(query)) {
      return emptyList();
    }
    return productSearchDao.search(query);
  }

tenslotte: frustratie van de dag / week, mot op een braq oud Dell toetsenbord van 2 euro werken, en de rechter shifttoets reageert niet goed meer, waardoor dubbelepunten regelmatig puntkomma's worden . Niet handig als je JS doet.

[Davio]

Ik heb hier ook bestanden vol met automatisch gegenereerde code.

Het wordt wel heel zielig als je dan Engelse comments krijgt bij Nederlandse methodenamen. Code is geërfd, anders had ik denk ik voor alles Engels gekozen, al blijft dat een gek iets.

[PiepPiep]

Jaap-Jan schreef op donderdag 24 mei 2012 @ 22:23:
[...]
Op school hebben we ook weleens zoiets geschreven als tweedejaars, ik heb zelfs de exacte code nog.

C:

char* printNum(unsigned char number) {
  unsigned char array[4];
  array[0] = ((number/(10 * 10)) % 10) + 48;
  array[1] = ((number/10) % 10) + 48;
  array[2] = (number % 10) + 48;
  array[3] = '\0';
  return &array;
}

't Was voor een microprocessor en met printf() zou het programmageheugen in één keer vol zijn.

Ehm, ik ben een beetje uit de pointers e.d., maar wordt hier array niet op de stack aangemaakt en gaat dit dus grondig fout als je daar een pointer naar returnt?.

[.oisyn]

Klopt. Maar daarnaast klopt er sowieso geen drol van, want hij returnt een pointer naar char[4] ipv pointer naar char, waarvan het type dus char(*)[4] is en geen char*. Nou heeft de pointer naar de array en de pointer naar het eerste element toevallig wel dezelfde waarde (beide beginnen immers op dat adres), maar toch zijn de types niet compatible.

[ Voor 48% gewijzigd door .oisyn op 25-05-2012 12:24 ]

[Quincy5]

YopY schreef op vrijdag 25 mei 2012 @ 09:25:
[...]

tenslotte: frustratie van de dag / week, mot op een braq oud Dell toetsenbord van 2 euro werken, en de rechter shifttoets reageert niet goed meer, waardoor dubbelepunten regelmatig puntkomma's worden . Niet handig als je JS doet.

Waarom kun je geen nieuwe krijgen? Bezuinigen op een tool van minder dan 10 euro, wat al gauw meer aan productiviteit kost? Daar zou ik echt niet tegen kunnen en desnoods zelf een ander toetsenbord regelen.

[Firesphere]

YopY schreef op vrijdag 25 mei 2012 @ 09:25:
tenslotte: frustratie van de dag / week, mot op een braq oud Dell toetsenbord van 2 euro werken, en de rechter shifttoets reageert niet goed meer, waardoor dubbelepunten regelmatig puntkomma's worden . Niet handig als je JS doet.

Als je met zulk slecht spul moet werken, dan moet je je beklagen, of zelf voor iets beters zorgen. Voor een paar tientjes een goed toetsenbord kopen scheelt heel veel. Anders heb je over een paar jaar een Carpaal tunnel syndroom of erger. Zit je ook niet op te wachten.
Kleine moeite, heel erg veel plezier.

Ehm, met Quincy dus.

[Patriot]

Davio schreef op vrijdag 25 mei 2012 @ 09:31:
Ik heb hier ook bestanden vol met automatisch gegenereerde code.

Het wordt wel heel zielig als je dan Engelse comments krijgt bij Nederlandse methodenamen. Code is geërfd, anders had ik denk ik voor alles Engels gekozen, al blijft dat een gek iets.

Oeh, is het weer tijd voor die discussie?

[Verwijderd]

Ik vond hem niet direct terug in dit topic dus hierbij: http://hatepaste.com

[CodeCaster]

quote: http://de.php.net/manual/en/function.count.php

count — Count all elements in an array, or something in an object

Ah, something. Bedankt.

[T-MOB]

Tsja, als je de helft quote kun je van alles een slecht voorbeeld maken. De volgende regel is namelijk:

For objects, if you have SPL installed, you can hook into count() by implementing interface Countable. The interface has exactly one method, Countable::count(), which returns the return value for the count() function.

[CodeCaster]

En wat is dan het gedrag als je geen SPL hebt geïnstalleerd, en/of als je object geen ICountable implementeert?

"Something" hoort gewoon niet op een man-page.

[ Voor 7% gewijzigd door CodeCaster op 01-06-2012 14:23 ]

[Gleighton]

Je bent wel erg veeleisend voor de php documentatie

[PdeBie]

niet echt een slecht programmeervoorbeeld, maar wel een leuke interactie 'error' tussen software en eindgebruiker.

De oude versie van onze software verzond middels FTP orders naar elkaar. De nieuwe software 'verzend' orders tussen gebruikers via een database. Dus: Klant A maakt order aan > order wordt opgeslagen in DB > Klant B ziet nieuwe order.

Nu trad er een error op bij het opslaan dus kreeg klant A een foutmelding in beeld 'order is niet opgeslagen'.
Zegt de klant:

Maakt mij niet uit dat hij niet opgeslagen is. Als hij maar verzonden is.

Heb de foutmelding maar even aangepast naar 'Order niet verzonden'

[ Voor 5% gewijzigd door PdeBie op 01-06-2012 15:15 ]

[Verwijderd]

Dan krijg je binnenkort:

Maakt mij niet uit dat hij niet verzonden is. Als hij maar opgeslagen is.

[Gomez12]

pdebie schreef op vrijdag 01 juni 2012 @ 15:14:

Heb de foutmelding maar even aangepast naar 'Order niet verzonden'

Ik hoop dat je logging nog wel accuraat is. Persoonlijk zou ik enkel de 1e foutmelding uitgebreid hebben, maar ok ik werk dan ook liever met foutnrs gevolgd door een textuele melding (dan kan de textuele melding compleet naar wens van de klant zijn, zolang hij het foutnr doorgeeft weten wij waar we moeten kijken)

Als de klant over 3 maanden opbelt met de melding dat er orders niet verzonden worden dan zou ik niet direct gaan kijken of ze wel opgeslagen worden, ik zou in 1e instantie dan enkel kijken naar het gedeelte wat de orders verzend.

[PdeBie]

verzenden = opslaan in dit geval. Het systeem werkt met 1 database waar zowel de verzender als ontvanger partij naar kijken. Er vind geen verzending van fysieke bestanden meer plaats zoals eerst via FTP.

Probleem alleen is, is dat wij met een gebruikersgroep werken die letterlijk met de muis naar de rechterbovenhoek gaan (ken je de reclame nog? ). Onze helpdesk werd platgebeld door deze gebruikers met de vraag waarom de order opgeslagen is en niet verzonden. Om te voorkomen dat we straks 1200 gebruikers moeten uitleggen wat het verschil tussen opslaan en verzenden is, hebben we besloten de tekst aan te passen naar een tekst die ze nog kenden uit het oude systeem.

Logging is uiteraard voldoende om voor ons te zien waarom een eventuele order niet opgeslagen is.

[Verwijderd]

Ik heb in de loop der jaren wel wat bagger gezien, dat weinig dingen zijn die me nog kunnen verbazen. Maar toch lukt het sommige personen toch nog wel:

PHP:

header('Content-Type: application/pdf');
header("Location:".DOC_ROOT.$filename." target: "_blank"");
echo $file;

Dafuq did I just read!?

[Ram0n]

Zucht... Collega kreeg een brief van zijn hostingpartij binnen met informatie over een nieuw beheerpakket voor klanten, en er werd doodleuk afgesloten met: "Uw inloggevens zijn niet veranderd, u kunt dus nog steeds inloggen met gebruikersnaam xxxxx en wachtwoord xxxx"

[Freeaqingme]

Het lijkt me inderdaad niet handig om je wachtwoord hetzelfde te maken als de eerste 4 karakters van je gebruikersnaam =)

[Jegorex]

Freeaqingme schreef op dinsdag 05 juni 2012 @ 21:37:
Het lijkt me inderdaad niet handig om je wachtwoord hetzelfde te maken als de eerste 4 karakters van je gebruikersnaam =)

Wie verwacht nou dat je gebruikersnaam en wachtwoord vrijwel identiek zijn? Perfect wachtwoord zou ik zeggen

[Verwijderd]

Zo heb ik mijn wachtwoord ingesteld op incorrect. Wanneer ik mijn wachtwoord vergeten ben vul ik gewoon iets random in en dan zeggen de meeste engelstalige websites zelf wel: "your password is incorrect."

[Verwijderd]

Slim, incorrect is wel het laatste wat ze zullen proberen

[Freeaqingme]

Ram0n schreef op dinsdag 05 juni 2012 @ 21:33:
Zucht... Collega kreeg een brief van zijn hostingpartij binnen met informatie over een nieuw beheerpakket voor klanten, en er werd doodleuk afgesloten met: "Uw inloggevens zijn niet veranderd, u kunt dus nog steeds inloggen met gebruikersnaam xxxxx en wachtwoord xxxx"

Without kidding, het is natuurlijk in- en intriest, zeker anno 2012. Dit zou voor mij reden zijn een andere leverancier te zoeken.

[azerty]

Verwijderd schreef op dinsdag 05 juni 2012 @ 22:24:
Zo heb ik mijn wachtwoord ingesteld op incorrect. Wanneer ik mijn wachtwoord vergeten ben vul ik gewoon iets random in en dan zeggen de meeste engelstalige websites zelf wel: "your password is incorrect."

off-topic: Dat heb je afgekeken van een post op memebase

Ben ik aan het werken aan het versturen van een counter via een $_POST, gebruik ik dit om te valideren of het wel een getal is:

PHP:

if(!is_int($_POST['count'])) return;

En maar blijven afvragen waarom het verzoek mislukt -_-

[Freeaqingme]

Ik gok zomaar dat je boven dit stukje code niet het volgende hebt staan?

$_POST['count'] = (int) $_POST['count'];

[azerty]

Freeaqingme schreef op dinsdag 05 juni 2012 @ 23:40:
Ik gok zomaar dat je boven dit stukje code niet het volgende hebt staan?

$_POST['count'] = (int) $_POST['count'];

Jup. Heb het uiteindelijk weggelaten omdat het in de huidige code toch geen probleem vormt moest het toch geen int zijn, komt verder alleen in een for lus voor (als $i<=$_POST['count'], dus geen probleem...)

Edit: zou beter stoppen en morgen verder werken ^^

[ Voor 5% gewijzigd door azerty op 05-06-2012 23:55 ]

[.oisyn]

Freeaqingme schreef op dinsdag 05 juni 2012 @ 23:40:
Ik gok zomaar dat je boven dit stukje code niet het volgende hebt staan?

$_POST['count'] = (int) $_POST['count'];

Waarna de hele check nutteloos wordt, want $_POST['count'] is vervolgens altijd een int, ook als het een string was.

[PiepPiep]

Verwijderd schreef op dinsdag 05 juni 2012 @ 22:24:
Zo heb ik mijn wachtwoord ingesteld op incorrect. Wanneer ik mijn wachtwoord vergeten ben vul ik gewoon iets random in en dan zeggen de meeste engelstalige websites zelf wel: "your password is incorrect."

Dan vind ik correct als password veel slimmer.
Als er dan iemand anders onder jouw naam probeerd in te loggen krijgen ze de melding dat het password correct is maar komen ze er niet in.
Dwaalspoor!

[spleethoven]

Dit bracht toch ook een grijns teweeg:

JavaScript:

//No coma on numbers
function nocoma(e){
..
}

Franstaligen en hun engels...

[Gimmeabrake]

Kwam zojuist dit tegen:

PHP:

list($invalidtimes, $incuficienttimeleft) = mysqli_fetch_array(mysqli_query($conn, "
   SELECT 
      TIME_TO_SEC('$starttime') >= TIME_TO_SEC('$endtime'), 
      $stats_xml->timeleft < (TIME_TO_SEC('$endtime') - TIME_TO_SEC('$starttime')) / 60"
   )
);

Iemand heeft dus ooit verzonnen dat hij met PHP variabelen in MySQL ging rekenen omdat hij het handiger vond. Oh, had ik al verteld dat deze variabelen uit een SOAP request komen en nergens een escape_string oid te vinden is?

[Tharulerz]

YouTube: How To Cancel Any Auction

Dit mag hier niet ontbreken vind ik... Je zou meer verwachten van een bedrijf als Blizzard