[C#]Errore operazioni su string

[Mattyfog]

Sto eseguendo questo codice su una string senza che mi compaia mai la message box di errore.
[code]
for(int a; paginahtml.IndexOf("<img") != -1; a++)
{
a = paginahtml.IndexOf("<img");

while(true)
{
a++;

if(paginahtml[a].ToString() == ">" & paginahtml[a-1].ToString() != @"\")
{
paginahtml.Insert(a - 1, @"\");
break;
}

if (a > paginahtml.Length)
MessageBox.Show("Errore", "Errore nella codifica del file xml", MessageBoxButton.OK, MessageBoxImage.Error);
}
}
[code]
Ricordo che la string contiene un file xml da modificare per poter eseguire il parsing senza errori.


In compenso ricevo su visual studio questo bel messaggio dopo un po' che il programma è in esecuzione e che non risponde:Impossibile eseguire la transizione dal contesto COM 0x3e83f8 al contesto COM 0x3e8568 per 60 secondi. Probabilmente il thread a cui appartiene il contesto/apartment di destinazione è impegnato nell'attesa senza distribuzione o nell'elaborazione di un'operazione a esecuzione prolungata che non prevede la distribuzione di messaggi Windows. Questa situazione in genere ha un impatto negativo sulle prestazioni e può causare una mancata risposta dall'applicazione o un accumulo continuato nel tempo dell'utilizzo della memoria. Per evitare questo problema, è necessario che durante le operazioni a esecuzione prolungata tutti i thread in modalità STA (Single Thread Apartment) utilizzino primitive di attesa con distribuzione (ad esempio CoWaitForMultipleHandles) e messaggi distribuiti in base a routine.

[||ElChE||88]

string.IndexOf(string value)
restituisce sempre il primo indice dell'occorrenza.
Se vuoi evitare il loop infinito devi usare
string.IndexOf(string value, int startIndex)
che cerca la sottostringa partendo da una determinata posizione.

[Mattyfog]

grazie domani provo ;)
Non ci avevo pensato :P

[Mattyfog]

Ora l'ho modificato così:

for (int a, b = 0; paginahtml.IndexOf("<img", b) != -1; b++)
{
a = paginahtml.IndexOf("<img", b);

while(true)
{
a++;

if(paginahtml[a].ToString() == ">" & paginahtml[a-1].ToString() != @"\")
{
paginahtml.Insert(a - 1, @"\");
break;
}

if (a > paginahtml.Length)
MessageBox.Show("Errore", "Errore nella codifica del file xml", MessageBoxButton.OK, MessageBoxImage.Error);
}
}

Il problema è che non inserisce mai il carattere '\' prima di '>'.
Eppure ho controllato ed indexof è != da -1 per cui dovrebbe partire il ciclo while e poichè dopo i caratteri '<img' sicuramente (ho controllato nel testo) c'è un carattere '>' sulla stessa riga dovrebbe venire modificato inserendo prima di quest'ultimo carattere un '/'.
Invece non succede. Perchè?

[astorcas]

Il tuo problema è che stai cercando di modificare una stringa, cosa che non puoi fare in quanto in C# le stringhe sono non modificabili.


paginahtml=paginahtml.Insert(a - 1, @"\");

[Mattyfog]

Grazie :)
Il problema direi è che avrei bisogno di un bel libro..

[Mattyfog]

Ok,
comunque ho modificato ancora il codice così:

for (int a = 0, b = 0; paginahtml.IndexOf("<img", b) != -1; b = a+1)
{
a = paginahtml.IndexOf("<img", b);

while(true)
{
a++;

if(paginahtml[a].ToString() == ">" & paginahtml[a-1].ToString() != @"\")
{
paginahtml = paginahtml.Insert(a - 1, @"\");
break;
}

if (a > paginahtml.Length)
MessageBox.Show("Errore", "Errore nella codifica del file xml", MessageBoxButton.OK, MessageBoxImage.Error);
}
}

Eppure paginahtml rimane identica anche a fine processo. In pratica questo script no fa niente.. :cry:

[astorcas]

non ho chiaro cosa vuoi fare però dando in pasto la stringa

<img src="pippo.gif"/>

ottengo

<img src="pippo.gif"\/>

qualcosa fa :)

[Mattyfog]

No scusa avevo scritto 'na cavolata ho sotituito @'\' con @'/'

for (int a = 0, b = 0; paginahtml.IndexOf("<img", b) != -1; b = a+1)
{
a = paginahtml.IndexOf("<img", b);

while(true)
{
a++;

if(paginahtml[a].ToString() == ">" & paginahtml[a-1].ToString() != @"/")
{
paginahtml = paginahtml.Insert(a - 1, @"/");
break;
}

if (a > paginahtml.Length)
MessageBox.Show("Errore", "Errore nella codifica del file xml", MessageBoxButton.OK, MessageBoxImage.Error);
}
}


Dando in pasto la stringa
<img src="pippo.gif">
dovrei ottenere
<img src="pippo.gif"/>

[astorcas]

paginahtml = paginahtml.Insert(a-1, @"/");

se la cambio con

paginahtml = paginahtml.Insert(a, @"/");

fa quello che dici.

Ti consiglio però, quando hai tempo, di guardarti un po' le regular expressions, renderesti molto più facile la cosa :)

[Mattyfog]

regular expression dici ad esempio string.Replace()?
il punto è che la cosa non è applicabile all'intero documento per cui non posso usarla..
se invece parli di altro allora vado a guardarlo :)