Das wäre die vernünftige Vorgehensweise. Meine Firma hat es quasi so gemacht. 6 Monate Probezeit, Gehalt erhöht sich nach der Probezeit automatisch und Vertrag auf 1 Jahr begrenzt. Da hat die Firma 2 Chancen einen bequem los zu werden. Mein Chef hat aber bis zur letzten Sekunde gewartet, mir den neuen Vertrag vor zu legen. Die Arge war sogar richtig am schwitzen und meinte ich müsste zu einem Termin vorbei kommen und mich Arbeitssuchend melden. Hab denen dann mitgeteilt, ich bekomme kein frei für den Termin und bla bla. Hab dann gesagt bekommen, ausnahmsweise geht das und ich müsste dann am 1. Tag sofort vorbei kommen. Hab dann noch mal ein 1 Jahres Vertrag bekommen und den unbefristeten habe ich dann am letzten Tag um 16 Uhr vorgelegt bekommen (der andere war so 3 Tage vor Ablauf). Da hat die Arge aber nicht genervt.
In meiner Bewerbungsphase habe ich einmal ein Probearbeiten bekommen und den habe ich vollkommen verkackt. Ansonsten frühe absagen und einige mit dem Hinweis, dass die halt keine Ressourcen haben, mir was bei zu bringen. Die Firma wo ich hin bin, hat gesagt, ich muss eh vieles lernen, da die nicht glauben, dass die groß jemanden mit Erfahrung finden werden, insbesondere in einer bezahlbaren Preisklasse. Stimmt auch, da das SDK privat ist und es kaum Firmen in Deutschland gibt, die es nutzen.
It really depends. Whitespaces are something most languages don’t care. The only people who care are enforcing style guides. Level 2 is the same but there it start to get more critical, because can you be sure that it makes no difference? Level 3 is critical. While it can help to eliminate code that probably didn’t caused the problem, it makes a difference. In code review this can make a difference. If a specific Hex number is well known, like of example 0x4711 and someone changes it to 18193 or even Binary, information to the programmer gets hidden. And even in style this makes a difference. When you have a flag Enum, the thing to use is binary or bit shift, because both is readable. Decimal is readable to a certain point. 4 bytes is fine but at the 5th I don’t know them by heart and can’t even spot them. Level 4 is irrelevant, when its on top of the file and bothering to hide it, is not necessary. Also this can be relevant. For example a while ago at our company we had code that needed to work with .NET 2 and we had parts with .NET 4 and at some point, new files had the using for LINQ, that isn’t available in .NET 2. This happened a lot.
The best solution is to have options and let the person using it decide. What I’m missing is to add my own ignore list. For example with our XML files, we have a date in them. The XML Class is badly written, because instead of having one date attribute for the first node, we have them on all. This is pretty irrelevant to show in a diff, because its not even used. Rewriting the Class is a big task, because its a core feature and can break everything, when one thing is missed.