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

@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:

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

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

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

Решение

Решение

Решение

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

@Alli_Lupin 5037 / 1064 / 149 Регистрация: 29.01.2013 Сообщений: 6,232
	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 5226 / 3472 / 1174 Регистрация: 21.03.2016 Сообщений: 8,300
	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