Hello again mnakkach,

I was thinking again and again the case and I found my previous solution Re: Merging 2 xlsx files to your problem not correct.

Why it is not correct let's assume that you have 2 lines on first spreadsheet all in colum A and also on second spreadsheet two lines on column A. The old script will work just fine as long as all the data are still in column A. In case that the second spreadsheet (the appending) data are in column B, even if the column B has only on line e.g. B1 then after the process the B1 will be passed to B5. Why? simply because I did a wrong calculation of the keys in the hash.

To resolve this issue, but unfortunately only for columns A-Z see the solution bellow:

#!/usr/bin/perl
use strict;
use warnings;
use Data::Dumper;
use feature 'say';
use Excel::Writer::XLSX;
use Spreadsheet::Read qw( ReadData );

# read from all spreadsheets
my @spreadsheets = ( 'test_file1.xlsx' , 'test_file2.xlsx' );

my %HoH;
foreach my $spreadsheet (@spreadsheets) {
    my $book = ReadData ($spreadsheet);

    my @sheets = @$book[0]->{sheets};

    foreach my $sheet (@sheets) {
    my @rows = Spreadsheet::Read::rows($book->[$sheet]);
    $HoH{$spreadsheet}{$sheet} = \@rows;
    }
}

my $outputXLSX = 'simple.xlsx';
my $workbook  = Excel::Writer::XLSX->new( $outputXLSX );
my $worksheet = $workbook->add_worksheet();

