Phpexcel cannot fix error: you tried to set a sheet active by the out of bounds index

Question

0.00/5 (No votes)

See more:

I have a PHP program that reads an XLS file's worksheets and converts every worksheet to a CSV file.

The error i get:
Fatal error: Uncaught PHPExcel_Exception: You tried to set a sheet active by the out of bounds index: 3. The actual number of sheets is 3. in C:\xampp\htdocs\Technocripa-php\PHPExcel\Classes\PHPExcel.php:695 Stack trace: #0 C:\xampp\htdocs\Technocripa-php\convertCSV.php(34): PHPExcel->setActiveSheetIndex(3) #1 C:\xampp\htdocs\Technocripa-php\convertCSV.php(19): convertXLStoCSV('uploads/Central...', 'Centralizator_f...') #2 {main} thrown in C:\xampp\htdocs\Technocripa-php\PHPExcel\Classes\PHPExcel.php on line 695

Code:

function convertXLStoCSV($infile, $outfile) // the function that converts the file
{
    $fileType = PHPExcel_IOFactory::identify($infile);
    $objReader = PHPExcel_IOFactory::createReader($fileType);

    $objReader->setReadDataOnly(true);
    $objPHPExcel = $objReader->load($infile);

    //$objPHPExcel->createSheet();

    $i = 0;
    while ($objPHPExcel->setActiveSheetIndex($i)){
        $objWorksheet = $objPHPExcel->getActiveSheet();

        $objWriter = PHPExcel_IOFactory::createWriter($objPHPExcel, 'CSV');
        $objWriter->save($outfile);

        $i++;

        //$sheetNames = $infile->getSheetNames();
    }
}
 ?>

What I have tried:

I have tried placing

$objPHPExcel->createSheet();

outside the loop, but the error persists but just modifies it's data:

Fatal error: Uncaught PHPExcel_Exception: You tried to set a sheet active by the out of bounds index: 4. The actual number of sheets is 4.

Posted 19-May-17 1:25am

fiftythr

Updated 17-Jun-20 5:46am

Add a Solution

2 solutions

Solution 2

hello thanks to your question I was able to solve the problem, you really were very close to the solution, it was just putting $ objPHPExcel-> createSheet (); inside the loop
I leave part of my code as an example:

for ($ j = 0; $ j <5; $ j ++) {
        // $ j = 0;

        if ($ j <> 0) {
            $ objPHPExcel-> createSheet ();
        }...
}

Posted 17-Jun-20 5:47am

Lucamen

Comments

CHill60 17-Jun-20 12:38pm

Actually the reason your code doesn't generate the exception has nothing to do with the loop. It's the

if ($ j <> 0) {

that stops the error

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

ZurdoDev · Accepted Answer · 2017-05-19T03:28:00

Solution 1

The error is very clear, isn't it? You are trying to activate a sheet that does not exist. Remember that indexes are 0 based so trying to access sheet with an index of 3 when there are only 3 sheets will not work. 3 sheets mean there are indexes 0, 1, and 2.

Posted 19-May-17 3:28am

ZurdoDev