[alg] Slechtste programmeervoorbeelden deel 5

[BoringDay]

Gomez12 schreef op donderdag 29 januari 2015 @ 21:58:
[...]

Het is en blijft PHP, dus waarom zou je dit enkel op een boolean gebruiken

Maar in loosely typed talen zou ik booleans toch expliciet (en dan niet met == maar met ===true) checken, iets te vaak meegemaakt dat iemand $var dacht te kunnen gebruiken voor een string / int en dan zo af en toe "vreemde" resultaten krijgen

Kan prima
return ($true) ? true : false;

niet assigned $true is (nil) 0 en anders ongelijk aan 0.

[ Voor 6% gewijzigd door BoringDay op 30-01-2015 19:39 ]

[Janoz]

Serieus? Jullie hebben het nog steeds over de vorm? De by far biggest WTF is dat er überhaupt een variabele met de naam 'true' is!

[.oisyn]

Precies. Hij moet natuurlijk gewoon $false heten.

[Verwijderd]

biglia schreef op dinsdag 13 januari 2015 @ 11:34:
C# door een Senior Developer. Converteren van int naar string voortdurend als volgt:

string _example = 1 + "";

Ik heb wel eens discussies met mensen die zichzelf "medior" (of zelfs senior) noemen maar wel dergelijke streken uithalen. Met als hoofdargument: "Het resultaat is toch hetzelfde, dus wat maakt het uit?". Of het klassieke: "Ja, dat andere vind ik niet zo mooi"-argument.

Tja.

[begintmeta]

.oisyn schreef op zaterdag 31 januari 2015 @ 00:07:
Precies. Hij moet natuurlijk gewoon $false heten.

Het beste kan je de variabelenaam afhankelijk maken van de waarde van de variabele, dus bij waarde false is het $false en bij waarde true is het $true.

[PierreAronnax]

Richh schreef op donderdag 29 januari 2015 @ 16:04:
Zolang je variabele een boolean is...

PHP:

if($var)

Waarom zou je er nog een == tussen zetten

Door deze aanname had ik afgelopen week ook een behoorlijke WTF.

Normaal kan je inderdaad zoiets doen:

C:

