function not printing

[gilgalbiblewhee]

I don't know if I missed something. The function is not printing at all.

$contents_of_page = file_get_contents('../bible1.html');

preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
$totaltds = count($tdInnerHTML[1]);
print_r($tdInnerHTML[1]);//prints all TD contents 

/********************************************************************************/
function recValues(){
    $tdValues = array();
    $tdValues = "";
    for($j = 0; $j < $totaltds; $j++){
        if($j % 10 != 0){
            $tdValues .= "'".$tdInnerHTML[1][$j];
            if($j < 9){
                $tdValues .= "', ";
                }else{
                $tdValues .= "'";
                }
            }
        }
    return $tdValues;
    }
    echo "<span style='font-weight: bold;'>hi".recValues()."hi</span><br />\n"; 

[visualAd]

That's because you use the return statement before the echo. I.e: the function exits before it can print anything and the echo statement is never executed.

[gilgalbiblewhee]

Ok so I moved some things in the function.
But another problem appears.The last $tdValues .= "'".$tdInnerHTML[1][$j]; shouldn't have ,' that follows it. I'm avoiding the multiples of 10 if($j % 10 != 0){

function recValues(){
$contents_of_page = file_get_contents('../bible1.html');
preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
$totaltds = count($tdInnerHTML[1]);
print_r($tdInnerHTML[1]);//prints all TD contents 
    $tdValues = array();
    $tdValues = "";
    for($j = 0; $j < $totaltds; $j++){
        if($j % 10 != 0){
            $tdValues .= "'".$tdInnerHTML[1][$j];
            $tdValues .= "', ";
            if($j % 10 != 0 + 9){
                $tdValues .= "'";
                $tdValues .= "<br />";
                }
            }
        }
    return $tdValues;
    }
    echo "<span style='font-weight: bold;'>".recValues()."</span><br />\n"; 

So it's this part of the code that has to be fixed. I want to insert at every (multiple of 10) + 9. In other words 19, 29, 39...

            if($j % 10 != 0 + 9){
                $tdValues .= "'";
                $tdValues .= "<br />";
                }
            } 

[visualAd]

Neither your code or you explanation make much sense. Please indent your code; I have done this for you below, it makes it easier for everyone and will mean your problem will get solved quicker.

Have a look at the explanations below:

<?php
function recValues()
{
    $contents_of_page = file_get_contents('../bible1.html');
    preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
    $totaltds = count($tdInnerHTML[1]);
    print_r($tdInnerHTML[1]); //prints all TD contents
    
    $tdValues = array(); /*** this line is pointless, you reset
            the value of the same variable on the line below ***/
    
        $tdValues = "";
    
    for($j = 0; $j < $totaltds; $j++){
        if($j % 10 != 0) {
            /*** $j is >= 1 and <= 9 ***/

                    $tdValues .= "'".$tdInnerHTML[1][$j];
                    $tdValues .= "', ";
            
            if($j % 10 != 0 + 9){ /*** isn't 0 + 9 == 9 ??? ***/
                /*** $j must be >= 1 and <= 8 */
                        $tdValues .= "'";
                        $tdValues .= "<br />";
                    }
                }
        }
        
    return $tdValues;
}

echo "<span style='font-weight: bold;'>".recValues()."</span><br />\n"; 

?>

I added the indentation too. The first if statement in your look will get executed for the values of $j (1,2,3,4,5,6,7,8,9,11,12,13,14,15,6,17,18,19,21 ....). The second if statement will get executed for these values of $j (1,2,3,4,5,6,7,8,11,12,13,14,15,16,17,18,21).

[gilgalbiblewhee]

Neither your code or you explanation make much sense. Please indent your code; I have done this for you below, it makes it easier for everyone and will mean your problem will get solved quicker.

Have a look at the explanations below:

<?php
function recValues()
{
    $contents_of_page = file_get_contents('../bible1.html');
    preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
    $totaltds = count($tdInnerHTML[1]);
    print_r($tdInnerHTML[1]); //prints all TD contents
    
    $tdValues = array(); /*** this line is pointless, you reset
            the value of the same variable on the line below ***/
    
        $tdValues = "";
    
    for($j = 0; $j < $totaltds; $j++){
        if($j % 10 != 0) {
            /*** $j is >= 1 and <= 9 ***/

                    $tdValues .= "'".$tdInnerHTML[1][$j];
                    $tdValues .= "', ";
            
            if($j % 10 != 0 + 9){ /*** isn't 0 + 9 == 9 ??? ***/
                /*** $j must be >= 1 and <= 8 */
                        $tdValues .= "'";
                        $tdValues .= "<br />";
                    }
                }
        }
        
    return $tdValues;
}

echo "<span style='font-weight: bold;'>".recValues()."</span><br />\n"; 

?>

I added the indentation too. The first if statement in your look will get executed for the values of $j (1,2,3,4,5,6,7,8,9,11,12,13,14,15,6,17,18,19,21 ....). The second if statement will get executed for these values of $j (1,2,3,4,5,6,7,8,11,12,13,14,15,16,17,18,21).

I did the following changes and it worked:

function recValues(){
    $contents_of_page = file_get_contents('../bible1.html');
    preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
    $totaltds = count($tdInnerHTML[1]);
    print_r($tdInnerHTML[1]);//prints all TD contents 
    $tdValues = array();
    $tdValues = "";
    for($j = 0; $j < $totaltds; $j++){
        if($j % 10 != 0){
            $tdValues .= "'".$tdInnerHTML[1][$j];
            if($j % 10 == 9){
                $tdValues .= "'";
                $tdValues .= "<br />";
                }else{
                $tdValues .= "', ";
                }
            }
        }
    return $tdValues;
    }
echo "<span style='font-weight: bold;'>".recValues()."</span><br />\n"; 

I get the following result:

'1', '1', 'gn', 'Genesis', '1', '1', '1', '1', 'In the beginning God created the heaven and the earth.'
'1', '1', 'gn', 'Genesis', '1', '1', '2', '2', 'And the earth was without form, and void; and darkness was upon the face of the deep. And the Spirit of God moved upon the face of the waters.'
'1', '1', 'gn', 'Genesis', '1', '1', '3', '3', 'And God said, Let there be light: and there was light.'
'1', '1', 'gn', 'Genesis', '1', '1', '4', '4', 'And God saw the light, that it was good: and God divided the light from the darkness.'
...

But what I want to add is $k being equal to each line so that:

echo "<span style='font-weight: bold;'>".recValues()."</span><br />\n"; 

will not give the whole chunk of results. I'm not sure how I should manage to do that.

[visualAd]

You are going to have to be move specific, I don't understand what you mean.

[gilgalbiblewhee]

You are going to have to be move specific, I don't understand what you mean.

I want to create an array of $tdValues:

$tdValues[0]
$tdValues[1]
$tdValues[2]
... 

each $tdValues[$k] should have one line:

'1', '1', 'gn', 'Genesis', '1', '1', '1', '1', 'In the beginning God created the heaven and the earth.'

Because later on I have:

for($k=0; $k < $totaltds; $k++)
    $sql = "INSERT INTO bible (".fields().") VALUES (".recValues().")";
    echo $sql."<br />\n";
} 

There is recValues(). $tdValues[$k] needs to somehow replace that.

[visualAd]

Just declare $tdValues as an array. $tdValues = array(); And instead of using the string concatenation operator .= to add to it, use the array append operator [].

    for($j = 0; $j < $totaltds; $j++){
        $line= '';
  
        /* build line data here */
        $tdValues[] = $line; // add a new element to the array.
    } 

P.s. are you taking this data from another site? If so, then please ensure that you have the permission of the site owner to copy their data. Otherwise you are breaking copyright law.

[gilgalbiblewhee]

Just declare $tdValues as an array. $tdValues = array(); And instead of using the string concatenation operator .= to add to it, use the array append operator [].

    for($j = 0; $j < $totaltds; $j++){
        $line= '';
  
        /* build line data here */
        $tdValues[] = $line; // add a new element to the array.
    } 

P.s. are you taking this data from another site? If so, then please ensure that you have the permission of the site owner to copy their data. Otherwise you are breaking copyright law.

No I'm not. What happened was that I accidentally erased my db table. Fortunately I had a copy of the table in HTML. So I'm trying to reconvert it to a db table.

Another problem was that I don't know how to upload a db table to an online server and connect to it. But I know how to connect to a created db on the online server. So I need this technique in both cases. I'm almost done with this.

I find that the solution you gave is getting complicated. This means I have to do a lot more reading than I can bear.

I decided to rework the function from scratch. THis is what I came up with:

function recValues(){
    $contents_of_page = file_get_contents('../bible1.html');
    preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
    $totaltds = count($tdInnerHTML[1]);
    $line = 0;
    $strLine = array();
    for($k=0; $k < $totaltds; $k++){
        if($k % 10 == 0){
            print("line ");
            $strLine[$line] = 
            }
        if($k % 10 != 0){
            $j = 0;
            $tdValues = array();
            $tdValues[$j] = $tdInnerHTML[1][$k];
            print("'");
            print("<span style='font-weight: bold; color: red'>".$tdValues[$j]."</span>");
            //print($tdInnerHTML[1][$k]);
            if($k % 10 == 9){
                print("'<br />\n");
            }else{
                print("', ");
            }
            //return $tdValues[$j];
            $j++;
            $line++;
        }
    } 

But I'm stuck. I need $strLine[$line] = to be equal to all this:

if($k % 10 != 0){
            $j = 0;
            $tdValues = array();
            $tdValues[$j] = $tdInnerHTML[1][$k];
            print("'");
            print("<span style='font-weight: bold; color: red'>".$tdValues[$j]."</span>");
            //print($tdInnerHTML[1][$k]);
            if($k % 10 == 9){
                print("'<br />\n");
            }else{
                print("', ");
            } 

[gilgalbiblewhee]

How about making $strLine[$line] equal to a function which would have:

if($k % 10 != 0){
            $j = 0;
            $tdValues = array();
            $tdValues[$j] = $tdInnerHTML[1][$k];
            print("'");
            print("<span style='font-weight: bold; color: red'>".$tdValues[$j]."</span>");
            //print($tdInnerHTML[1][$k]);
            if($k % 10 == 9){
                print("'<br />\n");
            }else{
                print("', ");
            } 

How does that work?

[gilgalbiblewhee]

I almost succeeded into building my sql statement. But here is the result:

line
0 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ()
'1',
'1', '1',
'1', '1', 'gn',
'1', '1', 'gn', 'Genesis',
'1', '1', 'gn', 'Genesis', '1',
'1', '1', 'gn', 'Genesis', '1', '1',
'1', '1', 'gn', 'Genesis', '1', '1', '1',
'1', '1', 'gn', 'Genesis', '1', '1', '1', '1',
'1', '1', 'gn', 'Genesis', '1', '1', '1', '1', 'In the beginning God created the heaven and the earth.'
9 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ('1', '1', 'gn', 'Genesis', '1', '1', '1', '1', 'In the beginning God created the heaven and the earth.')
line
'1',
'1', '1',
'1', '1', 'gn',
'1', '1', 'gn', 'Genesis',
'1', '1', 'gn', 'Genesis', '1',
'1', '1', 'gn', 'Genesis', '1', '1',
'1', '1', 'gn', 'Genesis', '1', '1', '2',
'1', '1', 'gn', 'Genesis', '1', '1', '2', '2',
18 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ('1', '1', 'gn', 'Genesis', '1', '1', '2', '2', )
'And the earth was without form, and void; and darkness was upon the face of the deep. And the Spirit of God moved upon the face of the waters.'
line
'1',
'1', '1',
'1', '1', 'gn',
'1', '1', 'gn', 'Genesis',
'1', '1', 'gn', 'Genesis', '1',
'1', '1', 'gn', 'Genesis', '1', '1',
'1', '1', 'gn', 'Genesis', '1', '1', '3',
27 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ('1', '1', 'gn', 'Genesis', '1', '1', '3', )
'3',
'3', 'And God said, Let there be light: and there was light.'

As you see there is a repetition. Instead it should be:

line

9 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ('1', '1', 'gn', 'Genesis', '1', '1', '1', '1', 'In the beginning God created the heaven and the earth.')
line

27 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ('1', '1', 'gn', 'Genesis', '1', '1', '1', '1', 'And the earth was without form, and void; and darkness was upon the face of the deep. And the Spirit of God moved upon the face of the waters.')
line

36 INSERT INTO bible (book, book_spoke, recordType, book_title, chapter, chapter_spoke, verse, verse_spoke, text_data) VALUES ('1', '1', 'gn', 'Genesis', '1', '1', '3', '3',
'And God said, Let there be light: and there was light.')

function recValues(){
    $contents_of_page = file_get_contents('../bible1.html');
    preg_match_all("#<td.*>(.+)</td#Ui", $contents_of_page, $tdInnerHTML);
    $totaltds = count($tdInnerHTML[1]);
    $line = 0;
    $strLine = array();
    for($k=0; $k < $totaltds; $k++){
        if($k % 10 == 0){
            print("line ");
            $strLine[$line] = "";
        }
        if($k % 10 != 0){
            $j = 0;
            $tdValues = array();
            $tdValues[$j] = $tdInnerHTML[1][$k];
            $strLine[$line] .= "'";
            //print("'");
            $strLine[$line] .= $tdValues[$j];
            //print("<span style='font-weight: bold; color: red'>".$tdValues[$j]."</span>");
            //print($tdInnerHTML[1][$k]);
            if($k % 10 == 9){
                $strLine[$line] .= "'";
                //print("'<br />\n");
            }else{
                $strLine[$line] .= "', ";
                //print("', ");
            }
        }
        
        print($strLine[$line]."<br />\n");
        //return $strLine[$line];
        $j++;
        if($k % 9 == 0){
            
            $sql = "INSERT INTO bible (".fields().") VALUES (".$strLine[$line].")";
            print("<span style='color: red;'>".$k." ".$sql."</span><br />\n");
            $line++;
        }
        
    }
    
}