Ви не увійшли. Будь ласка, увійдіть або зареєструйтесь.
Ласкаво просимо вас на україномовний форум з програмування, веб-дизайну, SEO та всього пов'язаного з інтернетом та комп'ютерами.
Будемо вдячні, якщо ви поділитись посиланням на Replace.org.ua на інших ресурсах.
Для того щоб створювати теми та надсилати повідомлення вам потрібно Зареєструватись.
Український форум програмістів → PHP → Порівняти зображення
В мене був подібний приклад на С# перед прийняттям на першу роботу 
Називався "знайти всі групи однакових за вмістом файлів у вказаній папці" (Адже однакові хеш-коди не гарантують однакового вмісту).
Перевірка в лоб виявилась вкрай повільною.
А той алгоритм, що я навів - буде працювати на порядок швидше.
Знаю про навантаження,можна було б звязати з бд.
В мене був подібний приклад на С# перед прийняттям на першу роботу
Перевірка в лоб виявилась вкрай повільною.
А той алгоритм, що я навів - буде працювати на порядок швидше.
наведіть приклад, коли два різні файли і мають однаковий хеш.
UPD. тут мається на увазі однакова довжина файлу? чи хеш за іншими ознаками генериться?
google "Колізія хеш-функції вікі"
для md5 я вже десь бачив такі приклади однакової довжини на вікіпедії, тільки не памятаю на якій саме сторінці
UPD. http://домен агресора/post/137316/
тут же на вікі радять дописувати сіль на початку або вкінці файлу перед хешуванням або конкатенувати підряд два різні хеші.
Це жах...
Розставте відступи, ваш код читати боляче.
Дайте нормальну назву функції (findPictureByContent, наприклад).
Визначати глобальні константи в функції, яка має інше призначення - не найкраща ідея.
Виводити повідомлення, а тим більше робити exit в функції - теж не варто, хай вона повертає щось адекватніше (наприклад шлях до малюнка, що вже існує чи був завантажений, чи false, якщо щось не так).
Про оптимізацію вже сказали - це ЖАХ.
Ну і при будь-якому хешуванні слід пам'ятати, що хеш - то засіб знаходити різні значення. А однакові треба перевіряти "в лоба", хоча, звісно, спосіб пана yarko значно швидший - якщо читати файли кластерами (512-4096 байт).
Щось типу такого
$imageGet = $DB->real_escape_string(file_get_contents($_FILES['filename']['tmp_name']));
$get_contents = $DB->query("SELECT * FROM images WHERE get_contents = '$imageGet' LIMIT 1");
if($get_contents->fetch_row() > 0)
{
$get_query = $DB->query("SELECT * FROM images WHERE get_contents = '$imageGet' LIMIT 1");
$get = $get_query->fetch_assoc();
//виводимо інфу про вже існуючий файл
}
else
{
//завантажуємо та пишемо його контент в бд
}
Ну і при будь-якому хешуванні слід пам'ятати, що хеш - то засіб знаходити різні значення. А однакові треба перевіряти "в лоба", хоча, звісно, спосіб пана yarko значно швидший - якщо читати файли кластерами (512-4096 байт).
Чому в лоба? Достатньо буде перевірити один такий кластер.
Це жах...
Розставте відступи, ваш код читати боляче.
Дайте нормальну назву функції (findPictureByContent, наприклад).
Визначати глобальні константи в функції, яка має інше призначення - не найкраща ідея.
Виводити повідомлення, а тим більше робити exit в функції - теж не варто, хай вона повертає щось адекватніше (наприклад шлях до малюнка, що вже існує чи був завантажений, чи false, якщо щось не так).
Про оптимізацію вже сказали - це ЖАХ.
Справа не в цьому,я такий код зазвичай в народ не пускаю,я не ставив питання про оптимізацію. Чого нападаєте,не розумію.
я такий код зазвичай в народ не пускаю
Варто було б продовжувати.
Ну з умови і було неявно сказано,шо потрібно створити велосипед).
Звичайно, що створення велосипедів для освітньої мети це добре, але у вас вийшло щось наподобі оцього:
Знаєте, дуже хотілося б подивитись на швидкість виконання, особливо коли на сервері вже є сотня схожих зображень по кілька мегабайтів.
при великій кількості підключень, буде суттєве навантаження на диск, цей код хоч і робочий, але... imho.
Насправді нині можна делегувати порівняння файлів файловій системі, яка знов-таки зробить це за допомогою геш функції. Але чи зможе компілятор зрозуміти що саме від нього хоче індус - складне питання.
Це жах...
Розставте відступи, ваш код читати боляче.
Дайте нормальну назву функції (findPictureByContent, наприклад).
Визначати глобальні константи в функції, яка має інше призначення - не найкраща ідея.
Виводити повідомлення, а тим більше робити exit в функції - теж не варто, хай вона повертає щось адекватніше (наприклад шлях до малюнка, що вже існує чи був завантажений, чи false, якщо щось не так).
Про оптимізацію вже сказали - це ЖАХ.
lol,оце так пояснення,оце так критика. "повертає щось адекватніше"
koala написав:Ну і при будь-якому хешуванні слід пам'ятати, що хеш - то засіб знаходити різні значення. А однакові треба перевіряти "в лоба", хоча, звісно, спосіб пана yarko значно швидший - якщо читати файли кластерами (512-4096 байт).
Чому в лоба? Достатньо буде перевірити один такий кластер.
Порівняйте так 2 bmp-файла 1024x768 із різними нижніми половинками.
lol,оце так пояснення,оце так критика. "повертає щось адекватніше"
Ну вибачте, я чомусь думав, що ви вже більше двох тижнів програмуванням займаєтеся.
quez написав:koala написав:Ну і при будь-якому хешуванні слід пам'ятати, що хеш - то засіб знаходити різні значення. А однакові треба перевіряти "в лоба", хоча, звісно, спосіб пана yarko значно швидший - якщо читати файли кластерами (512-4096 байт).
Чому в лоба? Достатньо буде перевірити один такий кластер.
Порівняйте так 2 bmp-файла 1024x768 із різними нижніми половинками.
У них різні хеші, до порівняння кластерів справа не доходить.
У них різні хеші, до порівняння кластерів справа не доходить.
В першому пості вказано не використовувати хешів
quez написав:У них різні хеші, до порівняння кластерів справа не доходить.
В першому пості вказано не використовувати хешів
Ну і при будь-якому хешуванні слід пам'ятати
Хіба порівняти 2 файли на одинаковість вмісту є хешуванням?