create a child object with initial data

JonB

@J-Hilk , @megido
Sorry, I did not see that QWidget *parent had an initialiser = nullptr from what the OP pasted as the definition. Then as @J-Hilk has offered, I would probably go for

Form::Form(const QString& date_initial_value = QDate::currentDate(), QWidget *parent = nullptr)

because of the convention (I believe) elsewhere that parent always comes last? If you do have any mandatory parameters they are going to have to precede parent (being optional) anyway, so it's probably best to get into this order habit.

megido

This post is deleted!

megido

header:

explicit Form(const QDate& date_initial_value = QDate::currentDate(), QWidget *parent = 0 );

cpp:

Form::Form(const QDate& date_initial_value, QWidget *parent):

works properly

Thanks

JonB

@megido
I am glad you have a solution. However, would you like to explain why you need that value to go ui->dateedit->setDate() in the constructor anyway? You have had to write extra, complicated constructor code to implement, and as I said it won't scale nicely if you add further parameters. I can't see why you would need to set its value as soon as the constructor, calling the setter explicitly after construction but before showing is the usual way to do things, so what's different about your case?

megido

@JonB
I just don't know another way if it exists

JonB

@megido
I can't go through it again. I said that your original way was fine, with no parameters

  Form *AF =  new Form;
  // here a few connects()
  AF->set_user_date(user_date);
  AF->show();

I told you that was fine. You then started saying the date was not initialised in the constructor. I replied:

I don't see what there is to say. Yes, the date is empty/default during the constructor, it gets changed to the desired date when the setter is called. What else would you expect, and what is the problem? Like I wrote, if you really need it to be set during the constructor you would have to add a parameter for it, but there is no obvious reason why you would need that.

You didn't say anything about why you have to have it initialized in the constructor. I do not see why you care about the log messages, they are no problem. You just started asking how to pass it as a parameter. So that is what I answered.

The best way is your original code. Answer why you care about when it is initilalized, what is unacceptable about this code?

jsulm

@megido Can you show how you implemented set_user_date()?

megido

@jsulm I have delete that. it was a private void

megido

@JonB yes, I have understood you. I too long time ago developed on Qt and forgot many. In this situation a just should call in the setter.

ui->dateedit->setDate(date);

but instead that I tried to update the variable before using it in the constructor. Because I thought what I really should predefine values in fields before the draw(It's usual for the WEB development).

jsulm

@megido said in create a child object with initial data:

I have delete that. it was a private void

Why not making it public and use instead of deleting?