Python-сообщество

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

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


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

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

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

 def ..
  
def ..
  
def main():
   ..
  
if __name__ == "__main__":
     main()

   
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)

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

FishHook

 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

 def longest_line(list_of_lines):
	return sorted(list_of_lines)[-1]

Отредактировано nassl (Фев. 7, 2024 15:33:27)

nassl

FishHook

  
     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

Понятно, что некропост, но так не проще? (я не настоящий программист, если что)

  
def longest_line(list_of_lines):
	return sorted(list_of_lines)[-1]

  
>>> lst = ['a', 'bbb', 'cc']
>>> max(lst, key=len)
'bbb'
>>>

nassl
но так не проще? (я не настоящий программист, если что)

Отредактировано py.user.next (Фев. 8, 2024 02:15:06)

py.user.next
Просто у нас тут тематика другая.

nassl
но если речь про миллионы строк, то numpy или чистый си, питон - это вроде не про скорость.

py.user.next
. Просто не знаешь ничего про питон.

nassl
Более того, я даже ни одной книжки по питону не читал. У меня другая профессия.

что мне подсказывает, что пирожок тебе не дали