Немного не разобрался в перегрузке []

[Borkot]

Привет всем!
Спасибо заранее всем! Если Я передаю по ссылке то и возвращать нужно ссылку.
Значит в фукции main() сделать присваивание.
Подчеркивает return s[k];Мне кажется что-то делаю не так.

файл MyString.h

#pragma once
#include"stdafx.h"
class MyString
{
	char s[255];
public:
	MyString(void);
	~MyString(void);
	void Show(void) const;
	void SetString();
	char &operator [](int k)const;
};

MyString.cpp

#include "MyString.h"


MyString::MyString(void)
{
	strcpy(s,"Привет люди человеки!");
}


MyString::~MyString(void)
{
}

void MyString::Show(void) const
{
  cout<<"s= "<<s<<endl<<flush;
}

void MyString::SetString()
{
   gets(s);
}
char &MyString::operator [](int k)const
{
	 int p=strlen(s);
	 if(k<0)
		 cout<<"\n"<<*s<<"\n";
	 else
	 {
		 if(k>p)
			 cout<<"\n"<<s[p-1]<<"\n";
		 else 
			 return s[k]; 
	 }
}

[Borkot]

в stdafx.h

#pragma once
#include<iostream>
#include<string.h>
using namespace std;

[Borkot]

Извините задачу забыл вылоджить задача такая обратится к массиву к конкретному элементу с помощью перегрузки оператора [] .ну и проверять что- бы не выходи за пределы.

[Abstraction]

Моя не понял, чего у Вас в operator[] делает cout.
Что печальнее, поскольку метод помечен как константный, то и s имеет тип const char[]. А поскольку Вы возвращаете неконстантную ссылку на элемент массива, компилятор начинает ругаться самыми чёрными словами.
С другой стороны, если возвращать константную ссылку, нельзя будет написать
myStr[2] = 'a';
Поэтому нужно - так (по-хорошему, надо бы кидать исключение за такие обращения, ну да ладно):

const char& MyString::operator[](int k)const {
  static char trash = '\0';
  if(k<0 || k>=strlen(s)){
    std::cerr << "MyString::operator[] - Index out of bounds!" << std::endl; 
    return trash;
  }
  return s[k];
}

char& MyString::operator[](int k) {
  return const_cast<char&>(const_cast<const MyString&>(*this)[k]);
}

[_SOKOL_]

поменяй
char &operator [](int k)const;
на
char *operator [](int k)const;

когда возвращаешь строку, то надо возвращать указатель, так как массив сам по себе и есть указатель

[waleri]

Сообщение от _SOKOL_

поменяй
char &operator [](int k)const;
на
char *operator [](int k)const;

когда возвращаешь строку, то надо возвращать указатель, так как массив сам по себе и есть указатель

В принципе можно так сделать, но не нужно, иначе надо будет помнить, что для этого класса результат [] отличается.
Если у нас есть char* ptr то какой тип возвращается при ptr[123]?
Если у нас есть char ptr[123] то какой тип возвращается при ptr[123]?
Тем более, что соль не в том, чтоб вернуть строку а символ из строки, при чем так, чтоб можно было использовать и как lvalue

[_SOKOL_]

Сообщение от waleri

В принципе можно так сделать, но не нужно, иначе надо будет помнить, что для этого класса результат [] отличается.
Если у нас есть char* ptr то какой тип возвращается при ptr[123]?
Если у нас есть char ptr[123] то какой тип возвращается при ptr[123]?
Тем более, что соль не в том, чтоб вернуть строку а символ из строки, при чем так, чтоб можно было использовать и как lvalue

в таком случае перегрузку делаем так
char operator[] (int k) const;
если нужно вернуть один символ, то возвращаем просто char, зачем возвращать ссылку или указатель?

[Abstraction]

если нужно вернуть один символ, то возвращаем просто char, зачем возвращать ссылку или указатель?

Тогда

нельзя будет написать
myStr[2] = 'a';

Точнее, написать-то будет можно, но вот результат окажется далёк от ожидаемого.

[denrubun]

sizeof(char) == 1 байт
sizeof(указатель) == 4 байт
ссылка это тоже указатель.
спрашивается: зачем возвращать ссылку?( только в случае char!!)

отсюда: const char operator[] (int k) const; (для rvalue)

для lvalue правильнее будет возвращать ссылку, т.к. с ней работать(синтаксически) так же как с символом, а вот с указателем как раз куча проблем и отличий возникает.

отсюда: char &operator[] (int k); (для lvalue)

два метода полюбому реализовывать придется от этого не уйти

[Abstraction]

спрашивается: зачем возвращать ссылку?( только в случае char!!)

Ну, во-первых, мы тогда лишаемся возможности в неконстантной версии метода сослаться на константную; во-вторых, отнюдь не очевидно, что Ваш вариант оптимальнее:
1) Возвращаем ссылку. В функции - взять адрес начала массива, прибавить требуемое смещение, вернуть адрес (thiscall возвращает результат в eax). В вызывающем коде - обратиться по адресу, прочитав значение в другой регистр.
2) Возвращаем значение. В функции - взять адрес начала массива, прибавить смещение, обратиться по адресу, вернуть значение. В вызывающем коде просто использовать значение.
Т.е. проделывается один и тот же набор действий, в общем-то.