Все ли хорошо в этом коде

@thechicho · Регистрация: 15.03.2015

Студворк — интернет-сервис помощи студентам

Предложите ваши варианты решения заданий
3. Все ли хорошо в этом коде?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
Файл legacy.c
int values[3];
 
Файл modern.cpp
#define LEGACY_SIZE 3
extern int *values;
 
class MyBlah {...};
 
class Adapter
{
public:
    Adapter()
    {
        for (int i = 0; i < LEGACY_SIZE; ++i)
            map_[values[i]] = new MyBlah (values[i]);
    }
private:
    std::map<int, MyBlah *> map_;
};

@Avazart · 18.08.2016, 16:43

Сообщение от _Ivana

нет индекса, удовлетворяющего условию - значит "нет", а не -1 - вы же сами против сишного наследия и за красоту и ясность абстракций?

Именно, но использование npos короче по записи, пускай не совсем ясно в общем случае, но именно для строк это вполне очевидно.

@Babysitter · 18.08.2016, 17:01

я даже вспомнил про ssize_t, которые как раз и должен обозначать что-то типа "беззнаковое или минус единица в случае ошибки". но это что-то из posix'а ..

@Voivoid · 18.08.2016, 18:12

Сообщение от hoggy

то, чего не должно быть, не должно быть в принципе.

Ага, аргумент уровня: "а давайте просто писать без багов"

HighPredator · 18.08.2016, 18:25

Voivoid, а что вам не нравится? Вполне себе верные вещи hoggy пишет.

@Voivoid · 18.08.2016, 18:42

Сообщение от HighPredator

Voivoid, а что вам не нравится? Вполне себе верные вещи hoggy пишет.

1) Не нравится предложение поменять int на size_t в коде цикла из оп-поста. Там надо или везде менять инты на size_t или ничего не трогать
2) Тут вопрос конечно спорный, но лично я предпочитаю использовать int вместо size_t ( там, где это конечно возможно и не требует cast'ов ). По причине более легкого отлова ошибок.

@avgoor · 18.08.2016, 19:01

Сообщение от _Ivana

вы же сами против сишного наследия и за красоту и ясность абстракций?

А вот чисто интересно: nullptr - это ужасное наследие си или красивая и ясная абстракция?

Добавлено через 4 минуты

Сообщение от Voivoid

Тут вопрос конечно спорный, но лично я предпочитаю использовать int вместо size_t

В студии, кстати, в шаблонах кода обычный фор с size_t, а ~~тайд~~ реверсный с интом. Но я тоже предпочитаю не использовать беззнаковые типы там, где без них можно обойтись.

@Voivoid · 18.08.2016, 19:12

Сообщение от _Ivana

В памяти всплывает светлый образ еще одного местного С++ гуру, который что-то писал про мэйбимонады А это снова они, родимые: нет индекса, удовлетворяющего условию - значит "нет", а не -1 - вы же сами против сишного наследия и за красоту и ясность абстракций?

Что сказать-то хотел?

Когда писалась стандартная библиотека, в плюсах еще не было optional'ов (аналог maybe из haskell ). И появятся они лишь в C++17, зато в бусте они уже довольно давно

@hoggy · 18.08.2016, 22:22

Сообщение от Avazart

И тем не менее это условность/договоренность что npos это -1 или макс значение, константа или еще что либо.

нет никаких договоренностей.
вы не можете закладываться на конкретное значение npos
и вас вообще не должно это никак волновать.

все что вам нужно знать,
что бы пользоваться - что npos - несуществующая позиция.

Добавлено через 2 минуты

Сообщение от Voivoid

Ага, аргумент уровня: "а давайте просто писать без багов"

мой аргумент был вполне конкретен:

Сообщение от hoggy

корректно оформленная - корректна с точки зрения стандарта,
и гарантирует инвариант.

вам не нужен знаковый инт там, где по смыслу нужно беззнаковое,
только для того, что бы валидировать входные значения.

Новые блоги и статьи Все статьи Все блоги /
Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .	интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .
Сам себя обучал rest api anaschu 02.07.2026 Педагогический лайфхак: Почему чистый REST API для ученика намного круче, чем готовые библиотеки Когда мы отказались от капризного JAR-файла AnyLogic и переписали код на стандартный HttpClient,. . .	rest api anylogic - выполнение модели на своём русском сайте anaschu 02.07.2026 Как подружиться с AnyLogic Cloud API, победить провайдеров и развернуться Java-бэкенд в Docker на бесплатном хостинге: Двухдневный лог борьбы Всем привет! Хочу поделиться свежим (и довольно. . .	Где деньги лежат kumehtar 02.07.2026 Это - японская подводная лодка I-52 (тип C2, кодовое имя Momi) вышла из Японии в марте 1944 года с миссией в оккупированную немцами Францию (Лорьян). Это была одна из «Янаги»-миссий по обмену. . .	Krabik для WoW 3.3.5a, многоязычный AmbA 02.07.2026 Допилил бота, думаю что окончательно. Изменения: - добавлена многоязычность - добавлено снятие скриншотов - добавлено поддержание бафов хождения по воде (для жреца, дк и шамана) - и так, по. . .

@Babysitter 245 / 139 / 53 Регистрация: 23.11.2015 Сообщений: 394
	18.08.2016, 17:01
	я даже вспомнил про ssize_t, которые как раз и должен обозначать что-то типа "беззнаковое или минус единица в случае ошибки". но это что-то из posix'а .. 0

HighPredator 6045 / 2160 / 753 Регистрация: 10.12.2010 Сообщений: 6,005 Записей в блоге: 3
	18.08.2016, 18:25
	Voivoid, а что вам не нравится? Вполне себе верные вещи hoggy пишет. 0