Уведомления

Группа в Telegram: @pythonsu

#1 Март 16, 2022 22:25:11

Rjugo
Зарегистрирован: 2022-03-16
Сообщения: 7
Репутация: +  0  -
Профиль   Отправить e-mail  

СРОЧНО помогите пожалуйста с оптимизацией кода

Доброго времени суток, уважаемые форумчане, несколько дней мучался над программой, сейчас она работает на на 0,015 секунды дольше чем должна, голова уже не варит от слова совсем, так что нужна помощь знающих людей, пожалуйста, прооптимизируйте мой горе код на сколько это возможно, я буду крайне Вам признателен

за ранее ОГРОМНОЕ СПАСИБО!
Задача с Timus 1287 “Каналы на марсе” (СМ прикрепленный файл)


Один из районов поверхности Марса покрыт идеальной сеткой каналов. Они делят поверхность на одинаковые квадратные клетки (кривизной поверхности мы здесь пренебрегаем). Сам район представляет собой идеальный квадрат, каждая сторона которого разбита на N клеток.
Как показали исследования археологической экспедиции, покрытый каналами район Марса занимала древняя страна под названием Йатик. Жители Йатика выращивали специальный злак — сир — который составлял основу их метаболизма. Сир бывает двух сортов — крупно- и мелкозернистый. Собственно, закат империи Йатика начался после гражданской войны между любителями этих сортов. До последнего времени, однако, не было установлено, какая из партий победила в этой кровопролитной войне. Результаты последней экспедиции на Марс позволяют надеяться на разгадку этой исторической тайны. Ученым удалось установить, сир какого сорта был последним высеян в той или иной клетке Йатика. Поскольку, по стародавней традиции, сир высевали в последовательности клеток (параллельные сторонам света, или идущие под углом 45° к ним), можно предположить, что сторонники победившей партии делали наиболее длинные посевы.
Исходные данные
В первой строке входа содержится единственное число N — размер квадрата (1 ≤ N ≤ 1400). Затем следуют N строк, каждая из которых состоит ровно из N символов. Буква “s” в i-й строке и j-м столбце означает, что в соответствующей клетке квадрата последним был высеян мелкозернистый сир, буква “S” означает, что последним был высеян крупнозернистый сир. Можно считать, что жители данного района не выращивали ничего кроме сира, и в каждой клетке района был засеян сир одного из этих двух сортов.
Результат
В первую строку нужно записать символ “s”, если в кровопролитной войне победила партия любитетей мелкозернистого сира, и символ “S”, если крупнозернистого. Если победителя определить невозможно, то первая строка должна содержать единственный символ “?”. Во второй строке должно быть записано целое число — максимальная длина посева сира одного сорта.

Прикреплённый файлы:
attachment 1287.py (9,5 KБ)

Офлайн

#2 Май 13, 2022 10:31:15

FishHook
От:
Зарегистрирован: 2011-01-08
Сообщения: 8312
Репутация: +  568  -
Профиль   Отправить e-mail  

СРОЧНО помогите пожалуйста с оптимизацией кода

Rjugo
Ваш код не может работать вообще никак потому что в строкaх 124, 135 и 143 у вас return вне всякой функции. Тот удивительный факт, что ваша программа не завершается с исключением, говорит о том, что ваша программа никогда не доходит до этих строк, а еще это говорит о том, что вы не тестировали все юз-кейсы своей программы. Что вам сейчас нужно сделать:
1. декомпозировать
на этом этапе ваша программа должна состоять из набора коротких (не более десяти строк) функций. Каждая функция выполняет одно и только одно элементарное действие и не зависит от некоего глобального состояния

 Нет!

 def scm(mat,lt):  # ФУНКЦИЯ, которая находит самые длинные неприрывные цепочки (среди строк передаваемой в качестве матрицы) из букв, возвращет букву, из которой состоит саммая длинная цепочка и колличесвто этой буквы в цепочке или знак "?" если в строчке равное колличесвто букв S и s (второй аргумент всегда 0, углубляться почему это так не буду)

 Да!
 def longest_line(matrix):  # ФУНКЦИЯ, которая находит самые длинные неприрывные цепочки 
  
def line_letter(line): # возвращет букву, из которой состоит цепочка
  
def letters_count(line, letter) # возвращет колличесвто буквы в цепочке или знак "?" если в строчке равное колличесвто букв S и s

2. весь код поместить в функции, в глобальной область должен быть только один вызов
 def ..
  
def ..
  
def main():
   ..
  
if __name__ == "__main__":
     main()

3. протестировать
каждую функцию вы должны протестировать отдельно. Если функции элементарны, то и тестировать их можно элементарно

 то есть если у нас есть функция, которая выбирает самую длинную строку из списка строк, например

   
def longest_line(list_of_lines):
    line = None 
    max_len = 0
        for l in list_of_lines:
           if len(l) > max_len:
              line = l
              max_len = len(l)
    return line

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

 assert longest_line(["aaa", "bbbbbb", "", "aa"]) == "bbbbbb"

сейчас же с вашим кодом ничего сделать нельзя, потому что он просто не работает как вы задумывали, а вы об этом и не знаете. Тут о скорости выполнения еще очень рано задумываться даже



Отредактировано FishHook (Май 13, 2022 10:33:50)

Офлайн

#3 Июнь 28, 2022 04:44:25

work2crowd
Зарегистрирован: 2020-12-02
Сообщения: 4
Репутация: +  0  -
Профиль   Отправить e-mail  

СРОЧНО помогите пожалуйста с оптимизацией кода

Хорошее объяснение, вывод нужно всё перепроверять по отдельным частям

Офлайн

Board footer

Модераторировать

Powered by DjangoBB

Lo-Fi Version