[alg] Slechtste programmeervoorbeelden deel 4

[Jeldert]

Ergens in oude code:

Visual Basic:

item = base64.Encode(Request("value"))
itemDecode = base64.Decode(item)

[nathanchunkie]

Wauw

code:

var Teller=null
function toontijd () {
var nu = new Date()
var maand = nu.getMonth()
if (maand == 0)
maand = "Januari";
else if (maand == 1)
maand = "Februari";
else if (maand == 2)
maand = "Maart";
else if (maand == 3)
maand = "April";
else if (maand == 4)
maand = "Mei";
else if (maand == 5)
maand = "Juni";
else if (maand == 6)
maand = "Juli";
else if (maand == 7)
maand = "Augustus";
else if (maand == 8)
maand = "September";
else if (maand == 9)
maand = "Oktober";
else if (maand == 10)
maand = "November";
else if (maand == 11)
maand = "December";

var datum = nu.getDate()

var dag = nu.getDay()
if (dag == 0)
dag = "Zondag";
else if (dag == 1)
dag = "Maandag";
else if (dag == 2)
dag = "Dinsdag";
else if (dag == 3)
dag = "Woensdag";
else if (dag == 4)
dag = "Donderdag";
else if (dag == 5)
dag = "Vrijdag";
else if (dag == 6)
dag = "Zaterdag";
var uren = nu.getHours()
if (uren <=9)
uren = "0"+uren
var minuten = nu.getMinutes()
if (minuten <=9)
minuten = "0"+minuten
var seconden = nu.getSeconds()
if (seconden <=9)
seconden = "0"+seconden
var tijdWaarde = "" + dag
tijdWaarde += " " + datum
tijdWaarde += " " + maand
tijdWaarde += " " + uren
tijdWaarde += ":"+minuten
tijdWaarde += ":"+seconden
document.getElementById('klokbalk').innerHTML=tijdWaarde;
teller =
setTimeout("toontijd()",1000)
}

[Verwijderd]

Jeldert schreef op dinsdag 25 september 2012 @ 15:02:
Ergens in oude code:

Visual Basic:

item = base64.Encode(Request("value"))
itemDecode = base64.Decode(item)

Wel eerst encoden, dat is wel zo veilig

[Bee.nl]

nathanchunkie schreef op zondag 30 september 2012 @ 18:57:
Wauw

Los van de gare datumopbouw, heeft die kerel schijnbaar ook nog nooit gehoord van een switch

[Firesphere]

Bee.nl schreef op zondag 30 september 2012 @ 19:02:
[...]

Los van de gare datumopbouw, heeft die kerel schijnbaar ook nog nooit gehoord van een switch

Waarom switchen als je het zelf kan typen?

[PeaceNlove]

AtleX schreef op vrijdag 21 september 2012 @ 11:08:
Weer een pareltje gevonden:

JavaScript:

// determine if user may edit <redacted>
var editAllowed = false;

Goed voor de veiligheid ja. Variabele op True gezet en ik kon los.

Doe ik ook wel hoor, maar dan check ik achteraf wel serverside of de editactie wel legitiem is. Dus dit is imho niet perse fout, maar je moet wel serverside een check hebben staan.

[F.West98]

PeaceNlove schreef op zondag 30 september 2012 @ 20:07:
[...]

Doe ik ook wel hoor, maar dan check ik achteraf wel serverside of de editactie wel legitiem is. Dus dit is imho niet perse fout, maar je moet wel serverside een check hebben staan.

Dat. Ik wil niet tigmiljoen scripts hebben
En ik serverside doe ik sowieso voor het geval mensen dat script los oproepen

nathanchunkie schreef op zondag 30 september 2012 @ 18:57:
Wauw
HUGE CODE

In PHP heb ik wel een oplossing, zonder switch.

PHP:

$maanden = array('NIETNULBASEDMAAND', 'Januari', 'Februari', 'Maart', 'April', 'Mei', 'Juni','Juli', 'Augustus', 'September', 'Oktober', 'November','December');
$dagen = array('NIETNULBASEDDAG', 'Maandag', 'Dinsdag', 'Woensdag', 'Donderdag', 'Vrijdag', 'Zaterdag', 'Zondag');
$maand = $maanden[date('n')];
$dag = $dagen[date('N')];

$datum = $dag.' '.date('d').' '.$maand.', '.date('H:i:s');
echo $datum

Of, als je geen kommagetallen wilt echo'en:

PHP:

setlocale(LC_ALL, 'nl_NL');
echo date('l d F, H:i:s');

Javascript ben ik niet zo in thuis

[ Voor 57% gewijzigd door F.West98 op 30-09-2012 21:12 ]

[.oisyn]

Dagen en maanden schrijf je in het Nederlands gewoon zonder hoofdletter.

[RaZ]

.oisyn schreef op zondag 30 september 2012 @ 23:03:
Dagen en maanden schrijf je in het Nederlands gewoon zonder hoofdletter.

Met als uitzondering als de zin met 1 van deze woorden begint. Dan is het dus netter om het eigenlijk fout te doen, zodat een zin als: "Januari was een koude maand." en "Zondag gingen we weer naar de kerk." er netter uitzien dan dat een zin niet met een hoofdletter begint.

[Verwijderd]

F.West98 schreef op zondag 30 september 2012 @ 21:02:
[...]

Dat. Ik wil niet tigmiljoen scripts hebben
En ik serverside doe ik sowieso voor het geval mensen dat script los oproepen


[...]

