Перегрузка конструктора, Передаются одинаковые типы, но разные по сути значения Опции

[QMainWindow]

Привет всем!
Столкнулся с такой проблемой: есть два конструктора класса

Plot2D(int topMargin, int rightMargin, int bottomMargin, int leftMargin, QPaintDevice *paintDevice);
Plot2D(int x, int y, int width, int height, QPaintDevice *paintDevice);

Видно, что в обоих передаваемые типы одинаковы, но по сути это совершенно разные переменные. Компилятор не пожет перегрузить мой конструктор. Как обычно выходят из подобных ситуаций? Добавить какие-то "отличительные" признаки не могу, т.к. они попросту не нужны.

[ilyabvt]

Как обычно выходят из подобных ситуаций?

Меняют объявления конструкторов. Иначе никак.

Компилятор не пожет перегрузить мой конструктор.

Я его прекрасно понимаю.
Вот объявление переменной:

Plot2D x(1, 2, 3, 4, image);

Угадайте какой конструктор я использовал?

[Алексей1153]

class Plot2D
{
    public:
    struct s_margins
    {
        int top___Margin;
        int right_Margin;
        int bottomMargin;
        int left__Margin

        s_margins()
        {
            *this=s_margins(0,0,0,0);
        }

        s_margins
        (
             int top___Margin
            ,int right_Margin
            ,int bottomMargin
            ,int left__Margin
        )
        :top___Margin(top___Margin)
        ,right_Margin(right_Margin)
        ,bottomMargin(bottomMargin)
        ,left__Margin(left__Margin)
        {
        }
    };

    struct s_placements
    {
        int x;
        int x;
        int w;
        int h

        s_placements()
        {
            *this=s_placements(0,0,0,0);
        }

        s_placements
        (
             int x
            ,int y
            ,int w
            ,int h
        )
        :x(x)
        ,y(y)
        ,w(w)
        ,h(h)
        {
        }
    };


    Plot2D(const s_margins& M, QPaintDevice *paintDevice);
    Plot2D(const s_placements& P, QPaintDevice *paintDevice);
}

[QMainWindow]

Ясно. Или QPaintDevice *paintDevice помять местами в одном...)

[Алексей1153]

QMainWindow, лучше сразу правильно делать )))

[BRE]

QMainWindow, лучше сразу правильно делать )))

Перед созданием заполнять структуру? Тяжелое наследие виндовс АПИ?

[Алексей1153]

BRE, апи тут ни при чём. Это во-первых. Во вторых, гораздо быстрее передать один указатель. В-третьих - удобнее, читабельнее и проще модифицировать, когда понадобится добавить ещё переменную

а в данном случае это вообще единственный выход

А вызов делать не сложнее:

было
F(x,y,z)

стало
F(s(x,y,z))

Сообщение отредактировал Алексей1153 - 16.3.2012, 14:17

[Влад]

Алексей1153, только вот тут лучше бы иначе:

  struct s_placements
    {
        // ..............
        s_placements()
        :x(0)
        ,y(0)
        ,w(0)
        ,h(0)
        {
        }

    };

или использовать делегирующий конструктор, если компилятор поддерживает C++11:

    struct s_placements
    {

        s_placements() : s_placements(0,0,0,0)
        {
        }

    };

[Алексей1153]

Влад, да, поленился )

а вот это

s_placements() : s_placements(0,0,0,0)

не приветствую!

Сообщение отредактировал Алексей1153 - 16.3.2012, 16:16

[BRE]

апи тут ни при чём. Это во-первых.

А что причем? Да, язык C++ не может иметь разные конструкторы с одинаковыми сигнатурами, но это же совсем не повод для таких решений.

Во вторых, гораздо быстрее передать один указатель.

А время заполнения структуры ты считаешь? А почему?

В-третьих - удобнее, читабельнее и проще модифицировать, когда понадобится добавить ещё переменную

Читабелнее? Тоже спорно.

F(s(x,y,z))

Читающему код придется разбираться, а что это за s такой? Объект это или функция, что-то возвращающая.

По мне, так вводить отдельную сущность на каждый чих это перебор. Если эту структуру можно выделить в отдельный класс - выделяй, но делать это для передачи параметров... это в лучших традициях "специалистов" микрософта.

Сообщение отредактировал BRE - 16.3.2012, 19:02