Почему MySQL вставляет 3 значения вместо 1? - PullRequest
1 голос
/ 23 октября 2009

Эй, ребята, у меня проблемы с тем, что кажется довольно простым делом. Позвольте мне рассказать вам, что я пытаюсь сделать, и затем я вставлю свой закомментированный код.

Я пытаюсь:

  1. Подключение к MySQL
  2. Вставить инкрементное значение (идентификатор)
  3. Временно сохранить это значение [я использовал mysql_insert_id()]
  4. Загрузите и измените размер изображения с помощью class.upload.php и присвойте ему имя, используя последний вставленный идентификатор
  5. Показать загруженный файл и разрешить пользователю скопировать его в миниатюру

Здесь весь ад распадается.

  1. Сохраните миниатюру, присвоив ей имя, используя последний вставленный идентификатор

Я думаю, что на самом деле происходит:

Запрос INSERT повторяется для каждого шага (загрузка, jcrop и отображение миниатюр), поэтому он вставляет 3 значения. Итак, что происходит, это дает [изначально] загруженному имени изображения 3, но ищет изображение 4 для создания миниатюры, а затем страница, которая показывает ошибку (которая должна показывать мне миниатюру), вставляет другую строку в MySQL. После всего этого, когда я в следующий раз захочу загрузить изображение, он получает начальный идентификатор загрузки 6.

Я схожу с ума, пытаясь использовать разные методы вставки строки, но кажется, что имя файла не проходит через каждый из этапов, используя любой другой метод. Я пытался использовать:

  • метод include_once для вставки строки

  • разделение процесса на 2 страницы, где t первая страница вставляет строку, затем отправляет его, используя скрытое поле

  • перемещение вставки mysql в одну из другие шаги (не проходит последний идентификатор в любой другой шаг)

Вот мой код:

<?php

   //connect to MySQL
   $dbname = "****";
   $username = "****";
   $password = "****";
   $hostname = "localhost"; 

  //connection to the database
  $dbhandle = mysql_connect($hostname, $username, $password)
   or die("<br/><h1>Unable to connect to MySQL, please contact support at support@michalkopanski.com</h1>");

  //select a database to work with
  $selected = mysql_select_db($dbname, $dbhandle)
    or die("Could not select database.");

    //insert an auto_incrementing value into the 'pictures' table
    if (!$result = mysql_query("INSERT INTO `pictures` VALUES ('')"))
  echo 'mysql error: '.mysql_error();

  $filename = mysql_insert_id();

?>
<?php

//submit coordinates of jcrop preview
if ($_POST['submit_coords'])
{

$targ_w = 180; // desired width of thumbnail
$targ_h = 60; // desired height of thumbnail
$jpeg_quality = 90; // desired jpeg quality
$src = 'f/'.$filename.'.jpg'; // get path to the resized image

// Fetch the uploaded jpeg file
$img_r = imageCreateFromJpeg($src);

//Use these proportions
$dst_r = ImageCreateTrueColor( $targ_w, $targ_h );

// Get coordinates from jcop, and create thumbnail
imagecopyresampled($dst_r,$img_r,0,0,$_POST['x'],$_POST['y'], $targ_w,$targ_h,$_POST['w'],$_POST['h']);

// save the thumbnail and echo the result
$output_file = 't/'.$filename.'.jpg';
imagejpeg($dst_r, $output_file, $jpeg_quality);
$results = <<<EOT
<img src="t/$filename.jpg"/>
EOT;
echo $results;
die();
}

// upload & resize mig image
if ($_POST['upload_image'] && $_FILES['image_upload'])
{
 require_once '../lib/jcrop/class.upload.php';

 $ih = new Upload($_FILES['image_upload']);
 if ($ih->uploaded) {
  //save full size
  $ih->file_new_name_body = $filename;
  $ih->file_new_name_ext = 'jpg';
  $ih->image_resize = true;
  $ih->image_x = 860;
  $ih->image_ratio_y = true;
  $ih->process('f/');
  if ($ih->processed) {

   $x = $ih->image_dst_x;
   $y = $ih->image_dst_y;
   echo 'image resized';

   $ih->clean();
  } else {
   echo 'error : ' . $ih->error;
  }

 }

 //if succesful show the thumbnail preview
 $output = <<<EOT
<img src="f/$filename.jpg" id="jcrop" />
<br />
<div style="overflow: hidden; width: 180px; height: 60px;">
<img src="f/$filename.jpg" id="preview" />
</div>
EOT;

// Show uploaded image, and allow jcrop
$head = <<<EOT
<link rel="stylesheet" href="../lib/jcrop/jquery.Jcrop.css" type="text/css" />
<script type="text/javascript" src="../lib/jcrop/jquery.min.js"></script>
<script type="text/javascript" src="../lib/jcrop/jquery.Jcrop.min.js"></script>
<script>
$(function(){
 function showPreview(coords)
 {
  var rx = 180 / coords.w;
  var ry = 60 / coords.h;

  $('#preview').css({
   width: Math.round(rx * $x) + 'px',
   height: Math.round(ry * $y) + 'px',
   marginLeft: '-' + Math.round(rx * coords.x) + 'px',
   marginTop: '-' + Math.round(ry * coords.y) + 'px'
  });
 };

 function insertCoords(c)
 {
  $('#x').val(c.x);
  $('#y').val(c.y);
  $('#x2').val(c.x2);
  $('#y2').val(c.y2);
  $('#w').val(c.w);
  $('#h').val(c.h);
 };


 $('#jcrop').Jcrop({
  onChange: showPreview,
  onSelect: insertCoords,
  aspectRatio: 3
 });
});
</script>
EOT;
}
?>

<html>
<head>
<?php if ($head) echo $head; ?>
</head>
<body>
<?php if (!$head) : ?>
<form action="" method="POST" enctype="multipart/form-data">
<input type="file" name="image_upload" />
<input type="submit" name="upload_image" value="click me" />
</form>
<?php else : ?>
<form action="" method="POST">
<input type="hidden" name="x" id="x" />
<input type="hidden" name="y" id="y" />
<input type="hidden" name="x2" id="x2" />
<input type="hidden" name="y2" id="y2" />
<input type="hidden" name="w" id="w" />
<input type="hidden" name="h" id="h" />
<input type="hidden" name="filename" id="filename" value="<?php echo $filename ?>" />
<input type="submit" name="submit_coords" value="gogo" />
</form>
<?php endif; ?>
<?php if ($output) echo $output; ?>
</body>
</html>

Пожалуйста, пожалуйста, если бы вы могли чем-то помочь или хотя бы предложить альтернативное решение, я был бы очень признателен.

Заранее большое спасибо.

Ответы [ 2 ]

4 голосов
/ 23 октября 2009

На первый взгляд, я бы предложил, чтобы верхние строки кода для вставки новой строки проходили непосредственно перед загрузкой кода. то есть после следующего:

if ($_POST['upload_image'] && $_FILES['image_upload'])
{
 require_once '../lib/jcrop/class.upload.php';
   //connect to MySQL
   $dbname = "****";
   $username = "****";
   $password = "****";
   $hostname = "localhost"; 

  //connection to the database
  $dbhandle = mysql_connect($hostname, $username, $password)
   or die("<br/><h1>Unable to connect to MySQL, please contact support at support@michalkopanski.com</h1>");

  //select a database to work with
  $selected = mysql_select_db($dbname, $dbhandle)
    or die("Could not select database.");

    //insert an auto_incrementing value into the 'pictures' table
    if (!$result = mysql_query("INSERT INTO `pictures` VALUES ('')"))
  echo 'mysql error: '.mysql_error();

  $filename = mysql_insert_id();

Тогда чуть меньше

if ($_POST['submit_coords'])
{

должно быть

$filename = $_POST['filename'];
0 голосов
/ 23 октября 2009

Из того, что я понял, ваш сценарий вызывается в целом для каждой из трех операций, и вы решаете, что делать с помощью операторов if, включая переменную $ _POST. Таким образом, для каждого вызова вы делаете вставку, поэтому вместо одного есть 3 значения.

Вы должны переместить ваш запрос MySQL INSERT в первый, если выполненная ветвь, а затем вернуть некоторую ссылку (скорее всего, идентификатор) в браузер, которая затем будет передана обратно на ваш сервер для следующих шагов. Не забывайте проверять эту ссылку каждый раз.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...