uint16_t interval = 1234;
if (interval)
{

Dus ik had een functie:

C:

bool wakeup(void)
{
    ...
    return interval;
}

Eventjes vergetend dat bool eigenlijk het volgende is:

C:

typedef uint8_t bool;

Dit gaat dus goed, behalve als interval precies 256 of 512 etc. is.

Gevolg: langzaam begonnen apparaten in het veld uit te vallen.

[DrBreakalot]

PierreAronnax schreef op zaterdag 31 januari 2015 @ 15:28:
[...]

Door deze aanname had ik afgelopen week ook een behoorlijke WTF.

Normaal kan je inderdaad zoiets doen:

C:

uint16_t interval = 1234;
if (interval)
{

Dus ik had een functie:

C:

bool wakeup(void)
{
    ...
    return interval;
}

Eventjes vergetend dat bool eigenlijk het volgende is:

C:

typedef uint8_t bool;

Dit gaat dus goed, behalve als interval precies 256 of 512 etc. is.

Gevolg: langzaam begonnen apparaten in het veld uit te vallen.

Daarom gebruik je de 'Cast to bool' operator

C:

bool wakeup(void)
{
    ...
    return !!interval;
}

[F.West98]

Even wat oude code van een jaar oud teruglezen:

C#:

int someInt;
try {
    int.TryParse(someString, out someInt);
} catch {
    // error handling
}

En ik me maar afvragen waarom die nooit fout ging

spoiler:

TryParse returnt een bool met of het gelukt is of niet en throwt niets. Parse throwt wel een exception als het fout gaat

[ Voor 25% gewijzigd door F.West98 op 01-02-2015 00:44 ]

[Alex)]

Intern doet TryParse niet veel anders dan dit hoor:

C#:

public bool TryParse(string input, out value)
{
  try
  {
    value = Int32.Parse(input);
    return true;
  }
  catch (FormatException)
  {
    return false;
  }
}

[F.West98]

TryParse geeft geen Exception intern. Het is sneller dan een Parse met een catch (Exceptions zijn duur). Het is volgens de decompiler juist andersom

C#:

[__DynamicallyInvokable]
    public static bool Parse(string value)
    {
      if (value == null)
        throw new ArgumentNullException("value");
      bool result = false;
      if (!bool.TryParse(value, out result))
        throw new FormatException(Environment.GetResourceString("Format_BadBoolean"));
      return result;
    }

    [__DynamicallyInvokable]
    public static bool TryParse(string value, out bool result)
    {
      result = false;
      if (value == null)
        return false;
      if ("True".Equals(value, StringComparison.OrdinalIgnoreCase))
      {
        result = true;
        return true;
      }
      if ("False".Equals(value, StringComparison.OrdinalIgnoreCase))
      {
        result = false;
        return true;
      }
      value = bool.TrimWhiteSpaceAndNull(value);
      if ("True".Equals(value, StringComparison.OrdinalIgnoreCase))
      {
        result = true;
        return true;
      }
      if (!"False".Equals(value, StringComparison.OrdinalIgnoreCase))
        return false;
      result = false;
      return true;
    }

[Soultaker]

Ook wel rare gegenereerde code trouwens. Waarom wordt op regel 25 wel een assignment aan result gegenereerd, maar voor regel 35 niet? En waarom staat die assignment wel op regel 36, terwijl de compiler de regel ervoor nog wist dat result gelijk aan false is op dat punt?

[ValHallASW]

De return value van de functie is of het parsen gelukt is, niet het resultaat Regel 35 is dus "return error", waar alle andere paden het resultaat instellen en "return success" doen.

[Soultaker]

Ik zie nu inderdaad dat regel 35 een negatie voor het if-statement heeft staan; die had ik even gemist. De code is in die zin dus wel logisch, aangezien "result" voor elke "return true" overschreven wordt.

Feit blijft wel dat "result" bovenaan op "false" geïnitialiseerd is, dus die waarde is ook gedefinieerd als de functie "false" returnt, en dat betekent dat regel 25 en 36 feitelijk overbodig zijn.

[P_Tingen]

begintmeta schreef op zaterdag 31 januari 2015 @ 15:03:
[...]

Het beste kan je de variabelenaam afhankelijk maken van de waarde van de variabele, dus bij waarde false is het $false en bij waarde true is het $true.

Bij ons moeten variabele altijd positief geformuleerd zijn. Een variabele als "NotPrinted" kan dus niet. Dit is om rare en slecht leesbare constructies in de code tegen te gaan. Iets als

code:

IF NOT NotPrinted THEN ...

Leest niet lekker en is vragen om problemen.

code:

IF Printed THEN ...

Is dan al een stuk beter. Een variabele met de naam $False zou er bij ons niet inkomen. Hooguit $IsFalse, maar ook dat is twijfelachtig.

[ValHallASW]

Soultaker schreef op zondag 01 februari 2015 @ 11:53:
Ik zie nu inderdaad dat regel 35 een negatie voor het if-statement heeft staan; die had ik even gemist. De code is in die zin dus wel logisch, aangezien "result" voor elke "return true" overschreven wordt.

Feit blijft wel dat "result" bovenaan op "false" geïnitialiseerd is, dus die waarde is ook gedefinieerd als de functie "false" returnt, en dat betekent dat regel 25 en 36 feitelijk overbodig zijn.

Nee, want ze zijn semantisch wel degelijk verschillend. De ene false is 'de waarde die teruggegeven wordt als het parsen mislukt', de andere false is 'de waarde die teruggegeven wordt als de waarde false is'. Die eerste had ook true kunnen zijn, 'geen verandering in waarde' , of (als het een nullable boolean was geweest) null.

Dat neem niet weg dat het gek is dat de compiler de extra assignments niet wegoptimaliseert...

[F.West98]

Het ging om het idee en niet om de implementatie

De implementatie op GitHub ziet er al veel logischer uit:

C#:

// Determines whether a String represents true or false.
//
public static Boolean Parse (String value) {
if (value==null) throw new ArgumentNullException("value");
Contract.EndContractBlock();
Boolean result = false;
if (!TryParse(value, out result)) {
throw new FormatException(Environment.GetResourceString("Format_BadBoolean"));
}
else {
return result;
}
}
// Determines whether a String represents true or false.
//
public static Boolean TryParse (String value, out Boolean result) {
result = false;
if (value==null) {
return false;
}
// For perf reasons, let's first see if they're equal, then do the
// trim to get rid of white space, and check again.
if (TrueLiteral.Equals(value, StringComparison.OrdinalIgnoreCase)) {
result = true;
return true;
}
if (FalseLiteral.Equals(value,StringComparison.OrdinalIgnoreCase)) {
result = false;
return true;
}
// Special case: Trim whitespace as well as null characters.
value = TrimWhiteSpaceAndNull(value);
if (TrueLiteral.Equals(value, StringComparison.OrdinalIgnoreCase)) {
result = true;
return true;
}
if (FalseLiteral.Equals(value,StringComparison.OrdinalIgnoreCase)) {
result = false;
return true;
}
return false;
}

[Soultaker]

ValHallASW schreef op zondag 01 februari 2015 @ 18:57:
Nee, want ze zijn semantisch wel degelijk verschillend. De ene false is 'de waarde die teruggegeven wordt als het parsen mislukt', de andere false is 'de waarde die teruggegeven wordt als de waarde false is'.

Dat is een puur filosofisch onderscheid; het is dezelfde waarde die je in beide gevallen teruggeeft, dus die hoef je dan niet nog eens te overschrijven.

Dat neem niet weg dat het gek is dat de compiler de extra assignments niet wegoptimaliseert...

Ik weet eerlijk gezegd niet precies wat de semantics zijn von out-parameters in C#. Zijn het extra return values, of references? In het tweede geval kan de compiler die optimalisatie niet uitvoeren omdat het in theorie mogelijk is dat String.Equals() die variabele tussendoor overschrijft.

[roy-t]

Ik vermoed omdat er ook het ref keyword is in C# dat het extra return values zijn. Maar 100% zeker weten doen ik het niet.

[.oisyn]

Het zijn onder water gewoon references, je kan er ook niet op overloaden. Maar je mag ze van de compiler niet lezen voor je ze schrijft, en ze moeten verplicht geschreven worden voor de functie retourneert.

C#:

using System;
using System.Threading;

namespace CsTest
{
    class Program
    {
        static AutoResetEvent evtMain = new AutoResetEvent(false), evtThread = new AutoResetEvent(false);

        static void Foo(out int i)
        {
            i = 1;
            evtMain.Set();
            evtThread.WaitOne();
            Console.WriteLine(i);
            i = 3;
            evtMain.Set();
            evtThread.WaitOne();
        }

        static void Main(string[] args)
        {
            int i = 0;
            Console.WriteLine(i);

            Thread t = new Thread(() => Foo(out i));
            t.Start();

            evtMain.WaitOne();
            Console.WriteLine(i);
            i = 2;

            evtThread.Set();
            evtMain.WaitOne();
            Console.WriteLine(i);

            evtThread.Set();

        }
    }
}

Dit print gewoon 0, 1, 2, 3. Maar volgens mij mag je daar niet op rekenen.

[DeluxZ]

Schaamteloze repost vanuit de Devschuur..

--------------------------
Yes, we hebben er weer een gevonden. Bezig met een project wat we overgenomen hebben van een andere partij...

C#:

[Serializable]
public class CreatePrizeViewModel
{
    [Required]
    public string Text { get; set; }
    [Required]
    public string Title { get; set; }
    [Required]
    public string Description { get; set; }
    [Required]
    public string SocialText { get; set; }
    [Required]
    public string StartDateTime { get; set; }
    [Required]
    public string EndDateTime { get; set; }
    [Required]
    [Range(0, Int64.MaxValue)]
    public int NumberOfPrizes { get; set; }
    [Required]
    public string Value { get; set; }
    public bool IsPublished { get; set; }
    [Required]
    public string SelectedPage { get; set; }
    [Required]
    public HttpPostedFileBase Photo { get; set; }
}

C#:

[HttpPost]
public async Task<ActionResult> Create(CreatePrizeViewModel model)
{
    DateTime startDateTime, endDateTime;
    if (!GetDateTimeFromString(model.StartDateTime, out startDateTime))
        ModelState.AddModelError("StartDateTime", new Exception("Invalid date and time, please supply a valid value"));
    if (!GetDateTimeFromString(model.EndDateTime, out endDateTime))
        ModelState.AddModelError("EndDateTime", new Exception("Invalid date and time, please supply a valid value"));

C#:

private bool GetDateTimeFromString(string input, out DateTime result)
{
    try
    {
        var splitDateAndTime = input.Split(' ');
        var splittedDate = splitDateAndTime[0].Split('/');
        var splittedTime = splitDateAndTime[1].Split(':');

        var day = int.Parse(splittedDate[0]);
        var month = int.Parse(splittedDate[1]);
        var year = int.Parse(splittedDate[2]);
        var hour = int.Parse(splittedTime[0]);
        var min = int.Parse(splittedTime[1]);

        result = new DateTime(year, month, day, hour, min, 0);
    }
    catch (Exception ex)
    {
        result = DateTime.MinValue;
        return false;
    }

    return true;
}

[Styxxy]

Want DateTime.TryParse is te moeilijk .

[ValHallASW]

Styxxy schreef op woensdag 04 februari 2015 @ 16:55:
Want DateTime.TryParse is te moeilijk .

DateTime.TryParse goed doen is nog niet helemaal triviaal. Een simpele

C#:

return DateTime.TryParse(input)

doet namelijk niet wat je wilt, en de documentatie uitpluizen tot je de goede variant te pakken hebt is nog niet helemaal triviaal.

spoiler:

Je moet namelijk ook de juiste CultureInfo meegeven, want anders wordt je datum opeens anders geïnterpreteerd als de landinstellingen van Windows anders staan... en dat is waarschijnlijk niet wat je wilt als je zo'n specifiek formaat aan het parsen bent. Dan kom je dus uit op iets als

[code=csharp]
CultureInfo nlNL = new CultureInfo("nl-NL");
return DateTime.TryParseExact(dateString, "d/M/yyyy H:m", nlNL, DateTimeStyles.None, out result);
[/code]

maar ik weet niet zeker of je daarmee alle relevante mogelijkheden (moet '04/02' ook werken, of alleen '4/2', of juist alleen '04/02'?) te pakken hebt.

[ Voor 42% gewijzigd door ValHallASW op 04-02-2015 17:07 ]

[Haan]

C#:

[Range(0, Int64.MaxValue)]
public int NumberOfPrizes { get; set; }

is ook wel een interessante range restrictie

[DeluxZ]

Haan schreef op donderdag 05 februari 2015 @ 09:44:

C#:

[Range(0, Int64.MaxValue)]
public int NumberOfPrizes { get; set; }

is ook wel een interessante range restrictie

Haha, ja dat is al veranderd naar Int16.MaxValue...

[afraca]

Ik vind dit stuk code zo heerlijk, heb er echt minutenlang om moeten lachen:

Python:

x = 23493
 
 
class SquareRootResult(Exception):
    def __init__(self, number, sqrt):
        self.number = number
        self.sqrt = sqrt
 
 
def Iterate(x, guess):
    guessedX = guess * guess
 
    difference = x - guessedX
    if difference > -0.0001 and difference < 0.0001:
        raise SquareRootResult(number=x, sqrt=guess)
 
    newGuess = 0.5 * (guess + (x / guess))
    return newGuess
 
 
def GetInitialGuess(x):
    # Unimportant code. From Wikipedia:
    # "one starts with an initial guess which is reasonably close to the true root"
    import math
    return math.sqrt(x)
 
 
numIterations = 0
 
try:
    guess = GetInitialGuess(x)
    while True:
        numIterations += 1
        guess = Iterate(x, guess)
except SquareRootResult as e:
    print "Square root of", e.number, "is", e.sqrt, "in", numIterations, "iterations."
except:
    print "This number has no square root after",  numIterations, "iterations."

De pracht van de comment `unimportant code` is mijn favoriet. En daarnaast natuurlijk de gekke control flow met Exception mishandeling.

[WernerL]

The fuck...
En zie ik goed dat x en guess bij iedere iteratie in de while true loop gelijk blijven?
Wat betekend dat hij continue dezelfde sqrt uitrekent.

En die try catch block..

[Firesphere]

WernerL schreef op zaterdag 14 februari 2015 @ 21:39:
The fuck...
En zie ik goed dat x en guess bij iedere iteratie in de while true loop gelijk blijven?
Wat betekend dat hij continue dezelfde sqrt uitrekent.

En die try catch block..

Nee, want x wordt geupdate in de iterate, dus na de aanroep is x verandert, dus is de aanroep iterate(x) gewoon een iteratie met een nieuwe x

[WernerL]

x wordt op regel 1 geinitialiseerd, ik zie nergens een regel code staan waar x een nieuwe waarde krijgt na regel 1. Of ik kijk er gigantisch overheen. :-)

//edit
guess wordt wel geupdate. Maar dat maakt de code niet minder WTF.

[ Voor 20% gewijzigd door WernerL op 14-02-2015 21:43 ]

[Firesphere]

Ow ja, ik keek er ook even overheen, x blijft inderdaad hetzelfde

[NESFreak]

afraca schreef op zaterdag 14 februari 2015 @ 21:22:
Ik vind dit stuk code zo heerlijk, heb er echt minutenlang om moeten lachen:

De pracht van de comment `unimportant code` is mijn favoriet. En daarnaast natuurlijk de gekke control flow met Exception mishandeling.

In wat voor context ben je deze code tegengekomen als ik vragen mag? Ik reken m'n wortels altijd met de volgende functie uit trouwens ;-)

C:

float Q_rsqrt( float number )
{
    long i;
    float x2, y;
    const float threehalfs = 1.5F;
 
    x2 = number * 0.5F;
    y  = number;
    i  = * ( long * ) &y;                       // evil floating point bit level hacking
    i  = 0x5f3759df - ( i >> 1 );               // what the fuck?
    y  = * ( float * ) &i;
    y  = y * ( threehalfs - ( x2 * y * y ) );   // 1st iteration
//      y  = y * ( threehalfs - ( x2 * y * y ) );   // 2nd iteration, this can be removed
 
    return y;
}

[WernerL]

Aah hij heeft Newton's method gebruikt. http://webcache.googleuse...&cd=1&hl=nl&ct=clnk&gl=nl

Daar staat inderdaad de regel "one starts with an initial guess". Dat Math.sqrt() in Python al een correcte uitkomst geeft was de desbetreffende auteur niet van op de hoogte blijkbaar. Ik ben ook wel benieuwd waar dit prachtig stukje prorgammeerstukje vandaan komt...

[afraca]

Mooi dat jullie er ook een beetje van kunnen genieten. Maar de eerlijkheid gebied mij te zeggen dat ik dit niet in de praktijk ben tegengekomen, maar op:
http://www.reddit.com/r/s...oot_finder_running_in_o1/

Dus inderdaad Newton's methode voor sqrt. Maar als je inderdaad zo kiest voor je initial guess, dan slaat de rest nergens meer op.

[RayNbow]

afraca schreef op zaterdag 14 februari 2015 @ 22:53:
Maar als je inderdaad zo kiest voor je initial guess, dan slaat de rest nergens meer op.

Dat hangt natuurlijk af van hoe goed de implementatie van Python's sqrt is.

[afraca]

Zolang we niet de complete code hebben, kunnen er nog veel spannendere dingen gebeuren dan dat de implementatie van Python niet precies genoeg is:

Python: awesome_math.py

def sqrt(x):
    return 42

Python:

import sys
sys.modules['math'] = __import__['awesome_math']

import math

RayNbow , je bent wel een beetje ambigu. Er is natuurlijk niet DE Python implementatie, wel die van CPython (wrapper naar de standaard C implementatie (welke standaard? ) ) , PyPy, Jython, IronPython enz.

[RayNbow]

afraca schreef op zondag 15 februari 2015 @ 09:34:
Zolang we niet de complete code hebben, kunnen er nog veel spannendere dingen gebeuren dan dat de implementatie van Python niet precies genoeg is:

Python: awesome_math.py

def sqrt(x):
    return 42

Die implementatie is nauwkeurig genoeg, imho.

[.oisyn]

NESFreak schreef op zaterdag 14 februari 2015 @ 22:27:
[...]


In wat voor context ben je deze code tegengekomen als ik vragen mag? Ik reken m'n wortels altijd met de volgende functie uit trouwens ;-)

C:

float Q_rsqrt( float number )
{
    long i;
    float x2, y;
    const float threehalfs = 1.5F;
 
    x2 = number * 0.5F;
    y  = number;
    i  = * ( long * ) &y;                       // evil floating point bit level hacking
    i  = 0x5f3759df - ( i >> 1 );               // what the fuck?
    y  = * ( float * ) &i;
    y  = y * ( threehalfs - ( x2 * y * y ) );   // 1st iteration
//      y  = y * ( threehalfs - ( x2 * y * y ) );   // 2nd iteration, this can be removed
 
    return y;
}

Die zat ook in Quake 1. Tegenwoordig is het hopeloos inefficient.

[F.West98]

Java:

private void someFunc(double mean) {
/**/
double EX = mean;
int k;
for(k=0;k+1 <= EX;k++);
/* iets met k */
}

En ik maar denken dat die ';' per ongeluk was en het stuk erna geloopt moest worden. Maar niets is minder waar.

[ Voor 3% gewijzigd door F.West98 op 20-02-2015 09:52 ]

[boe2]

.oisyn schreef op zondag 15 februari 2015 @ 10:03:
[...]


Die zat ook in Quake 1. Tegenwoordig is het hopeloos inefficient.

Echt? Ik heb onlangs nog eens met vectors zitten spelen in een ingame scripting taal (Wildstar) en had in iedere frame de afstand tussen twee punten nodig. Ik dacht dit dit de heilige graal was om snel de sqrt te berekenen

[NESFreak]

boe2 schreef op vrijdag 20 februari 2015 @ 10:07:
[...]

Echt? Ik heb onlangs nog eens met vectors zitten spelen in een ingame scripting taal (Wildstar) en had in iedere frame de afstand tussen twee punten nodig. Ik dacht dit dit de heilige graal was om snel de sqrt te berekenen

Zeker in een scripttaal wil je in principe altijd te build-ins gebruiken. Kijk even of je scripttaal een functie heeft die de norm van een vector voor je kan berekenen (of zelfs vectoren kan normalizeren, mocht dat je uiteindelijke doel zijn). Doordat je scripttaal per regel waarschijnlijk 1000 checks en casts doet (zeker als het interprented is), is minder code over het algemeen een stuk sneller.

fast_rsqrt(x) is een stukje code van voordat SSE instructies gemeengoed waren. Mocht je C(++) code hebben dan zijn gewone rsqrt statements samen met -O3 -ffastmath waarschijnlijk iets sneller.^{(citation needed)}

[hellfighter87]

humh

PHP:

function replace($str,$search,$replace) {
    return str_replace($search,$replace,$str);
}

[ Voor 6% gewijzigd door hellfighter87 op 20-02-2015 13:03 ]

[Merethil]

hellfighter87 schreef op vrijdag 20 februari 2015 @ 13:01:
humh

PHP:

function replace($str,$search,$replace) {
    return str_replace($search,$replace,$str);
}

Deze is tenminste in de juiste volgorde!

[aap]

NESFreak schreef op vrijdag 20 februari 2015 @ 10:57:
fast_rsqrt(x) is een stukje code van voordat SSE instructies gemeengoed waren. Mocht je C(++) code hebben dan zijn gewone rsqrt statements samen met -O3 -ffastmath waarschijnlijk iets sneller.^{(citation needed)}

Ik wed dat die SSE instructie precies hetzelfde doet als fast_sqrt, maar dan in hardware!

[Fordox]

De beste wachttijd in assembly ooit:

code:

nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop
nop

[.oisyn]

boe2 schreef op vrijdag 20 februari 2015 @ 10:07:
[...]

Echt? Ik heb onlangs nog eens met vectors zitten spelen in een ingame scripting taal (Wildstar) en had in iedere frame de afstand tussen twee punten nodig.

Elke frame maarliefst, nou nou . Met 60 fps (16,7ms per frame) op een 3GHz CPU heb je 50 miljoen cycles te besteden, denk je nou echt dat die paar cycles die je eventueel kunt winnen het verschil maken?

Het is traag omdat je een integer load doet na een floating point store, dat leidt tot een pipeline flush en dus een enorme stall. Het kan wel snel zijn, maar dan moet je het volledig in SSE uitvoeren (dan kun je de int en float operaties op hetzelfde register doen), maar dan kun je net zo goed de rsqrt instructie zelf gebruiken.

[F.West98]

Een lijst bijhouden met verschillende objecten die uiteindelijk allemaal een view uitspugen. Wat doe je dan?

Java:

private int FIRSTTYPE = 1;
private int SECONDTYPE = 2;
/*****/
private List<FirstType> firstTypeList;
private List<SecondType> secondTypeList;
private List<Integer> elementsList;

public void render() {
    /** Get views **/
    int firstTypeIndex, secondTypeIndex = 0;
    for(int type : elementsList) {
        switch(type) {
            case FIRSTTYPE: {
                FirstType element = firstTypeList.get(firstTypeIndex);
                firstTypeIndex++;
                doSomethingWith(element.getView());
            }
            case SECONDTYPE: {
                SecondType element = secondTypeList.get(secondTypeIndex);
                secondTypeIndex++;
                doSomethingWith(element.getView());
            }
        }
    }
}

Natuurlijk niet iets met een abstract class of anders een interface die wordt geëxtend/geïmplementeerd. Dat zou te gemakkelijk zijn.

[F.West98]

Vergeet de vorige post. Dit is echt enorm erg.

Java:

private void loop1() {
    try {
        loop2();
    } catch(StackOverflowError e) {
        loop1();
    }
}
private void loop2() {
    loop2();
}
public void wait() {
    try {
        loop1();
    } catch(StackOverflowError e) {
    }
}

Ik durf dit niet eens te runnen

[Webgnome]

F.West98 schreef op vrijdag 27 februari 2015 @ 02:21:
Vergeet de vorige post. Dit is echt enorm erg.

Java:

private void loop1() {
    try {
        loop2();
    } catch(StackOverflowError e) {
        loop1();
    }
}
private void loop2() {
    loop2();
}
public void wait() {
    try {
        loop1();
    } catch(StackOverflowError e) {
    }
}

Ik durf dit niet eens te runnen

En de medewerker die dit gemaakt heeft is nog in dienst? Of is dit een 'testje' ? Kan mij niet indenken waarom je dit soort code zou kunnen implementeren zonder zelf door te hebben dat dit nog niet helemaal de bedoeling is.

[boe2]

F.West98 schreef op vrijdag 27 februari 2015 @ 02:21:
Vergeet de vorige post. Dit is echt enorm erg.

Java:

private void loop1() {
    try {
        loop2();
    } catch(StackOverflowError e) {
        loop1();
    }
}
private void loop2() {
    loop2();
}
public void wait() {
    try {
        loop1();
    } catch(StackOverflowError e) {
    }
}

Ik durf dit niet eens te runnen

Lijkt me geschreven om bewust je pc te saboteren

[.oisyn]

Waarom? Er gebeurt niets hoor. Er gaat een thread out of stack space, daar merk je als gebruiker niets van.

[F.West98]

Onderdeel van een OS projectje ergens op Github die ik wilde gebruiken in mijn app... Nu toch maar eens verder zoeken.

[PrisonerOfPain]

.oisyn schreef op vrijdag 27 februari 2015 @ 10:36:
Waarom? Er gebeurt niets hoor. Er gaat een thread out of stack space, daar merk je als gebruiker niets van.

Dit had je op de PS3 moeten kunnen doen, dan wist je tenminste zeker dat je maar 256K stackspace nodig had op het moment dat de stackspace als last-resort ineens verlaagd moest worden (in plaats van willekeurig de heap te corrupten).

[Verwijderd]

-

[ Voor 99% gewijzigd door Verwijderd op 05-09-2023 11:17 ]

[Haijo]

Mag ik deze aandragen?

Java:

public static final int HUNDRED_AND_FIFTY_FOUR = 153;

[RayNbow]

Haijo schreef op dinsdag 03 maart 2015 @ 19:24:
Mag ik deze aandragen?

Java:

public static final int HUNDRED_AND_FIFTY_FOUR = 153;

Wat als dit gebruikt wordt om een item in een array te adresseren maar in het domein men bij 1 begint te tellen?

[Haijo]

1) Hij wordt nergens gebruikt
2) Deze applicatie zit vol met constanten die <GETAL> heten en <GETAL> bevatten.

