Форум программистов, компьютерный форум CyberForum.ru

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

Восстановить пароль Регистрация
 
 
thechicho
0 / 0 / 0
Регистрация: 15.03.2015
Сообщений: 14
Завершенные тесты: 1
17.08.2016, 10:31     Все ли хорошо в этом коде #1
Предложите ваши варианты решения заданий
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
 Аватар для Avazart
6893 / 5133 / 250
Регистрация: 10.12.2010
Сообщений: 22,561
Записей в блоге: 17
18.08.2016, 16:43     Все ли хорошо в этом коде #21
Цитата Сообщение от _Ivana Посмотреть сообщение
нет индекса, удовлетворяющего условию - значит "нет", а не -1 - вы же сами против сишного наследия и за красоту и ясность абстракций?
Именно, но использование npos короче по записи, пускай не совсем ясно в общем случае, но именно для строк это вполне очевидно.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Babysitter
 Аватар для Babysitter
78 / 103 / 34
Регистрация: 23.11.2015
Сообщений: 315
Завершенные тесты: 1
18.08.2016, 17:01     Все ли хорошо в этом коде #22
я даже вспомнил про ssize_t, которые как раз и должен обозначать что-то типа "беззнаковое или минус единица в случае ошибки". но это что-то из posix'а ..
Voivoid
 Аватар для Voivoid
580 / 256 / 12
Регистрация: 31.03.2013
Сообщений: 1,283
18.08.2016, 18:12     Все ли хорошо в этом коде #23
Цитата Сообщение от hoggy Посмотреть сообщение
то, чего не должно быть, не должно быть в принципе.
Ага, аргумент уровня: "а давайте просто писать без багов"
HighPredator
 Аватар для HighPredator
5342 / 1725 / 320
Регистрация: 10.12.2010
Сообщений: 5,108
Записей в блоге: 3
18.08.2016, 18:25     Все ли хорошо в этом коде #24
Voivoid, а что вам не нравится? Вполне себе верные вещи hoggy пишет.
Voivoid
 Аватар для Voivoid
580 / 256 / 12
Регистрация: 31.03.2013
Сообщений: 1,283
18.08.2016, 18:42     Все ли хорошо в этом коде #25
Цитата Сообщение от HighPredator Посмотреть сообщение
Voivoid, а что вам не нравится? Вполне себе верные вещи hoggy пишет.
1) Не нравится предложение поменять int на size_t в коде цикла из оп-поста. Там надо или везде менять инты на size_t или ничего не трогать
2) Тут вопрос конечно спорный, но лично я предпочитаю использовать int вместо size_t ( там, где это конечно возможно и не требует cast'ов ). По причине более легкого отлова ошибок.
avgoor
562 / 352 / 83
Регистрация: 05.12.2015
Сообщений: 1,137
18.08.2016, 19:01     Все ли хорошо в этом коде #26
Цитата Сообщение от _Ivana Посмотреть сообщение
вы же сами против сишного наследия и за красоту и ясность абстракций?
А вот чисто интересно: nullptr - это ужасное наследие си или красивая и ясная абстракция?

Добавлено через 4 минуты
Цитата Сообщение от Voivoid Посмотреть сообщение
Тут вопрос конечно спорный, но лично я предпочитаю использовать int вместо size_t
В студии, кстати, в шаблонах кода обычный фор с size_t, а тайд реверсный с интом. Но я тоже предпочитаю не использовать беззнаковые типы там, где без них можно обойтись.
Voivoid
 Аватар для Voivoid
580 / 256 / 12
Регистрация: 31.03.2013
Сообщений: 1,283
18.08.2016, 19:12     Все ли хорошо в этом коде #27
Цитата Сообщение от _Ivana Посмотреть сообщение
В памяти всплывает светлый образ еще одного местного С++ гуру, который что-то писал про мэйбимонады А это снова они, родимые: нет индекса, удовлетворяющего условию - значит "нет", а не -1 - вы же сами против сишного наследия и за красоту и ясность абстракций?
Что сказать-то хотел? Когда писалась стандартная библиотека, в плюсах еще не было optional'ов (аналог maybe из haskell ). И появятся они лишь в C++17, зато в бусте они уже довольно давно
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
18.08.2016, 22:22     Все ли хорошо в этом коде
Еще ссылки по теме:

C++ Что в этом коде не правильно?
C++ Как пронумеровать текст в этом коде?
C++ Как дописать цикл в этом коде?

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
hoggy
5031 / 2114 / 403
Регистрация: 15.11.2014
Сообщений: 4,799
Завершенные тесты: 1
18.08.2016, 22:22     Все ли хорошо в этом коде #28
Цитата Сообщение от Avazart Посмотреть сообщение
И тем не менее это условность/договоренность что npos это -1 или макс значение, константа или еще что либо.
нет никаких договоренностей.
вы не можете закладываться на конкретное значение npos
и вас вообще не должно это никак волновать.

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

Добавлено через 2 минуты
Цитата Сообщение от Voivoid Посмотреть сообщение
Ага, аргумент уровня: "а давайте просто писать без багов"

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

Цитата Сообщение от hoggy Посмотреть сообщение
корректно оформленная - корректна с точки зрения стандарта,
и гарантирует инвариант.
вам не нужен знаковый инт там, где по смыслу нужно беззнаковое,
только для того, что бы валидировать входные значения.
Yandex
Объявления
18.08.2016, 22:22     Все ли хорошо в этом коде
Ответ Создать тему
Опции темы

Текущее время: 15:34. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru