[alg] Slechtste programmeervoorbeelden deel 4

[PiepPiep]

We hadden hier ook zo'n logging foutje.
Een frontend communiceert met een backend d.m.v. soap.
Soms ging er iets mis en we willen dan weten wat, dus we loggen alle soapcalls.
Vervolgens kwamen we er achter dat het inloggen natuurlijk ook via soap gaat.
Nu staat er bij het inloggen een uitzondering voor het loggen

[Haan]

Alex) schreef op vrijdag 03 september 2010 @ 10:52:
[...]

Je moet niet de waardes van de variabelen laten outputten omdat je nooit weet wie die dingen leest. C# toont je het type en de naam van de parameter, maar niet de waarde.

En dat is vaak ook vreselijk irritant, omdat je dan niks aan de melding hebt. Leuk als ik te horen krijg dat een parameter een ongeldige waarde bevat, maar bied dan op z'n minst ook de mogelijkheid om te tonen wat die waarde dan was. Dan moet je gaan debuggen (vaak lastig of onmogelijk in productieomgeving), of hopen dat je goede logs hebt om info uit te vissen.

[PeterSelie]

Productieomgeving lijkt me dan ook totaal niet de aangewezen plek om een dergelijke situatie te gaan achterhalen.. Daarbij wil je dergelijke informatie vaak gewoon niet in plain-text opgeslagen hebben staan.

[ Voor 33% gewijzigd door PeterSelie op 03-09-2010 11:10 ]

[Stoffel]

Soms kan dat niet anders, hoe goed je ook probeert om de verschillende omgevingen hetzelfde te houden, je kunt altijd problemen hebben die zich alleen op productie voordoen. Dan is de keuze of niet oplossen of daar testen, en het eerste krijg je niet echt verkocht

[PeterSelie]

Stoffel schreef op vrijdag 03 september 2010 @ 11:11:
Soms kan dat niet anders, hoe goed je ook probeert om de verschillende omgevingen hetzelfde te houden, je kunt altijd problemen hebben die zich alleen op productie voordoen. Dan is de keuze of niet oplossen of daar testen, en het eerste krijg je niet echt verkocht

Dan nog is een probleem als verkeerde gebruikersnaam en wachtwoord prima te achterhalen, als je begint met het opnieuw invoeren van deze gegevens weet je in ieder geval dat het probleem daar niet gezocht hoeft te worden en hoef je de boel ook niet te loggen / weer te geven binnen een exception..

[418O2]

Laurens-R schreef op vrijdag 03 september 2010 @ 10:03:
... recursie is voor velen blijkbaar een moeilijk concept, maar ik denk niet dat je dan in een bedrijf software moet gaan ontwikkelen

http://www.google.nl/search?q=recursion

[Stoffel]

SoaDmaggot schreef op vrijdag 03 september 2010 @ 11:13:
[...]

Dan nog is een probleem als verkeerde gebruikersnaam en wachtwoord prima te achterhalen, als je begint met het opnieuw invoeren van deze gegevens weet je in ieder geval dat het probleem daar niet gezocht hoeft te worden en hoef je de boel ook niet te loggen / weer te geven binnen een exception..

Klopt, in deze situatie. Ik bedoelde meer dat je productie-issues in het algemeen soms noodgedwongen ook daar moet testen.

[PeterSelie]

Stoffel schreef op vrijdag 03 september 2010 @ 11:27:
[...]


Klopt, in deze situatie. Ik bedoelde meer dat je productie-issues in het algemeen soms noodgedwongen ook daar moet testen.

Klopt, je komt in (vooral grote projecten) zo nu en dan momenten tegen waarin je noodgedwongen op productie zal moeten testen, aangezien in veel gevallen deze omgevingen in bepaalde aspecten aanzienlijk verschillen van testomgevingen. Maar, in dit geval ging het om het weergeven of opslaan van login credentials

[Phoenix1337]

Tijdje terug ook wat leuks tegen gekomen:

PHP:

//dag
$today = date('d');
$yesterday = $today-1;
$daybeforeyesterday  = $today-2;

//maand precies hetzelfde

//jaar precies hetzelfde

Vraag me af hoe je daar ooit op komt.

[PiepPiep]

Is dat heel fout?
Als in de code erna in een lusje met today en yesterday vergeleken wordt zou het best snelheidswinst kunnen opleveren.

[CodeCaster]

PiepPiep schreef op vrijdag 03 september 2010 @ 12:38:
Is dat heel fout?
Als in de code erna in een lusje met today en yesterday vergeleken wordt zou het best snelheidswinst kunnen opleveren.

Ja? Wat als je een dag pakt die voor de derde van de maand ligt?

Klinkt een beetje als de papegaai van Herman Finkers:

Gaat ongeveer zo:
"En als ik twee augustus terugkom?"
- "Dan ben ik één augustus weg!"
"En als ik één augustus terugkom?"
- "... Dan ben ik half augustus weg!"