Java:

public static final int ONE = 1;
public static final int TWO = 2;
public static final int TWENTY = 20;

Maar ook strings en chars hebben dat:

Java:

public static final String SPACE=" ";
public static final String COMMA=",";
public static final String COMMA_SPACE=", ";

enzovoorts, enzovoorts. Mijn senior heeft het idee dat gedaan is om Sonar waarschuwingen over magic numbers te onderdrukken. Iets met klok en klepel dus.

[mitchdc]

Ik begon bij een nieuw bedrijf en opende een project...

C#:

const string ITEM = "<li><a href=\"{0}\">{1}</a></li>";

    protected void Page_Load(object sender, EventArgs e)
    {
        StringBuilder sb = new StringBuilder();

        using (DataConnection conn = new DataConnection(PublicationScope.Published))
        {
            IEnumerable<Layout.Navigation.BottomLink> bottomlinks = conn.Get<Layout.Navigation.BottomLink>().OrderBy(b => b.Position);

            foreach (Layout.Navigation.BottomLink link in bottomlinks)
            {
                PageNode node = conn.SitemapNavigator.GetPageNodeById(link.Page);
                if(node != null)
                    sb.AppendLine(string.Format(ITEM, node.Url, node.MenuTitle));
            }
        }

        lit_footernavigation.Text = sb.ToString();
    }

