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

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

23.02.2017, 22:57. Показов 983. Ответов 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,889
Записей в блоге: 12
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
Ответ Создать тему
Новые блоги и статьи
Модель микоризы: классовый агентный подход 3
anaschu 06.01.2026
aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR
ФедосеевПавел 06.01.2026
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .
Модель микоризы: классовый агентный подход 2
anaschu 06.01.2026
репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока
igorrr37 05.01.2026
/ * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .
Новый CodeBlocs. Версия 25.03
palva 04.01.2026
Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .
Модель микоризы: классовый агентный подход
anaschu 02.01.2026
Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
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? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru