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:

Codice:

---

<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:

Codice:

---

<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:

Codice:

---

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

---

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

Codice HTML:

---

<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:

Codice HTML:

---

<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>

---

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?

Citazione:

---

Originalmente inviato da davidenessi

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

Codice 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>';
?>

---

Citazione:

---

Originalmente inviato da davidenessi

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.

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

Quale errore?

Ho scritto manualmente questo:

Codice:

---

<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?

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

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

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.

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!

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/

Citazione:

---

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

---

Così:

Codice HTML:

---

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

---

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

Codice:

---

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

---

Se no, riscrivi comunque così:

Codice:

---

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

---

Quindi salva e ricarica il file.

Ciao!

Grazie ho fatto così e ora funziona.