#1 2010.06.07 17:43

WapBoss
Участник
Зарегистрирован: 2010.06.07
Сообщений: 9
Карма: 0
Профиль

Говнокод?!

Привет, Геморрой) какие у тебя замечания к моему коду?

Неактивен

#2 2010.06.07 19:21

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6593
Карма: 107
Профиль Веб-сайт

Re: Говнокод?!

задай вопрос так, чтобы он был понятен всем посетителям форума.

Неактивен

#3 2010.06.07 20:26

WapBoss
Участник
Зарегистрирован: 2010.06.07
Сообщений: 9
Карма: 0
Профиль

Re: Говнокод?!

Ёпт! Как тут всё сложно) вообщем, дорогие посетители, недавно вашему обозрению были представлены два моих скрипта matematikru и prazdniki (скрипты>разное). Так вот речь далее будет идти о них.

Геморрой, ты сказал, что у тебя есть замечания к коду данных скриптов, хотелось бы НАКОНЕЦ-ТО их услышать!

Неактивен

#4 2010.06.07 21:12

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6593
Карма: 107
Профиль Веб-сайт

Re: Говнокод?!

ну сравни, код скриптов, который был у тебя и который выложен в архив. (Я немного правлю скрипты всегда). Разметка HTML, а не XHTML, не отдавались HTTP заголовки, не было обработки входящих переменных. Просто передай туда не числа, а строку какую-нибудь. Деление на ноль. В праздниках мне лень было исправлять, но там нужны else if или switch, а не if. Двойные кавычки, там где нужны одинарные.

Неактивен

#5 2010.06.08 00:08

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Re: Говнокод?!

http://wapinet.ru/script/show.php?d=%2FRaznoe%2Fmatematikru.zip&f=day_time%2Fday_time.php
Ну хоть тут можно было заменить кавычки?


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#6 2010.06.08 00:28

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6593
Карма: 107
Профиль Веб-сайт

Re: Говнокод?!

Morgan, исправь и прикрепи к посту))

Неактивен

#7 2010.06.08 01:35

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Re: Говнокод?!

Впадлу


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#8 2010.06.08 08:42

WapBoss
Участник
Зарегистрирован: 2010.06.07
Сообщений: 9
Карма: 0
Профиль

Re: Говнокод?!

Спасибо, учту в дальнейших работах.

Насчет праздников не понял, там и так я использовал if, единственное, что не прописывал else ну и нах он нужен?

Неактивен

#9 2010.06.08 10:27

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6593
Карма: 107
Профиль Веб-сайт

Re: Говнокод?!

вот именно что нужен.
в случае с else if обработка условий закончилась бы при первом совпадении, в случае с if - каждое условие обязательно выполнится. очевидная экономия ресурсов.

Неактивен

#10 2010.06.08 11:10

WapBoss
Участник
Зарегистрирован: 2010.06.07
Сообщений: 9
Карма: 0
Профиль

Re: Говнокод?!

Хм... Не задумывался, спасибо.

Неактивен

Дополнительно

forum.wapinet.ru

PunBB Mod v0.6.2
0.010 s