|
5 / 5 / 0
Регистрация: 20.06.2016
Сообщений: 87
|
||||||
Как можно улучшить мой код ?23.02.2017, 22:57. Показов 998. Ответов 19
Метки нет (Все метки)
задача звучит так :
Мальчик Кирилл написал однажды на листе бумаги строчку, состоящую из больших и маленьких латинских букв, а после этого ушел играть в футбол. Когда он вернулся, то обнаружил, что его друг Дима написал под его строкой еще одну строчку такой же длины. Дима утверждает, что свою строчку он получил циклическим сдвигом строки Кирилла направо на несколько шагов(циклический сдвиг строки abcde на 2 позиции направо даст строку deabc). Однако Дима известен тем, что может случайно ошибиться в большом количестве вычислений, поэтому Кирилл в растерянности - верить ли Диме? Помогите ему! По данным строкам выведите минимально возможный размер сдвига или -1, если Дима ошибся. Входные данные Первые две строки входного файла INPUT.TXT содержат строки Кирилла и Димы соответственно. Строки состоят только из латинских символов. Длины строк одинаковы, не превышают 10000 и не равны 0. Выходные данные В выходной файл OUTPUT.TXT выведите единственное число - ответ на поставленную задачу. мой код
не проходит 11 тест т.к там огроничений 0.5 сек
0
|
||||||
| 23.02.2017, 22:57 | |
|
Ответы с готовыми решениями:
19
Улучшить и сжать код Я написал программу (вычисление среднего арифметического). Как можно ее улучшить? Что в этой программе можно изменить (что бы улучшить) И как ее усовершенствовать ? |
|
Модератор
|
|
| 24.02.2017, 09:11 | |
|
В современных Pascal длина строки достаточно велика.
Думаю, что что-то не так с алгоритмом - вложенный цикл на больших строках остановит что угодно. Как улучшение - не заниматься физическим сдвигом строки, а ограничиться вычислением смещения и сравнениями двух частей двух строк. Если и это не поможет, то попробовать строковые алгоритмы. Мне кажется, что подойдёт алгоритм Кнута-Морриса-Пратта http://www.e-maxx-ru.1gb.ru/algo/prefix_function
0
|
|
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
||||||||||||||||
| 24.02.2017, 09:31 | ||||||||||||||||
|
Замени
Для ускорения можно 3 строчку заменить, с проверкой на равенство первого символа
0
|
||||||||||||||||
|
Модератор
|
||||||
| 24.02.2017, 11:22 | ||||||
|
U-III,
В данном контексте Copy будет дорогим вызовом -- выделение памяти плюс копирование. Ради сравнения с другой строкой не стоит. Того же самого можно было достичь так
0
|
||||||
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
|||||||||||
| 24.02.2017, 12:02 | |||||||||||
|
bormant, copy в данном случае выгоднее,
ведь его используем только для сравнения. Добавлено через 24 минуты Ну или вообще так.
0
|
|||||||||||
|
Модератор
|
|
| 24.02.2017, 12:02 | |
|
U-III,
Вы точно понимаете, что делает Copy, RightStr, LeftStr? if s=Copy()+Copy() -- это создание 3-х временных строк, по одной на копию и одной для результата конкатенации, три копирования фрагментов памяти. Посимвольное сравнение тоже не блещет скоростью, но тут можно какой-то оптимизированный вариант выписать, сравнивая двойными словами за раз вместо побайтного варианта. Тем не менее, это не отменяет квадратичной сложности попыток проверить все смещения -- тупиковый вариант.
0
|
|
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
|
| 24.02.2017, 12:09 | |
|
bormant, Вы знаете как работает компилятор
Если условие (ss[i+1]=s[1]) не выполняется - то он даже не выполняет вторую часть условия and ((RightStr(ss,i+1)+LeftStr(ss,i)) = s) Добавлено через 1 минуту И по условию Важна скорость а не размер памяти. В том то и дело что за все проходы будут использованы 3 временные строки в памяти. Добавлено через 2 минуты При посимвольном сравнении строк длинной в 10 000 У вас будет выполнено 10 000 * 10 000 = 100 000 000 операций сравнения в максимальном случае и 10 000 в минимальном В моем случае только 10 000 в максимальном случае и 1 в минимальном Тут же вспомним про дерганья индекса массива, сравнения, лишние циклы. При словарном сравнении - вам еще нужно эти слова получить, что так же будет кушать проц и память.
0
|
|
|
Модератор
|
||||||
| 24.02.2017, 12:16 | ||||||
|
U-III,
копирование занимает существенное время и выполняется не 1 раз, а на каждой (почти) итерации. Вообще говоря, при наличии поддержки длинных строк, задачка тривиальная:
1
|
||||||
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
||||||
| 24.02.2017, 12:20 | ||||||
|
bormant, ))))))
Полностью согласен)))) Тоже хотел так написать сразу. Но это бы оборвало всю дискуссию) В данном примере
0
|
||||||
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
|
| 24.02.2017, 12:42 | |
|
bormant, будет 10 000 по моему примеру.
И 100 000 000 по вашему Добавлено через 1 минуту Если вы про циклы говорите, да в 9999 будет происходит сравнение строк Добавлено через 1 минуту Сравнить 9999 раз строки или 100 000 000 раз символы Добавлено через 33 секунды Давайте счас быстро напишу програмку с замерами при таком варианте Добавлено через 1 минуту Оформите ваш вариант как функцию (побайтовый) которая возвращает -1 или смещение Compare_STR2(s,ss:string):integer;
0
|
|
|
Модератор
|
||||||
| 24.02.2017, 12:47 | ||||||
|
U-III,
какой-то калькулятор ваш неправильный. Хотя, в случае строковых s1=s2 можно надеяться на цепочные операции сравнения, подставленные компилятором; но и в варианте сравнения без копирования никто не заставляет пользоваться циклом:
0
|
||||||
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
||||||
| 24.02.2017, 12:59 | ||||||
|
bormant, вызовите вы эту функцию 10 000 раз;
Вы написали тоже самое, что и я, без выделения памяти под временную переменную Вообще тема уже была ТУТ Вы же в ней и писали Добавлено через 3 минуты
В итоге банальный POS быстрее всего в выбранном варианте
0
|
||||||
|
Модератор
|
|
| 24.02.2017, 13:00 | |
|
U-III,
так то когда было... Всего ж не упомнишь. Если б оно как-то мои прежние знания поколебало, внесло б свежую струю в оценку борзодействия того или иного варианта, может и запомнилось бы хот бы этим. А так Copy()+Copy() аутсайдер, и ничего тут не изменить...
0
|
|
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
|
| 24.02.2017, 13:02 | |
|
0
|
|
|
Модератор
|
||
| 24.02.2017, 13:07 | ||
|
U-III,
это была демонстрация идеи сравнения по месту без копирования. Подобные циклы хорошо оптимизируются путем разбивки на сравнение стартовых невыровненных данных, сравнения выровненных данных по 4/8 байт за раз, сравнение хвоста. В конечном итоге Pos() делает именно это. Есть желание написать решение задачки для Turbo Pascal? Добавлено через 1 минуту
0
|
||
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
|
| 24.02.2017, 13:08 | |
|
Ответ. Уже нет.
Все уже и так проверено. Добавлено через 58 секунд В том варианте побайтовый вариант медленнее.
0
|
|
|
Модератор
|
||||||
| 24.02.2017, 13:42 | ||||||
|
В TP7 (а также и в FPC) можно так:
0
|
||||||
|
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
|
|
| 24.02.2017, 13:47 | |
|
bormant, Это тот же аналог pos для Pchar
StrPos (функция) (модуль Strings) Возвращает указатель на первое вхождение одной строки в другую.
0
|
|
|
Модератор
|
||
| 24.02.2017, 13:50 | ||
|
Просто иллюстрация, которую можно давать тем, кто жалуется на непригодность TP7 для простых заданий с длинными строками.
0
|
||
| 24.02.2017, 13:50 | |
|
Помогаю со студенческими работами здесь
20
Улучшить калькулятор, чтобы можно было продолжить операцию
Как можно улучшить код Как можно улучшить код (ООП)?
Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
Символьное дифференцирование
igorrr37 13.02.2026
/ *
Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2).
Унарный минус обозначается как !
*/
#include <iostream>
#include <stack>
#include <cctype>. . .
|
Камера 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, то после закрытия окошка. . .
|