Erweitern vom Duden um Entferne_Anzahl_Vorne, Entferne_Anzahl_Hinten #43
Conversation
lib/stdlib/Duden/Texte.ddp
Outdated
| t: "" | ||
| ] | ||
| Die öffentliche Funktion Entferne_Anzahl_Vorne mit den Parametern text und anzahl vom Typ Text Referenz und Zahl, gibt nichts zurück, macht: | ||
| Wenn nicht ((die Länge von text) größer als anzahl ist), dann: |
There was a problem hiding this comment.
Könnte zu Wenn die Länge von Text kleiner als, ode anzahl ist, ... vereinfacht (und ein wenig deutscher) werden
lib/stdlib/Duden/Texte.ddp
Outdated
| t: "" | ||
| ] | ||
| Die öffentliche Funktion Entferne_Anzahl_Hinten mit den Parametern text und anzahl vom Typ Text Referenz und Zahl, gibt nichts zurück, macht: | ||
| Wenn nicht ((die Länge von text) größer als anzahl ist), dann: |
lib/stdlib/Duden/Texte.ddp
Outdated
| Die öffentliche Funktion Entferne_Anzahl_Hinten mit den Parametern text und anzahl vom Typ Text Referenz und Zahl, gibt nichts zurück, macht: | ||
| Wenn nicht ((die Länge von text) größer als anzahl ist), dann: | ||
| Speichere "" in text. | ||
| Sonst speichere text von 1 bis (die Länge von text minus anzahl) in text. |
There was a problem hiding this comment.
Die Syntax hat sich im letzten Release zu t im Bereich von ... bis ... geändert, deshalb failen noch die Tests.
Es geht inzwischen sogar noch besser: text bis zum (die Länge von text minus anzahl). Element.
There was a problem hiding this comment.
Zusätzlich zu den Änderungen im Code wäre noch eine Variante praktisch, die den Text nicht verändert sondern eine Kopie mit den Änderungen zurückgibt.
z.B. <t> mit den ersten <n> Elementen entfernt o.ä..
Ansonsten siehts gut aus und ist eine praktische Erweiterung.
Schön zu sehen, dass jemand contributen möchte :).
lib/stdlib/Duden/Texte.ddp
Outdated
| Die öffentliche Funktion Entferne_Anzahl_Vorne mit den Parametern text und anzahl vom Typ Text Referenz und Zahl, gibt nichts zurück, macht: | ||
| Wenn nicht ((die Länge von text) größer als anzahl ist), dann: | ||
| Speichere "" in text. | ||
| Sonst speichere text von anzahl plus 1 bis (die Länge von text) in text. |
There was a problem hiding this comment.
Die Syntax hat sich im letzten Release zu t im Bereich von ... bis ... geändert, deshalb failen noch die Tests.
Es geht inzwischen sogar noch besser: text ab dem (anzahl plus 1). Element.
|
Soo, ich denke ich habe eure Vorschläge damit umgesetzt. |
|
|
||
| [Entferne_Anzahl_Vorne] | ||
| Der Text Entferne_Anzahl_Vorne_Text ist "Hello World!". | ||
| Schreibe Entferne_Anzahl_Vorne_Text mit den ersten 4 Buchstaben entfernt auf eine Zeile. [o World!] |
There was a problem hiding this comment.
Sollte Schreibe (Entferne_Anzahl_Vorne_Text mit den ersten 4 Buchstaben entfernt) auf eine Zeile. [o World!] sein.
In den anderen Aufrufen genau so.
|
Sieht super aus jetzt :). Die Tests failen noch wegen einem Syntax Fehler beim Aufrufen der Funktion als Argument zu Die Tests melden da leider keinen Fehler wegen eines Bugs auf der dev branch (, den ich gerade gefixt habe), also muss du wahrscheinlich kurz die base-branch updaten (https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/keeping-your-pull-request-in-sync-with-the-base-branch mir wird der Button irgendwie nicht angezeigt). Kannst du die Tests lokal bei dir ausführen? |
|
ich kann die tests leider nicht lokal ausführen, da ich llvm12 bei mir lokal nicht kompilieren kann. Da versuche ich gerade einen workaround für zu finden. Und da ich den master-branch und nicht den dev-branch geforkt habe, kann ich den auch nicht ganz so einfach synchronisieren. Auch das versuche ich gerade zu beheben. |
|
Ja, kein Ding. Ich denke mal du arbeitest unter WIndows, da wollten wir eh noch a) einen vernünftigen Contributor guide schreiben oder b) ein Docker Image erstellen. Falls du die Möglichkeit hast unter Linux zu arbeiten wäre es einfach, da kannst du einfach |
|
Ich arbeite tatsächlich unter linux. Entweder in wsl, wo ich allerdings nur llvm13 als älteste Version in den repos gesehen habe, oder auf archlinux, wo in den offiziellen repositories auch nur noch llvm14 oder höher ist. Und das kompilieren aus dem Source-Code schlägt wie gesagt fehl, da schaue ich mal ob ich den fehler finde. Vielleicht fehlt ja einfach nur eine dependency oder so. |
|
Ok, wenn du auf Linux bist kriegst du das schon hin. |
|
|
||
| [Entferne_Anzahl_Hinten_Mutierend] | ||
| Der Text Entferne_Anzahl_Hinten_Mutierend_Text ist "Hello world!". | ||
| Entferne 0 Buchstaben am Anfang von Entferne_Anzahl_Hinten_Mutierend_Text. |
There was a problem hiding this comment.
Sollte hier und die nächsten 5 Mal Ende statt Anfang sein, kleiner copy-paste Fehler
lib/stdlib/Duden/Texte.ddp
Outdated
| Gib "" zurück. | ||
| Gib text bis zum (die Länge von text minus anzahl). Element zurück. | ||
| Und kann so benutzt werden: | ||
| "<text> mit den letzten <anzahl> Buchstaben enfernt" |
lib/stdlib/Duden/Texte.ddp
Outdated
| Gib "" zurück. | ||
| Gib text ab dem (anzahl plus 1). Element zurück. | ||
| Und kann so benutzt werden: | ||
| "<text> mit den ersten <anzahl> Buchstaben enfernt" |
|
nvm, war kein bug sondern ein Typo in den Aliasen. Bei mir lokal funktionieren die tests jetzt mit den Änderungen die ich noch kommentiert hab. Darum ist es halt gut lokal zu testen vor der Pull-Request, sonst dauert sowas ewig bis man jede Kleinigkeit hat. Falls du Hilfe brauchst die Tests bei dir zum Laufen zu bringen melde dich, ich helfe gern! |
| @@ -3,6 +3,48 @@ Binde "Duden/Ausgabe" ein. | |||
|
|
|||
| Schreibe (einen leeren Text) auf eine Zeile. | |||
|
|
|||
| [Entferne_Anzahl_Vorne_Mutierend] | |||
| Der Text Entferne_Anzahl_Vorne_Mutierend_Text ist "Hello world!". | |||
There was a problem hiding this comment.
world sollte Großgeschrieben sein für expected.txt: "Hello World!"
| Schreibe (Entferne_Anzahl_Vorne_Text) auf eine Zeile. [Hello World!] | ||
|
|
||
| [Entferne_Anzahl_Hinten_Mutierend] | ||
| Der Text Entferne_Anzahl_Hinten_Mutierend_Text ist "Hello world!". |
There was a problem hiding this comment.
Ebenfalls world Großschreiben: "Hello World!"
|
Ich verspreche das waren die letzten zwei Typos, die ich gefunden hab (hoffentlich wirklich). |
|
jaa, sorry dafür. Das sind alles wirklich schon irgendwie peinliche Fehler, aber was erwarte ich auch wenn ich den Code um 2 Uhr nachts schreibe... 😅 |
|
aber LLVM sollte hoffentlich bald kompilliert sein, dann kann ich die tests auch endlich selbst laufen lassen |
|
Sind ja nur Kleinigkeiten. |
Erweitern vom Duden um Entferne_Anzahl_Vorne, Entferne_Anzahl_Hinten
Dies erweitert den Duden um die Funktionen
Entferne_Anzahl_VorneundEntferne_Anzahl_Hinten.Für #39.