dB6
Full Member | Редактировать | Профиль | Сообщение | Цитировать | Сообщить модератору mozers Цитата: Ребят, я не спец. Давайте объективно. Чей вариант более корректен в плане кода? Какой из вариантов лучше использовать в сборке? Понимаю что тяжело сравнивать 2 хороших варианта, но, пожалуйста, давайте выберем один. | Действительно "тяжело" сравнивать 2 таких варианта, поэтому возьми любой мерджер Хе... Отличие 1: Код: void SciTEBase::SelectionIntoFind(bool stripEol /*=true*/, bool needEncode /*=false*/) // Modified HSolo 23-11-2006 { | вместо Код: //! void SciTEBase::SelectionIntoFind(bool stripEol /*=true*/) { void SciTEBase::SelectionIntoFind(bool stripEol /*=true*/, bool inFiles /*=false*/) { //!-add-[FindInFiles] | Простите, я больше так не буду... Не буду называть переменную "inFiles", если можно назвать "needEncode".. Отличие 2: Код: if (needEncode) findWhat = EncodeString(sel); //!-add-[FindInFiles] // Modified HSolo 23-11-2006 else findWhat = sel; | вместо Код: //!-start-[FindInFiles] if (inFiles) findWhat = EncodeString(sel); else //!-end-[FindInFiles] findWhat = sel; | Я бы, кстати, поспорил, имхо старый вариант лучше. Отличие 3: Код: void SelectionIntoFind(bool stripEol = true, bool needEncode = false); // Modified HSolo 23-11-2006 | вместо Код: //! void SelectionIntoFind(bool stripEol = true); void SelectionIntoFind(bool stripEol = true, bool inFiles = false); //!-add-[FindInFiles] | Отличие 4: Код: SelectionIntoFind(true, true); // Modified HSolo 23-11-2006 | вместо Код: //! SelectionIntoFind(); SelectionIntoFind(true, true); //!-add-[FindInFiles] | Из 3-го и 4-го и так все понятно... Желаю поменьше самолюбия В общем, решайте, какой вариант более правильный, а главное рабочий Перепутал в 4-м местами, поправил. | Всего записей: 408 | Зарегистр. 09-02-2003 | Отправлено: 18:10 23-11-2006 | Исправлено: dB6, 18:13 23-11-2006 |
|