ASP.NET Webforms.

[kenneth]

Hij gebruikt een StringBuilder, beter dan string

[Jerrythafast]

Haijo schreef op woensdag 04 maart 2015 @ 11:06:
Maar ook strings en chars hebben dat:

Java:

public static final int SPACE=" ";
public static final int COMMA=",";
public static final int COMMA_SPACE=", ";

Zijn dat geldige ints?

[.oisyn]

In C wel als je enkele quotes gebruikt

[roeleboel]

.oisyn schreef op woensdag 04 maart 2015 @ 19:51:
In C wel als je enkele quotes gebruikt

Daar geloof ik niets van voor de derde uit die reeks

[Verwijderd]

Ik heb mezelf vandaag betrapt op dit stukje code. Ik schaam me.... diep......

code:

seoComp[field].isChanged = (seoComp[field].newVal != seoComp[field].oldVal ? true : false );

[ Voor 6% gewijzigd door Verwijderd op 04-03-2015 21:12 ]

[.oisyn]

roeleboel schreef op woensdag 04 maart 2015 @ 20:20:
[...]


Daar geloof ik niets van voor de derde uit die reeks

Dan geloof je dat toch niet . Het resultaat is overigens implementation-defined in die zin dat de volgorde van chars in de int niet vast staat. Onder gcc en msvc++ op x86 komt er 0x202c uit.

