in reply to Re^2: Moose type question
in thread Moose type question

That sounds like a union, so I tried it. Coercion works! ...but constraints don't?! Am I missing something?

It is not a union, it is an algebraic data type, there is a HUGE difference in terms of type safety and soundness.

Your second "Simple demonstration" seems to be a bug, I have brought it up in #moose-dev to discuss.

UPDATE:
Actually we figured out what is happening there, it is a fall through on an edge case. So if you look at this:

perl -Moose -e'has a=>(is=>"rw",isa=>"Undef|Undef",coerce=>1); my $x = + __PACKAGE__->new(a=>"abc"); print $x->a'
You will get Use of uninitialized value in print at -e line 3., which is happening because when Moose::Meta::TypeCoercion::Union attempts to coerce and doesnt find a proper coercion it returns 'undef'. Which in most cases would cause a type-check failure, but here actually passes the type check just fine.

Funny, because this perfectly illustrates why I really don't like undef. In this case we are trying to use it as a "no value", but your test case is using it as a valid value itself. The two ways of using it do not mesh well at all.

UPDATE(2):
See also Semipredicate Problem.

-stvn

Replies are listed 'Best First'.
Re^4: Moose type question
by ikegami (Patriarch) on Jun 15, 2010 at 15:38 UTC

    It is not a union, it is an algebraic data type

    I have no idea what that means. You described a type that could take one of two types of values, and that what ::Union does. I figured it would be a solution that falls within your mental model, and thus likely to work and to get promoted.

    Your second "Simple demonstration" seems to be a bug

    Filed as CPAN RT#58411

    Funny, because this perfectly illustrates why I really don't like undef.

    The problem has nothing to do with undef. The problem is sending control data in band. A value was reserved to mean "no coercion occurred". If the function had used zero instead of undef, it would suffer from the same bug.

    Undef is only a problem if you think it's ok for you and only you to use undef despite a long and continuing history of usefully using undef in Perl.

      I have no idea what that means.

      Algebraic Data types are mostly from functional languages. In spirit they are similar to C unions, but in practice they are very different. They are also type-safe meaning they fit into the theoretical end of the type system whereas in C the "type" is really more a way of telling the compiler how much space to allocate. If you have ever played with Haskell or OCaml you have likely encountered them, but they are most definitely not the same thing as a union (C style or Moose style).

      Filed as CPAN RT#58411

      Yes, thank you very much, we have been discussing the best way to solve this.

      The problem has nothing to do with undef.

      You are correct, it is the Semi-Predicate problem, which we clearly stepped into with this implementation.

      Undef is only a problem if you think it's ok for you and only you to use undef despite a long and continuing history of usefully using undef in Perl.

      Right, thats my problem though. That long and continuing history of use is inconsistent and at times contradictory. Just think of all the possible permutations of return;,  return undef; and  return (); and what each of them means in scalar and list context. Now there be dragons.

      -stvn

        we have been discussing the best way to solve this.

        An empty list cannot be placed in a scalar, so you could use an empty list.

        sub f { my $coercion_occurred = 1; # ... my $coerced_value = undef; # ... if ($coercion_occurred) { return $coerced_value; } else { return (); } } my $coercion_occurred = ($coerced_value) = f();