Требуется оптимизировать код

@makgeer · Регистрация: 24.04.2018

Студворк — интернет-сервис помощи студентам

Всем, доброго времени суток. Вот наверстал такой код!)))) Вопрос следующий: в каком направлении теперь двигаться, для оптимизации написанного. Хочу привести код к более читабельному виду. Просьба, без стеба).

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
#-*-coding:-utf-8-*-
 
var = int(input('1: Ru - Eng\n2: Eng - Ru\n'))
 
if var == 1:
    print('Конвертация из метрической в неметрическую')
    con = int(input('Что считаем: \n 1. Температура\n 2. Вес\n 3. Длина\n 4. Объем\n 5. Скорость\n'))
# Считаем температуру
    con == 1
    while con == 1:
        if con == 1:
            temperature = int(input(' 1. Цельсий -> Фаренгейт\n 2. Цельсий -> Кельвин\n'))
            if temperature == 1:
                value_temperature = float(input('Сколько градусов по Цельсию?:\n'))
                print(value_temperature, 'градусов по Цельсию составляют', round((((9/5) * value_temperature +32)), 2),'градусов по Фаренгейту')
            elif temperature == 2:
                value_temperature = float(input('Сколько градусов по Цельсию? :\n'))
                print(value_temperature, 'градусов по Цельсию составляют', round((value_temperature + 273.15), 2),'градусов по Кельвину')
            return(var)
    # Считаем вес
    con == 2
    while con == 2:
        if con == 2:
            mas = int(input(' 1. Килограммы -> фунты\n 2. Граммы -> унции \n'))
            if mas == 1:
                value_mas = float(input('Сколько килограмм? :\n'))
                print(value_mas, 'килограмм составляют', round((value_mas * 2.20462), 2), 'фунтов')
            elif mas == 2:
                value_mas = float(input('Сколько грамм? :\n'))
                print(value_mas, 'грамм составляют', round((value_mas * 0.035274), 2), 'унций')
            continue
    print (var)
               
    # Считаем длину
    con == 3
    while con == 3:
        if con == 3:
            dlina = int(input(' 1. Километры -> мили\n 2. Метры -> Ярды\n 3. Метры -> футы\n 4. Сантиметры -> дюймы\n 5. Миллиметры -> дюймы\n'))
            if dlina == 1:
                value_dlina = float(input('Сколько километров? :\n'))
                print(value_dlina, 'километров составляют', round((value_dlina * 1.60934), 2),'миль')
            elif dlina == 2:
                value_dlina = float(input('Сколько метров? :\n'))
                print(value_dlina,'метров составляют', round((value_dlina * 1.09361), 2), 'ярдов')
            elif dlina == 3:
                value_dlina = float(input('Сколько метров? :\n'))
                print(value_dlina,'метров составляют', round((value_dlina / 0.3048), 2), 'футов')
            elif dlina == 4:
                value_dlina = float(input('Сколько сантиметров? :\n'))
                print(value_dlina,'сантиметров составляют', round((value_dlina * 2.5400013716), 2), 'дюймов')
            elif dlina == 5:
                value_dlina = float(input('Сколько милиметров? :\n'))
                print(value_dlina,'миллиметров составляет', round((value_dlina / 0.0393701), 2) ,'дюймов')
            continue
    print (var)            
# Считаем объем
    con == 4
    while con == 4:
        if con == 4:
            space = int(input(' 1. Литры -> галлоны, \n 2. Литры -> пинты\n'))
            if space == 1:
                value_space = float(input('Сколько литров? : '))
                print(value_space,'литров составляет', round((value_space / 3.785411784), 2), 'галлонов')
            elif space == 2:
                value_space = float(input('Сколько литров? :\n'))
                print(value_space, 'литров составляет', round((value_space / 0.56826125), 2), 'пинт')
            elif convertation == 5:
                speed = float(input(' Сколько километров в час? : '))
                print(speed, 'километров в час составляет ', round((speed / 1.60934), 2), 'миль в час')
 
