Dubbio sul come formattare il codice

Vorrei sapere qual'è il metodo migliore per formattare questo script, ho provato nei 2 modi e funziona, ma qual'è il modo più corretto?

Questo css prende tutto:

 <span class="xxx">  
<script>	
//Feste dal 1-12 al 2-1		   	
function feste() {  
var calendario = new Date(); 
var giorno = calendario.getDate();
var mese = (calendario.getMonth()+1); // Gennaio=1	  
var nodate = 1;  
if (((giorno >=1) && (mese==4)) || ((giorno <=2) && (mese==1))) {
document.write('Buone Feste!'); var nodate = 0;
}
 } 	
feste();
  </script>
    </span>

Questo invece solo il richiamo:

<script>	
//Feste dal 1-12 al 2-1		   	
function feste() {  
var calendario = new Date(); 
var giorno = calendario.getDate();
var mese = (calendario.getMonth()+1); // Gennaio=1	  
var nodate = 1;  
if (((giorno >=1) && (mese==12)) || ((giorno <=2) && (mese==1))) {
document.write('Buone Feste!'); var nodate = 0;
}
 } 
</script>
 
    <span class="xxx">
      <script>	
        feste();
      </script>
    </span>

Ci sarebbe poi un altro modo che è quello di formattare direttamente il testo nel document.write:

document.write('<span class="xxx">Buone Feste!</span>'); var nodate = 0;

[mzanella]

La domanda non riguarda tanto la formattazione del codice, quanto piuttosto la struttura generale della pagina/script. Andiamo per gradi:

• hai definito una funzione, la cui invocazione scriverà del testo nella pagina: non importa dove la definisci, purché sia disponibile nello scope
• di conseguenza, colloca la definizione della funzione in un posto che sia "comodo" alla lettura del codice: certamente non mezzo all'HTML, piuttosto nello head o, meglio ancora, in un file JavaScript separato che potrai includere quando ne avrai necessità
• usa l'indentazione in modo ragionevole; non importa se usi 2 o 4 spazi, o la tabulazione, purché tu sia consistente (questa è la formattazione vera e propria), come nell'esempio in fondo
• meglio evitare parentesi ridondanti
• definisci una variabile locale nodate, la sovrascrivi dentro il corpo dell'if e poi non la usi: perché? Tanto vale rimuoverla
• in generale l'uso di document.write per generare dinamicamente contenuti non è una scelta molto popolare, meglio utilizzare le apposite funzioni per la manipolazione del DOM

Esempio

<script>
function feste() {
  var calendario = new Date();
  var giorno = calendario.getDate();
  var mese = calendario.getMonth() + 1;
  if ((giorno >= 1 && mese==12) || (giorno <= 2 && mese == 1)) {
    document.write('Buone Feste!');
  }
}
</script>
...
<span class="xxx">
  <script>
    feste();
  </script>
</span>

Ho messo così in un file e sembra funzionare:

<script>
function feste() {
  var calendario = new Date();
  var giorno = calendario.getDate();
  var mese = calendario.getMonth() + 1;
  if ((giorno >= 1 && mese==12) || (giorno <= 2 && mese == 1)) {
    document.write('<span class="feste">Buone Feste!</span>');
  }
}
 	
feste();
  </script>

[davidenessi]

Se scrivo il codice che avete postato ottengo questo errore di formattazione:

if ((giorno >= 1 && mese==12) || (giorno <= 2 &#038;#038)) {...}

come mai? Cosa posso fare per evitarlo?

Se scrivo il codice che avete postato ottengo questo errore di formattazione:

if ((giorno >= 1 && mese==12) || (giorno <= 2 &#038)) {...}

come mai? Cosa posso fare per evitarlo?

Non saprei, io l'ho messo in php

<?php
 //Feste dal 1-12 al 2-1        
   $giorno = date('d');
   $mese = date('m');
if (($giorno >= 1 && $mese == 12) || ($giorno <= 2 && $mese == 1))
   echo '<span class="feste">Buone Feste!</span>';
 ?>

[mzanella]

Se scrivo il codice che avete postato ottengo questo errore di formattazione:

if ((giorno >= 1 && mese==12) || (giorno <= 2 &#038)) {...}

come mai? Cosa posso fare per evitarlo?

A volte copiando e incollando pezzi di codice vengono inseriti caratteri o codifiche diversi da quelli attesi. Riscrivilo manualmente, almeno le parti di codice modificate.

[davidenessi]

L'ho riscritto manualmente ma mi da sempre lo stesso errore nella console quando avvio l'anteprima.

[mzanella]

Quale errore?

[davidenessi]

Ho scritto manualmente questo:

<script>
function feste(){
   var calendario = new Date();
   var giorno = calendario.getDate();
   var mese = calendario.getMonth() + 1;
   if ((giorno >=1 && mese == 5) || (giorno <= 2 && mese == 1)){
   document.write('<span class = "feste">Buone Feste!</span>');
}
}
feste();
</script>

Ma quando vado in anteprima nella console compare un errore Uncaught SyntaxError: Invalid or unexpected token e sembra che abbia trasformato && in &#038&#038. Come posso fare? Cosa sto sbagliando?

[mzanella]

Il codice di per se non ha problemi, probabilmente dipende dall'editor. Quale stai usando?

[davidenessi]

Sto inserendo il codice nell'editor integrato in altervista/wordpres per inserire gli articoli. Sto usando il blocco HTML personalizzato.

[mzanella]

Immagino che quegli editor siano pensati più per HTML che JavaScript, e quindi applichino la conversione dei caratteri speciali.
Se hai accesso a FTP prova a modificare il file localmente e caricarlo tramite FTP.

[alemoppo]

Hai cancellato la cache del tuo browser e quella di CloudFlare (se lo hai attivo?)

Ci puoi indicare la pagina così da vedere l'errore?

Ciao!

[davidenessi]

Ho caricato lo script via FTP ma ora non so come avviare lo script all'interno di un articolo.
Questo è il link alla pagina:

https://davidenessi.altervista.org/658-2/

[mzanella]

Ho caricato lo script via FTP ma ora non so come avviare lo script all'interno di un articolo.

Così:

<script src="percorso/dello/script/caricato.js"></script>

[alemoppo]

Se apri il file che hai caricato con blocco note, ti appare questo?

(giorno <= 2 && mese == 1)

Se no, riscrivi comunque così:

(giorno <= 2 && mese == 1)

Quindi salva e ricarica il file.

Ciao!

[davidenessi]

Grazie ho fatto così e ora funziona.