Использует D String Mixins для повторного использования кода анти-шаблон?

StackOverflow https://stackoverflow.com/questions/3300447

  •  26-09-2020
  •  | 
  •  

Вопрос

Для тех из вас, кто не знаком с D String Mixins, они в основном уклонны от времени с компиляцией. Вы можете предпринять любую строку времени компиляции (будь то буквальный или сгенерирован шаблоном MetaProgramming или оценки функций времени компиляции) и компилируйте его в качестве кода. Если вы используете простой строковый литерал, это в основном компилятор-автоматизированная копировальная вставка.

Если бы вы рассмотрели его Antipattern для использования строковых смеси литералов в качестве средства простого повторного использования кода, где другие методы факторинга не соответствуют правильному? С одной стороны, это в основном компилятор-автоматизированная буквальная копировальная и-паста, что означает, что после смешивания в экземплярах ничего не имеет ничего общего друг с другом. Плохие вещи произойдут (хотя во время компиляции, не во время выполнения), если символ в струнной микене сталкивается с символом в смешанном объеме. Это относительно неструктурировано в том, что можно, например, можно смешивать строку в середину функции, которая будет работать, если и только если переменные в объеме называются по определенной конвенции. Мистины также могут объявить переменные, которые затем могут использовать внешние области, поскольку они видят подходящие.

С другой стороны, потому что копилепочко-вставка является компилятором автоматизирована, существует одна точка истины для рассматриваемого кода на уровне источника, и если оно должно быть изменено, он должен быть изменен только в Одно место, и все остается в синхронизации. Струнные смесины также значительно упрощают повторное использование кода, который очень трудно для факторов каким-либо другим способом и в противном случае будет иметь очень высокую вероятность сокращения и вставленного вручную.

Нет правильного решения

Другие советы

Все критики, которые вы подняли, являются правдой.

независимо, он все еще превосходит ручной Copypaste.

На самом деле, у меня есть что-то похожее в моей библиотеке инструментов, расширение String Table.Пример кода, от реализации динамического значения трагера пути:

  T to(T)() {
    static if (!is(T == Scope)) {
      T value;
      if (flatType == FlatType.ScopeValue) value = sr.value().to!(T);
    }
    const string Table = `
                 | bool          | int         | string               | float   | Scope
      -----------+---------------+-------------+----------------------+---------+----------
      Boolean    | b             | b           | b?q{true}p:q{false}p | ø       | ø
      Integer    | i != 0        | i           | Format(i)            | i       | ø
      String     | s == q{true}p | atoi(s)     | s                    | atof(s) | ø
      Float      | ø             | cast(int) f | Format(f)            | f       | ø
      ScopeRef   | !!sr          | ø           | (sr?sr.fqn:q{(null:r)}p) | ø   | sr
      ScopeValue | value         | value       | value                | value   | sr`;
    mixin(ctTableUnrollColMajor(Table,
      `static if (is(T == $COL))
        switch (flatType) {
          $BODY
          default: throw new Exception(Format("Invalid type: ", flatType));
        }
      else `,
      `case FlatType.$ROW:
        static if (q{$CELL}p == "ø")
          throw new Exception(q{Cannot convert $ROW to $COL: }p~to!(string)~q{! }p);
        else return $CELL;
      `
    ).litstring_expand() ~ `static assert(false, "Unsupported type: "~T.stringof); `);
  }
.

Я уверен, что легко посмотреть, какой ужасный, избыточный беспорядок вложенных IFS и случаев случаев, которые были бы без струнных микс - таким образом, все уродство сосредоточено внизу, а фактическое поведение / em> функции легко считываться с первого взгляда.

While other, more elegant solutions may be better to use if you can, string mixins can be extremely useful. They allow for both code re-use and code generation. They're checked at compile time. The code that results is exactly the same as if you'de written it yourself by hand, so it's not any less safe than if you had written it yourself by hand.

The problem with string mixins is that they're harder to control than hand-written code in the sense that it's not physically laid out in your source in the same manner with line numbers clearly traceable to errors, and it may be harder to debug. For instance, take hello world with a string mixin:

import std.stdio;

void main()
{
    mixin(hello());
}

string hello()
{
    return "
    writeln(\"hello world\");
";
}

If we were to remove the semicolon after writeln(), then the error we got would be

d.d(7): found 'EOF' when expecting ';' following statement

The mixin is done on line 5. Line 7 is a blank line. So, the line number is of limited usefulness here. Now, this mixin is short enough that we could have put it on a single line and gotten it to say that the error was on the same line as the mixin, but with more complicated mixins, that obviously won't work. So, by using a string mixin, your ability to figure out where an error is is impaired. If the code is generated using CTFE, then it would become that much more difficult to figure out exactly what the code even looks like in order to figure out what's wrong with it. It's a lot like figuring out what code a C-style macro turns into, except that it could be worse because they could be generated rather than a direct replacement. However, they don't replace except where you explicitly tell them to, so they're much safer than C-style macros.

String mixins are totally safe, and there's nothing particularly wrong with them, but they do make maintenance harder in some ways. The corresponding hand-written code would be easier to debug. However, string mixins are powerful enough that they can do a lot of code generation for you and save you a lot of maintainence costs in that sense, and they allow you to re-use code, which can be a big maintanence gain as well.

So, whether using a string mixin is a good idea in a particular situation depends on that situation. I don't see anything particularly wrong with them, and I certainly wouldn't call them an anti-pattern, but there are both pros and cons to using them such that whether they're a good idea depends on what you're doing. In many cases, there are more elegant, cleaner solutions which would be better. In others, they're exactly what the doctor ordered.

Personally, I think that they're fantastic if you're looking to generate code, saving yourself the effort of having to write that code by hand, and possibly making it easier to generate correct code for a variety of situations and avoiding risking creating new bugs like you might have had you written it yourself in each of those places where you used the mixin. It also is one of the ways to just outright re-use code without having to worry about the cost of a function call or issues with the limits of single-inheritance or anything else that makes code re-use by calling functions or inheritance harder. You're simply copying and pasting the code into each place in a manner which makes it so that if you change the code, the changes will be properly pasted everywhere without you having to worry about tracking them all down like if you had hand copy and pasted.

So, use string mixins where appropriate, and it's probably best not to use them if they're not needed, but there's nothing really wrong with using them.

String mixin is like goto: it should be avoided where ever possible and should be used wherever required.

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top