my %hash;
foreach my $file ( sort keys %HoH ) {
    foreach my $sheet ( sort keys %{ $HoH{$file} } ) {
    $workbook->add_worksheet() if $sheet > 1;
    foreach my $i (1 .. scalar @{$HoH{$file}{$sheet}}) {
        foreach my $j (1 .. scalar @{$HoH{$file}{$sheet}[$i-1]}) {
        next if ($HoH{$file}{$sheet}[$i-1][$j-1] // '') eq '';
        if ( grep { $_ eq chr(64+$j) } keys %hash ) {
            if (grep { $_ eq $i} sort keys %{$hash{chr(64+$j)}}) {
            my $incrementRow = $i;
            while (grep { $_ eq $i} sort keys %{$hash{chr(64+$j)}}){
                $incrementRow++;
                last if ($i ne $incrementRow);
            }
            $worksheet->write(chr(64+$j) . $incrementRow,
                      ($HoH{$file}{$sheet}[$i-1][$j-1] // ''));
            $hash{chr(64+$j)}{$incrementRow} =
                ($HoH{$file}{$sheet}[$i-1][$j-1] // '');
            }
            else {
            $worksheet->write(chr(64+$j) . $i,
                      ($HoH{$file}{$sheet}[$i-1][$j-1] // ''));
            $hash{chr(64+$j)}{$i} =
                ($HoH{$file}{$sheet}[$i-1][$j-1] // '');
            }
        }
        else {
            # say "Column Index: " . $j . " Column Character: " . chr(
+64+$j) . " Text: " .  ($HoH{$file}{$sheet}[$i-1][$j-1] // '');
            $hash{chr(64+$j)}{$i} =
            ($HoH{$file}{$sheet}[$i-1][$j-1] // '');
            $worksheet->write(chr(64+$j) . $i,
                      ($HoH{$file}{$sheet}[$i-1][$j-1] // ''));
        }
        }
    }
    }
}

print Dumper \%hash;

$workbook->close;

# remove output file if you are
# going to rerun the script
# I used it for debuging purposes
# unlink($outputXLSX);

__END__

$ perl excel.pl
$VAR1 = {
          'Z' => {
                   '2' => 'Line Z1 File 2',
                   '1' => 'Line Z1 File 1',
                   '5' => 'Line Z5 File 1',
                   '6' => 'Line Z5 File 2'
                 },
          'A' => {
                   '2' => 'Line A1 file 2',
                   '3' => 'Line A2 file 2',
                   '1' => 'Line A1 file 1'
                 },
          'B' => {
                   '3' => 'Line B2 file 2',
                   '1' => 'Line B1 File 1',
                   '2' => 'Line B1 file 2'
                 }
        };

[download]

Update: I finally managed to put it together a solution that works also for longer columns like AA. Hope this script will help other people in future.

#!/usr/bin/perl
use strict;
use warnings;
use Data::Dumper;
use Excel::Writer::XLSX;
use Spreadsheet::Read qw( ReadData cr2cell );

# read from all spreadsheets
my @spreadsheets = ( 'test_file1.xlsx' , 'test_file2.xlsx' );

my %HoH;
foreach my $spreadsheet (@spreadsheets) {
    my $book = ReadData ($spreadsheet);

    my @sheets = @$book[0]->{sheets};

    foreach my $sheet (@sheets) {
    my @rows = Spreadsheet::Read::rows($book->[$sheet]);
    $HoH{$spreadsheet}{$sheet} = \@rows;
    }
}

my $outputXLSX = 'simple.xlsx';
my $workbook  = Excel::Writer::XLSX->new( $outputXLSX );
my $worksheet = $workbook->add_worksheet();

my %hash;
foreach my $file ( sort keys %HoH ) {
    foreach my $sheet ( sort keys %{ $HoH{$file} } ) {
    $workbook->add_worksheet() if $sheet > 1;
    foreach my $i (1 .. scalar @{$HoH{$file}{$sheet}}) {
        foreach my $j (1 .. scalar @{$HoH{$file}{$sheet}[$i-1]}) {
        next if ($HoH{$file}{$sheet}[$i-1][$j-1] // '') eq '';
        my $cell = cr2cell ($j, $i);
        my ($column, $row) = split /(?<=\D)(?=\d)/, $cell;
        if ( grep { $_ eq $column } keys %hash ) {
            if (grep { $_ eq $row} sort keys %{$hash{$column}}) {
            my $incrementRow = $row;
            while (grep { $_ eq $row} sort keys %{$hash{$column}}){
                $incrementRow++;
                last if ($row ne $incrementRow);
            }
            write2ExcelFile($column,
                    $incrementRow,
                    ($HoH{$file}{$sheet}[$i-1][$j-1] // ''));
            }
            else {
            write2ExcelFile($column,
                    $row,
                    ($HoH{$file}{$sheet}[$i-1][$j-1] // ''));
            }
        }
        else {
            write2ExcelFile($column,
                    $row,
                    ($HoH{$file}{$sheet}[$i-1][$j-1] // ''));
        }
        }
    }
    }
}

print Dumper \%hash;

$workbook->close;

sub write2ExcelFile {
    my ( $column , $row , $data ) = @_;
    $worksheet->write($column . $row, $data);
    $hash{$column}{$row} = $data;
    return;
}

# remove output file if you are
# going to rerun the script
# I used it for debuging purposes
# unlink($outputXLSX);

__END__

$ perl excel.pl
$VAR1 = {
          'B' => {
                   '3' => 'Line B2 file 2',
                   '1' => 'Line B1 File 1',
                   '2' => 'Line B1 file 2'
                 },
          'AA' => {
                    '2' => 'Line AA1 File 2',
                    '1' => 'Line AA1 File 1'
                  },
          'A' => {
                   '3' => 'Line A2 file 2',
                   '2' => 'Line A1 file 2',
                   '1' => 'Line A1 file 1'
                 },
          'Z' => {
                   '1' => 'Line Z1 File 1',
                   '2' => 'Line Z1 File 2',
                   '5' => 'Line Z5 File 1',
                   '6' => 'Line Z5 File 2'
                 }
        };

[download]

I will try to resolve the issue with the columns but so far I am stack, I will update as soon as I have resolved it.

Hope this helps, BR.

Seeking for Perl wisdom...on the process of learning...not there...yet!

In reply to Re: Merging 2 xlsx files (UPDATED) by thanos1983
in thread Merging 2 xlsx files by mnakkach

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":


  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.