in_array parameters

zimbabweo · April 2, 2013

I can't figure out what's wrong with this. The program works, but the deduplication doesn't work. The error I get is :

Warning: in_array() [function.in-array]: Wrong datatype for second argument in line 67.

I bolded line 67 below, which is an if (in_array ) statement.

in_array wants an array for the second argument, which I have, so don't see the problem. Any suggestions?

TIA
----------------------------------------------------------------------
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
   "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
   <title>Song Organizer</title>
   <meta http-equiv="content-type" content="text/html; charset=iso-8859-1" />
</head>
<body>
<h1>Song Organizer</h1>
<?php
   // handle parameters in the URL
   if (isset($_GET['action']))
   {
       if ((file_exists("SongOrganizer/songs.txt")) && (filesize("SongOrganizer/songs.txt") != 0))
       {
           $SongArray = file("SongOrganizer/songs.txt");

           // menu for Deduplication, Sort, Shuffle
           switch ($_GET['action'])
           {
               case   'Remove Duplicates':
                   $SongArray = array_unique($SongArray);
                   $SongArray = array_values($SongArray);
                   break;
               case   'Sort Ascending':
                   sort($SongArray);
                   break;
               case   'Shuffle':
                   shuffle($SongArray);
                   break;
           } // end of switch statement

           // save the song list after it has been modified
           if (count($SongArray) > 0)
           {
               $NewSongs = implode($SongArray);
               $SongStore = fopen("SongOrganizer/songs.txt", "wb");
               if ($SongStore === false)
                   echo "There was an error updating the song file\n";
               else
               {
                   fwrite($SongStore, $NewSongs);
                   fclose($SongStore);
               }
           }
           else
               unlink("SongOrganizer/songs.txt");
       }
   }

   // read song file data into the $ExistingSongs array
   if (isset($_POST['submit']))
   {
       $SongToAdd = stripslashes($_POST['SongName']) . "\n";
       $ExistingSongs = array();
       if (file_exists("SongOrganizer/songs.txt") && filesize("SongOrganizer/songs.txt") > 0)
       {
           $ExistingSongs = file("SongOrganizer/songs.txt");
       }
   }

   // check to see if the song name entered is already in the song list

// this is where it has a problem.
   if (in_array($SongToAdd, $ExistingSongs))
   {
       echo "<p>The song you entered already exists!<br />\n";
       echo "Your song was not added to the list.</p>";
   }
   // add new song to the list file
   else
   {
       $SongFile = fopen("SongOrganizer/songs.txt", "ab");
       if ($SongFile === false)
           echo "There was an error saving your message!\n";
       else
       {
           fwrite($SongFile, $SongToAdd);
           fclose($SongFile);
           echo "Your song has been added to the list.\n";
       }
   }

   // display the song list, or empty message if song list is empty
   if ((!file_exists("SongOrganizer/songs.txt")) || (filesize("SongOrganizer/songs.txt") == 0))
       echo "<p>There are no songs in the list.</p>\n";
   else
   {
       $SongArray = file("SongOrganizer/songs.txt");
       echo "<table border=\"1\" width=\"100%\" style=\"background-color:lightgray\">\n";
       foreach ($SongArray as $Song)
       {
           echo "<tr>\n";
           echo "<td>" . htmlentities($Song) . "</td>";
           echo "</tr>\n";
       }
       echo "</table>";
   }
?>


<p>
   <a href="SongOrganizer.php?action=Sort%20Ascending">SortSong List</a><br />
   <a href="SongOrganizer.php?action=Remove%20Duplicates">Remove Duplicate Songs</a><br />
   <a href="SongOrganizer.php?action=Shuffle">Shuffle Song List</a><br />
</p>


<form action="SongOrganizer.php" method="post">
   <p>Add a New Song</p>
   <p>Song Name: <input type="text" name="SongName" /></p>
   <p><input type="submit" name="submit" value="Add Song to List" />
   <input type="reset" name="reset" value="Reset Song Name" /></p>
</form>

</body>
</html>

requinix · April 2, 2013

Use [code] tags when posting code.

if (isset($_POST['submit']))

If that's not true then neither $SongToAdd nor $ExistingSongs will exist. You probably need to move the in_array() stuff up into that if block.

zimbabweo · April 2, 2013

Thanks for the reply. I just tried moving the in_array stuff into that block and it didn't work. In fact, I couldn't add new songs, so that breaks something else.

I think the submit button is working okay since when I add a new song name and click submit it does add it to the list. And my sort and shuffle buttons work. Everything works except the deduplication.

Sign In

in_array parameters

Recommended Posts

zimbabweo

Link to comment

Share on other sites

requinix

Link to comment

Share on other sites

zimbabweo

Link to comment

Share on other sites

Archived

Browse

Activity

Important Information