(Geen geluid hier, kan 't niet verifiëren )

[ Voor 35% gewijzigd door CodeCaster op 03-09-2010 12:45 ]

[PeterSelie]

PiepPiep schreef op vrijdag 03 september 2010 @ 12:38:
Is dat heel fout?
Als in de code erna in een lusje met today en yesterday vergeleken wordt zou het best snelheidswinst kunnen opleveren.

Ja dat is fout.. Stel dat date('d') 1 output? Zit je met eergister dus op -1 Augustus?

Spuit11..

[ Voor 3% gewijzigd door PeterSelie op 03-09-2010 12:41 ]

[Phoenix1337]

Het was schijnbaar nog nooit iemand opgevallen dat er op de 1ste en 2de van de maand, of de eerste 2 maanden van het jaar onjuiste data stond.

Met jaren is het eventueel nog wel te doen, maar netter is om gewoon iets met strtotime of mktime te doen.

[MueR]

Mensen die rekenen met de dag van de maand moeten ophouden met devven. Je dient daar gewoon netjes unix timestamps voor te gebruiken, dan kan je gewoon gebruik maken van standaard functies..

PHP:

$today = mktime(0,0,0);
// Deze zie je ook te veel:
$yesterday = $today - 86400;
// Waar je deze wil gebruiken:
$yesterday = strtotime('-1 day', $today);

[YopY]

Verwijderd schreef op vrijdag 03 september 2010 @ 10:23:
@Laurens-R: Hoe heb je het opgelost, de loop nog 10x gekopieerd? 13 niveau's zal wel genoeg zijn, zo niet gewoon nog iets vaker kopiëren.

imho bouw je er een codegenerator omheen die afhankelijk van het onderliggende OS en filesystem de max diepte van mappen vaststelt en net zolang loopt totdat er op het diepste niveau gezocht kan worden.

It's super effective! En je kunt 'code generation' op je CV zetten. Evt. zelfs 'recursive code generation based on OS and FS specific variables'.

Senior developer garantie

[Phyxion]

MueR schreef op vrijdag 03 september 2010 @ 12:43:
Mensen die rekenen met de dag van de maand moeten ophouden met devven. Je dient daar gewoon netjes unix timestamps voor te gebruiken, dan kan je gewoon gebruik maken van standaard functies..

PHP:

$today = mktime(0,0,0);
// Deze zie je ook te veel:
$yesterday = $today - 86400;
// Waar je deze wil gebruiken:
$yesterday = strtotime('-1 day', $today);

Dat laatste kan nog wel korter/anders, namelijk:

PHP:

$yesterday = strtotime('-1 day');
// Of zelfs:
$yesterday = strtotime('yesterday');
// Nog leukere varianten zijn:
$lastmonday = strtotime('last monday');

[PeterSelie]

Daarnaast zou je ook gewoon direct de goede timestamp kunnen creëren adhv mktime en zou je ook daar je berekening in door kunnen voeren. Zo blijf je een stuk flexibeler en wordt je gelijk geforceerd met die timestamp te blijven werken (tenzij je, wat je helaas nog redelijk vaak ziet, weer een strftime hieroverheen gaat gooien om vervolgens met die string verder te gaan werken / berekening..)

PHP:

mktime(0, 0, 0, date("n"), date("j") - 1, date("Y"))

[ Voor 50% gewijzigd door PeterSelie op 03-09-2010 14:03 ]

[BarôZZa]

Dat is inderdaad mijn favoriete methode. Ben ik trouwens de enige die de volgorde 'uur, minuut, seconde, maand, dag, jaar' zo irritant vind?

[jbdeiman]

Phyxion schreef op vrijdag 03 september 2010 @ 12:49:
[...]

Dat laatste kan nog wel korter/anders, namelijk:

PHP:

$yesterday = strtotime('-1 day');
// Of zelfs:
$yesterday = strtotime('yesterday');
// Nog leukere varianten zijn:
$lastmonday = strtotime('last monday');

Op zich is dat waar, maar waar bij jou altijd maar 1 datum (de huidige) hebt, kan je in de manier zoals die was opgezet natuurlijk ook andere data erin knallen.
Ik denk ook dat het voorbeeld op die manier was bedoeld, maar timestamp van vandaag pakken is dan wel het eenvoudigst om een voorbeeld weer te geven.

[Phyxion]

jbdeiman schreef op vrijdag 03 september 2010 @ 14:25:
[...]

Op zich is dat waar, maar waar bij jou altijd maar 1 datum (de huidige) hebt, kan je in de manier zoals die was opgezet natuurlijk ook andere data erin knallen.
Ik denk ook dat het voorbeeld op die manier was bedoeld, maar timestamp van vandaag pakken is dan wel het eenvoudigst om een voorbeeld weer te geven.

Ik zie het, je hebt inderdaad gelijk, dacht dat het alleen de bedoeling was van de huidige dag.
Hoe dan ook zou ik zoiets wel in een functie zetten want voor alles dat los aanroepen is ook wat overdreven

[PiepPiep]

Ah, ik had niet gezien dat de dag van de maand gepakt werd.
Als je de complete datum hebt en daar 1 dag af haalt en dan de dag van de maand bepaald gaat het wel goed natuurlijk.

[dragontje124]

jbdeiman schreef op vrijdag 03 september 2010 @ 14:25:
[...]

Op zich is dat waar, maar waar bij jou altijd maar 1 datum (de huidige) hebt, kan je in de manier zoals die was opgezet natuurlijk ook andere data erin knallen.
Ik denk ook dat het voorbeeld op die manier was bedoeld, maar timestamp van vandaag pakken is dan wel het eenvoudigst om een voorbeeld weer te geven.

dat is simpel op te lossen door de klok instellingen van je server eventjes te veranderen

PHP:

<?php
function changeTime($day,$month,$year,$hour,$minutes,$seconds) {
`date -s "$month.'/'.$day.'/'.$year.' '.$hour.':'.$minutes.':'.$seconds"`; // werkt alleen op linux, edit: niet dus 
}
?>

[ Voor 18% gewijzigd door dragontje124 op 05-09-2010 18:00 ]

[Alex)]

Dat script is al een WTF an sich

[dragontje124]

Alex) schreef op zondag 05 september 2010 @ 17:27:
Dat script is al een WTF an sich

past mooi in dit topic toch
(is gewoon een shell command met behulp van backticks, zie: http://nl.php.net/manual/en/language.operators.execution.php)

[ Voor 25% gewijzigd door dragontje124 op 05-09-2010 17:51 ]

[Verwijderd]

dragontje124 schreef op zondag 05 september 2010 @ 17:50:
[...]

past mooi in dit topic toch
(is gewoon een shell command met behulp van backticks, zie: http://nl.php.net/manual/en/language.operators.execution.php)

Past heel goed ja, zowel bedoeld als onbedoeld... Alleen je comment klopt niet

[dragontje124]

Verwijderd schreef op zondag 05 september 2010 @ 17:55:
[...]

Past heel goed ja, zowel bedoeld als onbedoeld... Alleen je comment klopt niet

ooh dat kan best kloppen ik haal dit stukje code net ergens anders vandaan, heb m ook niet getest.

[Verwijderd]

dragontje124 schreef op zondag 05 september 2010 @ 17:58:
[...]

ooh dat kan best kloppen ik haal dit stukje code net ergens anders vandaan, heb m ook niet getest.

Hmm, kleine tip dan: haal daar niks meer vandaan.

[dragontje124]

Verwijderd schreef op zondag 05 september 2010 @ 18:01:
[...]

Hmm, kleine tip dan: haal daar niks meer vandaan.

ach ja... google->willekeurige site is natuurlijk niet heel erg betrouwbaar nee, maar daar ging het me ook niet echt om

[ Voor 7% gewijzigd door dragontje124 op 05-09-2010 18:11 ]

[Alex)]

Toch leuk als ik $seconds een waarde kan geven als:

code:

"; rm -rf /*; "

[dragontje124]

Alex) schreef op zondag 05 september 2010 @ 18:29:
Toch leuk als ik $seconds een waarde kan geven als:

code:

"; rm -rf /*; "

PHP:

changeTime($_GET['day'],$_GET['month'],$_GET['year'],$_GET['hour'],$_GET['minutes'],$_GET['seconds']);

shell injection ??

[rrrandy]

Alex) schreef op zondag 05 september 2010 @ 18:29:
Toch leuk als ik $seconds een waarde kan geven als:

code:

"; rm -rf /*; "

Als je PHP scripts als root draait, dan verdien je het ook

[Dido]

PiepPiep schreef op vrijdag 03 september 2010 @ 10:56:
Nu staat er bij het inloggen een uitzondering voor het loggen

Hebben we hier een paar jaar terug aan de poot gehad: er was een general error log in het leven geroepen.

Een simpele server-funtie die aangeroepen kon worden vanuit de hele applicatie om een records in het error-log bestand weg te schrijven.

Alleen jammer dat op dat logbestand een unique key zat die niet altijd goed gevuld werd, en dat het error-logging programma zelf ook zijn errors logde door zichzelf aan te roepen...

[Verwijderd]

lol, zoiets heb ik ook eens gezien in een systeem.

De errors werden gelogd in een centrale errorlog (dat werkte wel heel goed) en eens in de zoveel tijd werd door middel van een cron de errorlog doorgestuurd naar de beheerders van de applicatie. Helaas gaf deze cron ook weer errors en kwamen die weer in dezelfde log terecht.

Via een soort cascade effect gaven meerdere errors dan weer minder errors in de cron en zodoende steeds minder errors totdat het weer status quo was. Erg komisch om te zien.

[Styxxy]