[ Voor 6% gewijzigd door .oisyn op 04-03-2015 21:25 ]

[Haijo]

Jerrythafast schreef op woensdag 04 maart 2015 @ 19:31:
[...]

Zijn dat geldige ints?

Haha, goed gezien. Koppiepastafaalactie. In onze applicatie zijn het natuurlijk strings.

[Stukfruit]

Verwijderd schreef op woensdag 04 maart 2015 @ 21:11:
Ik heb mezelf vandaag betrapt op dit stukje code. Ik schaam me.... diep......

code:

seoComp[field].isChanged = (seoComp[field].newVal != seoComp[field].oldVal ? true : false );

Als er drank in het spel was dan mag het nog hoor. Gewoon net iets te ver over de curve. Heeft iedereen wel eens

[ Voor 8% gewijzigd door Stukfruit op 04-03-2015 23:01 ]

[Verwijderd]

Geen drank in het spel. Stukje code vorige week geschreven, gisteren kwam ik er langs en zag direct 'hey, dat klopt niet...'

[Soultaker]

Nou ja, het klopt tenminste wel. Lelijke code die werkt is nog altijd beter dan code die niet werkt.

[svenhof]

Kwam dit tegen om een woord in full caps om te zetten in proper casing.

SQL:

 
SET upper(substring(UserFirst_Name, 1,1)) + REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(substring(lower(UserFirst_Name), 2,9000), ' a', ' A'), ' b', ' B'), ' c', ' C'), ' d', ' D'), ' e', ' E'), ' f', ' F'), ' g', ' G'), ' h', ' H'), ' i', ' I'), ' j', ' J'), ' k', ' K'), ' l', ' L'), ' m', ' M'), ' n', ' N'), ' o', ' O'), ' p', ' P'), ' q', ' Q'), ' r', ' R'), ' s', ' S'), ' t', ' T'), ' u', ' U'), ' v', ' V'), ' w', ' W'), ' x', ' X'), ' y', ' Y'), ' z', ' Z') end  