if var == 2:
    print('Конвертация из неметрической в метрическую')
    con = int(input('Что считаем: \n 1. Температура\n 2. Вес\n 3. Длина\n 4. Объем\n 5. Скорость\n '))
 
# Считаем температуру
    con == ()
    while con == 1:
        if con == 1:
            temperature = int(input(' 1. Фаренгейт -> Цельсий\n 2. Кельвин -> Цельсий\n'))
            if convertation == 1:
                value_temperature = float(input('Сколько градусов по Фаренгейту? :\n'))
                print(value_temperature, 'градусов по Фаренгейту = ', round(((5/9) * value_temperature - 32), 2), ' по Цельсию')
            elif convertation == 2:
                value_temperature = float(input('Сколько градусов по Кельвину? :\n'))
                print(value_temperature, 'градусов по Кельвину = ', value_temperature - 273.15, ' по Цельсию')
            continue
# Считаем вес
    con == 2
    while con == 2:
        if con == 2:
            massa = int(input(' 1. Фунты - килограммы\n 2. Унции - граммы\n'))
            if massa == 1:
                value_massa = float(input('Сколько фунтов? :\n'))
                print(value_massa, 'фунтов составляет', round((value_massa * 0.453592), 2),'килограмм')
            elif massa == 2:
                value_massa = float(input('Сколько унций? :\n'))
                print(value_massa, 'унций составляет', round((value_massa * 28.3), 2),'граммов')
            continue
# Считаем длину
    if con == 3:
        dlina = int(input(' 1. Мили -> километры\n 2. Ярды -> метры\n 3. Футы -> метры\n 4. Дюймы -> сантиметры\n'))
        if dlina == 1:
            value_dlina = float(input('Сколько миль? :\n'))
            print(value_dlina, 'миль =', round((value_dlina / 0.621371), 2), 'километров')
        elif dlina == 2:
            value_dlina = float(input('Сколько ярдов? :\n'))
            print(value_dlina, 'ярдов =', round((value_dlina / 0.9144), 2), 'метров')
        elif dlina == 3:
            value_dlina = float(input('Сколько футов? :\n'))
            print(value_dlina, 'футов =', round((value_dlina / 3.28084), 2), 'метров')
        elif dlina == 4:
            value_dlina = float(input('Сколько дюймов? :\n'))
            print(value_dlina, 'дюймов =', round((value_dlina / 0.393701), 2), 'сантиметров')
 
# Считаем объем
    if con == 4:
        space = int(input(' 1. Галлоны -> литры\n 2. Пинты -> литры\n'))
        if space == 1:
            value_space = float(input('Сколько галлонов? : \n'))
            print(value_space,'галлонов =', round((value_space * 3.785411784), 2), 'литров')
            if space == 2:
                value_space = float(input('Сколько пинт? : \n'))
                print(value_space, 'пинт =', round((value_space * 0.473176473), 2), 'литров')
 
# Считаем скорость
    if con == 5:
        speed = float(input('Сколько миль в час? :\n'))
        print(speed,'миль в час =', round((speed * 1.60934), 2),'километров')

Добавлено через 2 минуты
Прошу прощения, в коде ошибки, забыл убрать перед выкладкой.
Не обращайте внимания на окончания циклов!!!!

@Welemir1 · 24.09.2019, 16:42

твою же м...
ну ладно. до главы функции как я понимаю не дочитал...

Сообщение от makgeer

Вот наверстал

это тебе в хтмл

1) разбить код на выполняемые задачи и оформить их функциями
2) все повторяющееся - в функции
3) можно сваять общую функцию для выбора, которой передаем список возможных параметров

@makgeer · 24.09.2019, 17:13 **[ТС]**

Главу функции прочитал)), вот думаю как это теперь приводить к единому....
Я так и думал, сделать единую функцию обработки ввода, а в зависимости от требуемого, вызывать функцию конвертирования))). Спасибо за помощь)! Прошу прощения за изврат в коде. Только учусь!

@Alli_Lupin · 24.09.2019, 17:16

makgeer, учитесь. Когда что-то получится, присылайте результаты сюда. Мы снова посмотрим, покритикуем и посоветуем.

