Problem z wysyłaniem rekordów Opcje

[Mgorka]

Witam mam mały problemik mianowicie jeden formularz wysyła prawidło rekordy i wypełnia mi jedną z tabel bazy danych ale już drugi w ogóle nie chce wysłać rekordów do tej samej bazy danych. Korzystają z tego samego pliku .

[blooregard]

Stawiam na błąd w kodzie.

[xavaz]

Podeślij jakiś kod, co chcesz wstawiać, jak wygląda struktura bazy. Jak poprzednik stawiam na błąd w kodzie.

[Mgorka]

Plik który przetwarza wszystko,

[PHP] pobierz, plaintext 
<?php
require('config.php');
 
foreach ($_POST as $key => $value) {
  $$key = $value;
}
 
$conn = mysql_connect(SQL_HOST, SQL_USER, SQL_PASS)
  or die('Nie mogę połączyć się z bazą danych. ' . mysql_error());
mysql_select_db(SQL_DB, $conn);
 
switch ($action) {
  case "Utwórz postać":
    $sql = "INSERT IGNORE INTO char_zipcode (id, city, state) " .
           "VALUES ('$zip', '$city', '$state')";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    $sql = "INSERT INTO char_lair (id, zip_id, lair_addr) " .
           "VALUES (NULL, '$zip', '$address')";
    $result = mysql_query($sql) 
      or die(mysql_error());
    if ($result) {
      $lairid = mysql_insert_id($conn);
    }
 
    $sql = "INSERT INTO char_main (id,lair_id,alias,real_name,align) " .
           "VALUES (NULL, '$lairid', '$alias', '$name', '$align')";
    $result = mysql_query($sql) 
      or die(mysql_error());
    if ($result) {
      $charid = mysql_insert_id($conn);
    }
 
    if ($powers != "") {
      $val = "";
      foreach ($powers as $key => $id) {
        $val[] = "('$charid', '$id')";
      }
      $values = implode(',', $val);
      $sql = "INSERT IGNORE INTO char_power_link (char_id, power_id) " .
             "VALUES $values";
      $result = mysql_query($sql) 
        or die(mysql_error());
    }
 
    if ($enemies != '') {
      $val = "";
      foreach ($enemies as $key => $id) {
        $val[] = "('$charid', '$id')";
      }
      $values = implode(',', $val);
      if ($align = 'dobry') {
        $cols = '(good_id, bad_id)';
      } else {
        $cols = '(bad_id, good_id)';
      }
      $sql = "INSERT IGNORE INTO char_good_bad_link $cols " .
             "VALUES $values";
      $result = mysql_query($sql) 
        or die(mysql_error());
    }
 
    $redirect = 'charlist.php';
    break;
 
  case "Usuń postać":
    $sql = "DELETE FROM char_main, char_lair " .
           "USING char_main m, char_lair l " .
           "WHERE m.lair_id = l.id AND m.id = $cid";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    $sql = "DELETE FROM char_power_link WHERE char_id = $cid";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    $sql = "DELETE FROM char_good_bad_link " .
           "WHERE good_id = $cid OR bad_id = $cid";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    $redirect = 'charlist.php';
    break;
 
  case "Uaktualnij postać":
    $sql = "INSERT IGNORE INTO char_zipcode (id, city, state) " .
           "VALUES ('$zip', '$city', '$state')";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    $sql = "UPDATE char_lair l, char_main m " .
           "SET l.zip_id='$zip', l.lair_addr='$address', " .
           "alias='$alias', real_name='$name', align='$align' " .
           "WHERE m.id = $cid AND m.lair_id = l.id";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    $sql = "DELETE FROM char_power_link WHERE char_id = $cid";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    if ($powers != "") {
      $val = "";
      foreach ($powers as $key => $id) {
        $val[] = "('$cid', '$id')";
      }
      $values = implode(',', $val);
      $sql = "INSERT IGNORE INTO char_power_link (char_id, power_id) " .
             "VALUES $values";
      $result = mysql_query($sql) 
        or die(mysql_error());
    }
 
    $sql = "DELETE FROM char_good_bad_link " .
           "WHERE good_id = $cid OR bad_id = $cid";
    $result = mysql_query($sql) 
      or die(mysql_error());
 
    if ($enemies != '') {
      $val = "";
      foreach ($enemies as $key => $id) {
        $val[] = "('$cid', '$id')";
      }
      $values = implode(',', $val);
      if ($align == 'dobry') {
        $cols = '(good_id, bad_id)';
      } else {
        $cols = '(bad_id, good_id)';
      }
      $sql = "INSERT IGNORE INTO char_good_bad_link $cols " .
             "VALUES $values";
      $result = mysql_query($sql) 
        or die(mysql_error());
    }
 
    $redirect = 'charlist.php';
    break;
 
  case "Usuń moce":
    if ($powers != "") {
      $powerlist = implode(',', $powers);
 
      $sql = "DELETE FROM char_power WHERE id IN ($powerlist)";
      $result = mysql_query($sql) 
        or die(mysql_error());
 
      $sql = "DELETE FROM char_power_link " .
             "WHERE power_id IN ($powerlist)";
      $result = mysql_query($sql) 
        or die(mysql_error());
    }
 
    $redirect = 'poweredit.php';
    break;
 
  case "Dodaj moc":
    if ($newpower != '') {
      $sql = "INSERT IGNORE INTO char_power (id, power) " .
             "VALUES (NULL, '$newpower')";
      $result = mysql_query($sql) 
        or die(mysql_error());
    }
 
    $redirect = 'poweredit.php';
    break;
 
  default:
 
    $redirect = 'charlist.php';
}
header("Location: $redirect");
?>
[PHP] pobierz, plaintext 

formularz ma wstawiać dane użytkownika imię nazwisko pseudonim itd..

[blooregard]

To może jeszcze ten kod formularzy.

[Mgorka]

Formularz

[PHP] pobierz, plaintext 
<?php
require('config.php');
 
if (!isset($_GET['c']) || $_GET['c'] == '' || !is_numeric($_GET['c'])) {
  $char='0';
} else {
  $char = $_GET['c'];
}
$subtype = "Utwórz";
$subhead = "Wpisz dane postaci i kliknij przycisk " .
           "'$subtype postać.'";
$tablebg = '#EEEEFF';
 
$conn = mysql_connect(SQL_HOST, SQL_USER, SQL_PASS)
  or die('Nie mogę połączyć się z bazą danych. ' . mysql_error());
mysql_select_db(SQL_DB, $conn);
 
$sql = "SELECT id, power FROM char_power";
$result = mysql_query($sql)
  or die(mysql_error());
if (mysql_num_rows($result) > 0) {
  while ($row = mysql_fetch_array($result)) {
    $pwrlist[$row['id']] = $row['power'];
  }
}
 
$sql = "SELECT id, alias FROM char_main WHERE id != $char";
$result = mysql_query($sql) 
  or die(mysql_error());
if (mysql_num_rows($result) > 0) {
  $row = mysql_fetch_array($result);
  $charlist[$row['id']] = $row['alias'];
}
 
if ($char != '0') {
  $sql = "SELECT c.alias, c.real_name AS name, c.align, " .
         "l.lair_addr AS address, z.city, z.state, z.id AS zip " .
         "FROM char_main c, char_lair l, char_zipcode z " .
         "WHERE z.id = l.zip_id " .
         "AND c.lair_id = l.id " .
         "AND c.id = $char";
  $result = mysql_query($sql) 
    or die(mysql_error());
  $ch = mysql_fetch_array($result);
 
  if (is_array($ch)) {
    $subtype = "Uaktualnij";
    $tablebg = '#EEFFEE';
    $subhead = "Dokonaj edycji danych dla <i>" . $ch['alias'] . 
               "</i> i kliknij przycisk '$subtype postać.'";
 
    $sql = "SELECT p.id " .
           "FROM char_main c " .
           "JOIN char_power p " .
           "JOIN char_power_link pk " .
           "ON c.id = pk.char_id " .
           "AND p.id = pk.power_id " .
           "WHERE c.id = $char";
    $result = mysql_query($sql) 
      or die(mysql_error());
    if (mysql_num_rows($result) > 0) {
      while ($row = mysql_fetch_array($result)) {
        $powers[$row['id']] = 'selected';
      }
    }
 
    // pobierz listę wrogów postaci
    $sql = "SELECT n.id " .
           "FROM char_main c " .
           "JOIN char_good_bad_link gb " .
           "JOIN char_main n " .
           "ON (c.id = gb.good_id AND n.id = gb.bad_id) " .
           "OR (n.id = gb.good_id AND c.id = gb.bad_id) " .
           "WHERE c.id = $char";
    $result = mysql_query($sql) 
      or die(mysql_error());
    if (mysql_num_rows($result) > 0) {
      while ($row = mysql_fetch_array($result)) {
        $enemies[$row['id']] = 'selected';
      }
    }
  }
}
?>
 
<html>
<head>
<title>Edycja postaci</title>
</head>
<body>
<img src="CBA_Tiny.gif" align="left" hspace="10">
<h1>Witryna postaci<br />komiksowych</h1><br />
<h3><?php echo $subhead; ?></h3>
 
<form action="char_transact.php" name="theform" method="post">
<table border="0" cellpadding="15" bgcolor="<?php echo $tablebg; ?>">
  <tr>
    <td>Pseudonim postaci:</td>
    <td><input type="text" name="alias" size="41" 
          value="<?php if (isset($ch)) { echo $ch['alias']; } ?>">
    </td>
  </tr>
  <tr>
    <td>Rzeczywiste imię i nazwisko:</td>
    <td><input type="text" name="name" size="41" 
          value="<?php if (isset($ch)) { echo $ch['name']; } ?>">
    </td>
  </tr>
  <tr>
    <td>Moce:<br><font size="2" color="#990000">
      (Ctrl+klikniecie, aby<br>zaznaczyć wiele<br>mocy)</font>
    </td>
    <td>
      <select multiple name="powers[]" size="4">
<?php
foreach ($pwrlist as $key => $value) {
  echo "<option value=\"$key\" ";
  if (isset($powers) && array_key_exists($key,$powers)) {
    echo $powers[$key];
  }
  echo ">$value</option>\n";
}
?>
      </select>
    </td>
  </tr>
 
  <tr>
    <td>Lokalizacja kryjówki:<br><font size="2" color="#990000">
      (adres,<br>miasto, województwo, kod pocztowy)</font>
    </td>
    <td><input type="text" name="address" size="41" 
          value="<?php if (isset($ch)) { echo $ch['address']; } ?>"><br>
        <input type="text" name="city" 
          value="<?php if (isset($ch)) { echo $ch['city']; } ?>">
        <input type="text" name="state" size="2" 
          value="<?php if (isset($ch)) { echo $ch['state']; } ?>">
        <input type="text" name="zip" size="10" 
          value="<?php if (isset($ch)) { echo $ch['zip']; } ?>">
    </td>
  </tr>
 
  <tr>
    <td>Charakter:</td>
 
    <td>
      <input type="radio" name="align" value="dobry"
      <?php if (isset($ch)) { 
        echo($ch['align']=='dobry' ? ' checked' : ''); 
      } ?>>
      dobry<br>
      <input type="radio" name="align" value="zły"
      <?php if (isset($ch)) { 
        echo($ch['align']=='zły' ? ' checked' : ''); 
      } ?>>
      zły
    </td>
  </tr>
 
<?php if (isset($charlist) && is_array($charlist)) { ?>
  <tr>
    <td>Wrogowie:<br><font size="2" color="#990000">
      (Ctrl+kliknięcie, aby<br>zaznaczyć wielu<br>wrogów)</font>
    </td>
    <td>
      <select multiple name="enemies[]" size="4">
<?php
foreach ($charlist as $key => $value) {
  echo "<option value=\"$key\" ";
  if (isset($enemies)) {
    echo $enemies[$key];
  }
  echo ">$value</option>\n";
}
?>
      </select>
    </td>
  </tr>
<?php } ?>
  <tr>
    <td colspan="2">
      <input type="submit" name="action" 
        value="<?php echo $subtype; ?> postać">
      <input type="reset">
<?php if ($subtype == "Uaktualnij") { ?>
      &nbsp;&nbsp;&nbsp;&nbsp;
      <input type="submit" name="action" value="Usuń postać">
<?php } ?>
    </td>
  </tr>
</table>
<input type="hidden" name="cid" value="<?php echo $char; ?>">
</form>
<a href="charlist.php">Powróć do strony głównej</a>
</body>
</html>
[PHP] pobierz, plaintext 

[tehaha]

nie czytałem kodu ale wiem co to za książka bo sam ją kiedyś przerobiłem i niestety muszę Cię poinformować, że w kilku miejscach w tej książce jest błąd w przykładowych kodach i możliwe, że to właśnie dlatego Ci nie działa

aczkolwiek bardzo możliwe, że źle przepisałeś kod, na stronie wydawcy możesz pobrać pliki robocze do każdego działu, spróbuj użyć tamtych plików i zobacz czy dalej nie działa http://www.wrox.com/WileyCDA/WroxTitle/Beg...d-DOWNLOAD.html, jest tam zakładka Download Code

[Mgorka]

Musze ci przyznać że już w jednym miejscu znalazłem błąd w wcześniejszym rozdziale. A w zamian tej książki jaką polecasz??

[tehaha]

jak dla mnie ta książka była bardzo dobra, a błędy dodatkowo przyczyniły się do mojej edukacji bo zanim znalazłem błąd to cały tydzień analizowałem kod krok po kroku, po książce to już raczej manual php, manual mysql i artykuły na wortalach no i pisanie swoich skryptów:) poczytaj sobie jeszcze o zabezpieczaniu swoich skryptów przed atakami sql injection, xss i o walidacji danych bo niestety te tematy słabo były poruszane w tej książce

[Mgorka]

A czy coś może mieć z tym związek z kodowaniem pliku