Opened 5 years ago

Closed 5 years ago

#243 closed ожидаются исправления (задача сдана)

WW #4

Reported by: pshek.maksim Owned by: pshek.maksim
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description

test_try

Change History (5)

comment:1 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to pshek.maksim
Type: ожидается проверкаожидаются исправления

Попытка есть, честный +1 балл за корректный Makefile и структуру папок.

Стиль:

  1. stdio.h — стандартный заголовок, следует включать как <stdio.h>, а не "stdio.h"

comment:2 Changed 5 years ago by pshek.maksim

Owner: changed from pshek.maksim to Egor Suvorov
Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to pshek.maksim

Корректность 6.5/7:

  1. Выводятся лишние пробелы на экран.

Стиль 1.5/3:

  1. Делайте пустые строчки между функциями.
  2. Сделайте в main не иф с заглушением ошибки, а assert. Чтобы при неверных аргументах было громкое сообщение, а не молчание.
  3. Цепочка else if пишется так:
    if (a) {
        foo();
    } else if (b) {
        bar();
    } else if (c) {
        baz();
    } else {
        que();
    }
    
  4. a_size --> elements.
  5. *(s_size) — скобочки не нужны.
  6. strcmp(..) == 0 --> !strcmp(..)`
  7. merge можно сильно проще одним циклом. Не объявляйте переменные заранее. Пробелы вокруг бинарных операторов.

comment:4 Changed 5 years ago by Egor Suvorov

Type: ожидается проверкаожидаются исправления

comment:5 Changed 5 years ago by Egor Suvorov

Resolution: задача сдана
Status: assignedclosed

Кажется, так и осталось

Note: See TracTickets for help on using tickets.