[ Voor 7% gewijzigd door svenhof op 05-03-2015 15:08 ]

[Firesphere]

svenhof schreef op donderdag 05 maart 2015 @ 15:08:
Kwam dit tegen om een woord in full caps om te zetten in proper casing.

SQL:

 
SET upper(substring(UserFirst_Name, 1,1)) + REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(substring(lower(UserFirst_Name), 2,9000), ' a', ' A'), ' b', ' B'), ' c', ' C'), ' d', ' D'), ' e', ' E'), ' f', ' F'), ' g', ' G'), ' h', ' H'), ' i', ' I'), ' j', ' J'), ' k', ' K'), ' l', ' L'), ' m', ' M'), ' n', ' N'), ' o', ' O'), ' p', ' P'), ' q', ' Q'), ' r', ' R'), ' s', ' S'), ' t', ' T'), ' u', ' U'), ' v', ' V'), ' w', ' W'), ' x', ' X'), ' y', ' Y'), ' z', ' Z') end  

[roeleboel]

svenhof schreef op donderdag 05 maart 2015 @ 15:08:
Kwam dit tegen om een woord in full caps om te zetten in proper casing.

SQL:

 
SET upper(substring(UserFirst_Name, 1,1)) + REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(substring(lower(UserFirst_Name), 2,9000), ' a', ' A'), ' b', ' B'), ' c', ' C'), ' d', ' D'), ' e', ' E'), ' f', ' F'), ' g', ' G'), ' h', ' H'), ' i', ' I'), ' j', ' J'), ' k', ' K'), ' l', ' L'), ' m', ' M'), ' n', ' N'), ' o', ' O'), ' p', ' P'), ' q', ' Q'), ' r', ' R'), ' s', ' S'), ' t', ' T'), ' u', ' U'), ' v', ' V'), ' w', ' W'), ' x', ' X'), ' y', ' Y'), ' z', ' Z') end  

Vooral briljant omdat ze de functie 'lower' wel kennen, maar 'upper' duidelijk niet...

.oisyn schreef op woensdag 04 maart 2015 @ 21:23:
[...]

Dan geloof je dat toch niet . Het resultaat is overigens implementation-defined in die zin dat de volgorde van chars in de int niet vast staat. Onder gcc en msvc++ op x86 komt er 0x202c uit.

Om het met de woorden van Loekie te zeggen: asjemenou... Ik moet mijn geloof blijkbaar bijstellen...

[ Voor 18% gewijzigd door roeleboel op 05-03-2015 15:36 ]

[TheDevilOnLine]

-- nvm

[ Voor 101% gewijzigd door TheDevilOnLine op 05-03-2015 15:47 ]

[afraca]

TheDevilOnLine schreef op donderdag 05 maart 2015 @ 15:46:
-- nvm

offtopic:
Ik moet m'n leven herzien, ik ging dit parsen als een (long) option

[Acid_Burn]

roeleboel schreef op donderdag 05 maart 2015 @ 15:34:
[...]

Vooral briljant omdat ze de functie 'lower' wel kennen, maar 'upper' duidelijk niet...

'upper' is het 2e woord van het commando

[roeleboel]

Acid_Burn schreef op donderdag 05 maart 2015 @ 16:31:
[...]
'upper' is het 2e woord van het commando

ik was te verblind door al de replace's om mij dat nog te realizeren - dat maakt het zelfs des te erger

[ Voor 7% gewijzigd door roeleboel op 05-03-2015 16:50 ]

[Damic]

Ik kwam het volgende juist tegen in een kalender functie, volgens mij ergens diep in de nacht geschreven maar het doet wat het moet doen

PHP:

$start_tel=1;
/** Make empty boxes before the first day **/
    if($start_tel==1)
    {
        $template->assign_block_vars('CAL_BOX',array('week'=>date ("W", mktime (0,0,0,$month,$start_tel,$year)),));
        for ($i=0; $i<=$start_day; $i++)
        {
            $start_tel++;
            $template->assign_block_vars('CAL_E',array());
        }
    }

*Ik zien jusit dat ik start_tel wat verder gebruik, maar waarom die if

[ Voor 7% gewijzigd door Damic op 05-03-2015 21:22 ]

[F.West98]

Java:

private String[] states = ["state1", "state2", "state3"];

