С Новым годом! Форум программистов, компьютерный форум, киберфорум
Pascal (Паскаль)
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
5 / 5 / 0
Регистрация: 20.06.2016
Сообщений: 87

Как можно улучшить мой код ?

23.02.2017, 22:57. Показов 963. Ответов 19
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
задача звучит так :
Мальчик Кирилл написал однажды на листе бумаги строчку, состоящую из больших и маленьких латинских букв, а после этого ушел играть в футбол. Когда он вернулся, то обнаружил, что его друг Дима написал под его строкой еще одну строчку такой же длины. Дима утверждает, что свою строчку он получил циклическим сдвигом строки Кирилла направо на несколько шагов(циклический сдвиг строки abcde на 2 позиции направо даст строку deabc). Однако Дима известен тем, что может случайно ошибиться в большом количестве вычислений, поэтому Кирилл в растерянности - верить ли Диме? Помогите ему!

По данным строкам выведите минимально возможный размер сдвига или -1, если Дима ошибся.

Входные данные

Первые две строки входного файла INPUT.TXT содержат строки Кирилла и Димы соответственно. Строки состоят только из латинских символов. Длины строк одинаковы, не превышают 10000 и не равны 0.

Выходные данные

В выходной файл OUTPUT.TXT выведите единственное число - ответ на поставленную задачу.




мой код
Pascal
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
var
bol:boolean;
s,ss:string;
m:char;
j,i:integer;
begin
bol:=false;
 
readln(s);
readln(ss);
if s=ss then writeln(0)
else
begin
for i:=1 to length(s)-1 do begin
    m:=s[length(s)];
    for j:=length(s) downto 2 do s[j]:=s[j-1];
    s[1]:=m;
 
if s=ss then begin
bol:=true;
break;
end;
end;
if bol then writeln(i)
else
writeln(-1);
end;
end.

не проходит 11 тест т.к там огроничений 0.5 сек
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
23.02.2017, 22:57
Ответы с готовыми решениями:

Улучшить и сжать код
Ребят, плиз, гляньте - в чем можно улучшить код? Мож че лишнее есть? Дана строка. Вывести все буквы, встречающиеся не ме-нее двух раз....

Я написал программу (вычисление среднего арифметического). Как можно ее улучшить?
Написал простенькую программу по вычислению среднего арифметического из суммы не более чем двадцати вещественных чисел. Как можно сделать,...

Что в этой программе можно изменить (что бы улучшить) И как ее усовершенствовать ?
Program Kalkulator ; Label m1,m2; Var Operation,cal:byte; a,b:real; Begin m1: Writeln('1-сложить'); ...

19
Модератор
Эксперт по электронике
 Аватар для ФедосеевПавел
8647 / 4482 / 1669
Регистрация: 01.02.2015
Сообщений: 13,888
Записей в блоге: 11
24.02.2017, 09:11
В современных Pascal длина строки достаточно велика.
Думаю, что что-то не так с алгоритмом - вложенный цикл на больших строках остановит что угодно.
Как улучшение - не заниматься физическим сдвигом строки, а ограничиться вычислением смещения и сравнениями двух частей двух строк.
Если и это не поможет, то попробовать строковые алгоритмы. Мне кажется, что подойдёт алгоритм Кнута-Морриса-Пратта http://www.e-maxx-ru.1gb.ru/algo/prefix_function
0
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 09:31
Замени
Pascal
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
if s=ss then writeln(0)
else
begin
for i:=1 to length(s)-1 do begin
    m:=s[length(s)];
    for j:=length(s) downto 2 do s[j]:=s[j-1];
    s[1]:=m;
 
if s=ss then begin
bol:=true;
break;
end;
end;
if bol then writeln(i)
else
writeln(-1);
end;
на

Pascal
1
2
3
4
5
6
7
8
9
j:=length(s);
for i:=0 to length(s)-1 do  
  if (copy(s,i+1,j-i)+copy(s,1,i)) = ss then begin
    bol:=true;
    break;
  end;
if bol then writeln(i)
else
writeln(-1);
Добавлено через 4 минуты
Для ускорения можно 3 строчку заменить, с проверкой на равенство первого символа

