Не могу найти ошибку

AzatSh · October 2, 2008


....
<head>

<script language="javascript">

function getwidth(myPicture){
    newImg = new Image();
    newImg.src = myPicture;

    myWidht = newImg.width;
    myHeight = newImg.height;
    if (myWidht > 190 )
    {myHeight = myHeight*(190/myWidht);
      myWidht = 190;
     }
     if (myHeight > 140) {
     myWidht = myWidht*(140/myHeight);
     myHeight = 140;
     }
return myWidht
}

</script>


</head>
<body lang=RU link=blue vlink=purple style='tab-interval:35.4pt'>

<table border="1" width="100%">
 <tr>
  <td ><img src="r_s_220.jpg" width=getwidth("r_s_220.jpg")><br>баш</td>

.....

Ошибка в где то на уровне width=getwidth. Я не уверен что вообще можно так писать.. просто плохо знаю яваскрипт

homm · October 2, 2008

newImg.src = myPicture;

myWidht = newImg.width;

Между началом загрузки и тем моментом, когда ты хочешь получить размер должно пройти одно очень маленькое, но очень важное событие. Картинка должна загрузиться. За считаные такты процессора, это естественно не произойдет. У Image есть свойтво complete. Проверяй его по таймеру и счастье придет к тебе.

AzatSh · October 2, 2008

Пробовал делать так return 100..

все равно не работает

homm · October 2, 2008

Пробовал делать так return 100..
все равно не работает

А… Дак это потому что вы фигню полную пытаетесь сделать, так нельзя.

AzatSh · October 2, 2008

А… Дак это потому что вы фигню полную пытаетесь сделать, так нельзя.

Почему? А как можно тогда

Я просто убрал весь код в функции и сотавил только return 100, в чем фигня ?

Searcher · October 3, 2008

><img src="r_s_220.jpg" width=getwidth("r_s_220.jpg")>

Это полный бред. Почитайте мат.часть. Было бы хорошо, если бы в html можно было бы так внедрить Javascript Но, к сожалению, такой передачи параметров не предусмотрено. Можно по событиям запускать скрипт, по идентификатору объекта в скрипте передавать свойства и т.п. Но не так.

Сделайте так. Установите разные id для всех картинок, которые Вам нужно изменить: <img src="r_s_220.jpg" id="pic1">

В bodi пропишите запуск функции на событие onload. В функции по id (GetElementById("pic1")) обращаетесь ко всем изображениям и делаете с ними, что хотите.

newImg = new Image(); cоздает новый объект. Это нужно, если у Вас не загружена картинка. А так она уже будет в объекте с соответствующим id.

Если не секрет, какова конечная цель такого преобразования?

Edited October 3, 2008 by Searcher

AzatSh · October 3, 2008

Моя цель создать таблицу с ячейками фиксированного размера, чтобы картинки помещенные в ячейки не растягивали таблицу под свой размер. Т.к. размер картинок у всех разный необходимо как то уметь узнавать его для каждой картинки и подгонять свойства width и height

homm · October 3, 2008

Моя цель создать таблицу с ячейками фиксированного размера, чтобы картинки помещенные в ячейки не растягивали таблицу под свой размер. Т.к. размер картинок у всех разный необходимо как то уметь узнавать его для каждой картинки и подгонять свойства width и height

Ты пробовал не указывать значения windth и height?

AzatSh · October 3, 2008

Все получилось! Спасибо большое

Ты пробовал не указывать значения windth и height?

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

Searcher · October 3, 2008

В таком варианте есть минус.

Если картинка большая - грузиться она будет долго, и только потом уменьшаться. А если много картинок... Они подгоняются по размеру только после загрузки всех.

Только сейчас сообразил Есть еще вариант без скрипта! Вложить img в div, задать для div фиксированный размер, а для img высоту и ширину задать 100%.

AzatSh · October 3, 2008

Если картинка большая - грузиться она будет долго, и только потом уменьшаться.

Дело в том что у меня не сайт, а html документ с сохраненными картинками

поэтому грузиться будет быстро

Вложить img в div, задать для div фиксированный размер, а для img высоту и ширину задать 100%

разве в div можно задать размер?

Sign In

Не могу найти ошибку

Question

Link to comment

Share on other sites

10 answers to this question

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Обсуждения

Important Information