query_posts() should be avoided?

Question

I am reading that query_posts() should be avoided in favor of wp_query() and pre_get_posts(). I am not confident with messing with the

Answer 1

Creating a new WP_Query() object is always fine.

$sort= $_GET['sort'];

   if($sort == "pricelow"){
     $sort_args = array('meta_key' => 'price', 'orderby' => 'meta_value_num', 'order', 'ASC');
     $new_query = new WP_Query($sort_args);
    }

    blah blah blah...

No no no sorry about that. I didn't see the pre_get_posts hook.

The code in your question is good for hooking queries. As in described in WordPress Plugin API/Action Reference/pre_get_posts:

pre_get_posts runs before WP_Query has been setup.

So it hooks the default WP_Query() where you want (in your code, it changes WP_Query on GET request).

In your template files, use new WP_Query($args).