Pascal
1
if (s[i+1]=ss[1]) and ((copy(s,i+1,j-i)+copy(s,1,i)) = ss) then begin
Если символы не равны - дальнейшее сравнение производится не будет.
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 11:22
U-III,
В данном контексте Copy будет дорогим вызовом -- выделение памяти плюс копирование. Ради сравнения с другой строкой не стоит.
Того же самого можно было достичь так
Pascal
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function cmp(const p, q: String; d: Integer);
var j, k: Integer;
begin
  cmp:=False;
  k:=d+1;
  for j:=1 to Length(p)-d do begin
    if p[j]<>q[k] then Exit;
    Inc(k);
  end;
  k:=1;
  for j:=Length(p)-d+1 to Length(p) do begin
  begin
    if p[j]<>q[k] then Exit;
    Inc(k);
  end;
  Result:=True;
end;
Но это тупиковый путь, в худшем случае 10000 таких проходов не уложатся в отведённое время.
0
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 12:02
bormant, copy в данном случае выгоднее,
ведь его используем только для сравнения.

Добавлено через 24 минуты
Ну или вообще так.

Pascal
1
2
3
4
5
6
7
if length(s)=length(ss) then 
for i:=0 to length(ss)-1 do  
  if (ss[i+1]=s[1]) and ((RightStr(ss,i+1)+LeftStr(ss,i)) = s) then begin
    bol:=true;
    break;
  end;
if bol then writeln(i) else writeln(-1);
Добавлено через 10 минут
Pascal
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
var
  s,ss:string;
  j,i:integer;
begin
  readln(s); 
  readln(ss);
  j:=-1;
  if length(s)=length(ss) then 
    for i:=0 to length(ss)-1 do 
      if (ss[i+1]=s[1]) and ((RightStr(ss,i+1)+LeftStr(ss,i)) = s) then 
        begin
          j:=i;
          break;
        end;
writeln(j);
end.
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 12:02
U-III,
Вы точно понимаете, что делает Copy, RightStr, LeftStr?
if s=Copy()+Copy() -- это создание 3-х временных строк, по одной на копию и одной для результата конкатенации, три копирования фрагментов памяти.

Посимвольное сравнение тоже не блещет скоростью, но тут можно какой-то оптимизированный вариант выписать, сравнивая двойными словами за раз вместо побайтного варианта. Тем не менее, это не отменяет квадратичной сложности попыток проверить все смещения -- тупиковый вариант.
0
 Аватар для U-III
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
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 12:16
U-III,
копирование занимает существенное время и выполняется не 1 раз, а на каждой (почти) итерации.

Вообще говоря, при наличии поддержки длинных строк, задачка тривиальная:
Pascal
1
2
3
4
5
var q, p: String;
begin
  ReadLn(p); ReadLn(q);
  WriteLn(Pos(q,p+p)-1);
end.
1
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 12:20
bormant, ))))))
Полностью согласен))))
Тоже хотел так написать сразу. Но это бы оборвало всю дискуссию)
В данном примере

Pascal
1
2
3
4
5
6
7
8
var
  s,ss:string;
  j,i:integer;
begin
  readln(s); 
  readln(ss);
  writeln(Pos(s,ss+ss)-1);
end.
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 12:21
Цитата Сообщение от U-III Посмотреть сообщение
В моем случае только 10 000 в максимальном случае и 1 в минимальном
Вот вам пара строк: а(9999 раз)1, a(9998 раз)1а. Считайте ваш максимальный случай
0
 Аватар для U-III
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
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 12:47
U-III,
какой-то калькулятор ваш неправильный. Хотя, в случае строковых s1=s2 можно надеяться на цепочные операции сравнения, подставленные компилятором; но и в варианте сравнения без копирования никто не заставляет пользоваться циклом:
Pascal
1
2
3
4
5
6
7
uses SysUtils;
function cmp(const p, q: String; d: Integer): Boolean;
begin
  cmp:=
    CompareMem(@p[1],@q[d+1],Length(p)-d) and 
    ((d=0) or CompareMem(@p[Length(p)-d+1],@q[1],d));
end;
0
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 12:59
bormant, вызовите вы эту функцию 10 000 раз;
Вы написали тоже самое, что и я, без выделения памяти под временную переменную

Вообще тема уже была
ТУТ

Вы же в ней и писали

Добавлено через 3 минуты
Pascal
1
2
3
4
5
6
7
8
9
function cmp(s1,s2):integer;
var i,j:integer;
begin
s2:=s2+s2;
j:=length(s1);
I:=0;
while not CompareMem(@s1[1],@s2[1+i],j) and (i<j) do Inc(i);
result=i;
end;
Добавлено через 2 минуты
В итоге банальный POS быстрее всего в выбранном варианте
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 13:00
U-III,
так то когда было... Всего ж не упомнишь. Если б оно как-то мои прежние знания поколебало, внесло б свежую струю в оценку борзодействия того или иного варианта, может и запомнилось бы хот бы этим.
А так Copy()+Copy() аутсайдер, и ничего тут не изменить...
0
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 13:02
bormant,
Цитата Сообщение от bormant Посмотреть сообщение
Вот вам пара строк: а(9999 раз)1, a(9998 раз)1а
Оказывается оутсайдер побайтовый перебор.
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 13:07
U-III,
это была демонстрация идеи сравнения по месту без копирования. Подобные циклы хорошо оптимизируются путем разбивки на сравнение стартовых невыровненных данных, сравнения выровненных данных по 4/8 байт за раз, сравнение хвоста.
В конечном итоге Pos() делает именно это.

Есть желание написать решение задачки для Turbo Pascal?

Добавлено через 1 минуту
Цитата Сообщение от U-III Посмотреть сообщение
оутсайдер побайтовый перебор
Уверены? Точно не Copy()+Copy()?
0
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 13:08
Ответ. Уже нет.
Все уже и так проверено.

Добавлено через 58 секунд
В том варианте побайтовый вариант медленнее.
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 13:42
В TP7 (а также и в FPC) можно так:
Pascal
1
2
3
4
5
6
7
8
9
10
uses Strings;
var
  p: array [0..20000] of Char;
  q: array [0..10000] of Char;
  t: PChar;
begin
  ReadLn(p); ReadLn(q);
  t:=StrEnd(p); Move(p,t^,t-p); t:=StrPos(p,q);
  if t=nil then WriteLn(-1) else WriteLn(t-p);
end.
0
 Аватар для U-III
59 / 59 / 12
Регистрация: 24.10.2012
Сообщений: 170
24.02.2017, 13:47
bormant, Это тот же аналог pos для Pchar
StrPos (функция) (модуль Strings)
Возвращает указатель на первое вхождение одной строки в другую.
0
Модератор
Эксперт Pascal/DelphiЭксперт NIX
 Аватар для bormant
7816 / 4635 / 2837
Регистрация: 22.11.2013
Сообщений: 13,158
Записей в блоге: 1
24.02.2017, 13:50
Цитата Сообщение от U-III Посмотреть сообщение
Это тот же аналог pos для Pchar
Абсолютно верно.
Просто иллюстрация, которую можно давать тем, кто жалуется на непригодность TP7 для простых заданий с длинными строками.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
24.02.2017, 13:50
Помогаю со студенческими работами здесь

Улучшить калькулятор, чтобы можно было продолжить операцию
Написал простейший калькулятор,но нужно сделать так,что бы можно было продолжить операцию var a,b,rez:real;op:char; begin ...

Как можно улучшить мой код? Калькулятор
Как можно улучшить мой код? Калькулятор #include &lt;iostream&gt; using namespace std; int main (void) { int a, A, B; ...

Как можно улучшить код
Здравствуйте, вы, наверное слышали про метод шифрования - Атбаш, сам по себе он простой, вот его реализация: $alphabet =...

Как можно улучшить код (ООП)?
Здравствуйте форумчане. В книге Bhaskad Chaudhary &quot;Tkinter GUI Development Blueprints&quot; увидел реализацию вызова дочернего окна, из...

Как можно улучшить код работы с массивами?
users = users.each do |user| if user != nil and user != nil and user == true puts user end end #...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути
Programma_Boinc 01.01.2026
Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru