А что именно ты обновляешь после вставки?

inpost, другие таблицы...

хорошо бы было получить весь код с самими запросами и структурой таблиц, плюс логика, что куда и зачем происходит. Песочницы в php и mysql сколько я знаю нет, по этому просто придется оптимизировать код по принципу графов

lovesa
Какие вы похожие =)

lucasa
Покажи этот полный участок. С первого взгляда непонятно что да как =)

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

inpost, ща, попробую отформатировать

if(isset($_POST['Btn'],$_POST['Id'],$_POST['Result']))
{
   $sqlResult = mysql_query("SELECT * FROM `results` WHERE `id`='".mysql_real_escape_string($_POST['Id'])."'");
   if(mysql_num_rows($sqlResult) == 1)
   {
      $result = mysql_fetch_assoc($sqlResult);
      if($result)
      {
         if(mysql_num_rows(mysql_query("SELECT * FROM `result_user_view` WHERE `resultid`='".
                           mysql_real_escape_string($result['id'])."' AND `status`!='2'")) == $result['total_results_count'])
         {
            mysql_query("UPDATE `results` SET `status`='2' WHERE `id`='".mysql_real_escape_string($result['id'])."'");
            if(mysql_affected_rows() != 1)
            {
               $error = true;
            }
            else
            {
               $added = false;
            }
         }
         else
         {
            $createdDate = mysql_real_escape_string(gmdate($cfgFormat, time()));
            mysql_query("INSERT INTO `result_user_view`(`resultid`,`email`,`result`,`status`,`date`) 
                         VALUES('".mysql_real_escape_string($_POST['Id'])."','"
                         .mysql_real_escape_string($_SESSION['userlogin']).
                         "','".mysql_real_escape_string($_POST['Result'])."','1','".$createdDate."')");
            if(mysql_affected_rows() == 1)
            {
               mysql_query("UPDATE `results` SET `total_results_count`='".($result['total_results_count']+1)."' WHERE `id`='".mysql_real_escape_string($result['id'])."'");
               if(mysql_affected_rows() != 1)
               {
                  $error = true;
               }
               else
               {
                  $added = true;
               }
            }
            else $error = true;
         }
      }
      else $error = true;
   }
}

как-то так...

lovesa
а критичный запрос один подразумевается?

lovesa спасибо)
а альтернативы на русском случайно нет?

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

lucasa
1. В int попадает число или значение? Если число, то применяй (int), а не mysql_real_escape_sting.
2. А почему может вдруг не обновиться вторая таблица? Какую именно причину ты видишь? Перегрузки сервера, что на середине скрипта вырубится?

else
         {
            $createdDate = mysql_real_escape_string(gmdate($cfgFormat, time()));
            mysql_query("INSERT INTO `result_user_view`(`resultid`,`email`,`result`,`status`,`date`) 
                         VALUES('".mysql_real_escape_string($_POST['Id'])."','"
                         .mysql_real_escape_string($_SESSION['userlogin']).
                         "','".mysql_real_escape_string($_POST['Result'])."','1','".$createdDate."')");
            if(mysql_affected_rows() == 1)
            {
               mysql_query("UPDATE `results` SET `total_results_count`='".($result['total_results_count']+1)."' WHERE `id`='".mysql_real_escape_string($result['id'])."'");
               if(mysql_affected_rows() != 1)
               {
                  $error = true;
               }
               else
               {
                  $added = true;
               }
            }
            else $error = true;
         }

Здесь Insert если был, то обязательно должен быть UPDATE. Т.е. если UPDATE не прошел успешно, нужно делать DELETE Insertу...

И один из вопросов у меня был, можно ли как избежать этих многочисленных if'в?

вполне нормальная вложенность if-else. Не вижу смысла менять что-то.
Ты лучше объясни, чего именно ты боишься и хочешь добиться?

inpost
сорри, малость не понимаю про какой int идет речь?

Я думаю в данном случае тебе помогут транзакции и новая php конструкция исключений - exceptions: http://php.net/manual/en/language.exceptions.php.

работает это как то так: [code]

try {
    // начинаем транзакцию
    $db->query('BEGIN');

    // тут пошли запросы проверки и т.д
    $db->query('first query');
    $db->query('second query');
    $db->query('third query');

    // Если добрались до сюда, то подтверждаем изменения
    $db->query('COMMIT');
} catch (Exception $e) {
    //если была ошибка то 
    $db->query('ROLLBACK');
}

Но транзакции работают только на таблицах типа innodb

inpost
я хотела бы избавиться от этих бесконечных проверок... тут даже assert не напишешь, т.к. просто так оборвать цепочку нельзя...

а боюсь, что что-то помешает выполнится транзакции целиком, т.е. целостность данных попортится...

lovesa, спасибо, поизучаю!


у меня такого типа innodb - нет, даже не знаю, что это такое

mysql по дефолту если я не ошибаюсь использует тип MyIsam если не ошибаюсь, но при помощи обыкновенного phpmyadmin можно поменять тип таблицы на innodb и спокойно делать rollback`и, если в этом если необходимость. Но тогда данных кусок кода придется немного модифицировать по мануалу exceptions

Если запрос правильный, то врядли целостность будет повреждена.
ты получаешь $result['id'], в ней что содержится, порядковый номер (число) или текстовая строка? Мне кажется, что там число вроде 5, или 8, значит надо вставлять в таблицу так: .(int)$result['id'].
Пойми, пока нету такого, что твой сервер гнётся и хрустит от 10-и тысячного посетителя за последнюю секунду, то нет смысла думать о мелочных оптимизациях. Проверки проходят на первый взгляд логично и правильно, оставляй их такими.
И снова таки, целостность может повредиться только если в одну микросекунды вдруг сервер успеет выйти из строя, что будет врядли.

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

inpost
да, порядковый индекс
(int)$result['id'] - а почему так лучше делать?
просто данные идут из формы, вроде как их лучше обезопасить...

сервер то будет не мой, хз как он будет работать)

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

lovesa, да, надо тогда будет почитать про эти таблицы

lucasa
Потому что (int) - для числа, а string - для текстовой строки.

Меня тут сомнения стали одолевать. В примерах используется код:

if(mysql_affected_rows() > 0)

а я стала использовать сравнение на точное количество обновленных строк:

if(mysql_affected_rows() == 1)

а если в тоже самое время выполнится другой запрос, то эта функция может вернуть совсем иной результат...

как ей правильно пользоваться?

подниму темку, интересует ответ на последний вопрос, хелп!

lovesa