[alg] Slechtste programmeervoorbeelden deel 4

[HuHu]

mithras schreef op zondag 19 juni 2011 @ 13:58:
[...]


Nog een leuk slecht programmeervoorbeeld trouwens: er zit een goede flaw in onze eerste versie-webshop. Het proces is:

[..] > winkelmandje > betalen > terug naar site > facuur aanmaken + email sturen

Dat is met name leuk voor klanten die iDeal afbreken en nooit meer terugkomen op de site

Ik heb 'm zo:

[..] > winkelmandje > bevestig order + mail > betalen > terug naar site

En na het betalen komt er achteraf (offline / post-back) van de bank een melding en dan komt het stukje: factuur maken + mail.

Mensen die de betaling niet doen (of afbreken) hebben alsnog een mail met orderbevestiging gehad en kunnen daar zelf eventueel nog wat mee. Betalingen die na 5 dagen nog geen post-back hebben gehad komen als "expired" in het systeem en dan kan de secretaresse er achteraan (of je stuurt automatisch een mail voor een handmatige overschrijving / een nieuwe iDeal transactie).

[dev10]

mithras schreef op zondag 19 juni 2011 @ 13:58:
[...]


Nog een leuk slecht programmeervoorbeeld trouwens: er zit een goede flaw in onze eerste versie-webshop. Het proces is:

[..] > winkelmandje > betalen > terug naar site > facuur aanmaken + email sturen

Dat is met name leuk voor klanten die iDeal afbreken en nooit meer terugkomen op de site

Of mensen die na de iDEAL betaling gedaan te hebben niet teruggaan naar de webshop. Gebeurt bij ons regelmatig.

[ctrlaltdelbe]

mithras schreef op zondag 19 juni 2011 @ 13:16:
[...]
Gebruiken wij altijd in templates. In business logic is het dit:

PHP:

if (false === $model) {
  throw NotFoundException('Could not find blablabla');
}

Het is geen slecht programmeervoorbeeld, maar ik vind '==' operaties op booleans toch altijd maar een beetje meh. Zet dan gewoon

PHP:

if (!$model) {
//code 
}

[Manuel]

Nikolas schreef op zondag 19 juni 2011 @ 23:58:
[...]

Het is geen slecht programmeervoorbeeld, maar ik vind '==' operaties op booleans toch altijd maar een beetje meh. Zet dan gewoon

PHP:

if (!$model) {
//code 
}

Daarom maakt hij ook gebruik van type-matching. Als $model expliciet op false staat dan geldt die if pas, anders niet.

[CodeCaster]

Nikolas schreef op zondag 19 juni 2011 @ 23:58:
[...]

Het is geen slecht programmeervoorbeeld, maar ik vind '==' operaties op booleans toch altijd maar een beetje meh. Zet dan gewoon

PHP:

if (!$model) {
//code 
}

Dat zou zoals mijn bovenbuurman ook al aangaf een valide argument zijn bij een strongly typed taal, maar niet bij php. Wat als je model "0" is, gewoon omdat het kan? Daarnaast voorkomt hij een "assignment by typo" door false vooraan te plaatsen. Vergelijk:

PHP:

if ($model = false)
{
  // foo
}

PHP:

if ($model == false)
{
  // foo
}

[ Voor 18% gewijzigd door CodeCaster op 20-06-2011 00:15 ]

[Matis]

Yoda statement probeer ik er bij ons op de toko ook doorheen te drukken.

In php probeer ik overigens steeds vaker === te gebruiken om typedefinitie af te dwingen. Ook probeer ik vaak te casten, om het toch het gevoel van een strong typed taal te geven. Daarnaast kan een beetje intelligente IDE er ook beter mee omgaan

[Verwijderd]

Volgens mij herken je hieraan de mensen met meer ervaring.
En dus met minder vertrouwen in zichzelf en andere developers omdat er nu eenmaal foutjes worden gemaakt.

Bugs programmeren is zo makkelijk. Als je dat met een bepaalde stijl voor een deel kunt voorkomen dan levert het je veel tijd op. Bugs kosten tijd, geld en aanzien.

[ID-College]

Ik merk het ook idd! ik gebruik veel var_dump en kijk dan al naar het type.. Als ik het niet zeker weet, dan cast ik de waarde naar b.v. een int om vervolgens === te gebruiken..
Met name $_GET waarden die een getal moeten zijn (zoals id's) cast ik altijd direct naar een int.. Als de waarde dan 0 is, dan was de $_GET iig geen int

[Verwijderd]

ID-College schreef op maandag 20 juni 2011 @ 01:07:
Ik merk het ook idd! ik gebruik veel var_dump en kijk dan al naar het type.. Als ik het niet zeker weet, dan cast ik de waarde naar b.v. een int om vervolgens === te gebruiken..
Met name $_GET waarden die een getal moeten zijn (zoals id's) cast ik altijd direct naar een int.. Als de waarde dan 0 is, dan was de $_GET iig geen int

of 0

[Verwijderd]

Soms typ ik spontaan een functie op de volgende manier in PHP:

code:

public function foo(string $str) {...}

Om vervolgens een error te krijgen.. ohja doh

[.Gertjan.]

Matis schreef op maandag 20 juni 2011 @ 00:22:
Yoda statement probeer ik er bij ons op de toko ook doorheen te drukken.

In php probeer ik overigens steeds vaker === te gebruiken om typedefinitie af te dwingen. Ook probeer ik vaak te casten, om het toch het gevoel van een strong typed taal te geven. Daarnaast kan een beetje intelligente IDE er ook beter mee omgaan

Of je kiest gewoon een taal die echt strong typed is
[/knuppel in hoenderhok mode]

[Verwijderd]

FragFrog schreef op vrijdag 17 juni 2011 @ 16:36:
[...]

hilarisch!

Zo heb ik ooit toen ik nog kleine FragFrog was 3000 comments overschreven met "Dit is een test" door een WHERE clause te vergeten Testomgeving, wat is dat?

Ach dat is niets, vroeger, toen ik nog op mijn vorige werk werkte schreven wij soms sql's om updates uit te voeren op de database van de boekhouding (die draaiden afzonderlijk op de laptops van de boekhouders) om correcties uit te voeren. Ooit heeft er 1 iemand zo'n sql aangepast omdat hij dacht dat hij genoeg kon van sql want hij had een paar tutorials gelezen...

SQL:

delete from dbtransacties;
where boeking = 2;

pech voor hem dat we maar maandelijks backups hadden in ons hoofdkantoor

[Snake]

CodeCaster schreef op maandag 20 juni 2011 @ 00:13:
[...]

Dat zou zoals mijn bovenbuurman ook al aangaf een valide argument zijn bij een strongly typed taal, maar niet bij php. Wat als je model "0" is, gewoon omdat het kan? Daarnaast voorkomt hij een "assignment by typo" door false vooraan te plaatsen. Vergelijk:

PHP:

if ($model = false)
{
  // foo
}

PHP:

if ($model == false)
{
  // foo
}

Ik heb mijzelf aangeleerd van de constante voorop te zetten.

PHP:

if (false == $model)
{
  // foo
}

Dan kan je geen fouten maken, want false kan je niet toewijzen (hoop ik, het is en blijft PHP )

[Icekiller2k6]

Verwijderd schreef op maandag 20 juni 2011 @ 08:46:
[...]

SQL:

delete from dbtransacties;
where boeking = 2;

pech voor hem dat we maar maandelijks backups hadden in ons hoofdkantoor

auch... is dat niet een van de dingen die ingehamerd worden bij fatsoenlijke boeken/handleidingen?

Op meerdere regels en dan op het EINDE ; (bij oracle is dat toch zo)

[TweakBoy]

Icekiller2k6 schreef op maandag 20 juni 2011 @ 09:48:
[...]

auch... is dat niet een van de dingen die ingehamerd worden bij fatsoenlijke boeken/handleidingen?

Op meerdere regels en dan op het EINDE ; (bij oracle is dat toch zo)

Boekhouders zijn eigenwijs...

[.Gertjan.]

Snake schreef op maandag 20 juni 2011 @ 09:42:
[...]
Ik heb mijzelf aangeleerd van de constante voorop te zetten.

PHP:

if (false == $model)
{
  // foo
}

Dan kan je geen fouten maken, want false kan je niet toewijzen (hoop ik, het is en blijft PHP )

Dit is een goed punt. Ooit eens als tip gekregen op mijn opleiding bij C++. .NET straft je af als je

C#:

if(myVar = 5) { }

doet, maar andere talen staan het toe waarbij het gedrag nogal eens kon afwijken per taal.

Ooit gehoord dat uit een toewijzing een "true" terug komt en in andere talen komt er een 5 terug (de toegewezen waarde) waarbij de if alleen checkt of het niet null/0 is (bijvoorbeeld Javascript).

Dit stukje

C#:

if(5 = myVar) { } 

wordt direct afgestraft door (voor zover ik weet) iedere compiler. Het gaat alleen niet altijd goed bij menselijke parsers (aka ontwikkelaars), die kunnen dat soort constructies niet altijd lekker verwerken.

[.oisyn]

Verwijderd schreef op maandag 20 juni 2011 @ 08:46:
[...]


Ach dat is niets, vroeger, toen ik nog op mijn vorige werk werkte schreven wij soms sql's om updates uit te voeren op de database van de boekhouding (die draaiden afzonderlijk op de laptops van de boekhouders) om correcties uit te voeren. Ooit heeft er 1 iemand zo'n sql aangepast omdat hij dacht dat hij genoeg kon van sql want hij had een paar tutorials gelezen...

SQL:

delete from dbtransacties;
where boeking = 2;

pech voor hem dat we maar maandelijks backups hadden in ons hoofdkantoor

OMG this is so wrong on so many levels.
Boekhouddatabase op de laptop? WTF?
Maandelijke backups? WTF?
Database updaten met handgeschreven SQL queries? WTF?

[ Voor 8% gewijzigd door .oisyn op 20-06-2011 11:44 ]

[.oisyn]

.Gertjan. schreef op maandag 20 juni 2011 @ 10:06:
Ooit gehoord dat uit een toewijzing een "true" terug komt

Heb ik ook ooit gehoord. Van mensen hier op GoT die de taal niet snapten. Is er werkelijk een taal waarin een assignment een expression is die áltijd true oplevert (even afgezien zelfgedefnieerde assigment operators uiteraard)? Volgens mij bestaat die niet.

Btw, in C# kan het ook gewoon. De compiler klaagt alleen omdat het type van de assignment expression het type van de te assignen variabele is, en bij a = 5 is dat dus meestal int. Maar een if verwacht een boolean. Als de expressie gewoon een boolean oplevert vindt de compiler het prima:

C#:

if (a = true)
{
}

Nou is het natuurlijk wel weer zo dat je dan gewoon if(a) of if(!a) tikt als je a==true of a==false bedoelt.

[ Voor 41% gewijzigd door .oisyn op 20-06-2011 11:49 ]

[Verwijderd]

.oisyn schreef op maandag 20 juni 2011 @ 11:43:
[...]


OMG this is so wrong on so many levels.
Boekhouddatabase op de laptop? WTF?
Maandelijke backups? WTF?
Database updaten met handgeschreven SQL queries? WTF?

ja, op elke site een server plaatsen was niet echt intressant... en aangezien die boekhouders nogal veel over en weer vlogen moesten ze altijd hun boekhouding bijhebben
hadden de gegevens maar maandelijks nodig in hoofdkantoor, en meerdere backups laten bijhouden werd door de boekhouders als onprettig bevonden.
grootste ergernis van iedereen... maar het werkte (op af en toe en foutje na dus ) kregen er dus ook geen tijd voor om daar een applicatie voor te schrijven.

In elk bedrijf loopt niet alles perfect, maar zolang het blijft werken mag het meestal zo wel verder gaan... heb je las kleine bedrijven veel te snel groot worden

[ZpAz]

Verwijderd schreef op maandag 20 juni 2011 @ 01:21:
Soms typ ik spontaan een functie op de volgende manier in PHP:

code:

public function foo(string $str) {...}

Om vervolgens een error te krijgen.. ohja doh

In PHP 5.4 is type hinting wel mogelijk, enkel niet voor alles ofzo. Wel arrays, int's en objecten dacht ik.

[Verwijderd]

ZpAz schreef op maandag 20 juni 2011 @ 12:00:
[...]


In PHP 5.4 is type hinting wel mogelijk, enkel niet voor alles ofzo. Wel arrays, int's en objecten dacht ik.

Klopt. Maar niet voor strings Zou ook raar zijn omdat PHP een loosly typed language is. In principe is een Object ook een string als hij de functie __toString geimplementeerd heeft.

vanaf PHP 5.2+ toch?

[ Voor 4% gewijzigd door Verwijderd op 20-06-2011 12:10 ]

[.Gertjan.]

.oisyn schreef op maandag 20 juni 2011 @ 11:46:
[...]

Heb ik ook ooit gehoord. Van mensen hier op GoT die de taal niet snapten. Is er werkelijk een taal waarin een assignment een expression is die áltijd true oplevert (even afgezien zelfgedefnieerde assigment operators uiteraard)? Volgens mij bestaat die niet.

Denk ook niet dat ze bestaan, maar heb ooit iets opgevangen. Het zal inderdaad gaan om talen die waar de if kijkt of het geen null of 0 is, als je dan x = 5 neerzet komt er 5 uit en dat is dan in sommige talen ook een true. Of het ging over boolean assignments. Maar het statement: "als de assignment lukt komt er een true uit de if" heb ik toch met enige regelmaat aan moeten horen, maar waarschijnlijk is dat verwarring.

Overigens is er voor dat soort ifs wel iets te zeggen. Aan de ene kant zijn ze handig, in Axapta kun je zo zelfs een datarow in een if zetten waarbij het systeem kijkt of de row een ID heeft (en dus uit de DB komt). Aan de andere kant krijg je ook wel eens tricky en onleesbare code. Overigens is Axapta behoorlijk zuur als het om null's gaat, die kent hij namelijk niet, hij zal ook nooit een DBNull schrijven... Altijd leuk als je gaat interfacen...

Btw, in C# kan het ook gewoon. De compiler klaagt alleen omdat het type van de assignment expression het type van de te assignen variabele is, en bij a = 5 is dat dus meestal int. Maar een if verwacht een boolean. Als de expressie gewoon een boolean oplevert vindt de compiler het prima:

C#:

if (a = true)
{
}

Nou is het natuurlijk wel weer zo dat je dan gewoon if(a) of if(!a) tikt als je a==true of a==false bedoelt.

True statement was ik helemaal vergeten. Meestal vergelijk je inderdaad niet met == true. Vroeger deed ik dat wel (VB 6), maar toen was ik nog een klein .gertjan.-etje Toen had ik nog geen flauw benul hoe die boolean expressions werkte

[ZpAz]

Verwijderd schreef op maandag 20 juni 2011 @ 12:09:
[...]

Klopt. Maar niet voor strings Zou ook raar zijn omdat PHP een loosly typed language is. In principe is een Object ook een string als hij de functie __toString geimplementeerd heeft.

vanaf PHP 5.2+ toch?

5.0 en uitgebreid bij 5.1 blijkbaar zelfs al.

[YopY]

.Gertjan. schreef op maandag 20 juni 2011 @ 07:48:
[...]

Of je kiest gewoon een taal die echt strong typed is
\[/knuppel in hoenderhok mode]

Of je kiest een strong typed taal met type inference . Best of both worlds: typesafety wordt bepaald en afgedwongen tijdens compile-time, maar je hoeft niet elke var van een type te voorzien (bespaart typ- en denkwerk).

Scala:

  val x = 1 + 33 + 7 // joh, x is een int, duh.
  x.format(12) // compiler: Jonguh, wat doe je nou da ken toch nie?

En verder: talen motten gewoon assignments in if-statements niet toestaan, bam.

[ Voor 11% gewijzigd door YopY op 20-06-2011 14:15 ]

[Verwijderd]

Kan met c# ook gewoon hoor

C#:

     var personenDieSanderHeten = from p in personen where p.Naam == "Sander" select p;

Variabel is een...... IEnummerable van Personen

Daarna:

C#:

   var sander = personenDieSanderHeten.First();

variabel "sander" is een.... Persoon.

en daarna:

C#:

sander = "Sander";

Mag dan niet van de compiler.

[ Voor 37% gewijzigd door Verwijderd op 20-06-2011 14:20 ]

[RayNbow]

Alleen jammer dat Scala en C# aan local type inference doen...

[Verwijderd]

Achja, je kan altijd nog een Object teruggeven en die casten

[.Gertjan.]

Verwijderd schreef op maandag 20 juni 2011 @ 14:34:
Achja, je kan altijd nog een Object teruggeven en die casten

Leuk ja, en dan klapt ergens diep in de krochten van je library de boel om omdat het object wat je meegegeven hebt toch niet helemaal voldoet aan de verwachtingen

Echte bazen doen het gewoon met reflection. Data die je wilt weten als string constanten in je library opnemen en gewoon door je objects heen rennen tot je de betreffende value hebt gevonden. Is ook goed voor je performance

[MBV]

ontzettend leuk voor statische analyse, minder leuk als normale code dat gebruikt

[mithras]

HuHu schreef op zondag 19 juni 2011 @ 23:41:
[...]

Ik heb 'm zo:

[..] > winkelmandje > bevestig order + mail > betalen > terug naar site

En na het betalen komt er achteraf (offline / post-back) van de bank een melding en dan komt het stukje: factuur maken + mail.

Mensen die de betaling niet doen (of afbreken) hebben alsnog een mail met orderbevestiging gehad en kunnen daar zelf eventueel nog wat mee. Betalingen die na 5 dagen nog geen post-back hebben gehad komen als "expired" in het systeem en dan kan de secretaresse er achteraan (of je stuurt automatisch een mail voor een handmatige overschrijving / een nieuwe iDeal transactie).

Daarom was mijn post ook in het [alg] Slechtste programmeervoorbeelden deel 4 geplaatst

Matis schreef op maandag 20 juni 2011 @ 00:22:
Yoda statement probeer ik er bij ons op de toko ook doorheen te drukken.

Yoda

ZpAz schreef op maandag 20 juni 2011 @ 12:00:
[...]

In PHP 5.4 is type hinting wel mogelijk, enkel niet voor alles ofzo. Wel arrays, int's en objecten dacht ik.

Zoals later aangegeven is het 5.0+ voor type hinting (5.1 voor arrays) Dit geldt alleen voor non-scalar variabelen. In php 5.3.99 was dit wel opgenomen (scalar type hinting) maar later weer verwijderd door de verwachte BC breaks.

Verwijderd schreef op maandag 20 juni 2011 @ 12:09:
[...]

Klopt. Maar niet voor strings Zou ook raar zijn omdat PHP een loosly typed language is. In principe is een Object ook een string als hij de functie __toString geimplementeerd heeft.

Een object is geen string als Object::__toString() bestaat. Dat zou wat worden met een loose typed taal waar je scalars altijd kan casten....

[FragFrog]

Verwijderd schreef op maandag 20 juni 2011 @ 01:21:
Soms typ ik spontaan een functie op de volgende manier in PHP:

code:

public function foo(string $str) {...}

Om vervolgens een error te krijgen.. ohja doh

Het kan in principe wel; je zal dan zelf een string class moeten definieren en die gebruiken om je variabele te instantieren. Zoiets (quick'n'dirty):

PHP:

<?php
class String 
{
  private $text = '';
  
  public function __construct ($text) 
  {
    $this -> text = $text;
  }  
  
  public function __toString () 
  {
    return $this -> text;
  }
}

class Foo 
{
  public function bar (String $text) 
  {
    echo $text;
  }
}

$foo  =  new Foo();
$text =  new String("hello");
$foo  -> bar($text);     // Echo's "hello"
$foo  -> bar("hello");   // Catchable fatal error.

Heeft uiteraard in PHP geen enkel nut maar het illustreert wel mooi hoe je in elk geval type-safety kan afdwingen voor objecten. Het illustreert ook een kleine WTF: je kan in PHP classes definieren met dezelfde naam als interne types zodat je errors als deze krijgt:

Argument 1 passed to Foo::bar() must be an instance of string, string given

[MBV]

Heel leuk ook in PHP:

PHP:

define(blaat, 'blabla');

function blaat() {
  echo blaat;
}

class blaat {
  mekker() { echo blaat;}
}
$blaat = new blaat();
$blaat->mekker();

Dit is nog straight-forward, maar je kan echt de meest wazige code schrijven zonder 1 enkele warning

[ Voor 3% gewijzigd door MBV op 20-06-2011 18:26 ]

[.oisyn]

Je vergeet nog een namespace 'blaat'

[Davio]

Bij gebrek aan een Ceil-functie in een bepaalde taal (Progress OpenEdge), vandaag maar mijn eigen geschreven (en Floor, die was makkelijker).

Het is een slecht programmeervoorbeeld omdat elke zichzelf respecterende taal wel zo'n functie heeft. En omdat het een vrij vieze functie geworden is die niet altijd werkt.

Dat komt door de inherente quirks van floats, zo wordt 2.00000000001 ge-ceild naar '2'.

Ik maak er wel een uitdaging van: Bouw je eigen Ceil-functie, je hebt de volgende functies tot je beschikking:
- ROUND(expr, precision): Rondt expr af tot het in precision aangegeven aantal decimalen achter de komma, result is decimal
- TRUNCATE(expr, decimal_places): Kapt expr af tot het opgegeven aantal decimalen achter de komma, result is decimal
- MIN(a, b [,c...]) en MAX(a, b [,c...]): Spreekt voor zich, result is decimal als één van de argumenten decimal is
- De welbekende INTEGER(expr)-functie rondt in Progess af naar de nearest INT (dus INT(2.4) wordt 2, INT(2.6) wordt 3), in plaats van dat hij afkapt (wat de meeste talen doen)
- DECIMAL(expr): Spreekt voor zich

Succes!

PS: Hebben andere talen ook last van dit ceil-probleem met decimals en afronding?

[HuHu]

code:

FLOOR(a) = TRUNCATE(a, 0)
CEIL(a) = TRUNCATE(a + 1, 0)

Dat is toch niet zo moeilijk?

[ Voor 20% gewijzigd door HuHu op 20-06-2011 18:45 ]

[NMe]

MBV schreef op maandag 20 juni 2011 @ 18:26:
Heel leuk ook in PHP:

PHP:

define(blaat, 'blabla');

function blaat() {
  echo blaat;
}

class blaat {
  mekker() { echo blaat;}
}
$blaat = new blaat();
$blaat->mekker();

Dit is nog straight-forward, maar je kan echt de meest wazige code schrijven zonder 1 enkele warning

Op regel 1 krijg je toch op zijn minst een notice.

[mithras]

edit:
nvm

[ Voor 98% gewijzigd door mithras op 20-06-2011 19:33 ]

[Davio]

HuHu schreef op maandag 20 juni 2011 @ 18:45:

code:

FLOOR(a) = TRUNCATE(a, 0)
CEIL(a) = TRUNCATE(a + 1, 0)

Dat is toch niet zo moeilijk?

Werkt niet voor a = 12.00000000000 en dergelijke.

Daar moet 12 uitkomen, komt bij jou 13 uit.

[HuHu]

Davio schreef op maandag 20 juni 2011 @ 18:55:
[...]

Werkt niet voor a = 12.00000000000 en dergelijke.

Daar moet 12 uitkomen, komt bij jou 13 uit.

code:

CEIL(a) :
IF TRUNCATE(a) == a:
  RETURN a
ELSE:
  RETURN TRUNCATE(a + 1, 0)

[Davio]

@Huhu,

Op zoiets kwam ik ook uit, maar dan met INT, omdat er uit Ceil altijd een int moet komen.

Dus:

code:

CEIL(a):
 INT tmp = INTEGER(a) // 2.4 wordt dus 2, 2.6 wordt 3 etc
 IF DEC(tmp) < a THEN RETURN tmp + 1. // Alleen als er naar beneden is afgerond er nog 1 bij optellen
 RETURN tmp. // Voor de == case en als er naar boven is afgerond (tmp > a), hebben we al het goede antwoord

Ik heb het zo gedaan, omdat ik dan al een INT als returnwaarde heb.

[WernerL]

Vandaag op school voor het project aan het programmeren geweest.
Kom ik zojuist deze functie tegen van een klasgenoot van mij:

code:

private function bool_check(col:int):Boolean
        {
            if (col == 1)
                return true;
            else
                return false;
        }

wtf..

[ Voor 3% gewijzigd door WernerL op 20-06-2011 19:41 ]

[Alex)]

Verwijderd schreef op maandag 20 juni 2011 @ 14:17:
Kan met c# ook gewoon hoor

C#:

     var personenDieSanderHeten = from p in personen where p.Naam == "Sander" select p;

Variabel is een...... IEnummerable van Personen

Daarna:

C#:

   var sander = personenDieSanderHeten.First();

variabel "sander" is een.... Persoon.

en daarna:

C#:

sander = "Sander";

Mag dan niet van de compiler.

Kan wel, met implicit.

[ZpAz]

WernerL schreef op maandag 20 juni 2011 @ 19:40:
Vandaag op school voor het project aan het programmeren geweest.
Kom ik zojuist deze functie tegen van een klasgenoot van mij:

code:

private function bool_check(col:int):Boolean
        {
            if (col == 1)
                return true;
            else
                return false;
        }

wtf..

lawl

[Verwijderd]

lol.. de "private" maakt het nog grappiger..

[Onbekend]

Op zich is dat best netjes dat je elke functie zijn eigen handeling laat doen.
Nu vraag ik mij wel af hoe hij aan die 1 komt waarmee de functie true teruggeeft ......

[ Voor 11% gewijzigd door Onbekend op 20-06-2011 21:25 ]

[- peter -]

Ja, ziet er goed uit eigenlijk. Lekker OO.

[.oisyn]

HuHu schreef op maandag 20 juni 2011 @ 19:00:
[...]

code:

CEIL(a) :
IF TRUNCATE(a) == a:
  RETURN a
ELSE:
  RETURN TRUNCATE(a + 1, 0)

En nu nog negatieve getallen

code:

floor(x) := x > 0 ? integer(x) : -integer(-x);
ceil(x) := x > 0 ? -integer(-x) : integer(x);

[ Voor 5% gewijzigd door .oisyn op 20-06-2011 22:13 ]

[Davio]

.oisyn schreef op maandag 20 juni 2011 @ 22:11:
[...]

En nu nog negatieve getallen

code:

floor(x) := x > 0 ? integer(x) : -integer(-x);
ceil(x) := x > 0 ? -integer(-x) : integer(x);

Ja, maar jouw ceil kapt alles gewoon af (of rondt naar beneden en naar boven af), dat is met ceil niet de bedoeling.

[Woy]

Davio schreef op dinsdag 21 juni 2011 @ 09:17:
[...]

Ja, maar jouw ceil kapt alles gewoon af (of rondt naar beneden en naar boven af), dat is met ceil niet de bedoeling.

Het doet volgens mij precies wat CEIL moet doen. Het rond af naar het eerste gehele getal groter of gelijk aan de input

[Davio]

Woy schreef op dinsdag 21 juni 2011 @ 10:15:
[...]

Het doet volgens mij precies wat CEIL moet doen. Het rond af naar het eerste gehele getal groter of gelijk aan de input

Edit: Als ik als voorbeeld 2,4 neem, dan komt er uit -int(-2,4) de waarde 2, terwijl dit 3 moet zijn...

[ Voor 38% gewijzigd door Davio op 21-06-2011 11:05 ]

[bwerg]

WernerL schreef op maandag 20 juni 2011 @ 19:40:
Vandaag op school voor het project aan het programmeren geweest.
Kom ik zojuist deze functie tegen van een klasgenoot van mij:

code:

private function bool_check(col:int):Boolean
        {
            if (col == 1)
                return true;
            else
                return false;
        }

wtf..

Als ik dit soort dingen zie

code:

if (dingetje == true)

maak ik er ook altijd van

code:

if ((dingetje == true) == (dingetje == dingetje))

maar als om al die booleans bool_checks zouden staan is dat eigenlijk wel een stuk mooier ja.

[ Voor 7% gewijzigd door bwerg op 21-06-2011 11:03 ]

[PrisonerOfPain]

WernerL schreef op maandag 20 juni 2011 @ 19:40:
Vandaag op school voor het project aan het programmeren geweest.
Kom ik zojuist deze functie tegen van een klasgenoot van mij:

wtf..

Dat is lang geleden eprog ftw

[EddoH]

bwerg schreef op dinsdag 21 juni 2011 @ 11:01:
[...]

Als ik dit soort dingen zie

code:

if (dingetje == true)

maak ik er ook altijd van

code:

if ((dingetje == true) == (dingetje == dingetje))

maar als om al die booleans bool_checks zouden staan is dat eigenlijk wel een stuk mooier ja.

...ehhm.....huh?
Ligt het aan mij als ik niks van je post begrijp?

[.oisyn]

Davio schreef op dinsdag 21 juni 2011 @ 09:17:
[...]

Ja, maar jouw ceil kapt alles gewoon af (of rondt naar beneden en naar boven af), dat is met ceil niet de bedoeling.

Oh ja, duh. Ik dacht slim te zijn door 'm negatief te maken zodat hij "naar boven" gaat . Maar truncate(x) is natuurlijk altijd gelijk aan -truncate(-x)

[Wijnbo]

EddoH schreef op dinsdag 21 juni 2011 @ 11:20:
[...]


...ehhm.....huh?
Ligt het aan mij dat ik niks van je post begrijp?

Hij bedoelt denk ik dat hij er nog een extra nutteloze conditie aan toevoegt, zodat diegene die het gemaakt heeft ook zoiets heeft van "wtf" in de hoop dat hij het aanpast naar

code:

if (dingetje)

[ Voor 4% gewijzigd door Wijnbo op 21-06-2011 11:23 ]

[Icekiller2k6]

code:

if ((dingetje == true) == (dingetje == dingetje))

Als je dat niet begrijpt vul dan ff in wanneer de statement geldig zou zijn namelijk deze:

code:

if ((true == true) == (true== true))

waarbij dan

code:

(true==true)

true terug geeft dus je hebt dan opnieuw true == true ^^

[ Voor 23% gewijzigd door Icekiller2k6 op 21-06-2011 11:27 ]

[EddoH]

De code snap ik uiteraard wel. De bedoeling ontging me even

[Verwijderd]

Zelf kwam ik weer achter een stukje script (van mezelf) die ik eigenlijk best wel vaak toepas...

PHP:

if( $waarde == $waarde2 ) {
   // doe niets ;)
} else {
   /*
    * en hier staan dan andere code om mee verder te gaan
    */
}

Hoewel het niet fout is, is het vaak niet mogelijk om het anders op te lossen of ik ben er gewoon nog niet achter hoe.

[whoami]

Ehm ... Booleanse logica is toch simpel

code:

if( waarde != waarde2 )
{
}

[EddoH]

code:

<?php

if( $waarde < waarde2 || $waarde > waarde2)
{
}

?>

[Onbekend]

Ik hoop dat een hedendaagse compiler intern de code van EddoH vervangt door die van whoami.

[Grijze Vos]

Onbekend schreef op dinsdag 21 juni 2011 @ 12:20:
Ik hoop dat een hedendaagse compiler intern de code van EddoH vervangt door die van whoami.

Een compiler misschien, maar een php interpreter zeker niet.

[PrisonerOfPain]

Onbekend schreef op dinsdag 21 juni 2011 @ 12:20:
Ik hoop dat een hedendaagse compiler intern de code van EddoH vervangt door die van whoami.

Ik hoop dat een hedendaagse programmeur wat minder vertrouwen krijgt in z'n compiler

[roy-t]

Verwijderd schreef op dinsdag 21 juni 2011 @ 12:11:
Zelf kwam ik weer achter een stukje script (van mezelf) die ik eigenlijk best wel vaak toepas...

PHP:

if( $waarde == $waarde2 ) {
   // doe niets ;)
} else {
   /*
    * en hier staan dan andere code om mee verder te gaan
    */
}

Hoewel het niet fout is, is het vaak niet mogelijk om het anders op te lossen of ik ben er gewoon nog niet achter hoe.

Dit meen je toch niet serieus?!

[Onbekend]

PrisonerOfPain schreef op dinsdag 21 juni 2011 @ 12:30:
[...]
Ik hoop dat een hedendaagse programmeur wat minder vertrouwen krijgt in z'n compiler

Die compiler zorgt wel voor een werkend programma.
Maar als gebruiker wil je graag een vlot werkend programma. En als dit soort punten door de hele code heen zitten, kost het gewoon 3x zoveel rekenkracht....

[Wijnbo]

Voor geneste single-line if statements zou de doodstraf opnieuw ingevoerd moeten worden.

C#:

var a = b > c ? b : ( b > c + 1 ? 5 : c + 5)

Liever alle alternatieven door elkaar dan bovenstaande.

C#:

var temp = b > c + 1 ? 5 : c + 5
var a = b > c ? b : temp

Of voor mijn part

C#:

var a;
if(b > c)
{
    a = b;
}
else
{
    if(b > c + 1)
    {
        a = 5;
    }
    else
    {
        a = c + 5;
    }
}

Vooral i.c.m. wazige variabele namen.

developer.GetOverzicht() : Nullpointer exception.

[ Voor 9% gewijzigd door Wijnbo op 21-06-2011 13:17 ]

[WVL_KsZeN]

Ik kom net deze functies tegen in een macro die al jaren gebruikt wordt (geschreven door een collega die hier niet meer werkt).

code:

/*****************************************************************************/
/* Macro operation definitions                                               */
/*****************************************************************************/

 #define set_bit(ccol_var, ibit)    ccol_var  |= ibit
 #define reset_bit(ccol_var, ibit)    ccol_var  &= -(ibit)
 #define test_bit(ccol_var, ibit)    (ccol_var  & ibit)

En ik maar niet snappen waarom mijn programma niet werkte zoals het moest...

koekje voor de eerste die het ziet :^)