Enige tijd terug kreeg ik een iets oudere applicatie (C#) onder mijn hoede. Nuja, eigenlijk was het een programma dat twee jaar geleden werd gebruikt, maar daarna niet meer (men ging terug naar een papieren versie ). De reden dat het niet gebruikt werd, was omdat het programma toch wel vaak crashte. Ik dan maar eens gestart in Visual Studio met debugger. Al snel vond ik een van de oorzaken van de crashes:

C#:

        public void dgLopers_KeyUp(object sender, KeyEventArgs e)
        {
            e.SuppressKeyPress = true;
            e.Handled = true;
            switch (e.KeyCode)
            {
                //case Keys.F2: ChangeSelectedLoper(); break;
                case Keys.F5: LoadList(); break;
                case Keys.F3: AddNewLoper(); break;
                case Keys.Tab: AddSelectedToWachtlijst(); break;
                case Keys.Back: tbFilter.Text = tbFilter.Text.Remove(tbFilter.Text.Length - 1, 1);  break;
                default:
                    break;
            }
        }

(zie regel 11)

En die Keys.Back werd wel vaken gebruikt bij tekstboxes. Ik heb die dan toch mooi allemaal verwijderd en plots crashte het programma al 95% minder vaak.

Ik snap toch niet waarom iemand een backspace opnieuw zou willen definiëren .

[roy-t]

Wat zonde zeg dat je een heel programma schrijft, wat gewoon zijn werk doet enzo en dat je dan niet de moeite er in steekt om even een paar vage bugs eruit te halen. Vooral deze, die met een uurtje werken in debug mode zo gevonden had moeten worden.

"Invalid argument exception, index of -1" blaat blaat, je kijkt nog een keer naar die regel en je denkt WTF waarom deed ik dat ookalweer, fix klaar .

[mulder]

Is het niet een hele vage workaround omdat hij e.SuppressKeyPress = true & e.Handled = true doet?

[Verwijderd]

roy-t schreef op maandag 06 september 2010 @ 15:53:
Wat zonde zeg dat je een heel programma schrijft, wat gewoon zijn werk doet enzo en dat je dan niet de moeite er in steekt om even een paar vage bugs eruit te halen.

Daar heb je niet altijd de controle over natuurlijk

[CoolGamer]

Ik was code aan het schrijven om een webcam en microfoon te gebruiken binnen Flash. Toen kwam ik dit tegen.

Hoe krijgen ze dit verzonnen:

Flash ActionScript:

public static function getCamera(name:String = null):Camera

quote: http://livedocs.adobe.com...mera.html#getCamera%28%29

name:String (default = null) — To specify a value for this parameter, use the string representation of the zero-based index position within the Camera.names array. For example, to specify the third camera in the array, use Camera.getCamera("2").

Bij Microphone doen ze het wel logisch:

Flash ActionScript:

public static function getMicrophone(index:int = -1):Microphone

quote: http://livedocs.adobe.com....html#getMicrophone%28%29

index:int (default = -1) — The index value of the microphone.

[roy-t]

Hmm..

C#:

//lines is een string[]
lines[i] = (lines[i].ToCharArray()[nextIndex] = ' ').ToString();

Geweldige snippet die ik net geschreven heb, werkt helaas helemaal niet. Hoewel ik niet precies snap hoe. Het idee was om van lines[i] een char array te maken, hierin 1 karakter te vervangen en deze daarna weer naar een string te casten. Leuk idee .

[CoolGamer]

roy-t schreef op vrijdag 10 september 2010 @ 12:04:
Hmm..

C#:

//lines is een string[]
lines[i] = (lines[i].ToCharArray()[nextIndex] = ' ').ToString();

Geweldige snippet die ik net geschreven heb, werkt helaas helemaal niet. Hoewel ik niet precies snap hoe. Het idee was om van lines[i] een char array te maken, hierin 1 karakter te vervangen en deze daarna weer naar een string te casten. Leuk idee .

ToCharArray kopieert de characters. Die array aanpassen zorgt er dus niet voor dat je string wordt aangepast.

[Not Pingu]

Nou geweldig hoor. Een ToString() op een char array heeft natuurlijk alleen maar "System.char[]" als output. En hoe kun je nou in 1 statement een array index assignen en die array zelf teruggeven?

C#:

lines[i] = lines[i].Substring(0, nextIndex) + " " + lines[i].Substring(nextIndex + 1, lines[i].Length - (nextIndex + 1));

[CoolGamer]

Maar je roept ToString ook aan op een char ' ' (niet de chararray). Zo wordt elke string die je aanpast met deze functie " ". Lijkt me inderdaad niet wat je wilt.

[ Voor 41% gewijzigd door CoolGamer op 10-09-2010 12:33 ]

[roy-t]

Ja het was gewoon even iets dat ik snel schreef, vandaar dat het ook hier staat.

[bobo1on1]

Dido schreef op maandag 06 september 2010 @ 14:05:
[...]

Hebben we hier een paar jaar terug aan de poot gehad: er was een general error log in het leven geroepen.

Een simpele server-funtie die aangeroepen kon worden vanuit de hele applicatie om een records in het error-log bestand weg te schrijven.

Alleen jammer dat op dat logbestand een unique key zat die niet altijd goed gevuld werd, en dat het error-logging programma zelf ook zijn errors logde door zichzelf aan te roepen...

Dat is net zoiets als naar het logbestand een error schrijven dat het logbestand niet geopend kan worden (rechten staan fout oid), leuke recursie krijg je dan.

[Verwijderd]

Eentje van mezelf, heeft me 2 uur geduurt om nou precies te vinden waar het misging:

C++: alProjectBuilder

 
for( int i = 0; i < sliceSize; i++ )
{
    alSVOSlice slice = *m_sliceList[i];
    memmap.writeFile( &slice, sizeof( alSVOSlice ) );
}
 

Om vervolgens als alles klaar was dit stuk code aan te roepen:

C++: alProjectBuilder

 
                                                           
//here the destructor call of alSvoSlice :
alSVOSlice::~alSVOSlice()
{
    clear();
}

//which just calls this:
void alSVOSlice::clear()
{
    if( m_nodeList )
        delete[] m_nodeList;
    m_nodeList = NULL;
    m_shouldLoadFromDisk = true;
}
 

[ValHallASW]

Gokje: geen copy constructor gedefiniëerd? Of bedoel je dat je handmatig de destructor aanriep?

[Verwijderd]

ValHallASW schreef op zaterdag 11 september 2010 @ 13:38:
Gokje: geen copy constructor gedefiniëerd? Of bedoel je dat je handmatig de destructor aanriep?

idd copy constructor niet gedefiniëerd, juist vanwege de reden dat ik daadwerkelijk wilde werken met pointers, waarom ik dan toch een dereference gebruikt heb mag joost weten

[YopY]

De 'wtf' in dezen ontgaat me volledig, C++ snap ik maar nauwelijks . Ik heb er een project mee gewerkt en, zolang je het eenvoudig houdt (zoals ik deed) was het nog te doen.

[bobo1on1]

Die "if( m_nodeList )" is trouwens niet nodig bij C++

[Vinnienerd]

YopY schreef op zaterdag 11 september 2010 @ 19:48:
De 'wtf' in dezen ontgaat me volledig, C++ snap ik maar nauwelijks . Ik heb er een project mee gewerkt en, zolang je het eenvoudig houdt (zoals ik deed) was het nog te doen.

C++ is gewoon C met objects en templates, dan ben je d'r wel

[ValHallASW]

Vinnienerd schreef op zaterdag 11 september 2010 @ 22:22:
C++ is gewoon C met objects en templates, dan ben je d'r wel

Nou, nee. Dan moet je nog even de plaatsen van de mijnen in het mijnenveld weten, dán ben je er.

@YopY: de situatie is ook niet helemaal triviaal om uit te leggen. Wat er ongeveer gebeurt is dit:

1. Maak een nieuwe instance van alSVOSlice aan, 'sliceA'.
2. Roep een functie op sliceA aan die een stuk geheugen reserveert op plaats A, en zet daar de waarden [1,2,3] in. Dat stukje verwijzen we naar met de pointer P.
3. alSVOSlice sliceB = sliceA

sliceB heeft nu ook een pointer P. Je zou nu wellicht verwachten dat de waarden [1,2,3] naar een nieuw stuk geheugen zijn gekopiëerd, maar dat is niet het geval - sliceB.P verwijst nog steeds naar A!

Dit gaat goed, totdat sliceB uit scope gaat. Vervolgens wordt de destructor aangeroepen. In die destructor wordt het geheugen waar sliceB.P naar verwijst weer vrijgegeven - maar sliceA.P verwijst hier nog naar! Als je vervolgens dat geheugen via sliceA.P probeert te benaderen dan krijg je een segfault.

Sorry, op het moment kan ik geen manier bedenken om dit duidelijker uit te leggen. Ik hoop dat dit duidelijk genoeg is; je moet wel snappen hoe pointers werken om deze problematiek ueberhaupt te snappen.

[Verwijderd]

bobo1on1 schreef op zaterdag 11 september 2010 @ 22:05:
Die "if( m_nodeList )" is trouwens niet nodig bij C++

ik hergebruik het object later weer vandaar de specifieke 'clear' functie, dus op 'NULL' testen is wenselijk;)

[Jegorex]

Java:

        public void tick(long ticks) {
            while (ticks > 0) {
                x += Math.sin(angle) * speed;
                y += Math.cos(angle) * speed;
                ticks--;
            }
        }

[Devil]

