[JS] Dynamisch adsense laden

zaterdag 28 november 2009 14:06

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

Hallo,

Ik probeer via een javascriptje google adsense dynamisch in te laden op een pagina. Dat wil zeggen:hoewel de adsense hoog in mijn content staat, wil ik de ads pas onderaan de pagina inladen. Dit versnelt het inladen van de content en zorgt ervoor dat mijn site ook als adsense down is beschikbaar blijft. Ik gebruik hiervoor het volgende scriptje:

code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

<html xmlns="http://www.w3.org/1999/xhtml" lang="nl" xml:lang="nl">

<head>
<title>Dynamic adsense test</title>

<script type="text/javascript"><!--
window.onload = function() {
var string = new Array();
var string2 = new Array();

string.push('<!--');
string.push('google_ad_client = \"pub-inseryourcodehere\";');
string.push('/* Blah blah */');
string.push('google_ad_slot = "adslotnumber";');
string.push('google_ad_width = 336;');
string.push('google_ad_height = 280;');
string.push('//-->');
var writestring = string.join('\n');

var adsDiv = document.getElementById('adsense_here');

adScript1 = document.createElement("script");
adScript1.type="text/javascript";
adScript1.text=writestring;
adsDiv.appendChild(adScript1);

adScript2 = document.createElement("script");
adScript2.type="text/javascript";
adScript2.src="http://pagead2.googlesyndication.com/pagead/show_ads.js";
adsDiv.appendChild(adScript2);
}
//-->
</script>
</head>

<body>
<div id="adsense_here" style="width: 500px; height:500px;"></div>

<img src="http://www.free-pictures-photos.com/construction/construction-glt6.jpg" />
</body>
</html>

Zoals je zelf zult zien, werkt dit alleen voor geen meter. Waarom niet? Wie kan mij helpen?

maandag 30 november 2009 16:15

Acties:

0 Henk 'm!

Juup

1. Je gebruikt een variabele met de naam "string". Dis is onhandig en verwarrend omdat String een reserved word is in javascript.
2. Op regel 26 zet je de .text van een <script/>. Waarom? Doe dan eval() of beter nog, run de code gewoon rechtstreeks.
3. Waarom append je de adScript2 aan adsDiv? het is beter om hem aan de head te appenden.
4. Je kunt op GoT je syntax laten coloren met [ code=html][ /code]
5. Waarom bouw je je string zo raar op met een array? Lijkt op een premature optimization.

[ Voor 10% gewijzigd door Juup op 30-11-2009 16:19 ]

Een wappie is iemand die gevallen is voor de (jarenlange) Russische desinformatiecampagnes.
Wantrouwen en confirmation bias doen de rest.

maandag 30 november 2009 16:50

Acties:

0 Henk 'm!

Bosmonster

*zucht*

Javascript

Waarom zet je dat script niet gewoon onderaan je pagina? Ik zie zo aan je voorbeeld geen reden om die helemaal dynamisch op te bouwen?

Kijk als voorbeeld eens naar de Google Analytics snippet.

[ Voor 18% gewijzigd door Bosmonster op 30-11-2009 16:50 ]

Onderwerpen