Лингвофорум

Цитата: Светка12 от сентября 21, 2014, 10:24Хотя с твоим кодом как-то получаю  " Parse error: syntax error, unexpected end of file "

Символы <?php в конце убрать надо. Точнее, перенести их наверх, поставить самой первой строчкой.

Так, пока не будет новые ответы, я избегаю оставить тут новые сообщения, и прогресс как сказала можно будет видеть в http://iskragame.blogspot.co.il/

По дезайну с HTML я только изполсьзую <tr><td>. Всё остальное с помощью CSS конечно.

Да у тебя хорошая логика програмиста,  Клонопотам. Тут "деление с остатком" (%) точно подходит. Хотя с твоим кодом как-то получаю  " Parse error: syntax error, unexpected end of file "

Я буду перечитывать все и постараюсь исправить это.

Нет, знаешь что, я перепишусь код с помощью CSS и float. Это лучший вариант для такой таблицы.

ЦитироватьК сожалению, при таком подходе создастся большая горизонтальная строка, и аватары не влезут в экран.

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

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

Правильно, спасибо за замечание.

Цитата: Bhudh от сентября 21, 2014, 00:43
можно вообще от таблиц отказаться.

Это понятно, но мне нехотелось слишком отходить от оригинального кода.

Кстати, чтобы не было такого, что аватары не влезают в экран, можно вообще от таблиц отказаться.
Тогда они будут нормально переноситься на следующую строку.

Нормальный и рабочий — разные вещи

Да лан те, нормальный код, я примерно то же подсказал в блоггере, только писать не стал.

Я на этих выходных далеко от компьютера, в понедельние вечером посмотрю.

Что касается аватаров,

    foreach ($index as $avatars)
    {
    ?>
    <table>
    <tr>
    <td>
      <img src="images/<?php echo $avatars[0]; ?>.jpg"
      onclick="ChangeAvatar(this.id)" id="<?= $avatars[0] ?>">
     
     
      <script>
function ChangeAvatar(id) {
    document.getElementById("newimage").value = id;
    }
</script>
</td>
  <?php
  }
  ?> </tr> </table>
То здесь
    <table>
    <tr>
явно нужно вынести за foreach. Сейчас у тебя по сути создаётся по новой таблице для каждого аватара (правда, эти таблицы незакрытые — ведь тег </table> только один). Кроме того, у тебя функция ChangeAvatar переопределяется каждый раз, а ведь её нужно определить только один раз.

Итак, код должен принять примерно такой вид: (код не проверен!!! я не за своим обычным компьютером, у меня тут нет PHP):

    while ($row = $result->fetch_row()) {
        $index[]=($row);
    }
?>
    <table>
    <tr>
<?php
    foreach ($index as $avatars)
    {
    ?>
    <td>
      <img src="images/<?php echo $avatars[0]; ?>.jpg"
      onclick="ChangeAvatar(this.id)" id="<?= $avatars[0] ?>">
    </td>
  <?php
  }
  ?>
</tr>
</table>
<script>
function ChangeAvatar(id) {
    document.getElementById("newimage").value = id;
    }
</script>
<?php


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

    while ($row = $result->fetch_row()) {
        $index[]=($row);
    }
?>
    <table>
    <tr>
<?php
$n = 0;
    foreach ($index as $avatars)
    {
    $n++;
    if ($n % 4 == 0) {
    ?>
    </tr>
    <tr>
    <?
    }
    ?>
    <td>
      <img src="images/<?php echo $avatars[0]; ?>.jpg"
      onclick="ChangeAvatar(this.id)" id="<?= $avatars[0] ?>">
    </td>
    <?php
    }
    ?>
</tr>
</table>
<script>
function ChangeAvatar(id) {
    document.getElementById("newimage").value = id;
    }
</script>
<?php


К сожалению, весь код, который я пишу, не проверен, в нём могут быть ошибки!

Пока да, все открытый для просмотра.  Думаю что в продвинутой стадии закрою опцию посмотрить исходный код. Но пока, нечем мне хвалиться.

http://iskragame.blogspot.co.il/