anonymous@RULINUX.NET~# Last login: 2024-12-23 13:07:12
Регистрация Вход Новости | Разметка | Пользователи | Галерея | Форум | Статьи | Неподтвержденное | Трекер | Правила форума | F.A.Q. | Ссылки | Поиск
Форум - Rulinux.net [RSS]
[#] [Добавить метку] [Редактировать]
Скрыть

Прием патчей к исходникам LOR-NG

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

1. Повыбрасывать из кода в classes/*.class.php вызовы "mysql_close();" - насколько я понимаю, из-за этих вызовов каждый вызов метода класса будет переустанавливать соединение к БД.

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

2. Из метода pages::get_meta_data (classes/pages.class.php) следующие строки:

  2. if (mysql_numrows($meta_res)<=0){
  3. mysql_close();return -1;
  4. }
ибо возвращённое значение нигде не проверяется (и при пустом наборе данных даёт ошибки при использовании в foreach, например в incs/header.inc.php), а пустой массив - вполне адкватная репрезентация пустого набора данных

anonymous(*) (2009-05-11 18:03:12)

[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

> 1. Повыбрасывать из кода в classes/*.class.php вызовы "mysql_close();"

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

> 2. Из метода pages::get_meta_data (classes/pages.class.php) следующие строки:

ИМХО, лучше оставить return, mysql_close естественно выбросить. А тут - мало ли где понадобится его проверка

temy4(*)(2009-05-11 21:11:25)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

> мало ли где понадобится его проверка

-1 там говорит, что не нашлось строк в наборе данных. То же самое скажет и пустой массив. Если понадобится проверка - можно просто будет проверить длину массива.

anonymous(*)(2009-05-11 21:25:30)

[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

Я пытался унифицировать вывод ошибок функциями

temy4(*)(2009-05-11 21:59:31)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

> Нечего оправдываться и переводить стрелки на туксойда - никто же не говорю, что разработчиков надо убить прямо сейчас :)

Я говорю про то что времени не было их досканально просмотреть :-)

> это не оптимизация а то, что хотелось бы поиметь

пд оптимизацией я имел в виду минимизацию вызовов к СУБД

> Вообще, было бы неплохо если бы разрабы взяли бы за правило складывать сорцы в конце дня куда-нить в ФС сервера, раз уж cvs пока нет.

Да погоди, свн скоро будет уже

temy4(*)(2009-05-12 21:46:28)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

> Да погоди, свн скоро будет уже

Через неделю я уеду в отпуск, а за это время мог бы попробовать наваять трекер.пхп (в принципе я тоже на пыхе не писец).

anonymous(*)(2009-05-12 21:52:47)

[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

А можешь в forum.php заменить последовательность действий:

$ath = mysql_query("INSERT INTO forum_threads(name, user_name, posting_date, forum_id) VALUES('$title', '$username', '$posting_date', '$forumid')");

...

$tbl = mysql_query("SELECT thread_id FROM forum_threads WHERE name = '$title' AND posting_date = '$posting_date' AND forum_id = '$forumid'");

...

$thr = mysql_fetch_array($tbl);

$thread_id = $thr['thread_id'];

на:

$ath = mysql_query("INSERT INTO forum_....

$thread_id = mysql_insert_id($ath);

И не понял для чего после создания треда повторно брать дату для сообщения, а потом апдейтить тред новой датой. Наверное это можно закоментить нафиг.

Вообще, хотелось бы в базе иметь timestamp'ы и даты в полях типа timestamp а не в виде строки или целого. Можешь добавить колонку created_at типа timestamp в таблицы news, comments, forum_messages, forum_threads etc? Они будут автоматически инициализироваться текущим временем при создании записи.

И ещё хотелось бы поиметь полный набор сорцов, включая файлик view-message.php и х/з что там ещё было пропущено. Тестовый набор данных был бы тоже не вреден.

anonymous(*)(2009-05-12 07:45:41)

[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

форум Туксойд писал, я его исходники хоть и правил, но не оптимизировал

temy4(*)(2009-05-12 09:08:40)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

Нечего оправдываться и переводить стрелки на туксойда - никто же не говорю, что разработчиков надо убить прямо сейчас :)

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

Вообще, было бы неплохо если бы разрабы взяли бы за правило складывать сорцы в конце дня куда-нить в ФС сервера, раз уж cvs пока нет.

anonymous(*)(2009-05-12 13:46:02)

[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

  2. #ifndef nsHTMLReflowMetrics_h___
  3. #define nsHTMLReflowMetrics_h___
  5. #include <stdio.h>
  6. #include "nsISupports.h"
  7. #include "nsMargin.h"
  8. #include "nsRect.h"
  9. // for MOZ_MATHML
  10. #include "nsIRenderingContext.h" //to get struct nsBoundingMetrics
  12. //———————————————————————-
  14. // Option flags
  15. #define NS_REFLOW_CALC_MAX_WIDTH 0x0001
  16. #ifdef MOZ_MATHML
  17. #define NS_REFLOW_CALC_BOUNDING_METRICS 0x0002
  18. #endif
  20. /**
  21. * An nsCollapsingMargin represents a vertical collapsing margin between
  22. * blocks as described in section 8.3.1 of CSS2,
  23. * <URL: http://www.w3.org/TR/REC-CSS2/box.html#collapsing-margins >.
  24. *
  25. * All adjacent vertical margins collapse, and the resulting margin is
  26. * the sum of the largest positive margin included and the smallest (most
  27. * negative) negative margin included.
  28. */
  29. struct nsCollapsingMargin {
  30. private:
  31. nscoord mMostPos; // the largest positive margin included
  32. nscoord mMostNeg; // the smallest negative margin included
  34. public:
  35. nsCollapsingMargin()
  36. : mMostPos(0),
  37. mMostNeg(0)
  38. {
  39. }
  41. nsCollapsingMargin(const nsCollapsingMargin& aOther)
  42. : mMostPos(aOther.mMostPos),
  43. mMostNeg(aOther.mMostNeg)
  44. {
  45. }
  47. nsCollapsingMargin& operator=(const nsCollapsingMargin& aOther)
  48. {
  49. mMostPos = aOther.mMostPos;
  50. mMostNeg = aOther.mMostNeg;
  51. return *this;
  52. }
  54. void Include(nscoord aCoord)
  55. {
  56. if (aCoord > mMostPos)
  57. mMostPos = aCoord;
  58. else if (aCoord < mMostNeg)
  59. mMostNeg = aCoord;
  60. }
  62. void Include(const nsCollapsingMargin& aOther)
  63. {
  64. if (aOther.mMostPos > mMostPos)
  65. mMostPos = aOther.mMostPos;
  66. if (aOther.mMostNeg < mMostNeg)
  67. mMostNeg = aOther.mMostNeg;
  68. }
  70. void Zero()
  71. {
  72. mMostPos = 0;
  73. mMostNeg = 0;
  74. }
  76. PRBool IsZero() const
  77. {
  78. return (mMostPos == 0) && (mMostNeg == 0);
  79. }
  81. nscoord get() const
  82. {
  83. return mMostPos + mMostNeg;
  84. }
  85. };

temy4(*)(2009-05-13 00:36:56)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

  2. #ifndef nsHTMLReflowMetrics_h___
  3. #define nsHTMLReflowMetrics_h___
  5. #include <stdio.h>
  6. #include "nsISupports.h"
  7. #include "nsMargin.h"
  8. #include "nsRect.h"
  9. // for MOZ_MATHML
  10. #include "nsIRenderingContext.h" //to get struct nsBoundingMetrics
  12. //———————————————————————-
  14. // Option flags
  15. #define NS_REFLOW_CALC_MAX_WIDTH 0x0001
  16. #ifdef MOZ_MATHML
  17. #define NS_REFLOW_CALC_BOUNDING_METRICS 0x0002
  18. #endif
  20. /**
  21. * An nsCollapsingMargin represents a vertical collapsing margin between
  22. * blocks as described in section 8.3.1 of CSS2,
  23. * <URL: http://www.w3.org/TR/REC-CSS2/box.html#collapsing-margins >.
  24. *
  25. * All adjacent vertical margins collapse, and the resulting margin is
  26. * the sum of the largest positive margin included and the smallest (most
  27. * negative) negative margin included.
  28. */
  29. struct nsCollapsingMargin {
  30. private:
  31. nscoord mMostPos; // the largest positive margin included
  32. nscoord mMostNeg; // the smallest negative margin included
  34. public:
  35. nsCollapsingMargin()
  36. : mMostPos(0),
  37. mMostNeg(0)
  38. {
  39. }
  41. nsCollapsingMargin(const nsCollapsingMargin& aOther)
  42. : mMostPos(aOther.mMostPos),
  43. mMostNeg(aOther.mMostNeg)
  44. {
  45. }
  47. nsCollapsingMargin& operator=(const nsCollapsingMargin& aOther)
  48. {
  49. mMostPos = aOther.mMostPos;
  50. mMostNeg = aOther.mMostNeg;
  51. return *this;
  52. }
  54. void Include(nscoord aCoord)
  55. {
  56. if (aCoord > mMostPos)
  57. mMostPos = aCoord;
  58. else if (aCoord < mMostNeg)
  59. mMostNeg = aCoord;
  60. }
  62. void Include(const nsCollapsingMargin& aOther)
  63. {
  64. if (aOther.mMostPos > mMostPos)
  65. mMostPos = aOther.mMostPos;
  66. if (aOther.mMostNeg < mMostNeg)
  67. mMostNeg = aOther.mMostNeg;
  68. }
  70. void Zero()
  71. {
  72. mMostPos = 0;
  73. mMostNeg = 0;
  74. }
  76. PRBool IsZero() const
  77. {
  78. return (mMostPos == 0) && (mMostNeg == 0);
  79. }
  81. nscoord get() const
  82. {
  83. return mMostPos + mMostNeg;
  84. }
  85. };

temy4(*)(2009-05-13 00:38:13)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

  2. #ifndef nsHTMLReflowMetrics_h___
  3. #define nsHTMLReflowMetrics_h___
  5. #include <stdio.h>
  6. #include "nsISupports.h"
  7. #include "nsMargin.h"
  8. #include "nsRect.h"
  9. // for MOZ_MATHML
  10. #include "nsIRenderingContext.h" //to get struct nsBoundingMetrics
  12. //———————————————————————-
  14. // Option flags
  15. #define NS_REFLOW_CALC_MAX_WIDTH 0x0001
  16. #ifdef MOZ_MATHML
  17. #define NS_REFLOW_CALC_BOUNDING_METRICS 0x0002
  18. #endif
  20. /**
  21. * An nsCollapsingMargin represents a vertical collapsing margin between
  22. * blocks as described in section 8.3.1 of CSS2,
  23. * <URL: http://www.w3.org/TR/REC-CSS2/box.html#collapsing-margins >.
  24. *
  25. * All adjacent vertical margins collapse, and the resulting margin is
  26. * the sum of the largest positive margin included and the smallest (most
  27. * negative) negative margin included.
  28. */
  29. struct nsCollapsingMargin {
  30. private:
  31. nscoord mMostPos; // the largest positive margin included
  32. nscoord mMostNeg; // the smallest negative margin included
  34. public:
  35. nsCollapsingMargin()
  36. : mMostPos(0),
  37. mMostNeg(0)
  38. {
  39. }
  41. nsCollapsingMargin(const nsCollapsingMargin& aOther)
  42. : mMostPos(aOther.mMostPos),
  43. mMostNeg(aOther.mMostNeg)
  44. {
  45. }
  47. nsCollapsingMargin& operator=(const nsCollapsingMargin& aOther)
  48. {
  49. mMostPos = aOther.mMostPos;
  50. mMostNeg = aOther.mMostNeg;
  51. return *this;
  52. }
  54. void Include(nscoord aCoord)
  55. {
  56. if (aCoord > mMostPos)
  57. mMostPos = aCoord;
  58. else if (aCoord < mMostNeg)
  59. mMostNeg = aCoord;
  60. }
  62. void Include(const nsCollapsingMargin& aOther)
  63. {
  64. if (aOther.mMostPos > mMostPos)
  65. mMostPos = aOther.mMostPos;
  66. if (aOther.mMostNeg < mMostNeg)
  67. mMostNeg = aOther.mMostNeg;
  68. }
  70. void Zero()
  71. {
  72. mMostPos = 0;
  73. mMostNeg = 0;
  74. }
  76. PRBool IsZero() const
  77. {
  78. return (mMostPos == 0) && (mMostNeg == 0);
  79. }
  81. nscoord get() const
  82. {
  83. return mMostPos + mMostNeg;
  84. }
  85. };

temy4(*)(2009-05-13 00:39:25)

Opera/9.64 (X11; Linux i686; U; en) Presto/2.1.1
[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: Прием патчей к исходникам LOR-NG от anonymous 2009-05-11 18:03:12
avatar
Скрыть

Re:Прием патчей к исходникам LOR-NG

> Повыбрасывать из кода в classes/*.class.php вызовы "mysql_close();" - насколько я понимаю, из-за этих вызовов каждый вызов метода класса будет переустанавливать соединение к БД.

не, так было давно. Сейчас в действительности имеется пул соединений, и при запросе соединения с бд новое обычно не создаётся, а используется из пула. При mysql_close() соединение не закрывается, а возвращается в пул и может быть повторно использовано при новом запросе. Убирание вызовов mysql_close() в действительности мало на что повлияет, так как по завершении скрипта все его открытытые дескрипторы всё равно закрываются.

bugmaker(*)(2009-05-22 07:14:46)

Mozilla/5.0 (X11; U; Linux i686; ru; rv:1.9.0.4) Gecko/2008102920 Firefox/3.0.4
[Ответить на это сообщение]
Этот тред читают 3 пользователя:
Анонимных: 3
Зарегистрированных: 0


(c) 2010-2020 LOR-NG Developers Group
Powered by TimeMachine

Valid HTML 4.01 Transitional Правильный CSS!