|
14.09.2009, 15:51 | #1 |
Участник
|
Странное использование strfmt
Добрый день.
Наткнулся в коде одного из методов на следующий кусок кода: X++: //XXX Act_adjust_vend_Fix, XXX, 03.10.2008 --> //szDescription = strfmt("Накладная № %1 от %2", _custVendTrans.Invoice, this.getIvoiceDate(_custVendTrans)); //XXX Act_adjust_vend_Fix, XXX, 03.10.2008 <-- //XXX Act_adjust_vend_Fix, XXX, 03.10.2008 --> szDescription = strfmt("Накладная № %1 " + addDate, _custVendTrans.Invoice, this.getIvoiceDate(_custVendTrans)); //XXX Act_adjust_vend_Fix, XXX, 03.10.2008 <-- szDescription - текст, addDate - тоже текст, addDate = "от %2" Изменения сделаны, как видно, недавно, но зачем? Почему старый, более читаемый код, выполняющий тоже самое, закомментирован? Или разница все же есть? |
|
14.09.2009, 15:55 | #2 |
Участник
|
видимо текст в переменной addDate может меняться
|
|
|
За это сообщение автора поблагодарили: Silphidae (1). |
14.09.2009, 16:01 | #3 |
Участник
|
Ну, даже если addDate меняется, не проще ли менять текст в strfmt?
|
|
14.09.2009, 16:32 | #4 |
Member
|
Цитата:
Сообщение от Silphidae
...
Ну, даже если addDate меняется, не проще ли менять текст в strfmt? ... Как вариант "от ..." может и не быть вовсе.
__________________
С уважением, glibs® |
|
14.09.2009, 16:46 | #5 |
Участник
|
Цитата:
Тем более, дальше по коду я нашел вообще шедевральный момент: X++: addDate = ""; if (_custVendTrans.Invoice) szDescription = strfmt("Курсовая разница (Накладная № %1 от %2)" + addDate, _custVendTrans.Invoice, this.getIvoiceDate(_custVendTrans)); else szDescription = strfmt("Курсовая разница (Операция ГК № %1 (%2))", _custVendTrans.Voucher, _custVendTrans.Txt); |
|
|
За это сообщение автора поблагодарили: Alexx7 (1). |
14.09.2009, 17:24 | #6 |
Member
|
Цитата:
Сообщение от Silphidae
...
У нас в компании метками не пользуются ... Цитата:
Сообщение от Silphidae
...
можно банально писать strfmt("Накладная %1 от %2") или strfmt("Накладная %1") в зависимости от ситуации, а не составлять строку по кусочкам как в примере выше. ...
__________________
С уважением, glibs® |
|
|
За это сообщение автора поблагодарили: Silphidae (1). |
15.09.2009, 07:01 | #7 |
Сам.AX
|
|
|
14.09.2009, 16:11 | #8 |
Участник
|
можно и так
|
|