Parkie02 Posted January 6, 2014 Share Posted January 6, 2014 Can somebody maybe assist me. I am new to php mysql. Someone told me my html code is terrible. Here are some things they told me. You have table rows that aren't inside <table>..</table> tags You are not using end tags (such as </p>, </tr> The piece of code above should be inside the html <body>..</body> tags and not after the closing </html> tag. Can someone help me to fix these problems in my code please. Here is my code: <?php session_start(); ?> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"><head> <title>Complaints</title> <meta name="description" content=""> <meta name="keywords" content=""> <meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"> <link href="css/style.css" rel="stylesheet" type="text/css"> </head> <body> <div class="main"> <div class="page"> <div class="header"> <div class="header-img"> <h1>Who Didn't Pay</h1> <p> </p> </div> <div class="menu"> <ul> <li><a href="index.php">Home</a></li> <li><a href="registration.php">Register</a></li> <li><a href="complaint.php">Complaint</a></li> <li><a href="search.php">Search</a></li> <li><a href="#">Contact Us</a></li> <li><a href="login.php">Login</a></li> <li><a href="logout.php">Logout</a></li> </ul> </div> </div> <div class="content"> <div class="left-panel"> <div class="left-panel-in"> <h2 class="title">Complaints:</h2> <p> </p> <p> </p> <p> <form method="post" action="allcompanies.php?go" id="showallform"> <p> </p> <p> </p> <table width="600" border="1" cellpadding="1" cellspacing="1"> <tr> <th>Complaints</th> <tr> </form> </p> </body></html> <?php //connect to the database $db=mysql_connect ("localhost", "root", "") or die ('I cannot connect to the database because: ' . mysql_error()); //-select the database to use $mydb=mysql_select_db("whodidntpay"); //-query the database table $comp = $_GET['comp']; $sql="SELECT complain FROM complaint c WHERE c.d_name = '" . mysql_real_escape_string($comp) . "'"; //-run the query against the mysql query function $result=mysql_query($sql); //-count results $numrows=mysql_num_rows($result); echo "<p>" .$numrows . " results found </p>"; while($debtor=mysql_fetch_array($result)) { echo "<tr>"; echo "<td>".$debtor['complain']."</td>"; echo "</tr>"; } echo "<br/><a href=\"companydetails.php?company=$comp\">See more details</a>"; echo "<tr>"; echo "<tr>"; mysql_close($db); ?> Quote Link to comment Share on other sites More sharing options...
TinyI Posted January 6, 2014 Share Posted January 6, 2014 Hey, a few problems. Mostly, you're not closing tags. If you look around the <table> tag you have, you'll see that you opened up <tr>, but haven't closed it. Another problem - the php code you have running is outside of all the HTML - which is fine for processing any data from a form, but when you want to get data to display, you should always run the PHP first and write the php after, adding in PHP excerpts here and there. If you need me to go any further into this, message me and I'll go through it with you Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.