Комментарии: Решаем задачи Абрамян на Паскале. Array66

Автор: Bronislav

Bronislav — Fri, 03 Aug 2018 13:55:11 +0000

Если массив не содержит четных чисел, то в строке 19 ошибка выхода индекса за границу массива. Нужно поменять условия местами, чтобы проверялось сначала i <= N

Автор: Anton

Anton — Thu, 27 Apr 2017 05:56:49 +0000

Да, так лучше… 🙂 но всё таки считаю, что функция конкретно в этом случае излишня. Из за неё приходиться объявлять тип:

Type 
 MyArray = array [1..10] of integer;

1 2	Type MyArray = array [1..10] of integer;

Писать обвязку:

Procedure swap(k,n: integer; var a:MyArray);
Var
 i: integer;
Begin
...
End;

Procedure swap(k,n: integer; var a:MyArray);

Var

i: integer;

Begin

...

End;

И вызывается она в таком месте, что конструкция :

 For i:=1 to n do
  If (a[i] mod 2 = 0) then a[i]:=a[i]+k;

1 2	For i:=1 to n do If (a[i] mod 2 = 0) then a[i]:=a[i]+k;

Никак не ухудшила бы читаемость…
Но если это всё сделать то будет практически тоже самое, что и в шапке 🙂

Автор: Kirk

Kirk — Thu, 27 Apr 2017 05:20:25 +0000

Исправил 🙂

Program a66;

Type 
 MyArray = array [1..10] of integer;

Procedure swap(k,n: integer; var a:MyArray);
Var
 i: integer;
Begin
 For i:=1 to n do
  If (a[i] mod 2 = 0) then a[i]:=a[i]+k;
End;
  
Var
 a: MyArray;
 n,i,k: integer;
 
Begin
 k:=0;
 n:=random(1,10);
 Writeln('N=',n);
 Write('A: ');
 For i:=1 to n do
  Begin
   a[i]:=random(-10,10);
   Write(a[i],'  ');
   If (a[i] mod 2 = 0) and (k=0) then k:=a[i];
  End;
 Writeln();
 swap(k,n,a);
 Write('A: ');
 For i:=1 to n do Write(a[i],'  ');
End.

Program a66;

Type

MyArray = array [1..10] of integer;

Procedure swap(k,n: integer; var a:MyArray);

Var

i: integer;

Begin

For i:=1 to n do

If (a[i] mod 2 = 0) then a[i]:=a[i]+k;

End;

Var

a: MyArray;

n,i,k: integer;

Begin

k:=0;

n:=random(1,10);

Writeln('N=',n);

Write('A: ');

For i:=1 to n do

Begin

a[i]:=random(-10,10);

Write(a[i],' ');

If (a[i] mod 2 = 0) and (k=0) then k:=a[i];

End;

Writeln();

swap(k,n,a);

Write('A: ');

For i:=1 to n do Write(a[i],' ');

End.

Пришлось также поменять строчку #29:

   If (a[i] mod 2 = 0) and (k=0) then k:=a[i];

1	If (a[i] mod 2 = 0) and (k=0) then k:=a[i];

А то присваивалось новое значение k после увеличения первого четного

Автор: Kirk

Kirk — Thu, 27 Apr 2017 05:10:32 +0000

В ответ на Anton. Да уж.. Что то увлекся немного

Автор: Anton

Anton — Wed, 26 Apr 2017 09:00:40 +0000

For i:=1 to n do TempArray[i]:=a[k];
 For i:=1 to n do
  If (a[i] mod 2 = 0) then a[i]:=a[i]+TempArray[i];

For i:=1 to n do TempArray[i]:=a[k];

For i:=1 to n do

If (a[i] mod 2 = 0) then a[i]:=a[i]+TempArray[i];

Что то я не понял… вы создали массив состоящий из значения элемента a[k] а потом прибавляете …
В общем зачем?! почему не прибавлять a[k] вместо TempArray[i]? и вообще TempArray можно будет убрать..
Да и обвязка функции становиться больше тела функции…

Автор: Kirk

Kirk — Wed, 26 Apr 2017 06:34:42 +0000

Program a66;

Type 
 MyArray = array [1..10] of integer;

Procedure swap(k,n: integer; var a:MyArray);
Var
 TempArray: MyArray;
 i: integer;
Begin
 For i:=1 to n do TempArray[i]:=a[k];
 For i:=1 to n do
  If (a[i] mod 2 = 0) then a[i]:=a[i]+TempArray[i];
End;
  
Var
 a: MyArray;
 n,i,k: integer;
 
Begin
 k:=0;
 n:=random(1,10);
 Writeln('N=',n);
 Write('A: ');
 For i:=1 to n do
  Begin
   a[i]:=random(-10,10);
   Write(a[i],'  ');
   If (a[i] mod 2 = 0) and (k=0) then k:=i;
  End;
 Writeln();
 swap(k,n,a);
 Write('A: ');
 For i:=1 to n do Write(a[i],'  ');
End.

Program a66;

Type

MyArray = array [1..10] of integer;

Procedure swap(k,n: integer; var a:MyArray);

Var

TempArray: MyArray;

i: integer;

Begin

For i:=1 to n do TempArray[i]:=a[k];

For i:=1 to n do

If (a[i] mod 2 = 0) then a[i]:=a[i]+TempArray[i];

End;

Var

a: MyArray;

n,i,k: integer;

Begin

k:=0;

n:=random(1,10);

Writeln('N=',n);

Write('A: ');

For i:=1 to n do

Begin

a[i]:=random(-10,10);

Write(a[i],' ');

If (a[i] mod 2 = 0) and (k=0) then k:=i;

End;

Writeln();

swap(k,n,a);

Write('A: ');

For i:=1 to n do Write(a[i],' ');

End.