[ Voor 3% gewijzigd door WVL_KsZeN op 21-06-2011 13:42 ]

[FragFrog]

roy-t schreef op dinsdag 21 juni 2011 @ 12:33:
[...]


Dit meen je toch niet serieus?!

Inderdaad, variabelen genaamd 'waarde' en 'waarde2'.?!

_{Ach, je moet ergens beginnen; is er tenminste weer wat te lezen in dit topic}

[CodeCaster]

Wijnbo schreef op dinsdag 21 juni 2011 @ 13:15:


developer.GetOverzicht() : Nullpointer exception.

Hoezo, ben je geen developer?

[MBV]

whoami schreef op dinsdag 21 juni 2011 @ 12:12:
Ehm ... Booleanse logica is toch simpel

code:

if( waarde != waarde2 )
{
}

Zou ik niet zeker van zijn bij PHP, eerlijk gezegd

[PrisonerOfPain]

Onbekend schreef op dinsdag 21 juni 2011 @ 12:58:
[...]


Die compiler zorgt wel voor een werkend programma.
Maar als gebruiker wil je graag een vlot werkend programma. En als dit soort punten door de hele code heen zitten, kost het gewoon 3x zoveel rekenkracht....

Dan moet je ze niet intypen. Een compiler heeft maar een beperkte hoeveelheid optimalisaties die 'ie kan uitvoeren, de meeste zijn niet héél spectaculair en er word maar een beperkt aantal passes over je code gedaan. Daarbij wil het niet zeggen dat als je compiler het een keer automatisch goed optimaliseerd dat 'ie het de volgende keer ook doet, zeker niet als de omliggende code verandert. (Meestal is compiler output deterministisch als de input identiek is, maar zelfs dat is niet altijd het geval).

