C#•13mo ago

❔ ✅ A word with the length of > 5 will be reversed, e.g. hello -> olleh

Write a function that takes in a string of one or more words, and returns the same string, but with all five or more letter words reversed (Just like the name of this Kata). Strings passed in will consist of only letters and spaces. Spaces will be included only when more than one word is present. Examples:

spinWords( "Hey fellow warriors" ) => returns "Hey wollef sroirraw" 
spinWords( "This is a test") => returns "This is a test" 
spinWords( "This is another test" )=> returns "This is rehtona test"

spinWords( "Hey fellow warriors" ) => returns "Hey wollef sroirraw" 
spinWords( "This is a test") => returns "This is a test" 
spinWords( "This is another test" )=> returns "This is rehtona test"

my code:

using System.Collections.Generic;
using System.Linq;
using System;
using System.Text;



public class Kata
{
  public static string SpinWords(string sentence)
  {
    string[] words = sentence.Split(" ");
    foreach (string word in words){
      if (word.Length() >= 5){
      StringBuilder sb = new StringBuilder(word);
      words[word] = sb.Reverse();
        }
      string connected_string = words.join(' ');
      return connected_string;

    }
    
    string sen = SpinWords("this is a trial sentence.");
    Console.WriteLine(sen);
    
  }
}

using System.Collections.Generic;
using System.Linq;
using System;
using System.Text;



public class Kata
{
  public static string SpinWords(string sentence)
  {
    string[] words = sentence.Split(" ");
    foreach (string word in words){
      if (word.Length() >= 5){
      StringBuilder sb = new StringBuilder(word);
      words[word] = sb.Reverse();
        }
      string connected_string = words.join(' ');
      return connected_string;

    }
    
    string sen = SpinWords("this is a trial sentence.");
    Console.WriteLine(sen);
    
  }
}

error :

src/Solution.cs(14,16): error CS1955: Non-invocable member 'string.Length' cannot be used like a method.
src/Solution.cs(16,13): error CS0029: Cannot implicitly convert type 'string' to 'int'
src/Solution.cs(16,24): error CS1061: 'StringBuilder' does not contain a definition for 'Reverse' and no accessible extension method 'Reverse' accepting a first argument of type 'StringBuilder' could be found (are you missing a using directive or an assembly reference?)
src/Solution.cs(18,39): error CS1061: 'string[]' does not contain a definition for 'join' and no accessible extension method 'join' accepting a first argument of type 'string[]' could be found (are you missing a using directive or an assembly reference?)
src/Solution.cs(10,24): error CS0161: 'Kata.SpinWords(string)': not all code paths return a value

src/Solution.cs(14,16): error CS1955: Non-invocable member 'string.Length' cannot be used like a method.
src/Solution.cs(16,13): error CS0029: Cannot implicitly convert type 'string' to 'int'
src/Solution.cs(16,24): error CS1061: 'StringBuilder' does not contain a definition for 'Reverse' and no accessible extension method 'Reverse' accepting a first argument of type 'StringBuilder' could be found (are you missing a using directive or an assembly reference?)
src/Solution.cs(18,39): error CS1061: 'string[]' does not contain a definition for 'join' and no accessible extension method 'join' accepting a first argument of type 'string[]' could be found (are you missing a using directive or an assembly reference?)
src/Solution.cs(10,24): error CS0161: 'Kata.SpinWords(string)': not all code paths return a value

61 Replies

AhmedOP•13mo ago

but i still don't understand.

Jimmacle•13mo ago

what don't you understand?

AhmedOP•13mo ago

the error i'm making

Jimmacle•13mo ago

you wrote a bunch of code that isn't valid C#

AhmedOP•13mo ago

is it not?

Jimmacle•13mo ago

so going down your list of errors string has a Length property, not a Length() method you cannot convert a string to an int without using int.Parse StringBuilder doesn't have a reverse method, self explanatory string[] doesn't have a join method and finally your method is supposed to return something but doesn't

AhmedOP•13mo ago

how do i access the property? i dont think i was trying to convert

Jimmacle•13mo ago

by typing it exactly how the error shows

AhmedOP•13mo ago

dang string.Length

Jimmacle•13mo ago

word.Length no parentheses, because it's not a method

AhmedOP•13mo ago

fair enough how do i join string[] then

Jimmacle•13mo ago

all your errors mostly boil down to not researching what is actually available for each of these types string.Join("separator", array)

AhmedOP•13mo ago

i need to get the index of the word here

                words[word] = sb.Reverse();

                words[word] = sb.Reverse();

i'll do it myself but how do i reverse a string hello -> olleh

Jimmacle•13mo ago

https://www.google.com/search?q=c%23+reverse+a+string

🔎 c# reverse a string - Google Search

AhmedOP•13mo ago

can i reverse via an array put the word into an array and reverse through there

Jimmacle•13mo ago

what does your research tell you?

AhmedOP•13mo ago

yes but it doesn't show me how i can cut down from a string into a character set

Jimmacle•13mo ago

the first result of that google search tells you exactly how

AhmedOP•13mo ago

https://stackoverflow.com/questions/228038/best-way-to-reverse-a-string

Stack Overflow

Best way to reverse a string

