Программа должна удалять элементы кратные семи в массиве и оставшиеся сдвигать на место удаленных влево. Работает (удаляет и сдвигает) только первый найденный элемент, остальные элементы кратные семи почему-то не трогает. Я так понимаю напутал где-то с внешним циклом. Покажите ошибку, пожалуйста.
var a:array[1..20] of integer; i,e:integer; begin
randomize; for i:=1 to 20 do a[i]:=random(95)-50;
writeln('massiv: '); for i:=1 to 20 do write(a[i],' ');
writeln;
for i:=1 to 20 do begin if a[i] mod 7=0 then inc(e); if a[i] mod 7=0 then for i:=i to 20 do a[i]:=a[i+1]; end;
writeln('novyi massiv: '); for i:=1 to 20-e do write(a[i],' ');
writeln;
end.
volvo
20.10.2009 13:36
Цитата
for i:=1 to 20 do begin if a[i] mod 7=0 then inc(e); if a[i] mod 7=0 then for i:=i to 20 do a[i]:=a[i+1]; { <--- } end;
Как минимум в этом месте программа перестает работать правильно. Нельзя делать несколько циклов по одной и той же переменной, сделай второй цикл по J, например... Это первое. Второе - здесь внешний цикл желательно делать не For, а While, потому что у тебя сдвигаются элементы массива, так какой смысл идти по ВСЕМ элементам, до 20? Надо идти только до (20 - e), а это нереализуемо в For-е. Опять же, почему E не инициализировано? Проблем ищешь? Хочешь расскажу, что ты скажешь через некоторое время? Когда попытаешься сделать это задание отдельной процедурой?
Вячеслав Л.
21.10.2009 14:36
Цитата
Второе - здесь внешний цикл желательно делать не For, а While, потому что у тебя сдвигаются элементы массива, так какой смысл идти по ВСЕМ элементам, до 20? Надо идти только до (20 - e), а это нереализуемо в For-е.
Напиши конкретней, пожалуйста. Ты предлагаешь мне заменить вот этот цикл на while, и изменить условие на 20-e?
for i:=1 to 20 do (строка 16ая)
Я вроде попробовал так, ничего не изменилось.
Цитата
Опять же, почему E не инициализировано?
Что это значит?
ammaximus
21.10.2009 15:05
uses crt; const n=8; a:array[1..n] of integer =(7,7,3,4,7,7,5,7); var i,j,e:integer; begin clrscr;
{randomize; for i:=1 to 20 do a[i]:=random(95)-50;} writeln('massiv: '); for i:=1 to n do write(a[i],' '); writeln; {Инициализация} e:=0; i:=1; while (i<n-e+1) do begin if a[i] mod 7=0 then begin for j:=i to n-e do a[j]:=a[j+1]; inc(e); end else inc(i); end;
writeln('novyi massiv: '); for i:=1 to n-e do write(a[i],' ');
writeln; readkey; end.
Цикл while в этом случае позволяет уменьшить число итераций - отсекаются лишние. Инициализация - это когда переменной задают начальное значение. В Паскале инициализация происходит автоматически, но к этому лучше не привыкать. Например в коде сверху обнулять i обязательно.
Вячеслав Л.
22.10.2009 5:03
Спасибо, с циклами разобрался, программа работает.
2 ammaximus: спасибо за лаконичное объяснение, буду иметь в виду.
Это текстовая версия — только основной контент. Для просмотра полной версии этой страницы, пожалуйста, нажмите сюда.