Nuttige functie, 1 argument en 4 globals ofzo?

[CoolGamer]

Devil schreef op zondag 12 september 2010 @ 18:46:
Nuttige functie, 1 argument en 4 globals ofzo?

Als je meerdere malen hetzelfde optelt in een lus, zou je die lus ook kunnen vervangen door een vermenigvuldiging.

Java:

public void tick(long ticks) {
       if (ticks > 0) {
                x += Math.sin(angle) * speed * ticks;
                y += Math.cos(angle) * speed * ticks;                
       }
}

[Patriot]

TheCoolGamer schreef op zondag 12 september 2010 @ 18:50:
[...]

Als je meerdere malen hetzelfde optelt in een lus, zou je die lus ook kunnen vervangen door een vermenigvuldiging.

Java:

public void tick(long ticks) {
       if (ticks > 0) {
                x += Math.sin(angle) * speed * ticks;
                y += Math.cos(angle) * speed * ticks;                
       }
}

Euh, nee, je moet vermenigvuldigen met speed tot de macht ticks, anders is je uitkomst heel anders.

[CoolGamer]

Patriot schreef op zondag 12 september 2010 @ 18:54:
[...]
Euh, nee, je moet vermenigvuldigen met speed tot de macht ticks, anders is je uitkomst heel anders.

Dat was het geval zodra er *= had gestaan (i.p.v. +=).

[bobo1on1]

Verwijderd schreef op zondag 12 september 2010 @ 04:25:
[...]

ik hergebruik het object later weer vandaar de specifieke 'clear' functie, dus op 'NULL' testen is wenselijk;)

Aan delete[] kun je gewoon een NULL pointer doorgeven zonder problemen, dus de check hier is overbodig.

[Patriot]

TheCoolGamer schreef op zondag 12 september 2010 @ 19:00:
[...]

Dat was het geval zodra er *= had gestaan (i.p.v. +=).

Je hebt gelijk, mijn fout

[ValHallASW]

Mag ik voor de implementatie van OOP in IDL WTF-punten uitdelen?

IDL:

FUNCTION image::init, path
    self.path = path
    self.image = ptr_new(READ_IMAGE(path))
    return, 1
END

PRO image::cleanup
    ptr_free, self.image
END

FUNCTION image::getImage
    return, *self.image
END

PRO image__define
    namedStructure = {image, path:'', image:ptr_new()}
END

Nota bene: image::init moet een function zijn en 1 retourneren, anders doet 'ie het niet. Je destructor is /wel/ een procedure. Je definieert je klasse in de pro image__define, door een variabele genaamd namedStructure een struct toe te wijzen.


Ohja, en hoe gebruik je dat dan?

IDL:

x=obj_new('image', 'c:\whatever\..')

en, om x weer vrij te geven:

IDL:

obj_delete, x

Oh, yeah

[ Voor 18% gewijzigd door ValHallASW op 13-09-2010 10:23 ]

[Hydra]

Devil schreef op zondag 12 september 2010 @ 18:46:
Nuttige functie, 1 argument en 4 globals ofzo?

Ziet er meer uit als gewoon een busy wait. Als je in zo'n busy wait loop simpelweg een counter ophoogt wordt 'ie uitgeoptimaliseerd door de compiler.

In een van m'n eerste Qbasic programmatjes, 16 jaar terug ofzo, had gebruikte ik ook simpele loops om het programma een seconde te laten wachten. M'n 8086 deed ongeveer een seconde over 10000 iteraties. Als je dan upgrade naar een 80486 DX4 100Mhz (w00t!) werken je progsels niet meer

Hoop geleerd in die tijd, vooral hoe het niet moest

[Freeaqingme]

Daarom had je ook een turbo toets op je pc om ervoor te zorgen dat dat niet gebeurde.

Net zoals je anders in een halve seconde gameover was in pacman #goodOldTimes

[Hydra]

Freeaqingme schreef op dinsdag 14 september 2010 @ 13:07:
Daarom had je ook een turbo toets op je pc om ervoor te zorgen dat dat niet gebeurde.

Net zoals je anders in een halve seconde gameover was in pacman #goodOldTimes

Ik heb toen in dat spelletje bij startup gewoon ingebouwd dat 'ie 10k cycles testte om erachter te komen hoeveel tijd 'ie daarover deed

M'n 486 had geen turbo knop meer, m'n XT wel inderdaad. Die 486 draaide sowieso al op turbo. #goodoldtimesindeed

[.oisyn]

slowdos ftw

[AtleX]

Een pareltje in eigen code:

C#:

public sealed class Iets
    {
        public object Data
        {
            get;
            private set;
        }
    }

Knappe jongen die nog iets in Data krijgt. De SVN history laat trouwens wel zien dat dit ooit een veel uitgebreidere class was die bij het refactoren opgesplitst is. Dit is een dood overblijfsel en had al lang verwijderd moeten worden.

[Apache]

AtleX schreef op dinsdag 14 september 2010 @ 14:12:
Een pareltje in eigen code:

C#:

public sealed class Iets
    {
        public object Data
        {
            get;
            private set;
        }
    }

Knappe jongen die nog iets in Data krijgt. De SVN history laat trouwens wel zien dat dit ooit een veel uitgebreidere class was die bij het refactoren opgesplitst is. Dit is een dood overblijfsel en had al lang verwijderd moeten worden.

Ik weet niet hoe het in dotnet zit, maar volgens mij zijn er gelijkaardige reflection mogelijkheden als in java, en dan zou ik daar best nog wel een value ingefrot krijgen hoor

[RobIII]

AtleX schreef op dinsdag 14 september 2010 @ 14:12:
Een pareltje in eigen code:

Is dat de hele class of mist er nog een constructor oid?

[Verwijderd]

MySQL Tabel "Personen"

Kolom "id" varchar(16)



En even ter verheldering, de helft van de id's zijn numeriek, de andere helft zijn voornamen.



Edit: nog ééntje. Datumvelden als varchar.....

En deze applicatie heb ik geërft

[ Voor 71% gewijzigd door Verwijderd op 14-09-2010 16:35 ]