@makgeer · 24.09.2019, 22:09 **[ТС]**

😂 как закончу с функциями, сразу выложу)).

@makgeer · 26.09.2019, 16:42 **[ТС]**

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#-*-coding:utf-8-*-
def Foren():
    if temperature == 1:
        i = (int(input('ВВедите градусы С:\n')))
        print("Получаем:" + str(i*1.8+32))
    elif temperature == 2:
        i = (int(input('ВВедите градусы С:\n')))
        print('Получаем:' + str(i+273.15)) 
 
var = int(input('Что считаем:\n 1.RU/ENG\n 2.ENG/RU\n'))
# Choose a conversion system
if var == 1:
    print('Конвертация из метрической в Аглийскую:')
    con = int(input('Выбирете категорию:\n 1. Температура\n 2. Вес\n 3. Длина\n 4. Объем\n 5. Скорость\n'))
    if con == 1:
        temperature = (int(input('Выбирете единицы отображения:\n 1.Forengate\n 2.Kelvin\n')))
     
    print Foren()

Полагаю, как-то так все должно быть оформлено?

Добавлено через 7 минут

Сообщение от Welemir1

это тебе в хтмл
1) разбить код на выполняемые задачи и оформить их функциями
2) все повторяющееся - в функции
3) можно сваять общую функцию для выбора, которой передаем список возможных параметров

Вроде начинает получаться)!?

@Welemir1 · 26.09.2019, 16:42

Сообщение от makgeer

print Foren()

у тебя питон 2 чтоли? в третьем print()

функция Форен плохая!
1) название сделай понятнее, типа конвертер температуры (на английском конечно)
2) названия функций с маленькой буквы
3) функция зависит от внешних параметров (температура) - это плохо, принимай ее в аргументах
4) функция ничего не возвращает - возвращай результат, ничего не выводи, пусть выводит тот кому результат нужен

но в целом уже лучше

@makgeer · 26.09.2019, 20:16 **[ТС]**

Угу!

Добавлено через 3 часа 2 минуты

Сообщение от makgeer

temperature = int(input(' 1. Цельсий -> Фаренгейт\n 2. Цельсий -> Кельвин\n'))

Я правильно понимаю, в строке представлен кортеж? Или все же список?

Добавлено через 1 минуту
У меня питон 3.7

@Semen-Semenich · 26.09.2019, 20:31

Сообщение от makgeer

Я правильно понимаю, в строке представлен кортеж? Или все же список?

в строке представлена строка, но вы превели её к типу int значить это уже число. через инпут вы не введете кортеж или список

@makgeer · 26.09.2019, 20:37 **[ТС]**

Спасибо))

Добавлено через 2 минуты
Я же могу вызвать из функции часть кода, а не функцию целиком, т.е. обратиться к одному аргументу в функции?

@Damenikx · 26.09.2019, 20:39

makgeer, ммм, не совсем

Добавлено через 51 секунду
makgeer, если вызываешь функцию срабатывает весь код который в ней.

@Semen-Semenich · 26.09.2019, 20:39

нет. все что в функции это локально для функции и снаружи переменные не видны. если вы хотите такое сделать значить у вас неверно построен код

@makgeer · 26.09.2019, 20:44 **[ТС]**

Python
1
2
3
4
5
def coint():
                 forengate=(...)
                 Kelvin=(...)
              code of programm
              print forengate()

будет ли работать такая конструкция?

Добавлено через 1 минуту
Принял((, аргументы значить....
Будем думать))))))

Добавлено через 3 минуты
То есть в самой функции должно быть описано, что делать, если к тебе пришли!
Значит в самой функции я могу расписать ветвления, а потом просто написать простой код, с вызовом этой функции? Это правильно?

@Damenikx · 26.09.2019, 20:46

makgeer, не хочу перетягивать тебя на тёмную сторону, но пока никто не видит:

мммглобальные переменные, только тссс, а то меня поругают.

Добавлено через 1 минуту
Только сразу скажу, глобальные переменные замедляют работу программы.

@makgeer · 26.09.2019, 20:48 **[ТС]**

