Thread overview
How to make a generic function to take a class or struct by reference?
Mar 27, 2022
JN
Mar 27, 2022
drug
Mar 28, 2022
vit
Mar 28, 2022
Ali Çehreli
Mar 28, 2022
Ali Çehreli
March 27, 2022

I would like to have only one definition of getX if possible, because they both are doing the same thing. I can't remove the ref one, because without a ref it will pass the struct as a temporary and compiler won't like that.

import std.stdio;

struct Foo
{
    int x;

    void doStuff()
    {
        *getX(this) = 5;
    }
}

class Bar
{
    int x;

    void doStuff()
    {
        *getX(this) = 5;
    }
}

int* getX(T)(ref T t)
{
    return &t.x;
}

int* getX(T)(T t)
{
    return &t.x;
}

void main()
{
    Foo foo;
    Bar bar = new Bar();

    foo.doStuff();
    bar.doStuff();

    assert(foo.x == 5);
    assert(bar.x == 5);
}
March 27, 2022
Auto ref?

```D

int* getX(T)(auto ref T t)
{
    ...
```

March 27, 2022

On 3/27/22 12:27 PM, JN wrote:

>

int* getX(T)(T t)
{
    return &t.x;
}

Remove this one. It's a bad idea. It probably won't compile, but if it does, you will be using corrupted memory.

-Steve

March 28, 2022

On Sunday, 27 March 2022 at 16:27:44 UTC, JN wrote:

>

I would like to have only one definition of getX if possible, because they both are doing the same thing. I can't remove the ref one, because without a ref it will pass the struct as a temporary and compiler won't like that.

import std.stdio;

struct Foo
{
    int x;

    void doStuff()
    {
        *getX(this) = 5;
    }
}

class Bar
{
    int x;

    void doStuff()
    {
        *getX(this) = 5;
    }
}

int* getX(T)(ref T t)
{
    return &t.x;
}

int* getX(T)(T t)
{
    return &t.x;
}

void main()
{
    Foo foo;
    Bar bar = new Bar();

    foo.doStuff();
    bar.doStuff();

    assert(foo.x == 5);
    assert(bar.x == 5);
}

Try this:


int* getX(T)(return auto ref T t){
	static if(is(T == class))
		return &t.x;
	else static if(is(T == struct) && __traits(isRef, t))
		return &t.x;
	else
		static assert(0, "no impl " ~ T.stringof);
}

or:


int* getX(T)(return auto ref T t)
if(is(T == class) || (is(T == struct) && __traits(isRef, t))){
	return &t.x;
}
March 28, 2022
On 3/27/22 09:27, JN wrote:
> I would like to have only one definition of getX if possible, because
> they both are doing the same thing. I can't remove the ref one, because
> without a ref it will pass the struct as a temporary and compiler won't
> like that.

Combining all responses, the code at the bottom is a working example.

First, we should remove the by-value getX function but then class function fails expectedly:

class Bar
{
    int x;

    void doStuff()
    {
      *getX(this) = 5;  // <-- Compilation ERROR
    }
}

It may be worth going over why it fails to compile: 'this' happens to be of type Bar. Since classes are reference types, the 'this' is of type Bar (a reference) but it is an rvalue. (It is an rvalue because there is no one variable 'this' for this or any object.) rvalues cannot be passed by-reference; so the compilation fails.

So, assuming the by-value getX() is removed, the following would make the code compile:

class Bar
{
    int x;

    void doStuff()
    {
      auto this_ = this;
      *getX(this_) = 5;  // <-- Now compiles
    }
}

In that case we pass 'this_', which is clearly an lvalue and it can be passed by-reference. (lvalue becaues it is a variable sitting on the stack.)

But we don't want to do the above everywhere, so 'auto ref' is a solution. The reason is, it would copy the rvalue (the object reference) arguments and it wouldn't be an error because the copied rvalue would be another reference to the same class object and it would work.

import std.stdio;

struct Foo
{
    int x;

    void doStuff()
    {
      *getX(this) = 5;
    }
}

class Bar
{
    int x;

    void doStuff()
    {
      *getX(this) = 5;
    }
}

auto getX(T)(auto ref T t)
{
  return &t.x;
}

void main()
{
    Foo foo;
    Bar bar = new Bar();

    foo.doStuff();
    bar.doStuff();

    assert(foo.x == 5);
    assert(bar.x == 5);
}

Ali

March 28, 2022
On 3/27/22 22:32, vit wrote:

> int* getX(T)(return auto ref T t)
> if(is(T == class) || (is(T == struct) && __traits(isRef, t))){
>      return &t.x;
> }

I also think 'return' parameter is needed but I could not come up with an example where that 'return' provides any more safety. Compiler already warns without it. (I tried with and without -preview=dip1000 -preview=dip25).

Ali