[Hydra]

Verwijderd schreef op dinsdag 14 september 2010 @ 16:29:
En deze applicatie heb ik geërft

Te laat voor loonheronderhandelingen?

Edit: je moet het maar zo zien: met wat geluk zitten er een paar enorm domme fouten in die de performance om zeep helpen maar makkelijk te fixen zijn. Lekker makkelijk scoren by de 'higher-ups'

[ Voor 33% gewijzigd door Hydra op 14-09-2010 16:54 ]

[Sebazzz]

Apache schreef op dinsdag 14 september 2010 @ 16:12:
[...]


Ik weet niet hoe het in dotnet zit, maar volgens mij zijn er gelijkaardige reflection mogelijkheden als in java, en dan zou ik daar best nog wel een value ingefrot krijgen hoor

Reflection FTW

[Alex)]

Ah, sweet sweet Reflection. Ook al eens gebruikt om aan een helper-method van SharePoint te komen die als internal was gemarkeerd. Het enige wat het ding deed was een URL teruggeven, maar deed precies wat ik moest hebben.

Het was of 5 regels reflection gebruiken, of honderden regels copy/pasten want die ene method sprak weer andere internal methods en properties aan.

[Avalaxy]

Verwijderd schreef op dinsdag 14 september 2010 @ 16:29:
MySQL Tabel "Personen"

Kolom "id" varchar(16)



En even ter verheldering, de helft van de id's zijn numeriek, de andere helft zijn voornamen.



Edit: nog ééntje. Datumvelden als varchar.....

En deze applicatie heb ik geërft

Voornamen als primary key?

Damn Mag wel iemand terug naar school

[RobIII]

Avalaxy schreef op dinsdag 14 september 2010 @ 23:41:
Voornamen als primary key?

Damn Mag wel iemand terug naar school

Ja! Gebruik achternamen als PK! Of postcodes!

/sarcasm

[AtleX]

RobIII schreef op dinsdag 14 september 2010 @ 16:18:
[...]

Is dat de hele class of mist er nog een constructor oid?

Dit was de hele class.

[YopY]

Hydra schreef op dinsdag 14 september 2010 @ 16:53:
[...]

Te laat voor loonheronderhandelingen?

Och, ik zou in zo'n geval aandragen dat het onwerkbaar is om nieuwe features te ontwikkelen met een technische schuld zo hoog als die (oid, evt. met cijfers en grafieken aankomen). Dat je project in het eerste halfjaar het refactoren (of, 'normaal maken') van die applicatie is.

[Otherside1982]

RobIII schreef op dinsdag 14 september 2010 @ 23:49:
[...]

Ja! Gebruik achternamen als PK! Of postcodes!

/sarcasm

Gewoon een samenstelling gebruiken als PK: varchar(4096) met voornaam-naam-straat-nr-postcode-woonplaats-geboortedatum-telefoon-gsm-fax-email-gewicht-lengte
Dat is ook gewoon veel makkelijker om gegevens terug te vinden in de database, ook handig om te gebruiken in FKs, dan moet je niet steeds weer tabellen gaan joinen om die info eruit te halen.

[Janoz]

Alex) schreef op dinsdag 14 september 2010 @ 19:07:
Ah, sweet sweet Reflection. Ook al eens gebruikt om aan een helper-method van SharePoint te komen die als internal was gemarkeerd. Het enige wat het ding deed was een URL teruggeven, maar deed precies wat ik moest hebben.

Het was of 5 regels reflection gebruiken, of honderden regels copy/pasten want die ene method sprak weer andere internal methods en properties aan.

Zorg iig wel dat het duidelijk gedocumenteerd is en makkelijk terug gevonden kan worden. Dat zijn namelijk precies de plekken waar even goed naar gekeken moet worden bij een upgrade van het (sharepoint) platform. De voornaamste reden om dit niet in de publieke api op te nemen is (/lijkt mij) omdat ze het dan rustig kunnen aanpassen bij een volgende versie.

Persoonlijk gebruik ik wel reflectie, maar eigenlijk alleen in mijn unittests zodat ik wat mocks kan injecteren op plekken waar dat eigenlijk niet zou kunnen of het goed kunnen testen van methodes die ik eigenlijk toch echt private zou willen houden. (oh, en enkele libs zoals beanutils)

[YopY]

Otherside1982 schreef op woensdag 15 september 2010 @ 10:03:
[...]

Gewoon een samenstelling gebruiken als PK: varchar(4096) met voornaam-naam-straat-nr-postcode-woonplaats-geboortedatum-telefoon-gsm-fax-email-gewicht-lengte
Dat is ook gewoon veel makkelijker om gegevens terug te vinden in de database, ook handig om te gebruiken in FKs, dan moet je niet steeds weer tabellen gaan joinen om die info eruit te halen.

Eigenlijk zou je zoiets kunnen doen met persoonsgegevens. De waarden bepalen die voor elke persoon uniek zijn en aan de hand daarvan een unieke hash berekenen, die als ID gebruiken.

Want in veel gevallen is een ID gewoon een goedkope en efficiënte hack om van goed genormaliseerde PK's af te komen .

[Afvalzak]

Avalaxy schreef op dinsdag 14 september 2010 @ 23:41:
[...]


Voornamen als primary key?

Damn Mag wel iemand terug naar school

Woh, dat is wel heul ernstig zo erg ben ik ze (nog) niet tegen gekomen. Is zo'n beetje les 1: Databases...

[CodeCaster]

YopY schreef op woensdag 15 september 2010 @ 12:53:
[...]


Eigenlijk zou je zoiets kunnen doen met persoonsgegevens. De waarden bepalen die voor elke persoon uniek zijn en aan de hand daarvan een unieke hash berekenen, die als ID gebruiken.

Want in veel gevallen is een ID gewoon een goedkope en efficiënte hack om van goed genormaliseerde PK's af te komen .

Dat niet alleen, persoonsgegevens muteren nu eenmaal. Adressen, lengte, zelfs geslacht kan veranderen. Een ID is dus een prima oplossing.

