On Mon, 4 Jun 2012, Matt Simerson wrote:
> + my $count = $self->connection->notes('unrec_cmd_count') || 0;
> + $self->connection->notes('unrec_cmd_count',
> ++$count );
>
> Which does increment correctly.
>
> This also works (note the parens), which is how it was done before.
>
> + my $count = ($self->connection->notes('unrec_cmd_count') || 0) + 1;
> + $self->connection->notes('unrec_cmd_count',
> $count );
>
> Which one is easier to read and maintain? Both seem equal to the task.
I would prefer:
my $count = $self->connection->notes('unrec_cmd_count') || 0;
$self->connection->notes('unrec_cmd_count', $count + 1);
Do you have a reason for the unconventional whitespace formatting?