In PHP heb ik wel een oplossing, zonder switch.

PHP:

$maanden = array('NIETNULBASEDMAAND', 'Januari', 'Februari', 'Maart', 'April', 'Mei', 'Juni','Juli', 'Augustus', 'September', 'Oktober', 'November','December');
$dagen = array('NIETNULBASEDDAG', 'Maandag', 'Dinsdag', 'Woensdag', 'Donderdag', 'Vrijdag', 'Zaterdag', 'Zondag');
$maand = $maanden[date('n')];
$dag = $dagen[date('N')];

$datum = $dag.' '.date('d').' '.$maand.', '.date('H:i:s');
echo $datum

Of, als je geen kommagetallen wilt echo'en:

PHP:

setlocale(LC_ALL, 'nl_NL');
echo date('l d F, H:i:s');

Javascript ben ik niet zo in thuis

C#:

return DateTime.Now.ToString("F", new CultureInfo("nl-NL"));

Dat maakt het zelfs nog iets simpeler

[Alex)]

Gewoon de culture van de gebruiker gebruiken i.p.v. die opdringen.

Dit in web.config:

XML:

<globalization 
    culture="auto" 
    uiCulture="auto" 
    enableClientBasedCulture="true" />

Ik heb daar al eens een keer een hoop gebruikers gelukkig mee gemaakt.

...totdat bleek dat sommige validators (zoals de CompareValidator) stuk gaan wanneer de thousands seperator een spatie is (zoals bij Zweede cultures).

[ Voor 35% gewijzigd door Alex) op 30-09-2012 23:23 ]

[Verwijderd]

Dat is zo irritant als je in Spanje in een internetcafe zit en je geen laptop bij je hebt.

[Alex)]

Verwijderd schreef op zondag 30 september 2012 @ 23:24:
Dat is zo irritant als je in Spanje in een internetcafe zit en je geen laptop bij je hebt.

Dat is dan wel weer zo, maar daarvoor zou je nog een overrideknopje ergens kunnen maken, of je zou het kunnen koppelen aan een gebruikersprofiel (indien van toepassing). In mijn geval betrof het een intranetapplicatie die alleen vanaf klantspecifieke hardware gebruikt wordt.

[Verwijderd]

Wacht ff een hardwareafhankelijke webapplicatie? Dat is handig.

[Alex)]

Verwijderd schreef op zondag 30 september 2012 @ 23:29:
Wacht ff een hardwareafhankelijke webapplicatie? Dat is handig.

Wat ik bedoelde te zeggen: alleen toegankelijk vanaf laptops die door Corporate IT worden uitgegeven. En dus geen 3rd party hardware (zoals van consultants) of via internet. VPN was ook alleen mogelijk vanaf diezelfde laptops

[Keeper]

RaZ schreef op zondag 30 september 2012 @ 23:10:
Met als uitzondering als de zin met 1 van deze woorden begint. Dan is het dus netter om het eigenlijk fout te doen, zodat een zin als: "Januari was een koude maand." en "Zondag gingen we weer naar de kerk." er netter uitzien dan dat een zin niet met een hoofdletter begint.

Nu lijkt de kans dat je dat soort zinnen genereert op basis van een Date class/library/functie vrij klein (0 hoop ik zelfs), dus kan je beter gewoon wel met kleine letters werken als je een Nederlandse datum wilt printen.

Ik zou zelf echter altijd native implementatie hiervoor gebruiken, ipv zelf gaan kloten met arrays of voorbeelden zoals hierboven. In Javascript zit dat echter niet standaard, maar daar heb je gelukkig ook externe libs voor tegenwoordig.

[kenneth]

C#:

    public string MakeDocument(string templatefileName, string saveAs, DataTable table)
    {
        try
        {
            if (!File.Exists(templatefileName))
            {
                return "<error>";
            }

            CreateWordDocument(templatefileName, table, saveAs);
        }
        catch (Exception ex)
        {
            MessageBox.Show("Error in MakeDocument." + ex.Message);
            return "<error>";
        }

        return null;
    }

Dus "<error>" is fout en null is goed. Nee okay, duidelijk ...
En een MessageBox buiten de UI-layer ...

[NNF]

Iets opmerkelijks in ActionScript dat ik vandaag tegenkwam:

code:

_global.soundMuted = ExternalInterface.call("GetVariable", "AudioMuted", "" );

_global.soundMuted == 1 ? _global.soundMuted = true : _global.soundMuted = false;

_root.messages.text = _global.soundMuted;

switch(_global.soundMuted)
{
    case true:
        muteSounds();
        break;  
    case false:
        playSounds();
        break;
}

Het werkt natuurlijk wel, maar waarom zo omslachtig?

[ Voor 8% gewijzigd door NNF op 04-10-2012 23:06 ]

[Davio]

Omdat de switch dan eventueel nog uitgebreid kan worden met FileNotFound.

[bobo1on1]

Regel 3 vond ik ook wel wat verwarrend met de assignments binnen de ternary operator.

[ Voor 67% gewijzigd door bobo1on1 op 05-10-2012 00:19 ]

[Soultaker]

edit: Snel je post editen hè? Ik zag 't wel!

Regel 3 is een lelijke manier om te schrijven:

code:

if (_global.soundMuted == 1) {
    _global.soundMuted = true;
} else {
    _global.soundMuted = false;
}

... wat op zichzelf een lelijke manier is om te schrijven:

code:

_global.soundMuted = (_global.soundMuted == 1);

