Frankly it looks to me like you have no idea what you are supposed to be doing!
Yes, since we are told that "[math]x_1= 3x_4[/math], [math]x_2= 8+ x_4[/math], and [math]x_3= 2- 5x_4[/math] we have immediately that [math]\begin{bmatrix}x_1 \\ x_2 \\ x_3 \\ x_4 \end{bmatrix}= \begin{bmatrix} 3x_4 \\ 8+ x_4 \\ 2- 5x_4 \\ x_4 \end{bmatrix}[/math].
But why then did you set [math]x_4[/math] to 0?? The problem says that [math]x_4[/math] is "free" which means that it can be any number. Saying that a number is "free" certainly does NOT mean that it is 0!
I would say that [math]\begin{bmatrix} 3x_4 \\ 8+ x_4 \\ 2- 5x_4 \\ x_4 \end{bmatrix}[/math] is a perfectly good answer but some people might prefer to replace the "coordinate", [math]x_4[/math] with the "parameter", t:
[math]\begin{bmatrix} 3t \\ 8+ t \\ 2- 5t \\ t \end{bmatrix}[/math].
Some would prefer to write that as
[math]\begin{bmatrix}0 \\ 8 \\ 2 \\ 0 \end{bmatrix}+\begin{bmatrix} 3 \\ 1 \\ -5 \\ 1 \end{bmatrix}t[/math].