public void drawWindow(int state) {
/**/
switch(states[state]) {
    case "state1": {}
    case "state2": {}
    case "state3": {}
}
}

[ajakkes]

Damic schreef op donderdag 05 maart 2015 @ 21:19:
Ik kwam het volgende juist tegen in een kalender functie, volgens mij ergens diep in de nacht geschreven maar het doet wat het moet doen

PHP:

$start_tel=1;
/** Make empty boxes before the first day **/
    if($start_tel==1)
    {
        $template->assign_block_vars('CAL_BOX',array('week'=>date ("W", mktime (0,0,0,$month,$start_tel,$year)),));
        for ($i=0; $i<=$start_day; $i++)
        {
            $start_tel++;
            $template->assign_block_vars('CAL_E',array());
        }
    }

*Ik zien jusit dat ik start_tel wat verder gebruik, maar waarom die if

If (true) {

} ter voorbereiding van een verbetering die je al aan ziet komen maar nog niet gevraagd is.

Het ziet er uit alsof je een loopje van plan was maar niet nodig bleek.

[Damic]

Ik denk eerder de late uurtjes en vergeten weg te doen.

[Ralph Smeets]

Een collega vroeg net aan de junior java developers wat er mis is met de volgende code die hij tegenkwam in een project:

Java:

    public ArrayList<Category> categories;

    public ArrayList<Category> getTopCategories(int numberOfItems) {
         ArrayList<Category> returnList = new ArrayList<Category>();
         if (categories != null) {                           
             int counter = 0;
             while (counter < numberOfItems) {
                 counter++;
                 returnList.add(categories.get(counter));
             }
         }
         return returnList;
     }

Het erge is dat deze code waarschijnlijk niet eens door een junior is geschreven

spoiler:

Behalve de nullpointerexception die je krijgt als meer categorieën vraagt als er zijn, een variable genaamd counter gebruiken als een index, noem hem dan ook gewoon index, had de schrijver op zijn minst gebruik moeten maken van een iterator om door de lijst te lopen. Maar wist hij of zij van het bestaan van een iterator af, dan wist hij of zij ook dat hetzelfde bereikt kon worden met de method sublist, bijvoorbeeld:
public ArrayList<Category> categories;

public ArrayList<Category> getTopCategories(int numberOfItems) {
return categories.subList(0,( numberOfItems > categories.size() ? categories.size() : numberOfItems) -1);
}

[F.West98]

Dan hou ik toch wel weer van C#/.NET:

C#:

private List<Category> categories;
public List<Category> getTopCategories(int number) {
    return categories.Take(number);
}

[Caelorum]

Vraag is of je in C# überhaupt nog wel een methode wil schrijven aangezien categories in het voorbeeld public is

[Ralph Smeets]

Die natuurlijk ook nog

[F.West98]

Caelorum schreef op maandag 09 maart 2015 @ 17:15:
Vraag is of je in C# überhaupt nog wel een methode wil schrijven aangezien categories in het voorbeeld public is

Daarom had ik 'm private gemaakt

[WernerL]

Ik ben eens door mijn backups aan het bladeren die op een externe schijf staan, kom ik hele oude code tegen.

PHP:

$emailadres = $_POST['emailadres'];

$sql = mysql_query("SELECT * FROM nieuwsbrief_adressen WHERE emailadres = '$emailadres'");
if(mysql_num_rows($sql) > 0){
    $layout = str_replace("{content}", "U bent al aangemeld voor de nieuwsbrief.", $layout);
}else {
    mysql_query("INSERT INTO nieuwsbrief_adressen (emailadres) VALUES ('$emailadres')") or die(mysql_error());
    $layout = str_replace("{content}", "Het emailadres <b>" . $emailadres . "</b> is aangemeld voor de nieuwsbrief.", $layout);
}

[Hatsieflatsie]

WernerL schreef op donderdag 19 maart 2015 @ 19:37:
Ik ben eens door mijn backups aan het bladeren die op een externe schijf staan, kom ik hele oude code tegen.

PHP:

$emailadres = $_POST['emailadres'];

$sql = mysql_query("SELECT * FROM nieuwsbrief_adressen WHERE emailadres = '$emailadres'");
if(mysql_num_rows($sql) > 0){
    $layout = str_replace("{content}", "U bent al aangemeld voor de nieuwsbrief.", $layout);
}else {
    mysql_query("INSERT INTO nieuwsbrief_adressen (emailadres) VALUES ('$emailadres')") or die(mysql_error());
    $layout = str_replace("{content}", "Het emailadres <b>" . $emailadres . "</b> is aangemeld voor de nieuwsbrief.", $layout);
}

Wat is er daar verkeerd aan? .

Overigens, dit vind ik toch wel overwegend bad pratice:

Clojure:

(defn =  ([x] true)
  ([x y] (lang.Util/equals x y))
  ([x y & more]
   (if (= x y)
     (if (next more)
       (recur y (first more) (next more))
       (= y (first more)))
     false)))

[WernerL]

Het ging vooral om de sql injectie mogelijkheid, maar daar had ik gelukkig rekening mee gehouden

PHP:

//SQL injection security
foreach($_POST as $name => $value) {
    if(!is_array($_POST[$name])){
        if(get_magic_quotes_gpc()==0 && ($name != "layout" && $name != "content")) $_POST[$name] = mysql_real_escape_string($_POST[$name]);
    }
}

foreach($_GET as $name => $value) {
    if(!is_array($_GET[$name])){
        if(get_magic_quotes_gpc()==0) $_GET[$name] = mysql_real_escape_string($_GET[$name]);
        $_GET[$name] = htmlspecialchars($_GET[$name]);
    }
}

In mijn connect.php file

[Saven]

WernerL schreef op donderdag 19 maart 2015 @ 19:37:
Ik ben eens door mijn backups aan het bladeren die op een externe schijf staan, kom ik hele oude code tegen.

PHP:

$emailadres = $_POST['emailadres'];

$sql = mysql_query("SELECT * FROM nieuwsbrief_adressen WHERE emailadres = '$emailadres'");
if(mysql_num_rows($sql) > 0){
    $layout = str_replace("{content}", "U bent al aangemeld voor de nieuwsbrief.", $layout);
}else {
    mysql_query("INSERT INTO nieuwsbrief_adressen (emailadres) VALUES ('$emailadres')") or die(mysql_error());
    $layout = str_replace("{content}", "Het emailadres <b>" . $emailadres . "</b> is aangemeld voor de nieuwsbrief.", $layout);
}

