Improve the benchmark

diff --git a/samples/bench.pl b/samples/bench.pl
index 9b0f1735c5495155bae27be43c01254d36f751fe..28aa263dad40cba53e671b3aa8fae88346b4a87e 100644 (file)
--- a/samples/bench.pl
+++ b/samples/bench.pl
@@ -3,57 +3,98 @@
 use strict;
 use warnings;
 
-use Benchmark qw/cmpthese/;
+use Benchmark qw/:hireswallclock cmpthese/;
 
 use blib;
 
 my $count = -1;
 
-{
- my %h = (
-  a => 1,
- );
-
- cmpthese $count, {
-  fetch_hash_existing_av   => sub { $h{a} },
-  fetch_hash_existing_noav => sub { no autovivification; $h{a} },
- };
-}
+my @tests;
 
 {
  my %h = ();
 
- cmpthese $count, {
-  fetch_hash_nonexisting_av   => sub { $h{a} },
-  fetch_hash_nonexisting_noav => sub { no autovivification; $h{a} },
- };
+ push @tests, [
+  'Fetch a non-existing key from a hash',
+  {
+   av   => sub { $h{a} },
+   noav => sub { no autovivification; $h{a} },
+  }
+ ];
 }
 
 {
- my $x = {
-  a => 1,
- };
-
- cmpthese $count, {
-  fetch_hashref_existing_av   => sub { $x->{a} },
-  fetch_hashref_existing_noav => sub { no autovivification; $x->{a} },
- };
+ my %h = (a => 1);
+
+ push @tests, [
+  'Fetch an existing key from a hash',
+  {
+   av   => sub { $h{a} },
+   noav => sub { no autovivification; $h{a} },
+  }
+ ];
 }
 
 {
  my $x = { };
 
- cmpthese $count, {
-  fetch_hashref_nonexisting_av   => sub { $x->{a} },
-  fetch_hashref_nonexisting_noav => sub { no autovivification; $x->{a} },
- };
+ push @tests, [
+  'Fetch a non-existing key from a hash reference',
+  {
+   av          => sub { $x->{a} },
+   noav        => sub { no autovivification; $x->{a} },
+   noav_manual => sub { defined $x ? $x->{a} : undef },
+  }
+ ];
+}
+
+{
+ my $x = { a => 1 };
+
+ push @tests, [
+  'Fetch an existing key from a hash reference',
+  {
+   av          => sub { $x->{a} },
+   noav        => sub { no autovivification; $x->{a} },
+   noav_manual => sub { defined $x ? $x->{a} : undef },
+  }
+ ];
 }
 
 {
  my $x = { a => { b => { c => { d => 1 } } } };
 
- cmpthese $count, {
-  fetch_hashref4_existing_av   => sub { $x->{a}{b}{c}{d} },
-  fetch_hashref4_existing_noav => sub { no autovivification; $x->{a}{b}{c}{d} },
- };
+ push @tests, [
+  'Fetch a 4-levels deep existing key from a hash reference',
+  {
+   av          => sub { $x->{a}{b}{c}{d} },
+   noav        => sub { no autovivification; $x->{a}{b}{c}{d} },
+   noav_manual => sub { my $z; defined $x ? ($z = $x->{a}, defined $z ? ($z = $z->{b}, defined $z ? ($z = $z->{c}, defined $z ? $z->{d} : undef) : undef) : undef) : undef },
+  }
+ ];
+}
+
+{
+ my $x = { };
+ $x->{$_} = undef       for 100 .. 199;
+ $x->{$_} = { $_ => 1 } for 200 .. 299;
+ my $n = 0;
+
+ no warnings 'void';
+
+ push @tests, [
+  'Fetch 2-levels deep existing or non-existing keys from a hash reference',
+  {
+   inc         => sub { $n = ($n+1) % 300 },
+   av          => sub { $x->{$n}{$n}; $n = ($n+1) % 300 },
+   noav        => sub { no autovivification; $x->{$n}{$n}; $n = ($n+1) % 300 },
+   noav_manual => sub { my $z; defined $x ? ($z = $x->{a}, (defined $z ? $z->{b} : undef)) : undef; $n = ($n + 1) % 300 },
+  }
+ ];
+}
+
+for my $t (@tests) {
+ printf "--- %s ---\n", $t->[0];
+ cmpthese $count, $t->[1];
+ print "\n";
 }