ioanna.konova 47 Точки

## 11. Array Manipulator от Методи - съвет за качеството на решението

Здравейте, колеги,

Някой по-напреднал и с желание, може ли да разгледа решението на тази задача и да сподели обратна връзка за качеството на кода, дали е лесно четим, дали методите са добре структурирани и т. н.? Отбелязвам, че към настоящия момент съм на метериала с Методи.

Ще съм благодарна на всякакви градивни критики и съвети! :)

Условието на задачата:

## 11.*Array Manipulator

Trifon has finally become a junior developer and has received his first task. It’s about manipulating an array of integers. He is not quite happy about it, since he hates manipulating arrays. They are going to pay him a lot of money, though, and he is willing to give somebody half of it if to help him do his job. You, on the other hand, love arrays (and money) so you decide to try your luck.

The array may be manipulated by one of the following commands

• exchange {index} – splits the array after the given index, and exchanges the places of the two resulting sub-arrays. E.g. [1, 2, 3, 4, 5] -> exchange 2 -> result: [4, 5, 1, 2, 3]
• If the index is outside the boundaries of the array, print “Invalid index
• max even/odd– returns the INDEX of the max even/odd element -> [1, 4, 8, 2, 3] -> max odd -> print 4
• min even/odd – returns the INDEX of the min even/odd element -> [1, 4, 8, 2, 3] -> min even > print 3
• If there are two or more equal min/max elements, return the index of the rightmost one
• If a min/max even/odd element cannot be found, print “No matches”
• first {count} even/odd– returns the first {count} elements -> [1, 8, 2, 3] -> first 2 even -> print [8, 2]
• last {count} even/odd – returns the last {count} elements -> [1, 8, 2, 3] -> last 2 odd -> print [1, 3]
• If the count is greater than the array length, print “Invalid count
• If there are not enough elements to satisfy the count, print as many as you can. If there are zero even/odd elements, print an empty array “[]”
• end – stop taking input and print the final state of the array

### Input

• The input data should be read from the console.
• On the first line, the initial array is received as a line of integers, separated by a single space
• On the next lines, until the command “end” is received, you will receive the array manipulation commands
• The input data will always be valid and in the format described. There is no need to check it explicitly.

### Output

• The output should be printed on the console.
• On a separate line, print the output of the corresponding command
• On the last line, print the final array in square brackets with its elements separated by a comma and a space
• See the examples below to get a better understanding of your task

### Constraints

• The number of input lines will be in the range [2 … 50].
• The array elements will be integers in the range [0 … 1000].
• The number of elements will be in the range [1 .. 50]
• The split index will be an integer in the range [-231 … 231 – 1]
• first/last count will be an integer in the range [1 … 231 – 1]
• There will not be redundant whitespace anywhere in the input
• Allowed working time for your program: 0.1 seconds. Allowed memory: 16 MB.

### Examples

 Input Output 1 3 5 7 9 exchange 1 max odd min even first 2 odd last 2 even exchange 3 end 2 No matches [5, 7] [] [3, 5, 7, 9, 1] Input Output 1 10 100 1000 max even first 5 even exchange 10 min odd exchange 0 max even min even end 3 Invalid count Invalid index 0 2 0 [10, 100, 1000, 1] Input Output 1 10 100 1000 exchange 3 first 2 odd last 4 odd end [1] [1] [1, 10, 100, 1000]
Тагове:
2
MartinBG 4700 Точки

Поздравления за темата! Рядко има въпроси, касаещи качеството на кода. :)

Набързо нещата, които забелязах

- има твърде много код в main

- изнесете всяка команда в неин метод, който да се вика от while цикъла в main

- вместо if/else if .... може да се ползва switch по командата, всяка от които (ако вика метод) може да се разпише на 1 ред

- има повторяемост на кода за прочитане на следващата команда (5 пъти, ако не се лъжа). Може да се напише например така:

``````            while (true)
{
.Split(' ', StringSplitOptions.RemoveEmptyEntries);

if (currentCommand[0] == "end")
{
break;
}
//.....
}``````

- избягвайте използването на индекси, а вмeсто това използвайте именувани променливи. Например вместо викане на currentCommand[0] е по-добре да имаме string command = currentCommand[0] и после да ползваме само command

1
23/05/2022 21:50:19
Можем ли да използваме бисквитки?
Ние използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Можете да се съгласите с всички или част от тях.
Назад
Функционални
Използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Използваме „сесийни“ бисквитки, за да Ви идентифицираме временно. Те се пазят само по време на активната употреба на услугите ни. След излизане от приложението, затваряне на браузъра или мобилното устройство, данните се трият. Използваме бисквитки, за да предоставим опцията „Запомни Ме“, която Ви позволява да използвате нашите услуги без да предоставяте потребителско име и парола. Допълнително е възможно да използваме бисквитки за да съхраняваме различни малки настройки, като избор на езика, позиции на менюта и персонализирано съдържание. Използваме бисквитки и за измерване на маркетинговите ни усилия.
Рекламни
Използваме бисквитки, за да измерваме маркетинг ефективността ни, броене на посещения, както и за проследяването дали дадено електронно писмо е било отворено.