[JS] Array-value in verkorte if / else

[Verwijderd]

Ik gebruik de volgende code in mijn statistieken scriptje:

JavaScript:

var ie  = (agt.indexOf("msie") != -1);

var plugins = new Array();
plugins['acrobat']     = (ie) ? 'PDF.PdfCtrl.5' : 'application/pdf';
plugins['flash']       = (ie) ? 'ShockwaveFlash.ShockwaveFlash.1' : 'application/x-shockwave-flash';

Ik wil dit graag iets korter noteren:

JavaScript:

var ie  = (agt.indexOf("msie") != -1);

var plugins = [
 ['acrobat', ((ie) ? 'PDF.PdfCtrl.5' : 'application/pdf')],
 ['flash'  , ((ie) ? 'ShockwaveFlash.ShockwaveFlash.1' : 'application/x-shockwave-flash')]
];

Maar dit werkt niet. Kan iemand mij vertellen wat ik fout doe? Als ik een vaste waarde invul, doet 'ie het wel:

JavaScript:

var plugins = [
 ['acrobat', 'PDF.PdfCtrl.5'],
 ['flash'  , 'ShockwaveFlash.ShockwaveFlash.1']
];

[ Voor 4% gewijzigd door Verwijderd op 27-10-2007 00:52 ]

[Snake]

Als ik hier jouw code invoer, de middelste dus, met array in array, en ik gebruik Firebug, dan zie ik naderhand mooi de gevraagde inhoud in de var.

[ Voor 36% gewijzigd door Snake op 27-10-2007 01:00 ]

[crisp]

Gebruik geen array als hashmap maar een object. En dit is dan de juiste syntax:

JavaScript:

var plugins = {
    'acrobat': ie ? 'PDF.PdfCtrl.5' : 'application/pdf',
    'flash': ie ? 'ShockwaveFlash.ShockwaveFlash.1' : 'application/x-shockwave-flash'
}

Overigens is UA-sniffing hoogst onbetrouwbaar, dit is wel een mooie en betrouwbare manier om IE te detecteren:

JavaScript:

var ie = /*@cc_on!@*/false;

[ Voor 23% gewijzigd door crisp op 27-10-2007 09:17 ]

[Verwijderd]

@crisp - waarom werkt een object op deze manier wel en en array niet? Een array zo genoteerd werkte dan weer wel:

JavaScript:

var plugins = new Array();
plugins['acrobat']     = (ie) ? 'PDF.PdfCtrl.5' : 'application/pdf';
plugins['flash']       = (ie) ? 'ShockwaveFlash.ShockwaveFlash.1' : 'application/x-shockwave-flash';

Een hele originele manier om ie te detecteren trouwens. Dank je Ik heb altijd UA-sniffing gebruikt, omdat ik er vanuit ga dat echt niet zoveel mensen handmatig hun UA-string zullen veranderen...

[Verwijderd]

In hetzelfde statistiekenscriptje probeer ik de eventuele zoektermen van de referrer uit te lezen:

JavaScript:

detectSearch("http://www.google.com?q=dattum");

var sniffs = {
  '/google\./i' : 'q',
  '/yahoo\./i' : 'p'
}

function detectSearch(uri) {
  for (var sniff in sniffs) {
    if (uri.match(sniff)) {
      alert(getArg(uri, sniffs[sniff]));
    }
  }
}

function getArg(uri, arg) {
  var args = [];
  uri = uri.substr(uri.indexOf("?") + 1, uri.length);
  for (var i = 0; i < uri.split("&").length; i++) {
    if (uri.split("&")[i].split("=")[0] == arg) {
      return uri.split("&")[i].split("=")[1];
    }
  }
}

Maar ook hier gebeurt niets. Ik krijg geen error, maar wel een "searchterms undefined" boodschap. Wat doe ik nu weer verkeerd?

[JKVA]

Verwijderd schreef op zaterdag 27 oktober 2007 @ 14:09:
In hetzelfde statistiekenscriptje probeer ik de eventuele zoektermen van de referrer uit te lezen:

JavaScript:

detectSearch("http://www.google.com?q=dattum");

var sniffs = {
  '/google\./i' : 'q',
  '/yahoo\./i' : 'p'
}

function detectSearch(uri) {
  for (var sniff in sniffs) {
    if (uri.match(sniff)) {
      alert(getArg(uri, sniffs[sniff]));
    }
  }
}

function getArg(uri, arg) {
  var args = [];
  uri = uri.substr(uri.indexOf("?") + 1, uri.length);
  for (var i = 0; i < uri.split("&").length; i++) {
    if (uri.split("&")[i].split("=")[0] == arg) {
      return uri.split("&")[i].split("=")[1];
    }
  }
}

Maar ook hier gebeurt niets. Ik krijg geen error, maar wel een "searchterms undefined" boodschap. Wat doe ik nu weer verkeerd?

Geen idee, ik zie geen searchterms variabele in jouw code snippet.

[crisp]

Verwijderd schreef op zaterdag 27 oktober 2007 @ 12:53:
@crisp - waarom werkt een object op deze manier wel en en array niet? Een array zo genoteerd werkte dan weer wel:

JavaScript:

var plugins = new Array();
plugins['acrobat']     = (ie) ? 'PDF.PdfCtrl.5' : 'application/pdf';
plugins['flash']       = (ie) ? 'ShockwaveFlash.ShockwaveFlash.1' : 'application/x-shockwave-flash';

Omdat een array literal nu eenmaal deze syntax heeft:

JavaScript:

var array = [element0, element1, etc];

het is ook redelijk zinloos in javascript om voor associatieve keys een array te gebruiken; als je geen array-specifieke eigenschappen nodig hebt gebruik dan gewoon een object (een array is immers ook niet meer dan een object met wat specifieke eigenschappen en methods).
Daarbij wordt Array nog wel eens geprototyped wat weer problemen kan opleveren bij for-in constructs.

Een hele originele manier om ie te detecteren trouwens. Dank je Ik heb altijd UA-sniffing gebruikt, omdat ik er vanuit ga dat echt niet zoveel mensen handmatig hun UA-string zullen veranderen...

Aannames zijn de oorzaak van vele fuck-ups Opera heeft zich bijvoorbeeld heel lang standaard als IE geidentificeerd. Sowieso zal je moeten scannen op 'MSIE' en niet op 'msie'

Verwijderd schreef op zaterdag 27 oktober 2007 @ 14:09:
In hetzelfde statistiekenscriptje probeer ik de eventuele zoektermen van de referrer uit te lezen:

[...]

Maar ook hier gebeurt niets. Ik krijg geen error, maar wel een "searchterms undefined" boodschap. Wat doe ik nu weer verkeerd?

Ik denk dat je hier keylabels op een hele verkeerde manier gebruikt. Verder is een string natuurlijk geen regexp instance. Persoonlijk zou ik dit op zo'n manier doen:

JavaScript:

var snifs = [
    { regexp: /google\./i, arg: 'q' },
    { regexp: /yahoo\./i, arg: 'p' }
];

verder is je getArg functie zachtsgezegd nogal inefficient