[ Voor 3% gewijzigd door CodeCaster op 15-09-2010 13:12 ]

[_Apache_]

Een applicatie waarbij je moet opgeven welke kolommen in er in een csv export (db->csv) moeten terugkomen.

Wanneer er dus dus een van de kolommen een text veld is, waarin er geëntert is, is de uiteindelijke csv borked.

Nu kan je dus de kolommen opgeven als configuratie in een xml

<export_columns>replace(replace(replace(cast(columnname as varchar(8000)),CHAR(10),''),CHAR(13),''),CHAR(9),'')</export_columns>

Dit word vervolgens 1:1 overgenomen naar de query, die word uitgevoerd et voila, een goede csv.

[basvd]

Deze rommel

PHP:

$url = $_SERVER['PHP_SELF']."?".ereg_replace("&start=[0-9]*$", "", $_SERVER['QUERY_STRING'])."&start=".$vorige;

deels vervangen met

PHP:

$query= http_build_query( 
            array_merge(
                $_GET,
                array('start' => $vorige)
            )
        );

[Salandur]

_Apache_ schreef op woensdag 15 september 2010 @ 13:18:
Een applicatie waarbij je moet opgeven welke kolommen in er in een csv export (db->csv) moeten terugkomen.

Wanneer er dus dus een van de kolommen een text veld is, waarin er geëntert is, is de uiteindelijke csv borked.

Nu kan je dus de kolommen opgeven als configuratie in een xml

<export_columns>replace(replace(replace(cast(columnname as varchar(8000)),CHAR(10),''),CHAR(13),''),CHAR(9),'')</export_columns>

Dit word vervolgens 1:1 overgenomen naar de query, die word uitgevoerd et voila, een goede csv.

Meer een probleem met ongeldige escaping van content of het verwerken daarvan, dan borked csv lijkt me.

[YopY]

CodeCaster schreef op woensdag 15 september 2010 @ 13:12:
[...]

Dat niet alleen, persoonsgegevens muteren nu eenmaal. Adressen, lengte, zelfs geslacht kan veranderen. Een ID is dus een prima oplossing.

...da's waar. Voor- en achternamen kunnen ook veranderen.

Burgerservicenummer dan maar? Oh wacht, niet van toepassing voor kinderen < 16.

Kan DNA ook al muteren tegenwoordig? Wat is de korste en best indexeerbare manier om een DNA profiel op te slaan?

Maar je hebt gelijk natuurlijk, een gewoon ID is beter (en eenvoudiger) in heel veel gevallen.

En CSV is b0rked voor eigenlijk alles behalve... nouja, dingen zonder komma's / scheiders en zonder regeleindes enzo.

[mcDavid]

YopY schreef op woensdag 15 september 2010 @ 15:23:
[...]


...da's waar. Voor- en achternamen kunnen ook veranderen.

Burgerservicenummer dan maar? Oh wacht, niet van toepassing voor kinderen < 16.

Kan DNA ook al muteren tegenwoordig? Wat is de korste en best indexeerbare manier om een DNA profiel op te slaan?

Maar je hebt gelijk natuurlijk, een gewoon ID is beter (en eenvoudiger) in heel veel gevallen.

En CSV is b0rked voor eigenlijk alles behalve... nouja, dingen zonder komma's / scheiders en zonder regeleindes enzo.

BSN is ook niet veel meer dan een ID voor de databank van de overheid.

[NetForce1]

YopY schreef op woensdag 15 september 2010 @ 15:23:
[...]
Burgerservicenummer dan maar? Oh wacht, niet van toepassing voor kinderen < 16.

BSN krijg je tegenwoordig al wanneer je ingeschreven wordt bij de burgerlijke stand. Wat niet wegneemt dat er evengoed nog wel mensen zonder BSN kunnen zijn (buitenlanders bijv.).

[ Voor 14% gewijzigd door NetForce1 op 15-09-2010 19:26 ]

[Haan]

Hoe kom je er ook bij dat je dat pas vanaf 16 zou krijgen? Lang voor die tijd mag je toch ook al werken, gaat lastig zonder zo'n nummer

[Gomez12]

NetForce1 schreef op woensdag 15 september 2010 @ 19:25:
[...]

BSN krijg je tegenwoordig al wanneer je ingeschreven wordt bij de burgerlijke stand. Wat niet wegneemt dat er evengoed nog wel mensen zonder BSN kunnen zijn (buitenlanders bijv.).

Kan je met emigreren / remigreren dan ook een ander BSN krijgen? Of is het in theorie eigenlijk goed bruikbaar als PK.
In de praktijk zal het waardeloos als PK zijn ( ik zie het al voor me : een vertegenwoordiger vraagt aan binnendienst medewerker om een contact persoon in het systeem aan te maken : Pas als jij zijn BSN hebt kan ik hem aanmaken )
Nog even daargelaten dat ik een PK eigenlijk altijd gegeneerd wil hebben omdat handmatig ingevoerde data altijd wel eens veranderd moet worden ( tikfouten etc ) en ik een PK niet wil veranderen

[Dido]

Gomez12 schreef op woensdag 15 september 2010 @ 20:21:
Nog even daargelaten dat ik een PK eigenlijk altijd gegeneerd wil hebben

offtopic:
Hoe breng jij een PK in verlegenheid? Je hebt een verzameling compromitterende foto's van mogelijke PK's?

[naam]

Zelf vanmiddag ook een mooie geschreven:

C#:

static int getAverage(int[] numbers)
        {
            int begin = 0;
            int result = 0;
            foreach (int i in numbers)
            {
                begin += i;
            }

            result = (begin / numbers.Length);

            return result;
        }

Kan vast veel efficiënter, maar dit is deels leraartje-pesten aangezien ik de opdracht niet boeiend vind

[Sebazzz]

Als dat C# is, hou je je sowieso niet aan de guidelines (statische methodes moeten met een hoofdletter beginnen).

Oh en:

C#:

static int GetAverage(int[] numbers) {
            return numbers.Avg(); //uit me hoofd
        }

[ Voor 35% gewijzigd door Sebazzz op 15-09-2010 21:24 ]