Я думал уже о ней, но в моем случае, я могу нарваться на бесконечный цикл, очень не хочется))
Хотя очень заманчиво))))))))))

@Damenikx · 26.09.2019, 21:01

Не по теме:

makgeer, псс, парень, хочешь немного глобалок?

@Welemir1 · 27.09.2019, 07:32

Сообщение от makgeer

Значит в самой функции я могу расписать ветвления, а потом просто написать простой код, с вызовом этой функции? Это правильно?

да, ты начинаешь понимать суть функций, если сможешь делать их независимыми и маленькими, не использовать глобалок, то станешь хорошим программистом.

@makgeer · 28.09.2019, 20:28 **[ТС]**

Что-то начитает вырисовываться:

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#-*-coding:utf-8-*-
def cointforengate():
    i = (int(input('ВВедите градусы С:\n')))
    return("Получаем:" + str(i * 1.8 + 32))
def cointkelvin ():
    i = (int(input('ВВедите градусы С:\n')))
    return("Получаем:" + str(i + 273.15))
 
var = int(input('Что считаем:\n 1.RU/ENG\n 2.ENG/RU\n'))
# Choose a conversion system
if var == 1:
    print('Конвертация из метрической в Аглийскую:')
    con = int(input('Выбирете категорию:\n 1. Температура\n 2. Вес\n 3. Длина\n 4. Объем\n 5. Скорость\n'))
    if con == 1:
        temperature = (int(input('Выбирете единицы отображения:\n 1.Forengate\n 2.Kelvin\n')))
    if temperature == 1:
        print cointforengate()
    elif temperature == 2:
        print cointkelvin()

Добавлено через 1 минуту
А как сделать, чтобы программа не заканчивала работу после вывода функции?)))

@makgeer · 04.10.2019, 09:24 **[ТС]**

Подскажите, пожалуйста, как можно оптимизировать функции и возможно ли это? Был бы очень признателен за совет.

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
#-*-coding:utf-8-*-
# Cointing temperature
def cointemper():
    if temperature == 1:
        i = (float(input('Введите градусы С:\n')))
        return("Получаем:" + str(i*1.8+32))
    if temperature == 2:
        i = (float(input('Введите градусы С:\n')))
        return('Получаем:' + str(i+273.15))
# cointing mass
def coinmass():        
    if mass == 1:
        i = (float(input('Ввделите граммы:\n')))
        return('Получаем:' + str(i*0.035273998))
    if mass == 2:
        i = (float(input('Введите килограммы:\n')))
        return('Получаем:' + str(i*2.2046))
    if mass == 3:
        i = (float(input('Введите килограммы:\n')))
        return('Получаем:' + str(i*0.157473))
    if mass == 4:
        i = (float(input('Введите килограммы:\n')))
        return('Получаем:' + str(i/50.6))
    if mass == 5:
        i = (float(input('Введите килограммы:\n')))
        return('Получаем:' + str(i/45.3))
    if mass == 6:
        i = (float(input('Введите килограммы:\n')))
        return('Получаем:' + str(i/907.18))
    if mass == 7:
        i = (float(input('Введите килограммы:\n')))
        return('Получаем:' + str(i/1016))
#
def coinlght():
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())
    if lght == 1:
        i = (float(input('\n')))
        return('' + str())  
#general code                              
# Choose a conversion system
var = int(input('Что считаем:\n 1.RU/ENG\n 2.ENG/RU\n'))
if var == 1:
    print('Конвертация из метрической в Аглийскую:')
#Temperatura
    con = int(input('Выбирете категорию:\n 1. Температура\n 2. Вес\n 3. Длина\n 4. Объем\n 5. Скорость\n'))
    if con == 1:
        temperature = int(input('Выбирете единицы отображения:\n 1.Forengate\n 2.Kelvin\n'))
        if temperature == 1:
            print cointemper()
        elif temperature == 2:
            print cointemper()
    if con == 2:
#Mass
        mass = int(input('Выбирете единицы отображения:\n 1.OZ\n 2.Pound\n 3.Stone\n 4.Centner(GB)\n 5.Centner(USA)\n 6.Short tonn\n 7.Long tonn\n'))
        if mass == 1:
            print coinmass()
        elif mass == 2:
            print coinmass()
        elif mass == 3:
            print coinmass()
        elif mass == 4:
            print coinmass()
        elif mass == 5:
            print coinmass()
        elif mass == 6:
            print coinmass()
        elif mass == 7:
            print coinmass()
    if con == 3:
#Length
        lght = int(input('Выберите категорию:\ 1.Foot\n 2.Yard\n 3.Furlong\n 4.Mile\n 5.Naut. mile\n 6.Inch\n 7.Line\n 8.Point\n'))
        if lght==1:
            print

@Welemir1 · 04.10.2019, 09:43

makgeer, на примере

Сообщение от makgeer

cointemper():

а что если ввод принимать сразу? и сразу во флоат приводить? и только потом, в зависомти от условия возвращать варианты ответов?

а это чего вообще? после первой проверки lght остальные условия никогда не выполнятся ибо они аналогичны!

Сообщение от makgeer

if lght == 1:
* * * * i = (float(input('\n')))
* * * * return('' + str())
* * if lght == 1:
* * * * i = (float(input('\n')))
* * * * return('' + str())
* * if lght == 1:
* * * * i = (float(input('\n')))
* * * * return('' + str())
* * if lght == 1:
* * * * i = (float(input('\n')))
* * * * return('' + str())
* * if lght == 1:

в чем бы ты не писал, переходим на ПайЧарм, она подчеркнет такие косяки (с недостижимыми условиями)

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит переходные токи и напряжения на элементах схемы. . . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

Требуется оптимизировать код

Решение

Решение

Решение

@makgeer 2 / 2 / 0 Регистрация: 24.04.2018 Сообщений: 45 Записей в блоге: 4
	24.09.2019, 17:13 [ТС]
	Главу функции прочитал)), вот думаю как это теперь приводить к единому.... Я так и думал, сделать единую функцию обработки ввода, а в зависимости от требуемого, вызывать функцию конвертирования))). Спасибо за помощь)! Прошу прощения за изврат в коде. Только учусь! 0

@Alli_Lupin 5036 / 1063 / 149 Регистрация: 29.01.2013 Сообщений: 6,209
	24.09.2019, 17:16
	makgeer, учитесь. Когда что-то получится, присылайте результаты сюда. Мы снова посмотрим, покритикуем и посоветуем. 1

@makgeer 2 / 2 / 0 Регистрация: 24.04.2018 Сообщений: 45 Записей в блоге: 4
	24.09.2019, 22:09 [ТС]
	😂 как закончу с функциями, сразу выложу)). 0

@makgeer 2 / 2 / 0 Регистрация: 24.04.2018 Сообщений: 45 Записей в блоге: 4
	26.09.2019, 20:37 [ТС]
	Спасибо)) Добавлено через 2 минуты Я же могу вызвать из функции часть кода, а не функцию целиком, т.е. обратиться к одному аргументу в функции? 0

@Damenikx 955 / 341 / 114 Регистрация: 04.08.2018 Сообщений: 2,533
	26.09.2019, 20:39
	makgeer, ммм, не совсем Добавлено через 51 секунду makgeer, если вызываешь функцию срабатывает весь код который в ней. 1

@Semen-Semenich 5222 / 3469 / 1173 Регистрация: 21.03.2016 Сообщений: 8,295
	26.09.2019, 20:39
	нет. все что в функции это локально для функции и снаружи переменные не видны. если вы хотите такое сделать значить у вас неверно построен код 1

@makgeer 2 / 2 / 0 Регистрация: 24.04.2018 Сообщений: 45 Записей в блоге: 4
	26.09.2019, 20:48 [ТС]
	Я думал уже о ней, но в моем случае, я могу нарваться на бесконечный цикл, очень не хочется)) Хотя очень заманчиво)))))))))) 0

@Damenikx 955 / 341 / 114 Регистрация: 04.08.2018 Сообщений: 2,533
	26.09.2019, 21:01
	Не по теме: makgeer, псс, парень, хочешь немного глобалок? 0