diff --git a/changes.txt b/changes.txt new file mode 100644 index 0000000..b8f34d1 --- /dev/null +++ b/changes.txt @@ -0,0 +1,12 @@ +1. zmiana nazw iteratorów +2. usunięcie nadmiarowej funkcji +3. usunięcie nadmiarowych printów, oraz nadmiarowych słów z printów(np something went horribly wrong) +4. przeniesienie import na górę kodu i zmiana z importowania wszyskiego za pomocą "*" na import konktetnych funkcji +5. poprawa ogólnej estetyki kodu (wcięcia w odpowiednich miejscach, odpowiednie ilości odstępu między blokami kodu) +6. zmiana nazwy funkcji na bardziej zrozumiałą +7. wstawienie pętli for zamiast kilku takich samych operacji pod rząd +8. ususnięcie zmiennych globalnych +9. ujednolicenie nazw zmiennych +10. poprawienie komentarza, aby nie był jedną długą linijką, oraz skrócenie go by był bardziej zrozumiały +11. poszerzenie try-except +12. zmiana -1 na ":" przy zapisie pliku diff --git a/file157.py b/file157.py index 649c872..dfdbb2d 100644 --- a/file157.py +++ b/file157.py @@ -1,30 +1,37 @@ +from file87 import piContainer, calculate + + def enumerate(pi): - for hello in pi.a: - print(f'Next element: {hello}') -from file87 import * + for i in pi.a: + print(f'Next element: {i}') + + try: - my_pi = pi_container() - pi_gen = foo(5) - my_pi.mth(pi_gen.__next__()) - my_pi.mth(pi_gen.__next__()) - my_pi.mth(pi_gen.__next__()) - my_pi.mth(pi_gen.__next__()) - my_pi_2 = pi_container() - my_pi.mth(pi_gen.__next__()) - my_pi.mth(pi_gen.__next__()) - my_pi.mth(pi_gen.__next__()) + my_pi = piContainer() + pi_gen_1 = calculate(5) + + for i in range(6): + my_pi.mth(pi_gen_1.__next__()) + + my_pi_2 = piContainer() + + pi_gen_2 = calculate(194) + for i in range(23): + my_pi_2.mth(next(pi_gen_2)) + + my_pi_3 = piContainer() + pi_gen_3 = calculate(6) + my_pi_3.mth([i for i in list(pi_gen_3)]) + + print('my first pi') + enumerate(my_pi) + + print('my second pi') + enumerate(my_pi_2) + + new_file = open('pi_approximation.txt', 'w') + new_file.write(f'my best pi: {my_pi_3.a[:]}') + new_file.close() + except: - print('something went horribly wrong :(') -pIgEn3 = foo(194) -for the_variable_that_contains_next_approximations_of_pi_from_generator in range(23): - my_pi_2.mth(next(pIgEn3)) -my_pi_3 = pi_container() -pi_gen = foo(6) -my_pi_3.mth([i for i in list(pi_gen)]) -print('my first pi') -enumerate(my_pi) -print('my second pi') -enumerate(my_pi_2) -new_file = open('some-file.txt', 'w') -new_file.write(f'my best pi: {my_pi_3.a[-1]}') -new_file.close() + print('something went wrong') diff --git a/file87.py b/file87.py index 40e209e..881d9b7 100644 --- a/file87.py +++ b/file87.py @@ -1,8 +1,5 @@ -b = 0 -c = 0 - -class pi_container: +class piContainer: def __init__(self, a=list()): self.a = a @@ -11,25 +8,17 @@ def mth(self, x): self.a += x else: self.a.append(x) - - -def foo(x): - global b - global c +# this is a important operation in calculating pi that is +# provided in a separate file in this repository, +# please analyse this file before using! +def calculate(x): b = 0 - c=1 - for hello in range(x): - if hello % 2 == 0: - b += 4 / c#this is a very important operation in calculateing pi according to documentation that is provided in a seperate file in this repository, please analyse this file before using! + c = 1 + for i in range(x): + if i % 2 == 0: + b += 4 / c else: b -= 4 / c c += 2 yield b yield 'finished' - -def enumerate(pi: pi_container): - for hello in pi.a: - print(hello) - - -print('All functions are defined')