Ciao,

purtroppo non andava bene

Ho preparato una demo funzionante, vediamo se questa ti aiuta.
https://dotnetfiddle.net/KAAOlO

Nella demo, prima elimino l'occorrenza e poi rinumero quelle rimanenti.

il mio problema sta se cancello una che non sia l'ultima o se addirittura cancello proprio la prima
devo rinominare tutto

Beh, non è un problema. Prima elimini, poi rinumeri le restanti.
Ho l'impressione che tu stia provando a rinumerare prima di aver effettivamente rimosso le occorrenze.

Per la rimozione/aggiunta delle domande il problema non si pone perché la numerazione - da quanto ho capito - riguarda solo le occorrenze.
Il fatto che la proprietà "Occurrence" si trovi sulla domanda potrebbe essere un errore di modellazione. Nella demo infatti ho la proprietà "Numero" sull'oggetto "Occorrenza", che fa da contenitore per le domande, e non sulle domande stesse.

ciao,
Moreno

P.S.

certo é che io avrò rifatto 5 volte le funzione di controllo e sempre qualcosa non va bene a secondo di quello che cancello :(

Questo si poteva evitare usando il test driven development e/o spezzando il problema nelle sue parti elementari.
Ad esempio, questa funzionalità di editing avresti potuto implementarla con una task-based UI: se l'utente clicca il bottoncino "Elimina" per rimuovere l'occorrenza, tu fai una richiesta ajax per rimuovere quella e rinumerare le restanti. Poi l'utente aggiunge una domanda: fai un'altra richiesta ajax per aggiungerla al database. E così via: ogni operazione compiuta dall'utente che modifica il questionario corrisponde ad una singola richiesta ajax.

Comunque, rifarlo 5 volte può capitare a chi è costretto ad imparare da solo. Anch'io ho avuto momenti così. Avere un mentore che ti guida attraverso la formazione è un privilegio di pochi.

---

Modificato da BrightSoul il 27 agosto 2018 22.38 -

 foreach (var sec in NewModel.Sections)
            {
                bool prima = false;
                int old = -1;
                int contOcc = 0;
                int occAttuale = 0;

                foreach (var quest in sec.Questions)
                {

                    

                    if (old == -1)
                    {
                        old = 0;
                    }
                    else
                    {

                        old = occAttuale ;
                       
                       
                    }

                    occAttuale = quest.Occurance;
                   


                    if (old != occAttuale )
                    { 

                        if (prima == false && occAttuale > 0)
                        {
                            old = occAttuale ;
                            contOcc = 0;
                        }
                        else
                        {
                           
                            contOcc++;
                        }
                           
                    }

                    prima = true;

                    quest.Occurance = contOcc;
                }
            }