Kortom, als je wilt dat je programma snel is zul je toch echt zelf het meeste werk moeten verzetten.

[Verwijderd]

WVL_KsZeN schreef op dinsdag 21 juni 2011 @ 13:18:
Ik kom net deze functies tegen in een macro die al jaren gebruikt wordt (geschreven door een collega die hier niet meer werkt).

code:

/*****************************************************************************/
/* Macro operation definitions                                               */
/*****************************************************************************/

 #define set_bit(ccol_var, ibit)    ccol_var  |= ibit
 #define reset_bit(ccol_var, ibit)    ccol_var  &= -(ibit)
 #define test_bit(ccol_var, ibit)    (ccol_var  & ibit)

En ik maar niet snappen waarom mijn programma niet werkte zoals het moest...

koekje voor de eerste die het ziet :^)

Dit is verkeerd!

C:

#define reset_bit(ccol_var, ibit) ccol_var &= -(ibit)

Het moet denk ik dit zijn:

C:

#define reset_bit(ccol_var, ibit) ccol_var &= ~(ibit)

Krijg ik nu een koekje?

[.oisyn]

Ik zou er ook (1 << ibit) van maken. En daarnaast je macro's even afschermen met wat haakjes of een do ... while(0)

[xtra]

Verwijderd schreef op dinsdag 21 juni 2011 @ 12:11:
Zelf kwam ik weer achter een stukje script (van mezelf) die ik eigenlijk best wel vaak toepas...