Ach, zo zijn we allemaal begonnen in PHP

Hatsieflatsie schreef op donderdag 19 maart 2015 @ 19:49:
[...]

Wat is er daar verkeerd aan? .

Zo'n beetje alles wel?
Het werkt wel, maar hoe

[P_Tingen]

Ik kwam laatst zoiets tegen in ons Progress 4GL systeem

code:

IF cLocationList = "" THEN 
DO:
  FOR EACH location:
   iOnStock = iOnStock + location.onHand.
  END.
END.

ELSE IF NUM-ENTRIES(cLocationList) = 1 THEN
DO:
  FOR EACH location WHERE location.id = cLocationList:
   iOnStock = iOnStock + location.onHand.
  END.
END.

ELSE IF NUM-ENTRIES(cLocationList) > 1 THEN
DO:
  DO iLocation = 1 TO NUM-ENTRIES(cLocationList):
    FOR EACH location WHERE location.id = ENTRY(iLocation,cLocationList):
       iOnStock = iOnStock + location.onHand.
    END.
  END.
END.

De regel met iOnStock = iOnStock + location.onHand heb ik hier neergezet maar was in dit programma een blok code van ruim 100 regels die in ieder blok exact gelijk was.

Dat was al faal 1, want DRY.
Faal 2 zit hem in de drievoudige afhandeling, want eea kan simpeler:

code:

IF cLocationList = "" THEN cLocationList = "*".

DO iLocation = 1 TO NUM-ENTRIES(cLocationList):
  FOR EACH location WHERE location.id MATCHES ENTRY(iLocation,cLocationList):
     iOnStock = iOnStock + location.onHand.
  END.
END.

Ik vermoed dat de vorige ontwikkelaar per regel code betaald werd ....

[RayNbow]

Hatsieflatsie schreef op donderdag 19 maart 2015 @ 19:49:
Overigens, dit vind ik toch wel overwegend bad pratice:

Clojure:

(defn =  ([x] true)
  ([x y] (lang.Util/equals x y))
  ([x y & more]
   (if (= x y)
     (if (next more)
       (recur y (first more) (next more))
       (= y (first more)))
     false)))

Ben niet heel bekend met Clojure, maar wat doet deze functie met een lege lijst?

[Firesphere]

JavaScript:

var description = $(this).data('Title');
if(typeOf(description) === 'undefined'){
    description = description;
}

[Sebazzz]

Mag typeof wel met hoofdletter O en haakjes?

[.oisyn]

Waarom zou het niet mogen met haakjes . Het verwacht een expressie, om een expressie kun je haakjes zetten.

[Douweegbertje]

De table met wat constanten genaamd 'KONSTANT'. En nee, deze table was niet in 1995 gemaakt maar recent en ja, door iemand relatief gezien op leeftijd. En meneer wilde het ook niet aanpassen want "het hoort zo".

[PrisonerOfPain]

Douweegbertje schreef op maandag 06 april 2015 @ 01:52:
De table met wat constanten genaamd 'KONSTANT'. En nee, deze table was niet in 1995 gemaakt maar recent en ja, door iemand relatief gezien op leeftijd. En meneer wilde het ook niet aanpassen want "het hoort zo".

Rename 'm naar "k"

[Marcj]

Haijo schreef op woensdag 04 maart 2015 @ 11:06:
.... Mijn senior heeft het idee dat gedaan is om Sonar waarschuwingen over magic numbers te onderdrukken. Iets met klok en klepel dus.

Sonar, daar ben ik ook een aantal leuke 'fixes' door tegen gekomen. Onder andere dat we in java een Runnable lieten wrappen om zo afhandeling van RuntimeExceptions te kunnen doen. Maar op de code

Java:

try {
    wrappedRunnable.run();
} catch(RuntimeException ex) {
    // doe iets met logging ofzo
}

kwam Sonar natuurlijk met een waarschuwing dat de run methode direct werd aangeroepen. Dus dat is door een niet nader te noemen ontwikkelaar verbeterd tot:

Java:

new Thread() {
    public void run() {
        try {
            wrappedRunnable.run();
        } catch(RuntimeException ex) {
            // doe iets met logging ofzo
        }
    }
}.start();

Effectief dus het hele nut van de ThreadPool die we gebruikten hiermee ongedaan gemaakt! Aargh

[Haijo]

Ach ja, Sonar is op zich wel nuttig maar geen enkele garantie tot kwaliteit. Dat hangt, als altijd, van de kwaliteit van je ontwikkelaars af.

En deze ontwikkelaars kwamen uit India. Daar hebben ze vast goede rondlopen maar die hebben hier nooit gewerkt.

[Damic]

Gisteren nog een leuk stukje tegen gekomen: Divide by Zero niet aangedacht dat het daar kon voorkomen, dan maar een if then rond gebouwd

[NESFreak]

Damic schreef op zaterdag 18 april 2015 @ 23:15:
Gisteren nog een leuk stukje tegen gekomen: Divide by Zero niet aangedacht dat het daar kon voorkomen, dan maar een if then rond gebouwd

Integer of float? Is divide by close tot zero wel wenselijk? Als je het 'daar' niet verwacht, is er dan niet iets heel erg mis?

[Damic]

Divide by zero = error en ik had het eigenlijk niet verwacht dus maar if then structuurtje langs gepoot

Van

code:

iAvgValue /= iAvgCount
Dim x As Single = BasicStart+CSng(iAvgValue / FillProcent)

naar:

code:

Dim x As Single = BasicStart
If iAvgCount > 0 Then
iAvgValue /= iAvgCount
x += CSng(iAvgValue / FillProcent)
End If

[ Voor 78% gewijzigd door Damic op 19-04-2015 11:44 ]

[.oisyn]

Of gewoon delen door Max(iAvgCount, 1)

[incaz]

.oisyn: als de waarde van iAvgValue <> 0 geeft jouw deling ook een resultaat <> 0 en dus een andere waarde voor x dan het if-block.

Damic, als je het leuk vindt kun je nog 1 regeltje laten verdwijnen:

code:

Dim x As Single = BasicStart
If iAvgCount > 0 Then
    x += CSng(iAvgValue / (iAvgCount * FillProcent))
End If

[ Voor 3% gewijzigd door incaz op 19-04-2015 13:14 ]