From 79e6a7709b03b44c763d4a29a8e2991ab64e739f Mon Sep 17 00:00:00 2001
From: Christopher Fremond <christopher.fremond@gmail.com>
Date: Fri, 21 Jun 2019 19:12:40 +0100
Subject: [PATCH] fix of compact

---
 subjects/compact.en.md | 48 +++++++++++++++++++++++++++++++++---------
 subjects/compact.fr.md | 44 +++++++++++++++++++++++++++++++-------
 2 files changed, 74 insertions(+), 18 deletions(-)

diff --git a/subjects/compact.en.md b/subjects/compact.en.md
index 29184b971..5c637f851 100644
--- a/subjects/compact.en.md
+++ b/subjects/compact.en.md
@@ -1,15 +1,18 @@
-## compact
+## Compact
 
 ### Instructions
 
-Write a function `Compact` that takes a pointer to an array as parameter and overwrites the elements that points to `nil`.
+Write a function `Compact` that takes a pointer to a slice of strings as the argument.
+This function must:
 
-- Hint : This fonction exists in Ruby.
+- Return the number of elements with non-`nil`.
 
-### Expected function
+- Compact, i.e., delete the elements with `nil` in the slice.
+
+### Expected functions
 
 ```go
-func Compact(ptr *[]string, length int) int {
+func Compact(ptr *[]string) int {
 
 }
 ```
@@ -21,13 +24,29 @@ Here is a possible [program](TODO-LINK) to test your function :
 ```go
 package main
 
-import fmt
+import (
+       "fmt"
+
+       piscine ".."
+)
+
+const N = 6
 
 func main() {
-	array := []string{"hello", " ", "there", " ", "bye"}
+	arr := make([]string, N)
+	arr[0] = "a"
+	arr[2] = "b"
+	arr[4] = "c"
+
+	for _, v := range arr {
+		fmt.Println(v)
+	}
 
-	ptr := &array
-	fmt.Println(Compact(ptr, len(array)))
+	fmt.Println("Size after compacting:", piscine.Compact(&arr))
+
+	for _, v := range arr {
+		fmt.Println(v)
+	}
 }
 ```
 
@@ -36,6 +55,15 @@ And its output :
 ```console
 student@ubuntu:~/piscine/test$ go build
 student@ubuntu:~/piscine/test$ ./test
-3
+a
+
+b
+
+c
+
+Size after compacting: 3
+a
+b
+c
 student@ubuntu:~/piscine/test$
 ```
diff --git a/subjects/compact.fr.md b/subjects/compact.fr.md
index ed9c3efb6..aaeb5b8e3 100644
--- a/subjects/compact.fr.md
+++ b/subjects/compact.fr.md
@@ -2,14 +2,17 @@
 
 ### Instructions
 
-Écrire une fonction `Compact` qui prend un pointeur sur tableau comme paramètre et qui réécris sur les éléments qui pointent sur `nil`.
+Écrire une fonction `Compact` qui prend un pointeur sur slice de `strings` comme paramètre.
+Cette fonction doit:
 
-- Indice : Cette fonction existe in Ruby.
+- Retourner le nombre d'éléments avec des valeurs non-`nil`
+
+- Comprimer, c.à.d., effacer les éléments qui ont une valeur `nil` dans la slice.
 
 ### Fonction attendue
 
 ```go
-func Compact(ptr *[]string, length int) int {
+func Compact(ptr *[]string) int {
 
 }
 ```
@@ -21,13 +24,29 @@ Voici un éventuel [programme](TODO-LINK) pour tester votre fonction :
 ```go
 package main
 
-import fmt
+import (
+       "fmt"
+
+       piscine ".."
+)
+
+const N = 6
 
 func main() {
-	array := []string{"hello", " ", "there", " ", "bye"}
+	arr := make([]string, N)
+	arr[0] = "a"
+	arr[2] = "b"
+	arr[4] = "c"
+
+	for _, v := range arr {
+		fmt.Println(v)
+	}
 
-	ptr := &array
-	fmt.Println(Compact(ptr, len(array)))
+	fmt.Println("Size after compacting:", piscine.Compact(&arr))
+
+	for _, v := range arr {
+		fmt.Println(v)
+	}
 }
 ```
 
@@ -36,6 +55,15 @@ Et son résultat :
 ```console
 student@ubuntu:~/piscine/test$ go build
 student@ubuntu:~/piscine/test$ ./test
-3
+a
+
+b
+
+c
+
+Size after compacting: 3
+a
+b
+c
 student@ubuntu:~/piscine/test$
 ```