[Vinnienerd]

naam schreef op woensdag 15 september 2010 @ 21:20:
Zelf vanmiddag ook een mooie geschreven:

C#:

static int getAverage(int[] numbers)
        {
            int begin = 0;
            int result = 0;
            foreach (int i in numbers)
            {
                begin += i;
            }

            result = (begin / numbers.Length);

            return result;
        }

Kan vast veel efficiënter, maar dit is deels leraartje-pesten aangezien ik de opdracht niet boeiend vind

C#:

static int getAverage(int[] numbers)
        {
            int begin;
            foreach (int i in numbers)
            {
                begin += i;
            }

            return begin / numbers.Length;
        }

Scheelt weer geheugen

[Zeebonk]

naam schreef op woensdag 15 september 2010 @ 21:20:
Zelf vanmiddag ook een mooie geschreven:

C#:

static int getAverage(int[] numbers)
        {
            int begin = 0;
            int result = 0;
            foreach (int i in numbers)
            {
                begin += i;
            }

            result = (begin / numbers.Length);

            return result;
        }

Kan vast veel efficiënter, maar dit is deels leraartje-pesten aangezien ik de opdracht niet boeiend vind

Lekkere afronding...

EDIT: Geldt voor iedereen hierboven eigenlijk

[ Voor 4% gewijzigd door Zeebonk op 15-09-2010 21:32 ]

[Vinnienerd]

Wat ik trouwens ook niet snap is dat in Java alle getallen signed zijn. Da's toch hardstikke zonde als je veel getallen in geheugen hebt die nooit hoger dan 128 komen?

[Sebazzz]

Vinnienerd schreef op woensdag 15 september 2010 @ 21:25:
[...]
Scheelt weer geheugen

Dat meen je toch niet mag ik hopen, of mis ik het sarcasme?

Vinnienerd schreef op woensdag 15 september 2010 @ 21:31:
Wat ik trouwens ook niet snap is dat in Java alle getallen signed zijn. Da's toch hardstikke zonde als je veel getallen in geheugen hebt die nooit hoger dan 128 komen?

Dat is een designkeuze, en hoewel .NET wel uint,ulong,ushort en dergelijke heeft is het niet CLS compliant om deze objecten te gebruiken. Maar voor P/Invokes en dergelijke is het natuurlijk wel handig, met Java moet je dan moeilijk gaan doen. Overigens wordt in .NET zelf, bijna geen uint en dergelijke gebruikt. Ik ben zelf nog geen methode tegengekomen die wel een uint/ulong wilt en geen int/long.
Bovendien, met de computers van tegenwoordig, als een managed taal draaien geen probleem is, dan zijn die paar bitjes helemaal niet significant meer. Hetzelfde geldt natuurlijk voor veel dingen, voor veel dingen zou een short of zelfs een byte voldoen.

[ Voor 74% gewijzigd door Sebazzz op 15-09-2010 21:37 ]

[Zeebonk]

Vinnienerd schreef op woensdag 15 september 2010 @ 21:31:
Wat ik trouwens ook niet snap is dat in Java alle getallen signed zijn. Da's toch hardstikke zonde als je veel getallen in geheugen hebt die nooit hoger dan 128 komen?

Ik meen mij te herinneren dat het gebruiken van bytes, shorts en chars e.d helemaal geen geheugen bespaart omdat ze gewoon in 32bits blokken worden weggeschreven.

[sopsop]

Daarnaast lijkt mij dat je geen PK moet gebruiken die informatie bevat die onder privacywetgeving valt.

[Vinnienerd]

Sebazzz schreef op woensdag 15 september 2010 @ 21:32:
[...]

Dat meen je toch niet mag ik hopen, of mis ik het sarcasme?

Sorry, wat ik bedoelde: scheelt weer een extra instructie. En toen realiseerde ik me weer dat de compiler d'r vast weer een extra variable voor aanmaakt.

Laat maar

Dat is een designkeuze, en hoewel .NET wel uint,ulong,ushort en dergelijke heeft is het niet CLS compliant om deze objecten te gebruiken. Maar voor P/Invokes en dergelijke is het natuurlijk wel handig, met Java moet je dan moeilijk gaan doen. Overigens wordt in .NET zelf, bijna geen uint en dergelijke gebruikt. Ik ben zelf nog geen methode tegengekomen die wel een uint/ulong wilt en geen int/long.

Waarom is signedness niet in de CLS opgenomen? Als ik heel veel getallen gebruik die nooit onder 0 komen, dan is het toch zonde om de signed variant te pakken?

[ Voor 0% gewijzigd door Vinnienerd op 15-09-2010 21:42 . Reden: Ik moet sneller reageren ]

[.oisyn]

Sebazzz schreef op woensdag 15 september 2010 @ 21:32:
Bovendien, met de computers van tegenwoordig, als een managed taal draaien geen probleem is, dan zijn die paar bitjes helemaal niet significant meer. Hetzelfde geldt natuurlijk voor veel dingen, voor veel dingen zou een short of zelfs een byte voldoen.

Als je veel met data schuift is een zo klein mogelijk datatype in je classes en struct weldegelijk significant, ook in een managed taal.

[Sebazzz]

.oisyn schreef op woensdag 15 september 2010 @ 21:58:
[...]

Als je veel met data schuift is een zo klein mogelijk datatype in je classes en struct weldegelijk significant, ook in een managed taal.

Alleen als je dat doet, je van .NET methodes afblijft in het proces want die eten meestal alleen 32-bit of 64-bit integers dus dan raak je alsnog een hoop kwijt. De vraag is als geheugen zo belangrijk is, waarom je bepaalde componenten niet in een lagere taal zoals C++ implementeert.

Sorry, wat ik bedoelde: scheelt weer een extra instructie. En toen realiseerde ik me weer dat de compiler d'r vast weer een extra variable voor aanmaakt.

Nooit opgelet bij vakken zoals hardware of computersystemen? Nooit in ASM geprogrammeerd?

[ Voor 19% gewijzigd door Sebazzz op 15-09-2010 22:03 ]