🚀 go-pugleaf

On Feb 26, 6:23 pm, Skiner <printers.man...@gmail.com> wrote:
> Witam,
>  w ramach nauki pythona,napisałem sobie prosty skrypt, który w podanym
> pliku:
> dla kazdej lini wyszukuje podciag i zamienia go  na inny:
>
> dlaprzykładu linia "ab"  jest zamieniana na "abc"  ale juz linia "abc"
> jest pomijana.
>
> i tak:
>
> pobieram linie , robie z niej liste,  sprawdzam kazdy element i ew
> zamieniam,
>
> nastepnie zapisuje do innego pliku. Problem w tym,  ze edytuje pliki z
> kodem zrodlowym,  i  moj sposob pozbawia mnie wszelkich wciec, kazda
> linia zaczyna się od poczatku wiersza :/  jak to obejsc?
>
> ponizej fragment kodu( na razie bez zapisu do pliku):
>
> fp =open('plik.c','r');
> line1="";
> while 1:
>     line =fp.readline();
>     if line=='': break;
>     else:
>       data =line.split();
>       for i in range(len(data)):
>         if data[i]=='ab':
>           data[i]='abc';
>         line1= "".join(data);
>         print line1;
> """ i teraz otwieram nowy plik, zapisuje linia po linii, niestety
> usuwa biale znaki:/ """
> fp.close();
>
> TIA

ok, ale powiedz skąd wziąłeś pomysł na to by właśnie tak kodować?

dobra, po kolei zewnętrzna pętla to "while" a nie lepiej wrzucić od
razu "for", i kod czytelniejszy i python sam pilnuje by było coś do
odczytania.
po drugie, robisz split na stringu - tu tracisz wszelkie informacje,
które masz - po prostu spacje są tu usuwane i wynikowa tablica
takowych nie zawiera.

jeżeli chcesz zamienić jeden string na drugi, zrób to używając
odpowiednich funkcji - dedykowanych.
np. replace line.replace('ab', 'abc')
(ten przykład zamienia wystąpienia ab, wszędzie, a nie tylko jako
wyraz!)

oczywiście jeżeli potrzeba troszeczkę więcej elastyczności - to już
lepiej użyć modułu re

czyli:
In [1]: a = '    ab abc ocabcas'

In [2]: a.replace('ab', '123')
Out[2]: '    123 123c oc123cas'

In [3]: import re

In [4]: re.sub(r'ab', '123', a)
Out[4]: '    123 123c oc123cas'

In [5]: re.sub(r'\wab\w', '123', a)
Out[5]: '    ab abc o123as'

następna rzecz, to to, że masz zmienną która cały czas puchnie, przy
dużym pliku (większym od dostępnej pamięci i swapa) wywali to proces.
lepiej od razu zapisywać co potrzeba.

czyli taka poprawiona troszeczkę wersja:

import re
fp =open('plik.c','r')
fo = open('plik_nowy.c', 'w')
for line in fp.readlines():
    fo.write(re.sub(r'\wab\w', 'abc', line)
fp.close()
fo.close()

mniejsze, czytelniejsze i pythonowe ;)

p.s.
średników nie daje się na końcu ;)

p.p.s.
znaki końca linii (\n) były nietknięte, więc nie trzeba było ich
nigdzie dodawać.



pozdrawiam,
J