(Haakjes zijn hier natuurlijk niet verplicht, maar wel zo duidelijk.)

[ Voor 6% gewijzigd door Soultaker op 05-10-2012 00:25 ]

[Bee.nl]

F.West98 schreef op zondag 30 september 2012 @ 21:02:
In PHP heb ik wel een oplossing, zonder switch.

PHP:

$maanden = array('NIETNULBASEDMAAND', 'Januari', 'Februari', 'Maart', 'April', 'Mei', 'Juni','Juli', 'Augustus', 'September', 'Oktober', 'November','December');
$dagen = array('NIETNULBASEDDAG', 'Maandag', 'Dinsdag', 'Woensdag', 'Donderdag', 'Vrijdag', 'Zaterdag', 'Zondag');
$maand = $maanden[date('n')];
$dag = $dagen[date('N')];

Tip: als je de eerste waarde in de array een andere numerieke key meegeeft, dan telt php automatisch voor je door. Dus dan krijg je dit:

PHP:

$maanden = array(1=>'Januari', 'Februari', 'Maart', 'April', 'Mei', 'Juni','Juli', 'Augustus', 'September', 'Oktober', 'November','December');
var_dump($maanden);

/*
array
  1 => string 'Januari' (length=7)
  2 => string 'Februari' (length=8)
  3 => string 'Maart' (length=5)
  4 => string 'April' (length=5)
  5 => string 'Mei' (length=3)
  6 => string 'Juni' (length=4)
  7 => string 'Juli' (length=4)
  8 => string 'Augustus' (length=8)
  9 => string 'September' (length=9)
  10 => string 'Oktober' (length=7)
  11 => string 'November' (length=8)
  12 => string 'December' (length=8)
*/

Of, als je geen kommagetallen wilt echo'en:

PHP:

setlocale(LC_ALL, 'nl_NL');
echo date('l d F, H:i:s');

Dit kun je simpel oplossen door setlocale(LC_TIME, 'nl_NL') te gebruiken. Je kunt per categorie een andere locale erop nahouden als je dat wilt.

RaZ schreef op zondag 30 september 2012 @ 23:10:
[...]

Met als uitzondering als de zin met 1 van deze woorden begint. Dan is het dus netter om het eigenlijk fout te doen, zodat een zin als: "Januari was een koude maand." en "Zondag gingen we weer naar de kerk." er netter uitzien dan dat een zin niet met een hoofdletter begint.

Not sure if serious

In 99% van de gevallen weet je al of de datum wel of niet vooraan in de zin komt. Kwestie van ucfirst erover gooien, et voilà. Overigens terechte opmerking van .oisyn; dit wordt de laatste tijd steeds vaker fout geschreven.

[Mint]

Misschien wordt het tijd om te gaan slapen:

C#:

// pseudo
MySqlCommand cmd = new MySqlCommand();
cmd.CommandText = "query";
cmd.Parameters.AddWithValue("@param", object.property);
cmd.Prepare();
cmd.ExecuteNonQuery();
cmd.Dispose();

try {
 return true;
} 
catch (MySqlException ex)
{
 // doe iets met ex
return false;
}

En het maar raar vinden dat de exception nooit gethrowed werd. De fout was overigens een komma teveel in de query

[boe2]

mij lijkt juist dat de exception altijd zal gegooid worden als er iets mis is met die query? Hij komt niet eens tot aan de try/catch block.

[ Voor 40% gewijzigd door boe2 op 05-10-2012 09:25 ]

[Mint]

Boeboe schreef op vrijdag 05 oktober 2012 @ 09:11:
mij lijkt juist dat de exception altijd zal gegooid worden als er iets mis is met die query? Hij komt niet eens tot aan de try/catch block.

Er treedt wel een exception op, maar die wordt niet gehandled omdat de .ExecuteNonQuery() buiten de try/catch staat. Daardoor gaat de applicatie onderuit.

[Haan]

adyta schreef op vrijdag 05 oktober 2012 @ 11:43:
[...]


Er treedt wel een exception op, maar die wordt niet gehandled omdat de .ExecuteNonQuery() buiten de try/catch staat. Daardoor gaat de applicatie onderuit.

ergo: hij wordt gegooid

[Mint]

Haan schreef op vrijdag 05 oktober 2012 @ 12:06:
[...]

ergo: hij wordt gegooid

Precies. Mental note: volgende keer reactie beter lezen

[ Voor 3% gewijzigd door Mint op 05-10-2012 12:15 ]

[Geerrrt]

adyta schreef op vrijdag 05 oktober 2012 @ 12:15:
[...]


Precies. Mental note: volgende keer reactie beter lezen

Ja Stefan, beter lezen!

[ Voor 18% gewijzigd door Geerrrt op 05-10-2012 12:18 ]

[Mint]

GeertJam schreef op vrijdag 05 oktober 2012 @ 12:18:
[...]


Ja Stefan, beter lezen!

Hoi Geert

[bobo1on1]

Soultaker schreef op vrijdag 05 oktober 2012 @ 00:21:
edit: Snel je post editen hè? Ik zag 't wel!

Regel 3 is een lelijke manier om te schrijven:

code:

if (_global.soundMuted == 1) {
    _global.soundMuted = true;
} else {
    _global.soundMuted = false;
}

... wat op zichzelf een lelijke manier is om te schrijven:

code:

_global.soundMuted = (_global.soundMuted == 1);

(Haakjes zijn hier natuurlijk niet verplicht, maar wel zo duidelijk.)

Mja, alleen de ternary operator heeft ook een return value, dus eigenlijk is het meer iets als dit:

code:

bool meh(input)
{
  if (input == 1)
    return _global.soundMuted = true;
  else
    return _global.soundMuted = false;
}

_global.soundMuted = ExternalInterface.call("GetVariable", "AudioMuted", "" );

meh(_global.soundMuted);

[Soultaker]

Uhm, ten eerste levert mijn 1-regelige versie ook gewoon een waarde op, en ten tweede is dat hier eigenlijk irrelevant omdat die waarde nergens gebruikt wordt. Ik zie absoluut niet de meerwaarde van je meh() functie.

[gosse adema]

Een leuk stukje code gevonden:

---

//pseudo
input plaatsid;

plaatsnaam = "select p.naam from plaatsen p where p.id = plaatsid";

for l in ("select a.locatieid, a.naam from adressen a where lower(a.name) = lower(plaatsnaam))
{
verwerk a.locatieid, a.naam, plaatsnaam;
}

---

De name-kolom in de adressen tabel heeft geen index, dus met het toenemen van het aantal adressen ging de performance achteruit. Dit terwijl er een koppeltabel tussen beide tabellen aanwezig is.

Helaas zijn er wel plaatsen welke in meerdere landen voorkomen.

[Jegorex]

gosse adema schreef op zaterdag 06 oktober 2012 @ 22:06:
Een leuk stukje code gevonden:

...

Helaas zijn er wel plaatsen welke in meerdere landen voorkomen.

Er zijn zelfs plaatsen die meerdere keren in Nederland voorkomen.
Toen mijn ouders nog in Winsum/Gr woonden waren er regelmatig leveranciers die in Winsum/Fr stonden en de straat niet konden vinden.

[Alex)]

Daarom altijd op basis van postcode navigeren

[Caelorum]

Jegorex schreef op zaterdag 06 oktober 2012 @ 22:46:
[...] Er zijn zelfs plaatsen die meerdere keren in Nederland voorkomen.
Toen mijn ouders nog in Winsum/Gr woonden waren er regelmatig leveranciers die in Winsum/Fr stonden en de straat niet konden vinden.

Hengelo is ook zo'n mooie. Er is een klein dorpje en een stad. Op de een of andere manier gingen sommige mensen er vanuit dat je in het dorpje woont

[Hydra]

Caelorum schreef op zondag 07 oktober 2012 @ 11:27:
Hengelo is ook zo'n mooie. Er is een klein dorpje en een stad groot dorp. Op de een of andere manier gingen sommige mensen er vanuit dat je in het dorpje woont

[Caelorum]

Hydra schreef op maandag 08 oktober 2012 @ 11:49:
[...]

Tja. Ik vind een gemeente met meer dan 80.000 inwoners een stad en niet een klein dorp. Jammer genoeg zijn stadsrechten afgeschaft en bestaat er niet zoiets als het begrip stad in de wet.
Aan de andere kant. Als een gemeente als Groenlo met 10.000 inwoners stadsrechten heeft gekregen, mag je grotere gemeentes ook wel als stad zien lijkt mij

[Hydra]

Caelorum schreef op maandag 08 oktober 2012 @ 12:04:
Tja. Ik vind een gemeente met meer dan 80.000 inwoners een stad en niet een klein dorp. Jammer genoeg zijn stadsrechten afgeschaft en bestaat er niet zoiets als het begrip stad in de wet.
Aan de andere kant. Als een gemeente als Groenlo met 10.000 inwoners stadsrechten heeft gekregen, mag je grotere gemeentes ook wel als stad zien lijkt mij

Ik ben vanuit Twente naar Utrecht verhuist en sindsdien is er "de stad" en "de provincie" voor mij. Beetje uit de hoogte doen tegen als die boeren ten oosten van de A27 zegmaar

[Davio]

Jegorex schreef op zaterdag 06 oktober 2012 @ 22:46:
[...]

Er zijn zelfs plaatsen die meerdere keren in Nederland voorkomen.
Toen mijn ouders nog in Winsum/Gr woonden waren er regelmatig leveranciers die in Winsum/Fr stonden en de straat niet konden vinden.

Vlakbij waar ik woon is één van de vele "Spijk"-en van Nederland.
Zelfs één van de twee Spijk-en in Gelderland; de andere is waar de Rijn Nederland binnenkomt.

In totaal komt de naam Spijk vijf keer voor.

[ Voor 4% gewijzigd door Davio op 08-10-2012 13:34 ]

[afraca]

Jegorex schreef op zaterdag 06 oktober 2012 @ 22:46:
[...]

Er zijn zelfs plaatsen die meerdere keren in Nederland voorkomen.
Toen mijn ouders nog in Winsum/Gr woonden waren er regelmatig leveranciers die in Winsum/Fr stonden en de straat niet konden vinden.

Laatst nog m'n opa en oma in zo'n mooi Gronings accent dat horen vertellen over Den Ham, dat zo'n vrachtwagenchauffeur helemaal verkeerd zat.

[Hydra]

afraca schreef op maandag 08 oktober 2012 @ 13:50:
Laatst nog m'n opa en oma in zo'n mooi Gronings accent dat horen vertellen over Den Ham, dat zo'n vrachtwagenchauffeur helemaal verkeerd zat.

Als zo'n iemand naar "Kerkstraat 5 Den Ham" moet weet je ook gewoon dat het 50% van de keren misloopt

[MueR]

Genoeg off-topic gepost nu.

[Not Pingu]

Hoe split je een raw CSV-bestand op in een string array met 1 entry per regel? Zo dus:

C#:

//input: byte[] fileContent

DataTable csvDataTable = new DataTable();
string fileName = string.Format("{0}{1}.xls", Path.GetTempPath(), Path.GetRandomFileName());
File.WriteAllBytes(fileName, fileContent);
string[] csvData = File.ReadAllLines(fileName);

[ Voor 4% gewijzigd door Not Pingu op 09-10-2012 09:16 ]

[Hydra]

Huh? Hij schrijft eerst de data weg om daarna deze weer in te lezen? Iets met m'n zonnebril en hoe hij niks doet...

[ Voor 25% gewijzigd door Hydra op 08-10-2012 16:52 ]

[Not Pingu]

Zo zet hij in 1 stap de byte array om naar een string én split ie hem op line. Want op een live server heeft de application user natuurlijk schrijfrechten op de Temp map.
Even afgezien van het feit dat je er met een System.Text.Encoding.UTF8.GetString() en vervolgens een string.Split() ook bent en je dan geen file system operatie nodig hebt.
Die DataTable die bovenaan wordt gedeclareerd wordt overigens helemaal niet gebruikt.

[ Voor 67% gewijzigd door Not Pingu op 08-10-2012 16:59 ]

[Haan]

http://code.google.com/p/linqtoexcel/

[Rutix]

Haan schreef op vrijdag 12 oktober 2012 @ 12:30:
http://code.google.com/p/linqtoexcel/

Ik zie niet heel snel wat hier nou slecht aan is

[mcDavid]

Not Pingu schreef op maandag 08 oktober 2012 @ 16:49:
Hoe split je een raw CSV-bestand op in een string array met 1 entry per regel? Zo dus:

C#:

//input: byte[] fileContent

DataTable csvDataTable = new DataTable();
string fileName = string.Format("{0}{1}.xls", Path.GetTempPath(), Path.GetRandomFileName());
File.WriteAllBytes(fileName, fileContent);
string[] csvData = File.ReadAllLines(fileName);

zo zag ik laatst ergens het volgende:

PHP:

$xml = file_get_contents('feed.xml');
$json = json_encode($xml);
$array = json_decode($json,TRUE);

Vond het wel een creatieve oplossing maar simplexml_load_file() is toch iets korter

[Haan]

Rutix schreef op vrijdag 12 oktober 2012 @ 12:49:
[...]

Ik zie niet heel snel wat hier nou slecht aan is

Het is dan ook een reactie op de posts boven me

[Not Pingu]

Not Pingu schreef op maandag 08 oktober 2012 @ 16:49:
Hoe split je een raw CSV-bestand op in een string array met 1 entry per regel? Zo dus:

C#:

//input: byte[] fileContent

DataTable csvDataTable = new DataTable();
string fileName = string.Format("{0}{1}.xls", Path.GetTempPath(), Path.GetRandomFileName());
File.WriteAllBytes(fileName, fileContent);
string[] csvData = File.ReadAllLines(fileName);

Hoera, in de nieuwste release is bovenstaande code gerefactored!

C#:

string fileName = string.Format("{0}{1}.xls", Path.GetTempPath(), Path.GetRandomFileName());
File.WriteAllBytes(fileName, fileContent);

DataTable importDataTable = new DataTable();
using (MemoryStream stream = new MemoryStream(fileContent))
{
    parser.ReadCSV(stream, importDataTable);
}

foreach (DataRow r in importDataTable.Rows)
{
    //...
}

Dus we moeten de webserver nog steeds schrijfrechten op de Windows/Temp map geven, maar nu dient het helemáál nergens meer voor. En het blijft moeilijk he, een stringetje splitten.

Haan schreef op vrijdag 12 oktober 2012 @ 12:30:
http://code.google.com/p/linqtoexcel/

Heb je dan ook nog steeds MS Office nodig op de omgeving waar je die code wilt draaien, t.w. een webserver?

[Haan]

Not Pingu schreef op vrijdag 12 oktober 2012 @ 13:39:
[...]


Heb je dan ook nog steeds MS Office nodig op de omgeving waar je die code wilt draaien, t.w. een webserver?

Nee, alleen OleDB drivers.

Trouwens 'Path.GetRandomFilename()' Die kende ik nog niet!

[ Voor 67% gewijzigd door Haan op 12-10-2012 13:55 ]

[kenneth]

Haan schreef op vrijdag 12 oktober 2012 @ 12:30:
http://code.google.com/p/linqtoexcel/

Ziet er geweldig uit! Ik ga hem nooit gebruiken omdat ik voor eeuwig angstig ben aangezien hij in dit topic is gepost

[Davio]

kenneth schreef op vrijdag 12 oktober 2012 @ 14:01:
[...]

Ziet er geweldig uit! Ik ga hem nooit gebruiken omdat ik voor eeuwig angstig ben aangezien hij in dit topic is gepost

It's a trap!

[Haan]

kenneth schreef op vrijdag 12 oktober 2012 @ 14:01:
[...]

Ziet er geweldig uit! Ik ga hem nooit gebruiken omdat ik voor eeuwig angstig ben aangezien hij in dit topic is gepost

Zo dan? Haan in "Deel hier je favoriete NuGet packages"

[kenneth]

Dank

[Trokken]

Ik heb een stukje C gemaakt wat er zo uitziet:

code:

#include <stdio.h>
#include <stdlib.h>

int main()
{
  int A[10]={-3, 0, 0, 3, 5, 1, -6, 6, 3, -9};
  int x, t, i, l, e;
  int s = (sizeof(A)/sizeof(int));
  
  printf("index\t lengten\n"
         "----------------\n");
  x=0;
  
  for( x=0; x<=s-1; x++);
  {
          
        e=0;
        l=0;
        t=0;
        
        for( i=x; i<=(s-1); i++)
        {
                 
                 l++;
                 t = t+A[i];
                 
                 if(t==0 && e==0)
                 {
                         printf("\n%d\t %d",x, l);
                         e=1;
                 }
                 else if(t==0 && e==1)
                 {
                      printf(" %d", l);
                 }
                 
        }
  }
  printf("\n");
  system("PAUSE");  
  return 0;
}

Nu is het probleem dat ik niet in de tweede "for" loop kom. Met een printf instructie zie ik dat x 10 is voordat de eerste for loop is afgelopen maar x zou pas mogen verhogen nadat de tweede for loop is afgelopen lijkt mij.

Iemand enig idee hoe ik het voor elkaar heb gekregen dat de tweede for loop niet in werking gaat?

[Dido]

; op regel 14?

[Trokken]

Dido schreef op zondag 14 oktober 2012 @ 15:08:
; op regel 14?

Never mind

[MBV]

Zou je eens willen uitleggen waarom je je eigen code in het slechte voorbeelden topic neer hebt gezet? Ik kan wel uitleggen waarom dat een goed idee is, maargoed, benieuwd naar je zelfkennis

[ Voor 8% gewijzigd door MBV op 14-10-2012 17:48 ]

[telefoontoestel]

code:

X=0;

