Forum libero

- Scritto da:
> come si può migliorare questo codice?
> deve cercare le posizioni in cui una stringa
> (pattern senza espressioni regolari) in un'altra
> stringa
> data
>
> tipo come fa re.findall(), ma senza scomodare il
> motore che sta dietro il matcher delle regex che
> è
> lento.
>

> def allindices(string, sub, listindex=[],
> offset=0):
>     i = string.find(sub, offset)
>     while i >= 0:
>         listindex.append(i)
>         i = string.find(sub, i + 1)
>     return listindex
> 

il python fa cagare, ha l'indentazione obbligatoria

- Scritto da:
> come si può migliorare questo codice?
> deve cercare le posizioni in cui una stringa
> (pattern senza espressioni regolari) in un'altra
> stringa
> data
>
> tipo come fa re.findall(), ma senza scomodare il
> motore che sta dietro il matcher delle regex che
> è
> lento.
>

> def allindices(string, sub, listindex=[],
> offset=0):
>     i = string.find(sub, offset)
>     while i >= 0:
>         listindex.append(i)
>         i = string.find(sub, i + 1)
>     return listindex
> 

Questo codice e` buggato, il valore di default di un argomento "persiste".

Per il resto non c'e` molto che tu possa fare, supponendo che string.find (btw non e` un problema che una variabile si chiami come una classe?) ricostruisca l'automa ogni volta che la chiami potresti ottimizzare riusando lo stesso automa, ma se lo fai in python il guadagno non compensera` il rallentamento delle prestazioni rispetto all'implementazione nativa di find, tranne forse nel caso in cui sub sia molto lunga e piena di ripetizioni.