I've just had to write a string reverse function in C# 2.0 (i.e. LINQ not available) and came up with this: public string Reverse(string text) { char[] cArray = text.ToCharArray(); string

Jimmacle•13mo ago

public static string Reverse( string s )
{
    char[] charArray = s.ToCharArray();
    Array.Reverse(charArray);
    return new string(charArray);
}

public static string Reverse( string s )
{
    char[] charArray = s.ToCharArray();
    Array.Reverse(charArray);
    return new string(charArray);
}

AhmedOP•13mo ago

i'm here ye a char array how do i turn a char[] array into a string[] array so i can add it back to the original sentence string[] reversed_char = chars.ToString();

Jimmacle•13mo ago

you don't want that

AhmedOP•13mo ago

doesn't exist

Jimmacle•13mo ago

look at the method i shared here it does exactly what you want line 1 turns the string into an array of chars line 2 reverses that array line 3 creates a new string from that array

AhmedOP•13mo ago

can i just do string reversed_word = string(charArray);

Jimmacle•13mo ago

you tell me

AhmedOP•13mo ago

yes

Jimmacle•13mo ago

try it

AhmedOP•13mo ago

works

AhmedOP•13mo ago

AhmedOP•13mo ago

however oh wait now i gotta turn it into an array and then replace hmm

using System.Collections.Generic;
using System.Linq;
using System;
using System.Text;


public class Kata
{
  public static string SpinWords(string sentence)
  {

    string[] words = sentence.Split(" ");
   
        foreach (string word in words){
      
            if (word.Length >= 5){
      
                int indexOfWord = Array.IndexOf(words, word);
      
                char[] chars = word.ToCharArray();
      
                Array.Reverse(chars);

                string reversed_word = new string(chars);

                words[indexOfWord] = reversed_word;

            }

        }

        string final_sentence = word.Join(" ");

        return final_sentence;
    }

    
}

using System.Collections.Generic;
using System.Linq;
using System;
using System.Text;


public class Kata
{
  public static string SpinWords(string sentence)
  {

    string[] words = sentence.Split(" ");
   
        foreach (string word in words){
      
            if (word.Length >= 5){
      
                int indexOfWord = Array.IndexOf(words, word);
      
                char[] chars = word.ToCharArray();
      
                Array.Reverse(chars);

                string reversed_word = new string(chars);

                words[indexOfWord] = reversed_word;

            }

        }

        string final_sentence = word.Join(" ");

        return final_sentence;
    }

    
}

         string final_sentence = word.Join(" ");

         string final_sentence = word.Join(" ");

there's an error here what's wrong with it?

Jimmacle•13mo ago

what does your IDE say is wrong with it

AhmedOP•13mo ago

word doesn;t exist in this current context

Jimmacle•13mo ago

correct word only exists inside your loop you can't use it outside your loop

AhmedOP•13mo ago

Jimmacle•13mo ago

yes, look up how to correctly join an array of strings

AhmedOP•13mo ago

what i wanna return doesn't exist either string joinedString = String.Join(" ", words);

Angius•13mo ago

Well You commented out final_sentence variable

AhmedOP•13mo ago

done

Angius•13mo ago

So... the IDE is correct, it does not exist

AhmedOP•13mo ago

it does

Angius•13mo ago

Not in this scope $scopes

MODiX•13mo ago

scope A {
  thing a;
  scope B {
    thing b;
  }
}

scope A {
  thing a;
  scope B {
    thing b;
  }
}

thing a is available in scope A and scope B thing b is available only in scope B

AhmedOP•13mo ago

how do i make it exist outside the scope

Angius•13mo ago

int foo;
scope {
    foo = 69;
}
Console.Write($"The number is {foo}");

int foo;
scope {
    foo = 69;
}
Console.Write($"The number is {foo}");

Jimmacle•13mo ago

that is a solution, but i doubt that's the correct one based on the purpose of the code do you really want to rebuild your whole sentence every time you loop?

AhmedOP•13mo ago

Jimmacle•13mo ago

so why did you put it in the loop?

AhmedOP•13mo ago

Write a function that takes in a string of one or more words, and returns the same string, but with all five or more letter words reversed (Just like the name of this Kata). Strings passed in will consist of only letters and spaces. Spaces will be included only when more than one word is present.

Write a function that takes in a string of one or more words, and returns the same string, but with all five or more letter words reversed (Just like the name of this Kata). Strings passed in will consist of only letters and spaces. Spaces will be included only when more than one word is present.

i removed it from the loop how do i console.WriteLine now

Jimmacle•13mo ago

what do you mean?

AhmedOP•13mo ago

If i wanna use it Console.WriteLine(SpinWords("this is a trial sentence"));

Jimmacle•13mo ago

it looks like you know how to use it just fine

AhmedOP•13mo ago

no like which scope

Jimmacle•13mo ago

what do you mean that's a single line of code with no variables involved

AhmedOP•13mo ago

Jimmacle•13mo ago

you need to qualify it with the class name Kata.SpinWords("...")

AhmedOP•13mo ago

thank you :)) !close

Accord•13mo ago

Closed! Was this issue resolved? If so, run /close - otherwise I will mark this as stale and this post will be archived until there is new activity.

Gaming

Programming

❔ ✅ A word with the length of > 5 will be reversed, e.g. hello -> olleh