Discussion :

[sabrimi]

Bonjour à tous,

Au lieu de mettre une ligne texte de copyright dans chaque page de mon site, je me suis dit qu'il serait bien de le mettre dans un javascript.

J'ai essayé ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
var annee = parseInt('<?php echo date('Y'); ?>') ;
 
document.write('Copyright © 1998 - " + annee  + " Teams QI98 - Tous Droits Réservés</div>';);

Mais ça fonctionne pas.

J'ai pas d'erreur mais rien ne s'affiche.

Pourriez-vous me dire ce qui ne colle pas ?

Merci d'avance de vos réponses.

Cordialement
Sabrimi

[SpaceFrog]

Fais une simple recherche sur ce forum avec "document.write"

à part ça, il est ouvert ou le div ?

tu mélanges un peu hasardeusement js et php ...
ton passage de variable abbé à js est bon, mais son utilisation est douteuse
et ta hiérarchie de quote entre simples quotes ' et doubles quotes est juste un cauchemar

[Bovino]

Comment écrire un javascript impeccable ?

Certainement pas avec document.write() !

Mais je ne comprends vraiment pas pourquoi tu veux faire ça en JavaScript plutôt que directement en PHP, ça n'a aucun intérêt !

[sabrimi]

Merci les gars de votre réponse.

Pour te répondre Spacefrog, j'ai oublié d'enlever de div de fermeture.
J'ai rechercher pour le "document.write" mais j'ai pas trouvé l'exemple de texte qui pouvait inclure du php !!!
C'est vrai que c'est un vrai cauchemar... pour moi parce que je n'y connais rien en js.
Au départ j'ai trouvé ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
copyright=new Date();
update=copyright.getFullYear();
 
var copie;
	copie = "Copyright &copy 1998 - " + update + "  JL BOURDIN - Tous Droits Réservés ";
 
document.write( copie);

Alors je me suis demandé si "update" avait la même fonction que le code php que j'avais mis ?


--------------------------------

Pour te répondre aussi Bovino, j'avais fait ça en php et c'était impec.
Peut-être que ça n'a aucun intérêt, mais ça doit certainement en avoir puisque que je l'ai vu dans un code.

Cordialement
Sabrimi

[SpaceFrog]

mais ça doit certainement en avoir puisque que je l'ai vu dans un code.

J'en ai vu des codes pourris de toutes sortes !
Ce n'est pas pour cela qu'il faut les reproduire ...

En l'occurrence je confirme les dires de Beef
1- document.write
2- aucun interet de la faire en js ...

[sabrimi]

2- aucun interet de la faire en js ...

OK ! Alors, je vais laisser comme j'avais fait au départ.

Merci à tous.
Bonne journée.

Cordialement
Sabrimi

[Loceka]

Bon, pour expliquer un peu plus :
Il vaut mieux le faire en PHP parce que :

1. ça fait moins de donner à transférer sur le réseau
2. ça évite au navigateur un traitement inutile
3. c'est plus propre et tout aussi factorisable (<?php include 'copyright.php'; ?>)

Donc c'est pas impossible à faire en javascript mais ça n'a aucun intérêt.
Pour le faire propre je ferais un truc comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
function addCopyright() {
  var copyright = "Copyright © 1998 - " + (new Date()).getYear()  + " Teams QI98 - Tous Droits Réservés"
 
  var body = document.getElementsByTagName("BODY")[0];
  var copyrightDiv = document.createElement("DIV");
  copyrightDiv.setAttribute('id', 'copyright');
  var copyrightText = document.createTextNode(copyright);
 
  copyrightDiv.appendChild(copyrightText);
  body.appendChild(copyrightDiv);
}

[SylvainPV]

Mieux vaut ne même pas montrer de code en JS, surtout s'il affiche "'Copyright © 1998 - " + (new Date()).getYear() + " Teams QI98 - Tous Droits Réservés</div>'" en bas de ta page

[sabrimi]

Merci Loceka pour ce bout de code..

C'est vrai que c'est mieux que ce que j'avais fait !

Je vais me baser là-dessus parce que je pense que ce sera nettement mieux avec un include dans mes pages.

Merci encore pour cette exemple Loceka.

A bientôt.

Cordialement.
Sabrimi