for(x=0....

?

code:

i<=(s-1)

Zijn wel heel basale foutjes als je t mij vraagt

[Miyamoto]

Wat is daar precies fout aan volgens jou?
Behalve dat je het verkeerd hebt overgetikt...

[Webgnome]

nvm..

[ Voor 98% gewijzigd door Webgnome op 14-10-2012 22:14 ]

[jacobras]

Miyamoto schreef op zondag 14 oktober 2012 @ 21:34:
Wat is daar precies fout aan volgens jou?
Behalve dat je het verkeerd hebt overgetikt...

code:

int x, [..]

x=0;
  
  for( x=0;

Die x=0 op regel 3 (in Trokken's snippet regel 12) voegt niets toe. Je hebt al een integer, die zet je op 0 en de regel erna weer op 0

code:

i<=(s-1)

Normaal gebruik je dan:

code:

i<s;

[ Voor 39% gewijzigd door jacobras op 14-10-2012 22:31 ]

[ZaPPZion]

jacobras schreef op zondag 14 oktober 2012 @ 22:28:
Die x=0 op regel 3 (in Trokken's snippet regel 12) voegt niets toe. Je hebt al een integer, die zet je op 0 en de regel erna weer op 0

fout.

Hij heeft het over C. In C worden auto variables niet geinitialiseerd ivm execution speed. Dus de x wordt niet naar 0 geset bij het aanmaken van de variable.

[Patriot]

ZaPPZion schreef op maandag 15 oktober 2012 @ 00:47:
[...]


fout.

Fout

Vóór de for-loop zet hij hem op 0, en hij laat de for-loop hem wederom op 0 zetten.

[MueR]

Trokken schreef op zondag 14 oktober 2012 @ 15:05:
Ik heb een stukje C gemaakt wat er zo uitziet:

Wil je de volgende keer niet gaan helpdesken in dit topic? Daar is het pertinent niet voor bedoeld.

[jacobras]

ZaPPZion schreef op maandag 15 oktober 2012 @ 00:47:
[...]


fout.

Hij heeft het over C. In C worden auto variables niet geinitialiseerd ivm execution speed. Dus de x wordt niet naar 0 geset bij het aanmaken van de variable.

Dat weet ik, na int x is x nog niet geïnitialiseerd. Maar net voor de for-loop wordt x wél geïnitialiseerd, en vervolgens in de for-loop nogmaals.

[MBV]

MueR schreef op maandag 15 oktober 2012 @ 12:15:
[...]

Wil je de volgende keer niet gaan helpdesken in dit topic? Daar is het pertinent niet voor bedoeld.

Nou ja, ik vind het wel van toepassing op de code

- for-loop met dubbele initialisatie van i
- <= s-1
- variabele l die i volgt, maar niet gebruikt wordt
- een call naar system, waar een stdio call beter was geweest

en vast nog wel wat kleine dingetjes als je beter zoekt.

[Haan]

Het feit dat geen enkele variabele langer is dan 1 letter, is ook een goeie reden om in dit topic te mogen staan.

[MBV]

Voor een huiswerkopdracht vind ik dat nog wel gaan, niet echt aantoonbaar fout. Maar wel een sterke hint.

[Verwijderd]

code:

  int A[10]={-3, 0, 0, 3, 5, 1, -6, 6, 3, -9};
  int x, t, i, l, e;
  int s = (sizeof(A)/sizeof(int));

Lekkere variable namen heb je ook O.o
Hoe moet iemand die die code leest in godsnaam weten wat je daar doet.

[Merethil]

Verwijderd schreef op dinsdag 16 oktober 2012 @ 01:21:
[...]


Lekkere variable namen heb je ook O.o
Hoe moet iemand die die code leest in godsnaam weten wat je daar doet.

Dat was al gezegd... Gek genoeg doen scholen dat vaak tijdens voorbeelden. Ik had een docent die overal alleen maar "int i" gebruikte als tellertjes e.d. gewoon om het zo simpel mogelijk te houden.

En dan achteraf zeggen: "Ja dan moet je van die i wel even wat nuttigs maken, want nu is het eigenlijk onleesbaar!" Joh...

[RayNbow]

Een-letterige variabelenamen zijn toch niet per se fout?

Haskell:

map _ []      =  []
map f (x:xs)  =  f x : map f xs

[Haan]

Er zijn altijd uitzonderingen ja

JavaScript:

$ = jQuery;

[.oisyn]

Niets mis met 'i' als tellertje.

[MueR]

MBV schreef op maandag 15 oktober 2012 @ 12:55:
[...]

Nou ja, ik vind het wel van toepassing op de code

* MueR mept MBV. Je weet wat ik bedoel

[kenneth]

.oisyn schreef op dinsdag 16 oktober 2012 @ 12:03:
Niets mis met 'i' als tellertje.

Een oud-collega zei dat je ii moest gebruiken omdat je op i niet kan zoeken. Mijn punt dat je een heel ander probleem hebt als je op zo'n variable moet gaan zoeken is nooit echt geland ...

[bobo1on1]

En sowieso zou ik een goede editor gebruiken die op i als woord kan zoeken.

[Hydra]

kenneth schreef op dinsdag 16 oktober 2012 @ 12:39:
Een oud-collega zei dat je ii moest gebruiken omdat je op i niet kan zoeken. Mijn punt dat je een heel ander probleem hebt als je op zo'n variable moet gaan zoeken is nooit echt geland

Gewoon:

code:

public class Globals 
{
    public static int i;
}

En dan:

code:

for(Globals.i = 0;Globals.i < array.length;Globals.i++)
{

}

Scheelt geheugen!

[Verwijderd]

En wat als je op twee verschillende threads een teller hebt?

[lauwsa]

Ik gebruik "i" bijvoorbeeld alleen in een kleine/overzichtelijke loop, dan hoef je niet eens naar "i" te zoeken. Anders zie je na een tijd door de bomen het bos toch niet meer. Bij een groter stuk is het toch makkelijker om een naam te geven.

[Hydra]

Verwijderd schreef op dinsdag 16 oktober 2012 @ 16:05:
En wat als je op twee verschillende threads een teller hebt?

Da's de hele grap dus.

lauwsa schreef op dinsdag 16 oktober 2012 @ 16:05:
Ik gebruik "i" bijvoorbeeld alleen in een kleine/overzichtelijke loop, dan hoef je niet eens naar "i" te zoeken. Anders zie je na een tijd door de bomen het bos toch niet meer. Bij een groter stuk is het toch makkelijker om een naam te geven.

Sherlock

[ Voor 47% gewijzigd door Hydra op 16-10-2012 16:06 ]

[lauwsa]

Och, waarom het je zelf ingewikkeld maken als het ook overzichtelijk kan hé. "kijkt naar de hele mooie code met allemaal lettertjes die nergens op slaan. Kan je daar überhaupt hier mee werken zonder terug te kijken wat wat was."

Hydra schreef op dinsdag 16 oktober 2012 @ 16:06:
Sherlock

Eigenlijk had ik "door de letters de code niet meer kunnen zien" moeten zeggen . Ik zat trouwens meer te denken aan captain obvious .

[ Voor 86% gewijzigd door lauwsa op 16-10-2012 16:12 ]

[Merethil]

.oisyn schreef op dinsdag 16 oktober 2012 @ 12:03:
Niets mis met 'i' als tellertje.

Wel als het meerdere for-loops in elkaar zijn waar i gebruikt werd als tellertje én om dan door jagged arrays te lopen e.d....
Maar idd, ik pak zelf vaak genoeg $i als standaard voor m'n kleine for-loopjes of tellers die dagelijks langskomen.

Comments maken dat echter ook al een stuk overzichtelijker

[lauwsa]

Als je dan toch de moeite neemt om een command te type, waarom noem je hem dan eigelijk niet in één keer "fatsoenlijk". Want het zal niet overzichtelijk zijn als je een command nodig hebt, daar langs zal je IDE je aanvullen bij een langere namen. Dus zo veel meerwerk is het toch niet?

Tenminsten, naar mijn ervaring ( stelt niet veel voor, maar toch ) is het toch makkelijker om alles goed te benoemen in plaats van een command. Als je dan een tijd later de code bekijkt zie je meteen wat het is en heb je geen bras aan commands.

Edit:
Ik bedoel dus commands zoals, i staat hier voor de index van het slaap lietje in slaaplietjes. ( sorry, word een beetje moe van steeds de zelfde muziek op de radio )

[ Voor 16% gewijzigd door lauwsa op 16-10-2012 16:31 ]

[Merethil]

lauwsa schreef op dinsdag 16 oktober 2012 @ 16:23:
Als je dan toch de moeite neemt om een command te type, waarom noem je hem dan eigelijk niet in één keer "fatsoenlijk". Want het zal niet overzichtelijk zijn als je een command nodig hebt, daar langs zal je IDE je aanvullen bij een langere namen. Dus zo veel meerwerk is het toch niet?

Tenminsten, naar mijn ervaring ( stelt niet veel voor, maar toch ) is het toch makkelijker om alles goed te benoemen in plaats van een command. Als je dan een tijd later de code bekijkt zie je meteen wat het is en heb je geen bras aan commands.

Edit:
Ik bedoel dus commands zoals, i staat hier voor de index van het slaap lietje in slaaplietjes. ( sorry, word een beetje moe van steeds de zelfde muziek op de radio )

Ik zie dat meer als "variabele" maar dat zal ik dan wel zijn...

[roeleboel]

@Merethil: ik denk dat ie 'comments' bedoelt ipv commands...

[Merethil]

roeleboel schreef op dinsdag 16 oktober 2012 @ 16:50:
@Merethil: ik denk dat ie 'comments' bedoelt ipv commands...

Oh god ik denk dat je gelijk hebt. Ik ben ook moe blijkt het... Thanks for the heads up!

[lauwsa]

Oops, klopt. Dit bevestigt dat het laat is.

[Hydra]

lauwsa schreef op dinsdag 16 oktober 2012 @ 16:52:
Oops, klopt. Dit bevestigt dat het laat is.

Nouja, je maakt wel meer rare fouten die je teksten toch wat moeilijk te lezen maken. Zo laat is het niet, grijp nog maar een kop koffie

[Merethil]

Hydra schreef op dinsdag 16 oktober 2012 @ 16:53:
[...]


Nouja, je maakt wel meer rare fouten die je teksten toch wat moeilijk te lezen maken. Zo laat is het niet, grijp nog maar een kop koffie

Een kop loffie?

[bwerg]

bobo1on1 schreef op dinsdag 16 oktober 2012 @ 15:59:
En sowieso zou ik een goede editor gebruiken die op i als woord kan zoeken.

Waarom woorden zoeken, als je gewoon specifieke variabelen kan zoeken...

Ik zat laatst even met een antieke IDE te werken. Wil je daarin een variabele "i" hernoemen (met "find and replace all" in plaats van "refactor -> rename"), dan verandert hij mooi álle i's, in verschillende scopes, verschillende variabelen dus. Feest.

Ik snap mensen nooit die zeggen dat het prima programmeren is in notepad.

[ Voor 4% gewijzigd door bwerg op 16-10-2012 17:48 ]

[alwinuzz]

bwerg schreef op dinsdag 16 oktober 2012 @ 17:08:
[...]
Ik snap mensen nooit die zeggen dat het prima programmeren is in notepad.

Is prima te doen hoor: gewoon bovenaan het bestand een comment waar je in zet welke variable namen je al hebt gebruikt

[Hydra]

bwerg schreef op dinsdag 16 oktober 2012 @ 17:08:
Ik snap mensen nooit die zeggen dat het prima programmeren is in notepad.

Nooit de moeite genomen een fatsoenlijke IDE te leren kennen. "Noes! Allemaal rare knopjes!"

[MBV]

MueR schreef op dinsdag 16 oktober 2012 @ 12:35:
[...]

* MueR mept MBV. Je weet wat ik bedoel

Schot voor open doel trap ik er altijd graag in

Hydra schreef op dinsdag 16 oktober 2012 @ 17:14:
[...]


Nooit de moeite genomen een fatsoenlijke IDE te leren kennen. "Noes! Allemaal rare knopjes!"

Dan pak je VIM, geen knopje te bekennen! * MBV pakt popcorn en wacht op gezeur over emacs

[.oisyn]

Merethil schreef op dinsdag 16 oktober 2012 @ 16:18:
[...]

Wel als het meerdere for-loops in elkaar zijn waar i gebruikt werd als tellertje én om dan door jagged arrays te lopen e.d....

Dan nog steeds niet. Uiteraard is het tweede tellertje dan j. Of x en y.

[MBV]

Als de buitenste loop meer dan 1 scherm groot is*, vind ik alsnog dat je betekenisvolle variabelenamen moet verzinnen. En in een 3d-applicatie kunnen x, y, en z heel betekenisvol zijn, als ze daadwerkelijk als coordinaten of 'zoveelste item op de x-as' worden gebruikt.

Ik merk trouwens wel dat met code-completion mijn variabelenamen gemiddeld wat langer zijn geworden

*) een functie mag 2 schermen zijn, maar liever niet natuurlijk.

[ Voor 8% gewijzigd door MBV op 16-10-2012 19:54 ]

[Grijze Vos]

2 schermen? wat mij betreft mag dat enkel en alleen als het gaat om heel simpele maar verbose code, zoals mapping code van datamodel naar viewmodel oid. Normaliter ligt de grens toch wel bij 1 scherm voor 1 functie.

[Styxxy]

Grijze Vos schreef op dinsdag 16 oktober 2012 @ 20:19:
2 schermen? wat mij betreft mag dat enkel en alleen als het gaat om heel simpele maar verbose code, zoals mapping code van datamodel naar viewmodel oid. Normaliter ligt de grens toch wel bij 1 scherm voor 1 functie.

Ik kom toch wel vaak aan 2 schermen, maar ik documenteer ook aardig wat (en uitgebreid). Zeker stukken die complexere logica (business rules die speciaal zijn), is het vaak wel goed om te documenteren. Ik kom nog regelmatig in delen van anderen die niets hebben gedocumenteerd; moet ik de lastenboeken met bijhorende X aantal analyse documenten erbij nemen om het te kunnen snappen.

[sig69]

MBV schreef op dinsdag 16 oktober 2012 @ 19:53:
...
*) een functie mag 2 schermen zijn, maar liever niet natuurlijk.

2 schermen zegt ook niks natuurlijk. Welke resolutie? Welk font size? Op mijn scherm wordt dat een flinke functie namelijk