PHP:

if( $waarde == $waarde2 ) {
   // doe niets ;)
} else {
   /*
    * en hier staan dan andere code om mee verder te gaan
    */
}

Hoewel het niet fout is, is het vaak niet mogelijk om het anders op te lossen of ik ben er gewoon nog niet achter hoe.

PHP:

if( !( $waarde == $waarde2) ) {

als snelle oplossing. Vooral handig als er iets vaker || en && tussen de () staat.

[TJHeuvel]

PHP:

if($waarde != $waarde2)

[.oisyn]

!(a == b) => a != b
!(a > b) => a <= b
!(a && b) => !a || !b
!(a || b) => !a && !b

[Davio]

Hehe, DeMorgan's Law, jaar 2 van de uni, ik herinner het me nog.

[.oisyn]

Ik zal het vast ook wel gehad hebben maar dat kan ik me niet meer herinneren. Hier ben ik gewoon zelf achtergekomen

Het is ook heel logisch (duh, het ís logica ). Als iets alleen waar is zodra zowel a als b waar zijn, dan is het natuurlijk niet meer waar zodra een van de twee (of allebei) niet waar is. Ergo, !(a && b) => !a || !b

Als iets waar is als a waar is of b waar is (of allebei), dan is het natuurlijk niet waar als geen van beide waar is. Oftewel, !(a || b) => !a && !b

[ Voor 61% gewijzigd door .oisyn op 21-06-2011 14:47 ]

[PiepPiep]

Ik wil niet weten hoeveel mensen dit soort simpele logica niet meer volgen.
Ik had laatst trouwens ook een mooie uitspraak over mijn lievelings trui (valt net niet uit elkaar van slijtage), hij is ouder dan ik was toen ik hem kreeg. Snappen veel mensen ook niet

[Janoz]

Ach, de über nerd logica response op een vraag als "Wil je koffie of thee?" blijft natuurlijk:

Ja

[Davio]

Janoz schreef op dinsdag 21 juni 2011 @ 15:24:
Ach, de über nerd logica response op een vraag als "Wil je koffie of thee?" blijft natuurlijk:

Ja

Nog zo'n slechte koffiegrap: "Wat wil je erin?" "Koffie."

Ik vind de Professor Layton serie voor de DS wel leuk, zitten ook veel van die logica puzzeltjes in in de trant van "Vorig jaar scheelden mijn zus en ik 2 jaar, nu is ze 2x zo oud.....".

[EddoH]

"hoe oud ben ik?" edit: "hoe oud is m'n zus?" ..dus...
A: 4

Jeej ik win

[ Voor 35% gewijzigd door EddoH op 21-06-2011 15:55 ]

[.oisyn]

Vorig jaar scheelden mijn zus en ik 2 jaar, nu is ze 2x zo oud...

Die "vorig jaar" is ook zo lekker irrelevant

EddoH schreef op dinsdag 21 juni 2011 @ 15:36:
"hoe oud ben ik?"
A: 4

Jeej ik win

Euh nee, je bent 2.

[ Voor 42% gewijzigd door .oisyn op 21-06-2011 15:50 ]

[EddoH]

.oisyn schreef op dinsdag 21 juni 2011 @ 15:48:
[...]

Die "vorig jaar" is ook zo lekker irrelevant


[...]

Euh nee, je bent 2.

Wel als je die vorig jaar als irrelevant beschouwt ja.....

FAIL
Ik bedoelde dus, "hoe oud is m'n zus" in m'n antwoord

[ Voor 12% gewijzigd door EddoH op 21-06-2011 15:53 ]

[Davio]

Leuk als mensen antwoord proberen te geven op willekeurige getypete retorische voorbeeldvragen.

[.oisyn]

EddoH schreef op dinsdag 21 juni 2011 @ 15:52:
[...]


Wel als je die vorig jaar als irrelevant beschouwt ja.....

Want jij vindt dat als je vorig jaar 2 jaar scheelde dat dat nu niet meer zo is?

[kaesve]

waar het volgens mij om gaat is dat zuslief in 1 jaar tijd twee keer ouder is geworden. zus was vorig jaar dus 1 en nu 2. jij scheelt 2 jaar, dus je bent ofwel 0 ofwel 4

[kenneth]

Davio schreef op dinsdag 21 juni 2011 @ 15:54:
Leuk als mensen antwoord proberen te geven op willekeurige getypete retorische voorbeeldvragen.

Inderdaad, welkom op GoT

[EddoH]

.oisyn schreef op dinsdag 21 juni 2011 @ 15:57:
[...]

Want jij vindt dat als je vorig jaar 2 jaar scheelde dat dat nu niet meer zo is?

Anders had ik 0 kunnen zijn en mijn zus 2
mits 2x0= 2 is.


...Haha, ik stop ermee

[ Voor 8% gewijzigd door EddoH op 21-06-2011 16:01 ]

[Nappa]

Verwijderd schreef op dinsdag 21 juni 2011 @ 12:11:
Zelf kwam ik weer achter een stukje script (van mezelf) die ik eigenlijk best wel vaak toepas...

PHP:

if( $waarde == $waarde2 ) {
   // doe niets ;)
} else {
   /*
    * en hier staan dan andere code om mee verder te gaan
    */
}

Hoewel het niet fout is, is het vaak niet mogelijk om het anders op te lossen of ik ben er gewoon nog niet achter hoe.

Ik gebruik die constructie wel eens om een bepaalde conditie toe te lichten in commentaar als de reden niet heel duidelijk blijkt uit de code zelf, ik vind het wel handig

C#:

            if( conditie )
            {
                // In sommige gevallen bla... bijv. wanneer bla.
                // In dat geval willen we niet dat ... omdat...
            }
            else
            {
                // Code
            }

[.oisyn]

kaesve schreef op dinsdag 21 juni 2011 @ 15:58:
waar het volgens mij om gaat is dat zuslief in 1 jaar tijd twee keer ouder is geworden. zus was vorig jaar dus 1 en nu 2. jij scheelt 2 jaar, dus je bent ofwel 0 ofwel 4

Ah ja zo kun je het ook nog interpreteren. Ik interpreteerde het als "nu is ze 2x zo oud als jij"

[EddoH]

Begint een beetje op 2e coffee corner te lijken nu

On topic: kom net dit tegen, inclusief het commentaar...

C++:

if((signed int)len < (signed int)sizeof(int)) //..wtf

[PrisonerOfPain]

(Misschien handig om het type van len duidelijk te maken).

[EddoH]

hmm, mja, zou kunnen, vind ik alleen neit de mooise oplossing.. sowieso 'len' als signed.....
Ben wel benieuwd wie de "wtf" erbij heeft gezet

[Soultaker]

Als len signed is, moet je juist het resultaat van sizeof() casten, anders mis je negatieve waarden van len, omdat bij gelijke grootte signed types naar unsigned geconverteerd worden in plaats van andersom (en sizeof(int) obviously klein genoeg is dat de conversie veilig is).

De cast van len zou ik dan wél achterwege laten. En signed int schrijven in plaats van int; tja, ik zou het overbodig vinden, maar het is op zich ook correct en het maakt het wel heel expliciet dat het om signedness te doen is.

[ Voor 29% gewijzigd door Soultaker op 21-06-2011 16:26 ]

[EddoH]

Het is op zich wel te snappen aangezien het achterliggende framework alle sizes en lengths als signed ehandeld (wat ik de eigenlijke wtf vind).
Ik vind het alleen lelijke code. De combi met het commentaar maakt het alleen nog maar slechter

[.oisyn]

Ik ben zelf niet zo'n fan van unsigned, het mixen van signed en unsigned variabelen zorgt vaker dan eens voor problemen.

[Verwijderd]

Ik heb m'n koekje nog niet gehad

[EddoH]

.oisyn schreef op dinsdag 21 juni 2011 @ 16:35:
Ik ben zelf niet zo'n fan van unsigned, het mixen van signed en unsigned variabelen zorgen vaker dan eens voor problemen.

Omdat bestaande code vaak ook signed gebruikt doe ik dat ook, om problemen te voorkomen. Maar ik vind len's en sizes als size een beetje tegen mn gevoel indruisen. Sowieso gooi je de helft van je range weg, en kun je mooi constructies zoals bovenstaand verwachten als je wel een keer unsigned nterug krijgt / nodig hebt.

[EddoH]

Verwijderd schreef op dinsdag 21 juni 2011 @ 14:16:

Krijg ik nu een koekje?

Vragen betekent niet krijgen